refactor(volunteer-apply): 봉사 지원 조회 리팩토링

 - getById

Author: leebs0521

src/main/java/com/somemore/domains/volunteerapply/service/VolunteerApplyQueryService.java

@@ -8,6 +8,7 @@
 import com.somemore.domains.volunteerapply.dto.response.VolunteerApplySummaryResponseDto;
 import com.somemore.domains.volunteerapply.repository.VolunteerApplyRepository;
 import com.somemore.domains.volunteerapply.usecase.VolunteerApplyQueryUseCase;
+import com.somemore.global.exception.ExceptionMessage;
 import com.somemore.global.exception.NoSuchElementException;
 import java.util.List;
 import java.util.UUID;
@@ -26,46 +27,47 @@ public class VolunteerApplyQueryService implements VolunteerApplyQueryUseCase {
     private final VolunteerApplyRepository volunteerApplyRepository;
 
     @Override
-    public List<UUID> getVolunteerIdsByRecruitIds(List<Long> recruitIds) {
-        return volunteerApplyRepository.findVolunteerIdsByRecruitIds(recruitIds);
+    public VolunteerApply getById(Long id) {
+        return volunteerApplyRepository.findById(id).orElseThrow(
+                () -> new NoSuchElementException(NOT_EXISTS_VOLUNTEER_APPLY));
     }
 
     @Override
     public VolunteerApply getByRecruitIdAndVolunteerId(Long recruitId, UUID volunteerId) {
-        return volunteerApplyRepository.findByRecruitIdAndVolunteerId(recruitId, volunteerId)
-                .orElseThrow(
+        return volunteerApplyRepository.findByRecruitIdAndVolunteerId(recruitId, volunteerId).orElseThrow(
                         () -> new NoSuchElementException(NOT_EXISTS_VOLUNTEER_APPLY));
     }
 
     @Override
     public VolunteerApplySummaryResponseDto getSummaryByRecruitId(Long recruitId) {
 
-        List<VolunteerApply> applies = volunteerApplyRepository.findAllByRecruitId(
-                recruitId);
+        List<VolunteerApply> applies = volunteerApplyRepository.findAllByRecruitId(recruitId);
 
         return VolunteerApplySummaryResponseDto.from(applies);
     }
 
     @Override
-    public VolunteerApplyResponseDto getVolunteerApplyByRecruitIdAndVolunteerId(Long recruitId,
-            UUID volunteerId) {
+    public VolunteerApplyResponseDto getVolunteerApplyByRecruitIdAndVolunteerId(Long recruitId, UUID volunteerId) {
         VolunteerApply apply = getByRecruitIdAndVolunteerId(recruitId, volunteerId);
 
         return VolunteerApplyResponseDto.from(apply);
     }
 
     @Override
-    public Page<VolunteerApply> getAllByRecruitId(Long recruitId,
-            VolunteerApplySearchCondition condition) {
+    public Page<VolunteerApply> getAllByRecruitId(Long recruitId, VolunteerApplySearchCondition condition) {
         return volunteerApplyRepository.findAllByRecruitId(recruitId, condition);
     }
 
     @Override
-    public Page<VolunteerApply> getAllByVolunteerId(UUID volunteerId,
-            VolunteerApplySearchCondition condition) {
+    public Page<VolunteerApply> getAllByVolunteerId(UUID volunteerId, VolunteerApplySearchCondition condition) {
         return volunteerApplyRepository.findAllByVolunteerId(volunteerId, condition);
     }
 
+    @Override
+    public List<UUID> getVolunteerIdsByRecruitIds(List<Long> recruitIds) {
+        return volunteerApplyRepository.findVolunteerIdsByRecruitIds(recruitIds);
+    }
+
     @Override
     public List<VolunteerApply> getAllByIds(List<Long> ids) {
         return volunteerApplyRepository.findAllByIds(ids);

src/main/java/com/somemore/domains/volunteerapply/usecase/VolunteerApplyQueryUseCase.java

@@ -1,30 +1,30 @@
 package com.somemore.domains.volunteerapply.usecase;
 
+import com.somemore.domains.volunteerapply.domain.VolunteerApply;
 import com.somemore.domains.volunteerapply.dto.condition.VolunteerApplySearchCondition;
 import com.somemore.domains.volunteerapply.dto.response.VolunteerApplyResponseDto;
 import com.somemore.domains.volunteerapply.dto.response.VolunteerApplySummaryResponseDto;
-import com.somemore.domains.volunteerapply.domain.VolunteerApply;
+import org.springframework.data.domain.Page;
 
 import java.util.List;
 import java.util.UUID;
-import org.springframework.data.domain.Page;
 
 public interface VolunteerApplyQueryUseCase {
 
-    List<UUID> getVolunteerIdsByRecruitIds(List<Long> recruitIds);
+    VolunteerApply getById(Long id);
 
     VolunteerApply getByRecruitIdAndVolunteerId(Long recruitId, UUID volunteerId);
 
     VolunteerApplySummaryResponseDto getSummaryByRecruitId(Long recruitId);
 
-    VolunteerApplyResponseDto getVolunteerApplyByRecruitIdAndVolunteerId(Long recruitId,
-                                                                         UUID volunteerId);
+    VolunteerApplyResponseDto getVolunteerApplyByRecruitIdAndVolunteerId(Long recruitId, UUID volunteerId);
 
-    Page<VolunteerApply> getAllByRecruitId(Long recruitId, VolunteerApplySearchCondition condition);
-
-    Page<VolunteerApply> getAllByVolunteerId(UUID volunteerId,
-            VolunteerApplySearchCondition condition);
+    List<UUID> getVolunteerIdsByRecruitIds(List<Long> recruitIds);
 
     List<VolunteerApply> getAllByIds(List<Long> ids);
 
+    Page<VolunteerApply> getAllByRecruitId(Long recruitId, VolunteerApplySearchCondition condition);
+
+    Page<VolunteerApply> getAllByVolunteerId(UUID volunteerId, VolunteerApplySearchCondition condition);
+
 }