Prep Calendar project 첫 번째 코드 리뷰

달력과제 제출한 것에 대한 코드 리뷰가 2일 전에 달렸다. 뜨든.

근데 나의 고질병 중에 하나인 아 어떡해 못보겠다 병이 도져서 오늘에서야 열어봤다.

이 것들은 다음 번 코드 작성시에 더 신경쓰도록 하자.

 

• README File 
  • 컴퓨터 구성, 설치법, 사용법, 저작권, 사용권, 배포자 연락처, 알려진 버그, 트러블 슈팅, 체인지 로그 등으로 구성되어 있다.
  • 약간 코드의 사용설명서 + 히스토리를 담고 있는 내용?
  • 자세한 점은 위키피디아 참조.  
• Array 재활용
  • monthName, monthDays를 따로따로 Array 로 만들어서 지정해줬는데,
  • Array 안에 key & value를 이용하여 하나의 큰 array를 사용하는 것을 권장해주셨다. 
    • const months = {
        JAN:  {
          name:  'JAN',
          lastday: 31
        },
      .....
      } 
  • 확실히 내가 C++ 할 때의 코딩 스타일로만  JS를 접근하고 있다. JS는 어레이 안에 키랑 밸류를 줄 수 있다. 

• 변수명의 명확한 선언.
  • today를 줄여서 t 라는 변수를 썼는데, 재정의를 위해 검색한다면 정말 밑도 끝도 없을 변수명이다...
  • 주석을 달아서 변수를 설명하기 보다는 처음부터 변수명을 좀 더 직관적으로 만들어주자!
• 함수 앞에 공백없음.
  • 일반적으로 함수의 첫줄에는 공백을 두지 않습니다!
• CSS naming 은 꼬치구이식으로 사용합시다.
  • frontend (x), front-end (o)
  • Naming convention
• console.log 찍어본 흔적을 남기지 말자 ^^;

-> 이 외에, 가끔가다가 코딩 스타일이 다른 포인트들이 있었다. (인덴트가 이상하게 되어있다거나, 연산자를 어디선 죄다 붙여쓰고 어디선 죄다 스페이스 넣어서 깔끔쓰) 이건 저번에  prep 과제 끝냈을 때 느꼈던, '나중에는 없다, 매순간 최선을 다해 fitting해야한다' 와 일맥상통하는 개선점이다. 

 

 

 

한편, 내가 더 공부해야할 부분은 다음과 같다. 

 

• Var, Let, Const 의 차이점 // var는 hoisting 관련해서 유의해야하고, 이거말고 let 을 더 쓴다는 점을 과제 종료 후에 알게되었다. 
• InnerHTML, innerText, textContent의 차이점 
• 함수의 선언 & 실행의 차이점
  • function(event)와 function() 둘 다 왜 똑같이 동작했을까?
  • Callback 함수의 정의 및 실행