Merge pull request #103 from SynergyX-AI-Pattern/refactor/#102_backtest_rounding

Refactor/#102 backtest rounding

Author: Soomxn

src/main/java/com/synergyx/trading/service/backtestService/BacktestServiceImpl.java

@@ -21,7 +21,7 @@
 import org.springframework.data.domain.Pageable;
 import org.springframework.transaction.annotation.Transactional;
 import com.synergyx.trading.service.backtestService.client.BacktestClientService;
-
+import static com.synergyx.trading.util.NumberUtil.round;
 import java.time.LocalDate;
 import java.time.LocalDateTime;
 import java.util.List;
@@ -97,15 +97,15 @@ public BacktestResponseDTO.BacktestExecutionDTO runBacktest(Long userId, Long pa
                 .matchedCount(saved.getMatchedCount())
                 .startDate(saved.getStartDate())
                 .endDate(saved.getEndDate())
-                .winRate(saved.getWinRate())
-                .averageReturn(saved.getAverageReturn())
-                .maxReturn(saved.getMaxReturn())
+                .winRate(round(saved.getWinRate()))
+                .averageReturn(round(saved.getAverageReturn()))
+                .maxReturn(round(saved.getMaxReturn()))
                 .maxReturnDate(saved.getMaxReturnDate())
-                .minReturn(saved.getMinReturn())
+                .minReturn(round(saved.getMinReturn()))
                 .minReturnDate(saved.getMinReturnDate())
-                .totalReturn(saved.getTotalReturn())
+                .totalReturn(round(saved.getTotalReturn()))
                 .lastMatchedDate(saved.getLastMatchedDate())
-                .lastMatchedReturn(saved.getLastMatchedReturn())
+                .lastMatchedReturn(round(saved.getLastMatchedReturn()))
                 .highlightRange(
                         (saved.getHighlightFromDate() != null && saved.getHighlightToDate() != null)
                                 ? BacktestResponseDTO.HighlightRangeDTO.builder()
@@ -139,16 +139,16 @@ public BacktestResponseDTO.BacktestResultDetailDTO getBacktestResultDetail(Long
                 .executedAt(backtest.getExecutedAt())
                 .startDate(backtest.getStartDate())
                 .endDate(backtest.getEndDate())
-                .winRate(backtest.getWinRate())
-                .averageReturn(backtest.getAverageReturn())
+                .winRate(round(backtest.getWinRate()))
+                .averageReturn(round(backtest.getAverageReturn()))
                 .matchedCount(backtest.getMatchedCount())
                 .maxReturnDate(backtest.getMaxReturnDate())
-                .maxReturn(backtest.getMaxReturn())
+                .maxReturn(round(backtest.getMaxReturn()))
                 .minReturnDate(backtest.getMinReturnDate())
-                .minReturn(backtest.getMinReturn())
+                .minReturn(round(backtest.getMinReturn()))
                 .lastMatchedDate(backtest.getLastMatchedDate())
-                .lastMatchedReturn(backtest.getLastMatchedReturn())
-                .totalReturn(backtest.getTotalReturn())
+                .lastMatchedReturn(round(backtest.getLastMatchedReturn()))
+                .totalReturn(round(backtest.getTotalReturn()))
                 .highlightRange(
                         (backtest.getHighlightFromDate() != null && backtest.getHighlightToDate() != null)
                                 ? BacktestResponseDTO.HighlightRangeDTO.builder()
@@ -178,8 +178,8 @@ public Page<BacktestResponseDTO.BacktestSummaryDTO> getBacktestResultList(Long u
                 .backtestId(bt.getId())
                 .stockName(bt.getStock().getName())
                 .executedAt(bt.getExecutedAt())
-                .winRate(bt.getWinRate())
-                .averageReturn(bt.getAverageReturn())
+                .winRate(round(bt.getWinRate()))
+                .averageReturn(round(bt.getAverageReturn()))
                 .matchedCount(bt.getMatchedCount())
                 .build());
     }

src/main/java/com/synergyx/trading/service/backtestService/ranking/BacktestRankingService.java

@@ -16,6 +16,7 @@
 import java.util.concurrent.atomic.AtomicInteger;
 import java.util.function.Function;
 import java.util.stream.Collectors;
+import static com.synergyx.trading.util.NumberUtil.round;
 
 @Service
 @RequiredArgsConstructor
@@ -65,6 +66,13 @@ public List<BacktestRankingDTO> getRanking(int limit) {
         // 순위 부여
         assignRanks(rankings);
 
+        // 반올림 처리
+        rankings.forEach(r -> {
+            r.setWinRate(round(r.getWinRate()));
+            r.setAverageReturn(round(r.getAverageReturn()));
+            r.setMaxReturn(round(r.getMaxReturn()));
+        });
+
         return rankings;
     }
 }

src/main/java/com/synergyx/trading/service/patternService/PatternQueryServiceImpl.java

@@ -13,6 +13,7 @@
 
 import java.util.List;
 import java.util.Optional;
+import static com.synergyx.trading.util.NumberUtil.round;
 
 @Service
 @RequiredArgsConstructor
@@ -40,8 +41,8 @@ public List<PatternResponseDTO.PatternDTO> getPatternList(Long userId) {
                     List<PatternResponseDTO.BacktestSummaryDTO> summaries = backtests.stream()
                             .map(b -> PatternResponseDTO.BacktestSummaryDTO.builder()
                                     .stockName(b.getStock().getName())
-                                    .averageReturn(b.getAverageReturn())
-                                    .winRate(b.getWinRate())
+                                    .averageReturn(round(b.getAverageReturn()))
+                                    .winRate(round(b.getWinRate()))
                                     .matchedCount(b.getMatchedCount())
                                     .executedAt(b.getExecutedAt())
                                     .build())
@@ -81,10 +82,10 @@ public PatternResponseDTO.PatternDetailDTO getPatternDetail(Long userId, Long pa
                 .executedAt(bt.getExecutedAt())
                 .startDate(bt.getStartDate())
                 .endDate(bt.getEndDate())
-                .winRate(bt.getWinRate())
-                .averageReturn(bt.getAverageReturn())
+                .winRate(round(bt.getWinRate()))
+                .averageReturn(round(bt.getAverageReturn()))
                 .matchedCount(bt.getMatchedCount())
-                .maxReturn(bt.getMaxReturn())
+                .maxReturn(round(bt.getMaxReturn()))
                 .maxReturnDate(bt.getMaxReturnDate())
                 .build()
         ).orElse(null);

src/main/java/com/synergyx/trading/util/NumberUtil.java

@@ -0,0 +1,23 @@
+package com.synergyx.trading.util;
+
+import java.math.BigDecimal;
+import java.math.RoundingMode;
+
+/**
+ * 숫자 관련 공통 유틸리티 클래스
+ */
+public class NumberUtil {
+
+    /**
+     * 값을 소수점 둘째 자리까지 반올림합니다.
+     *
+     * @param value Double 값
+     * @return 소수점 둘째 자리까지 반올림된 값
+     */
+    public static Double round(Double value) {
+        if (value == null) return null;
+        return BigDecimal.valueOf(value)
+                .setScale(2, RoundingMode.HALF_UP)  // 둘째 자리에서 반올림
+                .doubleValue();
+    }
+}