[Feature] 이동봉사자 마이페이지 메인 정보 조회 API 수정 #184
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
kyeong-hyeok
approved these changes
May 5, 2024
| case PROGRESSING -> progressingCount = count; | ||
| case COMPLETED -> completedCount = count; | ||
| } | ||
| } |
There was a problem hiding this comment.
저는 중개자 홈 화면 정보 조회할 때
Map<PostStatus, Long> countOfPostStatus = customPostRepository.getCountOfPostStatus(intermediary.getId(), null);
IntermediaryGetHomeResponse response = IntermediaryGetHomeResponse.of(
intermediary,
countOfPostStatus.getOrDefault(PostStatus.RECRUITING, 0L),
countOfPostStatus.getOrDefault(PostStatus.WAITING, 0L),
countOfPostStatus.getOrDefault(PostStatus.PROGRESSING, 0L),
countOfPostStatus.getOrDefault(PostStatus.COMPLETED, 0L));
Map으로 가져왔었는데 Tuple로도 이렇게 한 번에 매핑해서 넘겨줄 수 있을까요?!
There was a problem hiding this comment.
오호 중개자 홈 화면에도 건수 조회하는 기능이 있었죠!
조금 찾아보니, Map은 키-값 쌍을 저장하기 때문에 키를 사용하여 값을 찾을 때 편리한 장점이 있고 Tuple은 간단하고 빠르게 여러 값을 그룹화할 때 사용하는 장점이 주요 장점인 것 같아요.
건수 3~4가지를 가져오기 때문에 큰 차이는 없어 보입니다! 덕분에 Tuple에서 바로 Map 만들어 가져오는 방법을 리마인드했습니다 👍
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 #181
📝 작업 내용
💬 리뷰 요구 사항