[TIL] [4주차] [20221208] 4주차 회고 - 코드리뷰와 리팩토링 / package.json, EC2 set locale, express sanitizer

오늘은 4주차 마지막 날로, 주차 시험을 치뤘다. 끝나고 나니까 그렇게 어려운 문제는 아니었는데, 에러 메세지에 낚여서 SQL row query 문법을 찾느라 시간을 많이 버렸다. 그래도 시간 내에 잘 풀었다.

오후에는 조별과제 포스팅을 했고, 저녁부터 어제 기술 매니저님께 받은 리뷰 대로 코드를 수정했다. 아래는 코드리뷰를 받은 내용이다.

 

• app.js에서 router 불러 오는 것은 ./routes/index.js만  index에서 나머지 라우터를 계층 순서에 맞게 라우팅 해주기
  • app -> index -> user / posts / comments 순으로 계층구조 정리할 것
• app.js에서 최종 error handdling 미들웨어 추가
• 보통 파일 이름은 단어 하나로 사용
  • authAndUser -> auth와 user 파일로 분리 또는 보통 회원가입 및 로그인 묶어서 user로 씀
• router 스크립트에는 router만 담기도록.
  • 만약, router 파일에서 router 외에, 별도의 정의한 함수를 모듈로 사용하려면, util 폴더에 별도 파일로 만들어서 관리
  • 아니라면, router 안에서 함수를 정의하고 해당 파일 안에서만 사용
• await이 .then()을 대체하기 위해서 나온 것이니까, async await을 사용한다면, await을 사용할 것

 

위의 리뷰 내용을 반영하여 리팩토링 하였고, 진행하면서 보였던 중복 코드들도 모두 정리해주었다.

Refactor: 22.12.07 기술매니저님 피드백 반영 · jn33-dev/hanghae-week4@da06608

 

코드리뷰 시간에 추가로 더 조언해주신 내용들 아래 공유한다.

• SQL deleteOn 옵션 더 알아보기 / 보통 구현하는 서비스에 따라서 해당 옵션을 많이 사용함
• package.json
  • “script” 지정해서 사용 가능 (node app.js와 같은 명령문을 지정해 놓고, 사용 가능)
  • “devDependency” : 계발 단계에서만 필요한 패키지를 -D 옵션으로 설치
• package-lock.json : 배포할 때, dependent한 package들의 버전을 저장해 두는 문서. 매우 중요하므로, 깃헙에 무조건 같이 올릴 것!

package.json의 dependencies와 devDependencies의 차이점에 대한 정리 | imkh.dev

 

• RDS에서는 db region 설정이 가능함.
  • EC2에서 지역 시간대로 설정하려면, EC2 set locale

AWS EC2 Timezone, Locale 변경하기

 

• sanitizer: req.body 값을 검증해주는 보안

express-sanitizer

 

어떤 코드를 짜던, 정답은 없는데, 내가 쓰는 코드가 내가 서비스 할 서비스에 효율적인 코드인지 고민해보고, 왜 그렇게 짰는지 답할 수 있어야 함!!