[7주차/윤샘] 워크북 제출합니다

[tjdals-dbs]

✅ 실습 체크리스트

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

✅ 컨벤션 체크리스트

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

📌 주안점

[YoungJJun]

7주차 피드백

1. 검증 어노테이션 붙여서 검증하는 로직 전체적으로 잘 구현되었습니다!

2. 이외의 모든 미션에 대해서도 요구사항에 맞게 구현되었어요.

3. MissionConverter 에 toMission()

   미션 생성 요청을 받아 요청 DTO를 실제 Mission 객체로 변환하는 메서드로 이해했습니다.

   생성되는 Mission의 title이 임의값으로 하드코딩 되어있어요. 이 부분은 하드코딩을 할 필요가 없다고 생각하는데 확인해주세요!

   수정할 경우 DTO에 title 필드 추가해서 변환하면 될 것 같아요.

4. MissionService 에 getMissionsByStoreId()

   missionList에 빈 리스트가 들어올 수 있는 구조일까요?

   missionList가 비어있을 수 있으면 List<Mission> content = missionList.getContent(); 에서 content가 비어있을 수 있습니다.

   Mission lastMission = content.get(content.size() -1 );

   그러면 해당 코드가 IndexOutOfBoundsException 발생할 수 있을 것 같아요.

5. MissionSuccessCode

   STATUS_UPDATED, MISSION_CREATED 모두 MISSION200_2로 중복입니다.

   중복을 해결해주시면 될 것 같은데 200_N으로 차이를 두는것도 좋지만 현재 상황에서는 미션 생성을 MISSION201_N으로 설정하고 HttpStatus 또한 CREATED로 바꾸는게 적절할 것 같네요.

6. MissionService 의 커서 페이징 맨 마지막 코드

   nextCursor = lastMission.getId() + ":" + lastMission.getId(); 두 값이 동일한데 실수이시면 수정해주시고 의도하신거면 설명이 필요할 것 같아요.

7. MissionReqDTO.GetMyMissionInProgress

   각 값에 @NotNull 만 붙어있는데 다른 위치에서 검증이 이루어지지 않고 있다면 @Min(0), @Positive 등 검증 어노테이션 추가해주면 좋을 것 같습니다.

---

윤샘 7주차 수고하셨어요~ 리뷰 참고해서 개선해주시면 좋을 것 같습니다!
내용이 어려웠을 것 같은데 미션 잘 진행해주셨고 앞으로도 화이팅입니다.