[refactor] 회원가입 API 리팩토링 + 카카오 소셜 로그인 관련 리팩토링 #70
Conversation
|
""" Walkthrough회원 가입 관련 DTO, 엔티티, 서비스, 카카오 로그인 서비스, 그리고 통합 테스트에서 사용자 생성 시 필수 입력값과 필드가 대폭 축소되었습니다. 닉네임, 이름, 생년월일, 성별, 사용자 유형 등이 제거되고, 이메일, 비밀번호, 로그인 유형만 필수로 남았습니다. 닉네임은 별도의 setter로 설정합니다. 또한, 카카오 로그인 서비스에서 ObjectMapper와 WebClient가 주입되고, 예외 처리 범위가 구체화되었습니다. 사용자 장소 컨트롤러에서는 인증된 사용자 ID 검증 로직이 명확해졌습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant UserService
participant User
Client->>UserService: 회원가입 요청(email, password, loginType)
UserService->>User: User.createUser(email, password, loginType)
User-->>UserService: User 객체 반환
UserService-->>Client: 회원가입 처리 결과 반환
sequenceDiagram
participant Kakao
participant KakaoLoginService
participant User
Kakao->>KakaoLoginService: 카카오 로그인 요청
KakaoLoginService->>User: User.createUser(email, "", LoginType.KAKAO)
KakaoLoginService->>User: setNickname(nickname)
KakaoLoginService->>User: updateProfileImageUrl(url)
KakaoLoginService-->>Kakao: 로그인 처리 결과 반환
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. ✨ Finishing Touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
src/main/java/com/wayble/server/user/entity/User.java (1)
88-90: 닉네임 setter 메서드 추가를 고려해보세요.public setter는 캡슐화를 깨뜨릴 수 있지만 현재 리팩토링에는 필요합니다. 닉네임 유효성 검증을 추가하는 것을 고려해보세요.
public void setNickname(String nickname) { + if (nickname != null && nickname.length() > 8) { + throw new IllegalArgumentException("닉네임은 8자를 초과할 수 없습니다"); + } this.nickname = nickname; }src/main/java/com/wayble/server/user/dto/UserRegisterRequestDto.java (1)
12-13: 비밀번호 유효성 검증 제거 검토 필요비밀번호 최소 길이 검증이 제거되었습니다. 보안 요구사항에 따라 최소 길이나 복잡성 검증이 필요한지 검토해보세요.
@NotBlank(message = "비밀번호는 필수입니다") +@Size(min = 8, message = "비밀번호는 최소 8자 이상이어야 합니다") String password,
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
src/main/java/com/wayble/server/user/dto/UserRegisterRequestDto.java(1 hunks)src/main/java/com/wayble/server/user/entity/User.java(2 hunks)src/main/java/com/wayble/server/user/service/UserService.java(1 hunks)src/main/java/com/wayble/server/user/service/auth/KakaoLoginService.java(2 hunks)src/test/java/com/wayble/server/explore/WaybleZoneRecommendApiIntegrationTest.java(1 hunks)
🔇 Additional comments (8)
src/main/java/com/wayble/server/user/service/auth/KakaoLoginService.java (2)
45-50: 사용자 생성 로직 단순화가 잘 적용되었습니다.카카오 소셜 로그인을 위한 사용자 생성이 새로운 패턴에 맞게 올바르게 수정되었습니다. 필수 정보만으로 사용자를 생성하고 닉네임을 별도로 설정하는 방식이 적절합니다.
60-60: Lombok @Getter로 getUserType() 자동 생성됨 확인
User.java클래스에 클래스 레벨로 선언된@Getter어노테이션이userType필드의getUserType()메서드를 자동 생성하므로
user.getUserType().name()호출은 정상입니다. 별도 수정이 필요 없습니다.src/main/java/com/wayble/server/user/service/UserService.java (1)
24-28: 회원가입 로직 단순화가 올바르게 구현되었습니다.필수 정보만으로 사용자를 생성하는 새로운 패턴이 정확히 적용되었습니다. 비밀번호 암호화와 중복 사용자 체크 로직이 그대로 유지되어 기존 기능이 보장됩니다.
src/test/java/com/wayble/server/explore/WaybleZoneRecommendApiIntegrationTest.java (2)
136-142: 테스트에서 사용자 생성 패턴이 올바르게 적용되었습니다.새로운 사용자 생성 방식에 맞게 테스트 코드가 정확히 수정되었습니다. 이메일 형식도 개선되었고 닉네임을 별도로 설정하는 패턴이 일관되게 적용되었습니다.
152-154: 검증 결과: NPE 발생 우려 없음
- AgeGroup.fromBirthDate(null) 호출 시 내부에서
if (birthDate == null) return OTHERS;로 처리하여 NPE 발생하지 않습니다.- User.createUser 빌더는 birthDate, gender를 설정하지 않아 두 필드가 null이지만,
user.getGender()는 단순히 null을 반환할 뿐 메서드 호출로 인한 NPE가 발생하지 않습니다.원 리뷰 코멘트의 null 참조 가능성 지적은 코드상에서 실제 예외를 일으키지 않으므로 무시해도 좋습니다.
Likely an incorrect or invalid review comment.
src/main/java/com/wayble/server/user/entity/User.java (2)
31-31: 닉네임 필드 nullable 처리가 올바르게 적용되었습니다.새로운 회원가입 플로우에서 닉네임이 선택적 정보가 된 것에 맞게 데이터베이스 제약조건이 적절히 수정되었습니다.
71-82: 사용자 생성 팩토리 메서드 단순화가 효과적으로 구현되었습니다.필수 정보만으로 사용자를 생성하는 새로운 요구사항에 맞게 createUser 메서드가 올바르게 단순화되었습니다. userType을 GENERAL로 기본 설정하는 것도 적절합니다.
src/main/java/com/wayble/server/user/dto/UserRegisterRequestDto.java (1)
7-17: 회원가입 DTO 단순화가 요구사항에 맞게 잘 구현되었습니다.필수 정보만으로 회원가입을 처리하는 새로운 API 스펙에 맞게 DTO가 올바르게 단순화되었습니다. LoginType에 대한 주석도 API 문서화에 도움이 됩니다.
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
src/main/java/com/wayble/server/user/controller/UserPlaceController.java (1)
66-73: 동일한 인증 로직 패턴 적용
saveUserPlace메소드와 동일한 개선된 인증 로직이 일관성 있게 적용되었습니다. 타입 안전성과 예외 처리가 잘 구현되어 있습니다.위에서 언급한 것처럼, 공통 유틸리티 메소드로 추출하면 코드 중복을 해결할 수 있습니다.
🧹 Nitpick comments (1)
src/main/java/com/wayble/server/user/controller/UserPlaceController.java (1)
40-47: 인증 로직 개선 - 타입 안전성 향상Authentication 객체에서 principal을 안전하게 추출하는 로직이 잘 구현되었습니다. instanceof 체크를 통해 ClassCastException을 방지하고 명시적인 예외 처리를 추가한 점이 좋습니다.
하지만 동일한 패턴이
getUserPlaces메소드에서도 반복되어 코드 중복이 발생합니다. 공통 유틸리티 메소드로 추출하는 것을 고려해보세요.private Long getAuthenticatedUserId() { Authentication authentication = SecurityContextHolder.getContext().getAuthentication(); if (!(authentication.getPrincipal() instanceof Long)) { throw new ApplicationException(UserErrorCase.FORBIDDEN); } return (Long) authentication.getPrincipal(); }이후 각 메소드에서는 다음과 같이 사용할 수 있습니다:
Long tokenUserId = getAuthenticatedUserId(); if (!userId.equals(tokenUserId)) { throw new ApplicationException(UserErrorCase.FORBIDDEN); }
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (5)
src/main/java/com/wayble/server/user/controller/UserPlaceController.java(3 hunks)src/main/java/com/wayble/server/user/dto/KakaoLoginRequestDto.java(1 hunks)src/main/java/com/wayble/server/user/dto/KakaoUserInfoDto.java(1 hunks)src/main/java/com/wayble/server/user/entity/User.java(2 hunks)src/main/java/com/wayble/server/user/service/auth/KakaoLoginService.java(6 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
- src/main/java/com/wayble/server/user/service/auth/KakaoLoginService.java
- src/main/java/com/wayble/server/user/entity/User.java
🧰 Additional context used
🧬 Code Graph Analysis (1)
src/main/java/com/wayble/server/user/dto/KakaoUserInfoDto.java (1)
src/main/java/com/wayble/server/user/entity/User.java (1)
Getter(14-91)
🔇 Additional comments (6)
src/main/java/com/wayble/server/user/dto/KakaoLoginRequestDto.java (1)
3-6: 유효성 검증 추가가 적절합니다.카카오 로그인 요청에서 액세스 토큰에 대한
@NotBlank유효성 검증 추가는 보안과 데이터 무결성 측면에서 중요한 개선사항입니다. 빈 문자열이나 공백만 있는 토큰을 사전에 차단하여 불필요한 API 호출을 방지할 수 있습니다.src/main/java/com/wayble/server/user/controller/UserPlaceController.java (1)
14-14: Import 추가 승인Authentication 인터페이스 import가 적절히 추가되었습니다.
src/main/java/com/wayble/server/user/dto/KakaoUserInfoDto.java (4)
5-5: 필요한 import 추가가 적절합니다.@JsonProperty 어노테이션 사용을 위한 import 추가가 올바르게 되었습니다.
13-14: Java 네이밍 컨벤션 개선이 우수합니다.필드명을 camelCase로 변경하면서 @JsonProperty 어노테이션으로 JSON 호환성을 유지한 점이 좋습니다. 코드 가독성이 향상되었습니다.
26-27: 일관된 네이밍 컨벤션 적용이 우수합니다.profileImageUrl 필드명 변경이 kakaoAccount와 일관성 있게 적용되었고, User 엔티티의
updateProfileImageUrl()메소드와도 네이밍이 일치합니다.
9-30: 전체적으로 우수한 리팩토링입니다.이 변경사항들은 다음과 같은 장점을 제공합니다:
- Java 네이밍 컨벤션 준수로 코드 가독성 향상
- @JsonProperty를 통한 API 호환성 유지
- 불변 객체 패턴 적용 (Setter 제거)으로 데이터 안정성 확보
- 사용자 등록 API 리팩토링 목표와 일치하는 구조 개선
회원가입 및 카카오 로그인 리팩토링의 일환으로 적절한 변경입니다.
연관된 이슈
#68 , #58
작업 내용
테스트 결과
바뀐 회원가입 로직으로 swagger 테스트
리뷰 요구사항 (선택)
Summary by CodeRabbit
New Features
Refactor
Tests