[김민식] sprint 7#12
Hidden character warning
Conversation
broBbang2
requested review from
Hoooooou-Jun and
iamjaeholee
and removed request for
iamjaeholee
|
PR템플릿이 사라져서... 이렇게 리뷰 남깁니다.. |
Hoooooou-Jun
left a comment
Hoooooou-Jun
left a comment
There was a problem hiding this comment.
안녕하세요 민식님. 코드리뷰 마무리되었습니다!
전반적으로 민식님은 요구사항에 맞게 거의 모든 리스트업을 충족해 주셨습니다. 고생 많으셨습니다.
더불어 나름 에러 처리에 대한 구분들이 확실하게 되어있어서 500 처리를 많이 줄여주셨습니다.
제가 체크한 부분들, 그리고 그 외 공통된 부분들 찾으셔서 같이 개선하면 좋을 부분들이라 해당 부분 참고하셔서 개선해주시면 좋을 것 같습니다!
백엔드에 대해 전반적으로 이해가 안되시는 것 같은데..ㅎ 일단은 지금처럼 에이전트 도움 받으시면서 계속 스프린트 수행해주시고 후에 나중에 개념 반복해서 정리하시는거 권유 드립니다. 일단은 지금은 최대한 많이 따라치시고 나중에 해당 문법이나 express 계열 라이브러리 작성법들을 익히시면서 습득하시는게 좋을 것 같습니다. 잘 하고 있으십니다!!
수고 많으셨습니다!
|
|
||
| const articleIdNum = parseInt(articleId, 10); | ||
| if (isNaN(articleIdNum)) { | ||
| return res.status(400).json({ |
There was a problem hiding this comment.
400 처리에 대해 해당 부분은 미들웨어로 추출해 재사용하시면 좋을 것 같습니다!
| try { | ||
| const { articleId } = req.params; | ||
|
|
||
| const articleIdNum = parseInt(articleId, 10); |
There was a problem hiding this comment.
해당 부분이 들어오는지에 대한 체크도 해주시면 좋을 것 같습니다!
| }); | ||
|
|
||
| export default router; | ||
|
|
There was a problem hiding this comment.
전반적으로 로직들이 라우트에 많이 뭉쳐있는데 로직들을 서비스레이어로 따로 분리해 나누거나 자주 쓰는 코드는 유틸 폴더로 따로 구분해 작성해주시면 좋을 것 같습니다!
Hoooooou-Jun
merged commit 40b1b8b
into
codeit-sprint-fullstack:node-김민식
2 participants
기본 요구사항
중고마켓
[ ] mongoDB에서 PostgreSQL을 사용하도록 코드를 마이그레이션 해주세요.
공통
[ ] PostgreSQL를 이용해 주세요.
[ ] 데이터 모델 간의 관계를 고려하여 onDelete를 설정해 주세요.
[ ] 데이터베이스 시딩 코드를 작성해 주세요.
[ ] 각 API에 적절한 에러 처리를 해 주세요.
[ ] 각 API 응답에 적절한 상태 코드를 리턴하도록 해 주세요.
자유게시판
[ ] Article 스키마를 작성해 주세요.
[ ] id, title, content, createdAt, updatedAt 필드를 가집니다.
[ ] 게시글 등록 API를 만들어 주세요.
[ ] title, content를 입력해 게시글을 등록합니다.
[ ] 게시글 조회 API를 만들어 주세요.
[ ] id, title, content, createdAt를 조회합니다.
[ ] 게시글 수정 API를 만들어 주세요.
[ ] 게시글 삭제 API를 만들어 주세요.
[ ] 게시글 목록 조회 API를 만들어 주세요.
[ ] id, title, content, createdAt를 조회합니다.
[ ] offset 방식의 페이지네이션 기능을 포함해 주세요.
[ ] 최신순(recent)으로 정렬할 수 있습니다.
[ ] title, content에 포함된 단어로 검색할 수 있습니다.
댓글
[ ] 댓글 등록 API를 만들어 주세요.
[ ] content를 입력하여 댓글을 등록합니다.
[ ] 중고마켓, 자유게시판 댓글 등록 API를 따로 만들어 주세요.
[ ] 댓글 수정 API를 만들어 주세요.
[ ] PATCH 메서드를 사용해 주세요.
[ ] 댓글 삭제 API를 만들어 주세요.
[ ] 댓글 목록 조회 API를 만들어 주세요.
[ ] id, content, createdAt 를 조회합니다.
[ ] cursor 방식의 페이지네이션 기능을 포함해 주세요.
[ ] 중고마켓, 자유게시판 댓글 목록 조회 API를 따로 만들어 주세요.