♻️ refactor: dto 이름 변경(StatWithNicknameAndUserId -> StatWithUserSummary)

Author: jiwon1217

backend/src/main/java/io/f1/backend/domain/stat/dao/StatJpaRepository.java

@@ -1,7 +1,7 @@
 package io.f1.backend.domain.stat.dao;
 
 import io.f1.backend.domain.stat.dto.StatWithNickname;
-import io.f1.backend.domain.stat.dto.StatWithNicknameAndUserId;
+import io.f1.backend.domain.stat.dto.StatWithUserSummary;
 import io.f1.backend.domain.stat.entity.Stat;
 
 import org.springframework.data.domain.Page;
@@ -33,12 +33,12 @@ public interface StatJpaRepository extends JpaRepository<Stat, Long> {
     @Query(
             """
             SELECT
-            		new io.f1.backend.domain.stat.dto.StatWithNicknameAndUserId
+            		new io.f1.backend.domain.stat.dto.StatWithUserSummary
             				(u.id, u.nickname, s.totalGames, s.winningGames, s.score)
             FROM
             		Stat s JOIN s.user u
             """)
-    List<StatWithNicknameAndUserId> findAllStatWithNicknameAndUserId();
+    List<StatWithUserSummary> findAllStatWithUserSummary();
 
     @Modifying
     @Query(
@@ -66,11 +66,11 @@ public interface StatJpaRepository extends JpaRepository<Stat, Long> {
 
     @Query(
             """
-            SELECT new io.f1.backend.domain.stat.dto.StatWithNicknameAndUserId(
+            SELECT new io.f1.backend.domain.stat.dto.StatWithUserSummary(
                 u.id, u.nickname, s.totalGames, s.winningGames, s.score
             )
             FROM Stat s JOIN s.user u
             WHERE u.id = :userId
             """)
-    Optional<StatWithNicknameAndUserId> findByUserId(long userId);
+    Optional<StatWithUserSummary> findStatWithUserSummary(long userId);
 }

backend/src/main/java/io/f1/backend/domain/stat/dao/StatRedisRepository.java

@@ -4,7 +4,7 @@
 
 import io.f1.backend.domain.stat.dto.StatPageResponse;
 import io.f1.backend.domain.stat.dto.StatResponse;
-import io.f1.backend.domain.stat.dto.StatWithNicknameAndUserId;
+import io.f1.backend.domain.stat.dto.StatWithUserSummary;
 import io.f1.backend.domain.user.dto.MyPageInfo;
 
 import lombok.RequiredArgsConstructor;
@@ -44,7 +44,7 @@ public void setup() {
         valueOps = redisTemplate.opsForValue();
     }
 
-    public void initialize(StatWithNicknameAndUserId stat) {
+    public void initialize(StatWithUserSummary stat) {
         String statUserKey = getStatUserKey(stat.userId());
         String statNicknameKey = getStatNickname(stat.nickname());
 

backend/src/main/java/io/f1/backend/domain/stat/dao/StatRepositoryAdapter.java

@@ -4,7 +4,7 @@
 
 import io.f1.backend.domain.stat.dto.StatPageResponse;
 import io.f1.backend.domain.stat.dto.StatWithNickname;
-import io.f1.backend.domain.stat.dto.StatWithNicknameAndUserId;
+import io.f1.backend.domain.stat.dto.StatWithUserSummary;
 import io.f1.backend.domain.user.dto.MyPageInfo;
 import io.f1.backend.global.exception.CustomException;
 import io.f1.backend.global.exception.errorcode.RoomErrorCode;
@@ -56,7 +56,7 @@ public StatPageResponse getRanksByNickname(String nickname, int pageSize) {
 
     @Override
     public void addUser(long userId, String nickname) {
-        redisRepository.initialize(new StatWithNicknameAndUserId(userId, nickname, 0, 0, 0));
+        redisRepository.initialize(new StatWithUserSummary(userId, nickname, 0, 0, 0));
     }
 
     @Override
@@ -80,7 +80,7 @@ public void removeUser(long userId) {
     }
 
     private void warmingRedis() {
-        jpaRepository.findAllStatWithNicknameAndUserId().forEach(redisRepository::initialize);
+        jpaRepository.findAllStatWithUserSummary().forEach(redisRepository::initialize);
     }
 
     private Pageable getPageableFromNickname(String nickname, int pageSize) {
@@ -108,16 +108,16 @@ public MyPageInfo getMyPageByUserId(long userId) {
             log.error("Redis miss, fallback to MySQL for userId={}", userId, e);
         }
 
-        StatWithNicknameAndUserId stat = findStatByUserId(userId);
+        StatWithUserSummary stat = findStatByUserId(userId);
         long rank = jpaRepository.countByScoreGreaterThan(stat.score()) + 1;
 
         return new MyPageInfo(
                 stat.nickname(), rank, stat.totalGames(), stat.winningGames(), stat.score());
     }
 
-    private StatWithNicknameAndUserId findStatByUserId(long userId) {
+    private StatWithUserSummary findStatByUserId(long userId) {
         return jpaRepository
-                .findByUserId(userId)
+                .findStatWithUserSummary(userId)
                 .orElseThrow(() -> new CustomException(UserErrorCode.USER_NOT_FOUND));
     }
 }

backend/src/main/java/io/f1/backend/domain/stat/dto/StatWithNicknameAndUserId.java renamed to backend/src/main/java/io/f1/backend/domain/stat/dto/StatWithUserSummary.java

@@ -1,4 +1,4 @@
 package io.f1.backend.domain.stat.dto;
 
-public record StatWithNicknameAndUserId(
+public record StatWithUserSummary(
         long userId, String nickname, long totalGames, long winningGames, long score) {}

backend/src/test/java/io/f1/backend/domain/stat/RedisStatBrowserTest.java

@@ -26,7 +26,7 @@
 import io.f1.backend.domain.stat.dao.StatJpaRepository;
 import io.f1.backend.domain.stat.dao.StatRepositoryAdapter;
 import io.f1.backend.domain.stat.dto.StatWithNickname;
-import io.f1.backend.domain.stat.dto.StatWithNicknameAndUserId;
+import io.f1.backend.domain.stat.dto.StatWithUserSummary;
 import io.f1.backend.domain.user.dao.UserRepository;
 import io.f1.backend.domain.user.dto.AuthenticationUser;
 import io.f1.backend.domain.user.dto.SignupRequest;
@@ -186,9 +186,9 @@ private MockHttpSession getMockSession(User user, boolean signup) {
     }
 
     private void warmingRedisOneUser(User user) {
-        StatWithNicknameAndUserId mockStat =
-                new StatWithNicknameAndUserId(user.getId(), user.getNickname(), 10, 10, 100);
-        given(statJpaRepository.findAllStatWithNicknameAndUserId()).willReturn(List.of(mockStat));
+        StatWithUserSummary mockStat =
+                new StatWithUserSummary(user.getId(), user.getNickname(), 10, 10, 100);
+        given(statJpaRepository.findAllStatWithUserSummary()).willReturn(List.of(mockStat));
         statRepositoryAdapter.setup();
     }
 }