[Week1] Code Review Summary

1번째 과제 코드 리뷰에서는 기본적인 것들에 대해 피드백을 받았다.

완전 기본 중에 기본인데 convention을 제대로 확인하지 않고 과제를 제출하여 많이 반성했다.

(너무 기본적인거라 민망했다.😅😅)

 

개선해야 할 사항

if와 else if의 앞뒤는 한 칸 띄우기

// bad
if(){}

// good
if () {}

 

주석

주석을 일일이 달 필요는 없다. 변수 네이밍, 함수 네이밍과 로직이 잘 짜여있다면 더욱더 주석을 달 필요가 없다.

만약 주석을 달아야 될 것 같다면 지금 구현한 코드가 가동성있게 작성한 것인지 고민해볼 필요가 있다. 주석은 가능한 최소한으로 줄이는 게 좋다.

 

객체 선언에 세미콜론 붙이기

(생각치 못한 곳에서 세미콜론을 빼먹었었다...)

 

코드의 일관성

string 값을 적을 때 '(single quote)와 "(double quote)를 번갈아서 써두었다. 일관성을 지키도록 노력하자!

 

 

추가로 공부하면 좋을 것들

String.prototype.includes()의 작동 방법 & 시간 복잡도

String.prototype.includes()가 내부적으로 어떻게 작동하는지 한번 알아볼 필요가 있다고 조언해주셨다.

시간 복잡도란?

자바스크립트에서 Big O(시간 복잡도)란?

Array Method 시간 복잡도

Time complexity Big 0 for Javascript Array methods and examples.

 

String.prototype.indexOf()의 작동 방법

String.prototype.indexOf()가 내부적으로 어떻게 작동되는지 알아두면 좋을 것 같다는 코멘트를 남겨주셨다.
특히 지금처럼 `while`이나 `for`문과 같은 반복문 안에 `indexOf()`나 `includes()` 와 같은 메서드를 쓰는 경우는 좀 더 조심할 필요가 있다고 말씀하셨다.

참고) String.prototype.indexOf()는 Array.prototype.indexOf()와 작동방식이 같다.

 

기타

CSS 속성 순서 -  모질라에서 제안한 css 속성 순서 

CSS 속성 순서 | Today Yurim Learned