june2
commented
1 year ago -
네이밍 컨벤션을지켜주세요, 첫글자 대소문자가 혼합되어있네요. https://github.com/hoonbongEE/NB_LV4/blob/ad3c26e91962cf649c1fe55f811276a9680db7b3/models/comments.js#L24-L40
-
파일명도 켄벤션을 지켜주세요. 단수 복수 혼합되어 있습니다.
-
불피요한 주석은 삭제 해주세요. 코드로 알수 있는 내용은 굳이 주석을 달지 말아주세요. 오히려 가독성을 떨어뜨립니다. https://github.com/hoonbongEE/NB_LV4/blob/ad3c26e91962cf649c1fe55f811276a9680db7b3/routes/comments.js#L12
-
존재 하지 않을때는 404 not found에러가 더 적합합니다. https://github.com/hoonbongEE/NB_LV4/blob/ad3c26e91962cf649c1fe55f811276a9680db7b3/routes/comments.js#L75-L77
-
post에 likes필드를 추가 해서 증감하는게 아니라 like 엔티티와 join을 해야합니다. like 갯수를 가져오기 위해서 group by count 쿼리 작성이 필요합니다. 해당 내용 참고해보세요!https://stackoverflow.com/questions/42470383/sequelize-query-with-count-in-inner-join https://github.com/hoonbongEE/NB_LV4/blob/ad3c26e91962cf649c1fe55f811276a9680db7b3/models/posts.js#L48-L51
-
update를 title, content따로 하지않고, 한번에 쿼리를 호출해서 변경해야합니다. https://github.com/hoonbongEE/NB_LV4/blob/ad3c26e91962cf649c1fe55f811276a9680db7b3/routes/posts.js#L113-L118
-
ERD작성은 잘하셨는데 관계표시가 누락되어있네요.
-
커밋 메시지도 남겨져있지 않네요. 커밋로그를 잘남기는 습관을 가져보아요. (https://yozm.wishket.com/magazine/detail/1974/)
[x] 내 프로젝트의 ERD 작성하기 [x] 댓글 목록 조회 API [x] 댓글 작성 API [x] 댓글 수정 API [x] 댓글 삭제 API [x] 게시글 좋아요 API [] 전체 게시글 목록 조회(좋아요갯수) API [] 게시글 조회(좋아요갯수) API