Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
eomsung
added
매운맛🔥
jjjwodls
reviewed
Dec 11, 2024
| } = req.query; | ||
| const offset = (page - 1) * pageSize; | ||
|
|
||
| if (page < 1 || pageSize < 1) { |
There was a problem hiding this comment.
클라이언트에서 숫자 타입인지 체크해준 부분은 확인 했습니다
서버에서도 추가로 확인해주시는걸 권장드립니다
서버로의 요청은 외부에서 올 수 있기때문에 불필요한 예외 방지를 위해 서버에서도 유효성 체크가 필요합니다.
| "/products/:id", | ||
| asyncHandler(async (req, res) => { | ||
| const id = req.params.id; | ||
| const product = await Product.findById(id).select("-updatedAt"); |
There was a problem hiding this comment.
검색을 한 후 존재하지 않는 경우 예외 처리를 해주는 로직을 추가해주세요!
| "/products/:id", | ||
| asyncHandler(async (req, res) => { | ||
| const id = req.params.id; | ||
| const product = await Product.findById(id); |
There was a problem hiding this comment.
해당 부분도 마찬가지로 product 를 업데이트 하기 전에 조회가 제대로 되어
존재하는지 체크를 한 후 진행해주세요!
| app.delete( | ||
| "/products/:id", | ||
| asyncHandler(async (req, res) => { | ||
| const id = req.params.id; |
There was a problem hiding this comment.
여기도 마찬가지로 삭제하려는 product 가 존재하는지 체크해주세요!
|
성민님 스프린트 고생 많으셨어요 백엔드 쪽도 분리해서 잘 작업해주셨네요 코드를 분리하여 백엔드, 프론트엔드를 구분한건 좋습니다 유효성 검증 및 데이터 체크하는 부분만 조금만 더 봐주시면 될 것 같습니다 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
요구사항
기본
중고마켓
Product 스키마를 작성해 주세요.
id, name, description, price, tags, createdAt, updatedAt필드를 가집니다.
필요한 필드가 있다면 자유롭게 추가해 주세요.
상품 등록 API를 만들어 주세요.
name, description, price, tags를 입력하여 상품을 등록합니다.
상품 상세 조회 API를 만들어 주세요.
id, name, description, price, tags, createdAt를 조회합니다.
상품 수정 API를 만들어 주세요.
PATCH 메서드를 사용해 주세요.
상품 삭제 API를 만들어 주세요.
상품 목록 조회 API를 만들어 주세요.
id, name, price, createdAt를 조회합니다.
offset 방식의 페이지네이션 기능을 포함해 주세요.
최신순(recent)으로 정렬할 수 있습니다.
name, description에 포함된 단어로 검색할 수 있습니다.
각 API에 적절한 에러 처리를 해 주세요.
각 API 응답에 적절한 상태 코드를 리턴하도록 해 주세요.
. env 파일에 환경 변수를 설정해 주세요.
CORS를 설정해 주세요.
render.com로 배포해 주세요.
MongoDB를 활용해 주세요.
주요 변경사항
.env에는 DATABASE_URL(mongodb)와 PORT=3000 정보가 담겨있습니다.
https://four-sprint-mission-be-cfjg.onrender.com
스크린샷
멘토에게