issuebombom / nodejs_personal_project_layered_architecture

0 stars 0 forks source link

lv5 과제 리뷰 #3

Open Ho-yeong opened 1 year ago

Ho-yeong commented 1 year ago

종합의견

readme 파일을 잘 작성해주셨습니다. 커밋 메세지도 좀 더 자세하게 사용해봅시다.
service 단계에서는 에러핸들링이 보이지않습니다. 효율적인 에러핸들링 방법을 적용해봅시다.
Profile image, refresh token 등 테이블을 세세하게 분류해놓았는데 테이블의 정규화를 많이 할수록 데이터 관리가 힘들어집니다.
이제 코드 그자체의 공부보다는 필수요소가 빠지지않도록, 효율적인 코드의 진행 생각해봅시다.

Ho-yeong commented 1 year ago

https://github.com/issuebombom/nodejs_personal_project_layered_architecture/blob/0fa397a8156b070d144730def2649e97a44369f6/repositories/posts.repository.js#L9-L23 like를 알기 위해서 like의 모든 정보를 불러오고 있습니다. count만 필요한 거라면 비용이 적은 쿼리를 사용합시다.

https://github.com/issuebombom/nodejs_personal_project_layered_architecture/blob/0fa397a8156b070d144730def2649e97a44369f6/services/users.service.js#L56-L71 if 조건문에서 중괄호 생략시 가독성이 떨어집니다.