[7주차/레오] 워크북 제출합니다

[yongyong213]

✅ 실습 체크리스트

• 이론 학습을 완료하셨나요?
• 미션 요구사항을 모두 이해하셨나요?
• 실습을 수행하기 위한 공부를 완료하셨나요?
• 실습 요구사항을 모두 완료하셨나요?

✅ 컨벤션 체크리스트

• 디렉토리 구조 컨벤션을 잘 지켰나요?
• pr 제목을 컨벤션에 맞게 작성하였나요?
• pr에 해당되는 이슈를 연결하였나요?(중요)
• 적절한 라벨을 설정하였나요?
• 파트장에게 code review를 요청하기 위해 reviewer를 등록하였나요?
• 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?(매우 중요!)

📌 주안점

[YoungJJun]

7주차 피드백

1. 우선 이전주차 피드백 관련해서 잘 수정해주신 것 같아요! 👍

2. (미션) 내가 진행중인 미션 조회 API

   미션 요구사항에서 사용자ID를 @RequestBody에서 받으라고 명시하고 있습니다. 현재 @RequestParam 으로 구현되어있는데 결국 토큰 구현 이후에 모두 수정되어야 하는 내용입니다.

   따라서 제 생각에는 값을 전달받는 두 방식 모두 익숙하시다면 현 상태 유지해도 된다고 생각해요.

   만약에 @RequestBody로 받는것과 @RequestParam 으로 받는거에 있어서 차이를 잘 모르겠거나 헷갈리시면 Body로 변경하시면서 공부해보시는걸 추천드려요!

3. 페이지네이션 구현과정에 @RequestParam Integer pageNumber 등이 있는데 validation 이 빠져있어요. @Min(1) ,@Positive 같은걸로 validation 추가해주시면 좋을 것 같아요.

4. 페이지네이션 DTO를 만드신점 좋습니다!

   data, pageNumber, pageSize 내부에 이렇게 3가지 데이터가 들어가는데 해당 데이터만으로는 프론트 입장에서 몇페이지까지 버튼을 만들어야 하는지 알 수 없을 것 같아요.

   totalPages, totalElements 등을 추가해야 프론트 화면에 실제 존재하는 페이지만큼만 버튼을 만들 수 있습니다.

   만약에 해당 값이 없으면 프론트에서는 당장 가능한 페이지는 모두 보여줘야 하고 9페이지까지 존재하는데 10페이지를 요청하는 상황이 발생할 수 있어요. → 이 경우 data가 비어있는 상태로 결과가 반환될 수 있는데 UX에 좋지 않을 것 같습니다.

5. 내가 생성한 리뷰 조회, RequestBody 검증 어노테이션은 아직 제대로 구현되지 않은것 같아서 키워드 제출하실때 Swagger 캡쳐본 올려주시면 확인할게용.

6. MissionSuccessCode

   CREATED(HttpStatus.OK, "MISSION200_3", "성공적으로 미션 생성 완료."),

   → HttpStatus에 CREATED 도 있습니다. OK 보다 CREATED가 적절할 것 같아요.

---

레오 7주차 수고하셨습니다~ 남은 미션 시도해보시고 머지해주시면 됩니다!
담주도 화이팅이에요.