refactor : 불필요한 코드 삭제 및 구조 개선

Author: seungwookc97

src/main/java/com/back/domain/user/entity/User.java

@@ -37,9 +37,6 @@ public class User {
     @Column(unique = true, length = 100)
     private String oauthId;     // OAuth 제공자별 고유 ID (예: kakao_123456789)
 
-    @Column(length = 20)
-    private String provider;    // OAuth 제공자 (KAKAO, GOOGLE, NAVER)
-
     private Double abvDegree;   // 알콜도수(회원 등급)
 
     @CreatedDate    // JPA Auditing 적용

src/main/java/com/back/domain/user/service/UserService.java

@@ -23,22 +23,6 @@ public User findById(Long id) {
                 .orElseThrow(() -> new IllegalArgumentException("User not found. id=" + id));
     }
 
-    // 소셜로그인으로 회원가입 & 회원 정보 수정
-    public RsData<User> modifyOrJoin(String oauthId, String email, String nickname) {
-
-        //oauthId로 기존 회원인지 확인
-        User User = userRepository.findByOauthId(oauthId).orElse(null);
-
-        // 기존 회원이 아니면 소셜로그인으로 회원가입 진행
-        if(User == null) {
-            User = joinSocial(oauthId, email, nickname);
-            return new RsData<>(201, "회원가입이 완료되었습니다.", User);
-        }
-
-        // 기존 회원이면 회원 정보 수정
-        modifySocial(User, nickname);
-        return new RsData<>(200, "회원 정보가 수정되었습니다.", User);
-    }
 
     public User joinSocial(String oauthId, String email, String nickname){
         userRepository.findByOauthId(oauthId)
@@ -52,44 +36,32 @@ public User joinSocial(String oauthId, String email, String nickname){
         User user = User.builder()
                 .email(email)
                 .nickname(uniqueNickname)
-                .profileImgUrl(null)
                 .abvDegree(0.0)
+                .createdAt(LocalDateTime.now())
+                .updatedAt(LocalDateTime.now())
                 .role("USER")
                 .oauthId(oauthId)
                 .build();
 
         return userRepository.save(user);
     }
 
-    public void modifySocial(User user, String nickname){
-        user.setNickname(nickname);
-        userRepository.save(user);
-    }
-
-    public RsData<User> findOrCreateOAuthUser(String oauthId, String email, String nickname, String provider) {
+    @Transactional
+    public RsData<User> findOrCreateOAuthUser(String oauthId, String email, String nickname) {
         Optional<User> existingUser = userRepository.findByOauthId(oauthId);
 
         if (existingUser.isPresent()) {
             // 기존 사용자 업데이트 (이메일만 업데이트)
             User user = existingUser.get();
             user.setEmail(email);
-            return RsData.of(200, "기존 사용자 정보 업데이트", userRepository.save(user));
+            return RsData.of(200, "회원 정보가 업데이트 되었습니다", user); //더티체킹
         } else {
-            // 새 사용자 생성 - 고유한 닉네임 생성
-            String uniqueNickname = generateUniqueNickname(nickname);
-            User newUser = User.builder()
-                    .oauthId(oauthId)
-                    .email(email)
-                    .nickname(uniqueNickname)
-                    .provider(provider)
-                    .role("USER")
-                    .createdAt(LocalDateTime.now())
-                    .build();
-            return RsData.of(201, "새 사용자 생성", userRepository.save(newUser));
+            User newUser = joinSocial(oauthId, email, nickname);
+            return RsData.of(201, "사용자가 생성되었습니다", newUser);
         }
     }
 
-    private String generateUniqueNickname(String baseNickname) {
+    public String generateUniqueNickname(String baseNickname) {
         // null이거나 빈 문자열인 경우 기본값 설정
         if (baseNickname == null || baseNickname.trim().isEmpty()) {
             baseNickname = "User";

src/main/java/com/back/global/security/CustomOAuth2UserService.java

@@ -2,8 +2,8 @@
 
 import com.back.domain.user.entity.User;
 import com.back.domain.user.service.UserService;
+import com.back.global.rsData.RsData;
 import lombok.RequiredArgsConstructor;
-import lombok.extern.slf4j.Slf4j;
 import org.springframework.security.oauth2.client.userinfo.DefaultOAuth2UserService;
 import org.springframework.security.oauth2.client.userinfo.OAuth2UserRequest;
 import org.springframework.security.oauth2.core.OAuth2AuthenticationException;
@@ -15,7 +15,6 @@
 
 @Service
 @RequiredArgsConstructor
-@Slf4j
 public class CustomOAuth2UserService extends DefaultOAuth2UserService {
     private final UserService userService;
 
@@ -56,26 +55,22 @@ public OAuth2User loadUser(OAuth2UserRequest userRequest) throws OAuth2Authentic
 
         // OAuth ID를 제공자와 함께 저장 (예: kakao_123456789)
         String uniqueOauthId = providerTypeCode.toLowerCase() + "_" + oauthUserId;
+        RsData<User> rsData = userService.findOrCreateOAuthUser(uniqueOauthId, email, nickname);
 
-        log.debug("OAuth2 user info - oauthUserId: {}, email: {}, nickname: {}, provider: {}",
-                  oauthUserId, email, nickname, providerTypeCode);
-
-        User user = userService.findOrCreateOAuthUser(uniqueOauthId, email, nickname, providerTypeCode).data();
+        if (rsData.code()<200 || rsData.code()>299) {
+            throw new OAuth2AuthenticationException("사용자 생성/조회 실패: " + rsData.message());
+        }
 
-        log.debug("User from DB - id: {}, email: {}, nickname: {}",
-                  user.getId(), user.getEmail(), user.getNickname());
+        User user = rsData.data();
 
-        // null 체크 및 기본값 설정
         String userEmail = user.getEmail() != null && !user.getEmail().trim().isEmpty()
-                          ? user.getEmail() : "[email protected]";
-        String userNickname = user.getNickname() != null && !user.getNickname().trim().isEmpty()
-                             ? user.getNickname() : "Unknown User";
+                ? user.getEmail() : "unknown";
 
         // securityContext
         return new SecurityUser(
                 user.getId(),
                 userEmail,
-                userNickname,
+                user.getNickname(),
                 user.getAuthorities(),
                 oAuth2User.getAttributes()
         );