[Feature] 공고 강아지 성별, 몸무게 제거 및 API 수정 #160
Merged
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
6 tasks
hojeong2747
approved these changes
Apr 18, 2024
There was a problem hiding this comment.
우리 초반 erd 설계할 때 되게 오래 고민하던 기억이 나네요. 하나 짤 때도 많이 고민했는데 정규화 고려해서 강아지 도메인을 분리할지 말지 고민하고 1안, 2안, 3안 나눴던 기억이 새록새록..
덕분에 많이 고민해서 설계한 erd 덕분에 유지보수하기 편한 것 같아요! 비즈니스 로직, 서비스 단에서 로직 변경 사항이 없고 dto에서 불필요한 필드만 제거하면 되니 '유지보수가 용이하다'는 것도 느껴보네요. 비즈니스 로직과 dto 변환을 분리한 것도 장점을 확실히 알 수 있게 되었어요.
수고하셨습니다~!
| private String name; // 강아지 이름 | ||
| @Column(nullable = false) | ||
| private DogSize size; // 강아지 사이즈 | ||
| @Column(nullable = false) | ||
| private DogGender gender; // 강아지 성별 | ||
| private Float weight; // 강아지 몸무게 |
| @@ -166,7 +166,7 @@ public PostIntermediaryGetOneResponse getIntermediaryOnePost(Long postId) { | |||
| .select(Projections.constructor(PostIntermediaryGetOneResponse.class, | |||
| post.id, postImage.image, post.status, post.departureLoc, post.arrivalLoc, | |||
| post.startDate, post.endDate, post.pickUpTime, post.isKennel, post.content, | |||
| dog.name, dog.size, dog.gender, dog.weight, dog.specifics, | |||
| dog.name, dog.size, dog.specifics, | |||
There was a problem hiding this comment.
해당 엔티티 사용하던 DTO와 비즈니스 로직에서도 필드 제거만 해주고
| @@ -217,7 +217,7 @@ void setUp() { | |||
| images.add("image2"); | |||
| PostVolunteerGetOneResponse response = new PostVolunteerGetOneResponse(1L, "mainImage", images, "모집중", "서울시 성북구", "서울시 중랑구", | |||
| startDate, endDate, "12:00", true, "이동봉사 공고", "봄이", DogSize.SMALL.getKey(), | |||
| DogGender.FEMALE.getKey(), 5.1f, "ㄱㅇㅇ", 1L, "profileImage", "이동봉사 중개", true); | |||
| "ㄱㅇㅇ", 1L, "profileImage", "이동봉사 중개", true); | |||
|
설계 및 1차 개발 이후에 추가적으로 수정되는 부분을 개발하면서 편리한 부분을 느낄 수 있었던 거 같아요! 좋습니다. |
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.
💡 연관된 이슈
close #159
📝 작업 내용
💬 리뷰 요구 사항