FrontEnd Js부분 코드 리뷰 정리

코드리뷰 1차

이번에 현장실습 나온 회사의 팀장님께서 코드 리뷰를 해주셨따. frontend 부분에서
항상 많은 부족함을 느끼는데 좋은 기회였다. 프로젝트는 크롬 익스텐션인데
html,js,css가 주되다 보니 거의 프론트 영역에 가까웠다.

js 부분

• main 부분은 main 용 이름으로 파일을 구성해라
  • index.js, index.html, app.js, app.html과 같이

 

 

• 변수, 함수 이름 등을 동사 + 명사로 구성해라
  • 지금까지 app을 생성하는 함수면 app_create라 이름을 지었는데 create_app으로 변경하였다.

 

 

 

• element의 class id 부분에서 왠만하면 class를 사용해라
  • 아무래도 id는 고유값이기 때문에 제약이 많다.

 

 

 

• 함수들의 나열 순서는 호출부는 나중에 선언부는 먼저 오게 하라.
  • 예를 들어 init 함수에서 call_name이라는 함수를 호출한다고 하면 call_name을 먼저 선언하고 init은 나중에 작성한다.

 

 

 

• js에는 dictionary라는 명칭보다 object라는 명칭을 쓰자.
  • js에는 dictionary라는 개념이 없다고 한다.

 

 

 

• 스크립트형 언어에서 순수함수와 비순수 함수를 구별해서 return을 활용하자
  • 비순수 함수란 화면에 직접적인 영향을 끼치는 것으로 이럴 때는 요소를 return해서 main부에 작성한다.

 

 

 

• if 문 안에 비교에서 왠만하면 '==' 대신 '==='을 사용하자
  • 타입 때문에 오류가 생기는 경우가 많다.

 

 

 

 

• if 문을 사용할 때는 else를 쓰지 않게 작성하도록 노력하자
  • 만약 이중 if 문으로 구성되어 있다면 첫번쨰 if에서 return을 활용해 filtering 한다.

 

 

 

 

• event 호출 부분에서 왠만하면 this대신 event를 활용하자.
  • 이 부분은 취향차이인듯 한데 회사 표준을 따라가는 것이 좋을것 같다. 왠만하면 모든 부분에서 표준을 통일시키자.

 

 

 

• 전역변수는 파일의 가장 상단부에 위치시킨다.
  • 전역변수를 가장 앞에두어야 가독성이 좋다.

 

 

 

• js로 style을 조작해야 하는 경우가 있따면 css style class를 따로 만들고 add class를 한다.
  • 한눈에 보기도 어렵고 나중에 style로 지정하는 방식이 있을떄는 헷길리기 마련이다.

 

 

 

• insertHtml이나 js를 통해 html을 삽입해야 하는 상황이면 일반 string이 아닌 역따음표 `를 사용한다.
  • 가장 좋은 배움인 것 같다. 지금까지 노가다 한 자신이 미워진다. 역따음표 사용해보면 너무너무 편하다.