과제 리뷰 - Githubissues

june2 commented 1 year ago

[x] 내 프로젝트의 ERD 작성하기 [x] 댓글 목록 조회 API [x] 댓글 작성 API [x] 댓글 수정 API [x] 댓글 삭제 API [x] 게시글 좋아요 API [x] 전체 게시글 목록 조회(좋아요갯수) API [x] 게시글 조회(좋아요갯수) API

jiahnkim230502 commented 1 year ago

임의로 빠뜨린 ERD까지 작성해서 깃 레파지토리에 올려두었습니다. 가능하시다면 다른 피드백도 주시면 감사하겠습니다!

june2 commented 1 year ago

검토해야할 과제가 많아서 채점 먼저하고 나중에 피드백 드리고 있어요. 다음부터 참고 해주세요!

jiahnkim230502 commented 1 year ago

넵 알겠습니다.

june2 commented 1 year ago

네이밍 컨벤션을 통일시켜주세요. 첫글자에 대소문자가 혼합되어있네요 https://github.com/jiahnkim230502/LV4_PROJECT/blob/6cd784d1a16675416dcdbe3aa5592887746f5776/models/comments.js#L27-L37
코드에서 바로 파악이되는 불필요한 주석은 삭제해주세요. 오히려 가독성을 떨어뜨립니다. https://github.com/jiahnkim230502/LV4_PROJECT/blob/6cd784d1a16675416dcdbe3aa5592887746f5776/models/comments.js#L27-L30
post에 nickname을 저장하는게 아니라 user테이블을 join해서 nickname을 조회해야합니다. user가 nickname을 변경했을경우 데이터가 다르게 보이기 때문이에요 https://github.com/jiahnkim230502/LV4_PROJECT/blob/6cd784d1a16675416dcdbe3aa5592887746f5776/models/posts.js#L25-L44
user와 userInfo 엔티티를 1:1관계로 분리한 이유가있나요? 지금상황에서는 user 엔티티에 모두 포함해도 무관할거같네요
파일명 컨벤션을 지켜주세요. 복수/단수 혼합되어있네요.
ERD 작성 잘하셨어요. 그런데 post, commet에 관계그림이 1:N이어야하는데 잘못되었네요.
커밋 컨벤션을 잘 지키는 습관을 들여보아요 (https://yozm.wishket.com/magazine/detail/1974/)

jiahnkim230502 / LV4_PROJECT