[코드 리뷰] 좋은 기회로 얻은 JS관련 코드 리뷰2

 

1. 얼리리턴

 

일정 조건을 걸어 함수의 뒷 내용이 실행되지 않도록 할 수 있다!

 

=> 가독성 측면에서 좋음

봐도 위 보다 아래가 가독성이 좋은 걸 볼 수 있다 ㅎㅎ...

 

 

2. 유지보수 용이한 코드를 작성하자.

 

해당 미션에는 2가지의 form이 있어 모든 폼의 새로고침을 막으려 formEvent함수를 만들었다.

그러나 해당 미션에는 괜찮지만 만약 다른 form에서 submit하는 상황이 생긴다면 돌아가지 않을 가능성이 있다.

아무 생각없이 새로고침 다 막아야지! 하고 진행했던 것 같다.

 

 

3. EOL(End Of Line)

ESLint, Prettier 설정에서 EOL에 대해 한 번 찾아보라고 말씀하셨다.

"endOfLine": "auto"

으로 설정할 수 있고, 자동으로 파일 맨 끝에 개행이 되게끔 해주는 것이다!

 

 

4. Style은 CSS에서 작성

js에서 스타일을 조정했다. 그러나 생각해보니 css파일이 따로 있었는데 왜 그랬는 지 모르겠다 ㅎㅎ..

 

 

 

+ 배열의 filter의 존재를 잠깐 잊은 게 어이가 없었다.. (map, reduce, forEach만 생각함)..

+ 함수 및 변수의 네임을 좀 더 명확하게 작성하자! (저번 주에도 들었던 소리지만 좀 더 명확하게 해야겠다 ㅎㅎ..)