refactor(location): Location 조회 수정 기능 리팩토링

Author: leebs0521

src/main/java/com/somemore/location/service/command/UpdateLocationService.java

@@ -7,7 +7,6 @@
 import com.somemore.location.dto.request.LocationUpdateRequestDto;
 import com.somemore.location.repository.LocationRepository;
 import com.somemore.location.usecase.command.UpdateLocationUseCase;
-import com.somemore.location.usecase.query.LocationQueryUseCase;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
@@ -17,14 +16,17 @@
 @Service
 public class UpdateLocationService implements UpdateLocationUseCase {
 
-    private final LocationQueryUseCase locationQueryUseCase;
     private final LocationRepository locationRepository;
 
     @Override
     public void updateLocation(LocationUpdateRequestDto requestDto, Long locationId) {
-        Location location = locationQueryUseCase.findById(locationId)
-            .orElseThrow(() -> new BadRequestException(NOT_EXISTS_LOCATION.getMessage()));
+        Location location = getLocation(locationId);
         location.updateWith(requestDto);
         locationRepository.save(location);
     }
+
+    private Location getLocation(Long locationId) {
+        return locationRepository.findById(locationId)
+            .orElseThrow(() -> new BadRequestException(NOT_EXISTS_LOCATION.getMessage()));
+    }
 }

src/main/java/com/somemore/location/service/query/LocationQueryService.java

@@ -1,9 +1,11 @@
 package com.somemore.location.service.query;
 
+import static com.somemore.global.exception.ExceptionMessage.NOT_EXISTS_LOCATION;
+
+import com.somemore.global.exception.BadRequestException;
 import com.somemore.location.domain.Location;
 import com.somemore.location.repository.LocationRepository;
 import com.somemore.location.usecase.query.LocationQueryUseCase;
-import java.util.Optional;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
@@ -16,8 +18,14 @@ public class LocationQueryService implements LocationQueryUseCase {
     private final LocationRepository locationRepository;
 
     @Override
-    public Optional<Location> findById(Long id) {
-        return locationRepository.findById(id);
+    public Location getById(Long id) {
+        return getLocation(id);
+    }
+
+    private Location getLocation(Long id) {
+        return locationRepository.findById(id).orElseThrow(
+            () -> new BadRequestException(NOT_EXISTS_LOCATION.getMessage())
+        );
     }
 
 }

src/main/java/com/somemore/location/usecase/query/LocationQueryUseCase.java

@@ -1,10 +1,9 @@
 package com.somemore.location.usecase.query;
 
 import com.somemore.location.domain.Location;
-import java.util.Optional;
 
 public interface LocationQueryUseCase {
 
-    Optional<Location> findById(Long id);
+    Location getById(Long id);
 
 }