Conversation
YONGJAEEE
reviewed
May 6, 2026
Contributor
YONGJAEEE
left a comment
YONGJAEEE
left a comment
There was a problem hiding this comment.
징니 API 코드는 바로 적용해도 될만큼 시간 많이 쓰며 제대로 고민하고 필요한 기능들 많이 추가한거 같아 .. !
내가 빼먹은 부분이 참 많구나 ...
고생했어 ~~
| .point(member.getPoint()) | ||
| .birth(member.getBirth()) | ||
| .address(member.getAddress()) | ||
| .build(); |
Contributor
There was a problem hiding this comment.
여기는 메서드에 존재하는 코드가 적어서 괜찮은거 같은데,
DTO <-> Entity 변환 코드는 컨버터 클래스로 분리하던가, 메서드로 분리하는게 좋다고 생각해
변환 로직을 별도로 구분하게 되면,
- 재사용성
- 확장성
- 협업 편리성 (정해진 규칙대로 개발 가능)
등의 이점을 챙길 수 있다고 생각해
| } | ||
|
|
||
| List<MissionDTO.Mission> result = missions.stream() | ||
| .map(m -> new MissionDTO.Mission( |
Contributor
There was a problem hiding this comment.
이런 map 구문에서는 특히 변환 로직을 빼서 얻는 장점이 더 클거같아
| if( | ||
| request.content() == null || request.content().isBlank() | ||
| || request.star() == null || request.star().isNaN() | ||
| ){ |
Contributor
There was a problem hiding this comment.
OR 조건이 한줄에 너무 많은거 같다는 생각이 드네
- content 조건과 star 조건문 두개로 분리
- 별도 함수로 분리
- Request Class 안에 isValid 같은 함수 추가 (코틀린 확장함수 쓰면 진짜 편함, 다만 자바니까 오히려 DTO의 역할이 애매해질듯...)
요런 방법들이 있을거 같아
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
🛠️ 작업 사항
📸 관련 이미지 (스크린샷 또는 동영상)
회원가입 API 응답
사용자 정보 조회 API
해당 지역 내 모든 미션 조회 API
내가 진행중/진행완료 인 미션 조회 API
내가 달성한 미션의 개수 조회
리뷰 생성하기 API
💬 기타 설명
아직 스웨거랑 폴더구조 정리를 못했어요 !!!!
특히 폴더 구조는 고민이 많이 필요해서 스터디 때 얘기해보고 수정하고싶어요 !!!