[refac] apply code review

src/main/java/org/hankki/hankkiserver/api/report/controller/ReportController.java

@@ -2,10 +2,9 @@
 
 import lombok.RequiredArgsConstructor;
 import org.hankki.hankkiserver.api.dto.HankkiResponse;
-import org.hankki.hankkiserver.api.report.service.ReportService;
+import org.hankki.hankkiserver.api.report.service.ReportQueryService;
 import org.hankki.hankkiserver.api.report.service.response.ReportServiceResponse;
 import org.hankki.hankkiserver.common.code.CommonSuccessCode;
-import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RestController;
@@ -15,12 +14,11 @@
 @RequestMapping("/api/v1")
 public class ReportController {
 
-    private final ReportService reportService;
+    private final ReportQueryService reportQueryService;
 
     @GetMapping("/reports/count")
     public HankkiResponse<ReportServiceResponse> getReportCounts() {
-        ReportServiceResponse response = new ReportServiceResponse(reportService.getMyReportSequence());
-        return HankkiResponse.success(CommonSuccessCode.OK, response);
+        return HankkiResponse.success(CommonSuccessCode.OK, ReportServiceResponse.of(reportQueryService.getMyReportSequence()));
 
     }
 }

src/main/java/org/hankki/hankkiserver/api/report/service/ReportFinder.java

@@ -3,6 +3,7 @@
 import lombok.RequiredArgsConstructor;
 import org.hankki.hankkiserver.domain.report.repository.ReportRepository;
 import org.springframework.stereotype.Component;
+import org.springframework.transaction.annotation.Transactional;
 
 @Component
 @RequiredArgsConstructor

src/main/java/org/hankki/hankkiserver/api/report/service/ReportService.java → src/main/java/org/hankki/hankkiserver/api/report/service/ReportQueryService.java

@@ -2,14 +2,18 @@
 
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
 
 @Service
 @RequiredArgsConstructor
-public class ReportService {
+public class ReportQueryService {
+
+    private static final long REPORT_SEQUENCE = 1;
 
     private final ReportFinder reportFinder;
 
+    @Transactional(readOnly = true)
     public long getMyReportSequence() {
-        return reportFinder.getReportCount() + 1;
+        return reportFinder.getReportCount() + REPORT_SEQUENCE;
     }
 }

src/main/java/org/hankki/hankkiserver/api/report/service/response/ReportServiceResponse.java

@@ -3,4 +3,7 @@
 public record ReportServiceResponse(
         long count
 ) {
+    public static ReportServiceResponse of(long count) {
+        return new ReportServiceResponse(count);
+    }
 }