Conversation
- if email already exists, return ture
|
""" Walkthrough이메일 중복 확인 기능이 추가되었습니다. 신규 요청/응답 DTO, 서비스 메서드, API 엔드포인트가 도입되었으며, 관련 에러 코드와 성공 코드, 스웨거 문서, 예외 처리 로직이 확장되었습니다. 기존 코드에는 경미한 포맷팅 변경만 발생했습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant AuthApi
participant AuthService
participant UserRepository
Client->>AuthApi: POST /api/auth/register/check-email (CheckEmailDuplicatedRequest)
AuthApi->>AuthService: checkEmailDuplicated(request)
AuthService->>UserRepository: existsByEmail(email)
UserRepository-->>AuthService: boolean (존재 여부)
AuthService-->>AuthApi: CheckEmailDuplicatedResponse
AuthApi-->>Client: BaseResponse<CheckEmailDuplicatedResponse> (성공/실패 코드 포함)
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~18 minutes Suggested labels
Suggested reviewers
Poem
Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
✨ Finishing Touches
🧪 Generate unit tests
🪧 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: 1
🧹 Nitpick comments (3)
src/main/java/com/blockguard/server/domain/auth/application/AuthService.java (1)
55-60: 메서드 구현이 올바르지만 매개변수명 개선이 필요합니다.이메일 중복 체크 로직은 올바르게 구현되었습니다. 하지만 매개변수명과 클래스명이 동일하여 가독성이 떨어집니다.
매개변수명을 개선하여 가독성을 향상시킬 수 있습니다:
-public CheckEmailDuplicatedResponse checkEmailDuplicated(CheckEmailDuplicatedRequest CheckEmailDuplicatedRequest) { - boolean isDuplicated = userRepository.findByEmail(CheckEmailDuplicatedRequest.getEmail()).isPresent(); +public CheckEmailDuplicatedResponse checkEmailDuplicated(CheckEmailDuplicatedRequest request) { + boolean isDuplicated = userRepository.findByEmail(request.getEmail()).isPresent();src/main/java/com/blockguard/server/global/exception/GlobalExceptionHandler.java (1)
26-44: 유효성 검사 예외 처리가 잘 구현되었지만 개선 가능합니다.유효성 검사 예외를 적절히 처리하고 이메일 형식과 일반 유효성 검사를 구분하여 처리하는 로직이 좋습니다. 다만 하드코딩된 문자열 비교 부분을 개선할 수 있습니다.
하드코딩된 문자열 대신 상수를 사용하도록 개선하는 것을 고려해보세요:
+ private static final String EMAIL_FORMAT_ERROR_KEYWORD = "이메일 형식"; + @ExceptionHandler(MethodArgumentNotValidException.class) protected ResponseEntity<ErrorResponse> handleValidationException(MethodArgumentNotValidException ex) { String message = ex.getBindingResult().getFieldErrors().stream() .findFirst() .map(DefaultMessageSourceResolvable::getDefaultMessage) .orElse("유효성 검사에 실패했습니다."); log.warn("[ValidationException] message: {}", message); - if (message.contains("이메일 형식")) { + if (message.contains(EMAIL_FORMAT_ERROR_KEYWORD)) { return ResponseEntity .status(ErrorCode.INVALID_EMAIL_TYPE.getStatus()) .body(ErrorResponse.of(ErrorCode.INVALID_EMAIL_TYPE)); }src/main/java/com/blockguard/server/domain/auth/api/AuthApi.java (1)
4-4: 와일드카드 import 사용을 재고해보세요.와일드카드 import
import com.blockguard.server.domain.user.dto.request.*;는 코드의 명확성을 떨어뜨릴 수 있습니다. 특히 사용되는 클래스가 명확하지 않을 때 가독성이 저하됩니다.명시적인 import를 사용하는 것을 고려해보세요:
-import com.blockguard.server.domain.user.dto.request.*; +import com.blockguard.server.domain.user.dto.request.CheckEmailDuplicatedRequest; +import com.blockguard.server.domain.user.dto.request.FindEmailRequest; +import com.blockguard.server.domain.user.dto.request.FindPasswordRequest; +import com.blockguard.server.domain.user.dto.request.LoginRequest; +import com.blockguard.server.domain.user.dto.request.RegisterRequest;
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (8)
src/main/java/com/blockguard/server/domain/auth/api/AuthApi.java(3 hunks)src/main/java/com/blockguard/server/domain/auth/application/AuthService.java(3 hunks)src/main/java/com/blockguard/server/domain/user/dto/request/CheckEmailDuplicatedRequest.java(1 hunks)src/main/java/com/blockguard/server/domain/user/dto/response/CheckEmailDuplicatedResponse.java(1 hunks)src/main/java/com/blockguard/server/global/common/codes/ErrorCode.java(2 hunks)src/main/java/com/blockguard/server/global/common/codes/SuccessCode.java(1 hunks)src/main/java/com/blockguard/server/global/config/swagger/SwaggerResponseDescription.java(1 hunks)src/main/java/com/blockguard/server/global/exception/GlobalExceptionHandler.java(2 hunks)
🧰 Additional context used
🧬 Code Graph Analysis (4)
src/main/java/com/blockguard/server/global/exception/GlobalExceptionHandler.java (2)
src/main/java/com/blockguard/server/global/exception/BusinessExceptionHandler.java (1)
BusinessExceptionHandler(7-12)src/main/java/com/blockguard/server/global/config/resolver/CurrentUserArgumentResolver.java (1)
CurrentUserArgumentResolver(15-41)
src/main/java/com/blockguard/server/domain/user/dto/response/CheckEmailDuplicatedResponse.java (5)
src/main/java/com/blockguard/server/domain/auth/domain/JwtToken.java (1)
Builder(8-16)src/main/java/com/blockguard/server/domain/user/dto/request/CheckEmailDuplicatedRequest.java (1)
Getter(8-14)src/main/java/com/blockguard/server/domain/user/dto/response/FindEmailResponse.java (1)
FindEmailResponse(8-14)src/main/java/com/blockguard/server/domain/user/dto/response/RegisterResponse.java (1)
RegisterResponse(7-12)src/main/java/com/blockguard/server/domain/user/dto/response/LoginResponse.java (1)
LoginResponse(8-14)
src/main/java/com/blockguard/server/global/config/swagger/SwaggerResponseDescription.java (2)
src/main/java/com/blockguard/server/global/config/swagger/CustomExceptionDescription.java (1)
CustomExceptionDescription(8-12)src/main/java/com/blockguard/server/global/config/swagger/SwaggerConfig.java (4)
SwaggerConfig(28-139)generateErrorCodeResponseExample(80-98)addGlobalErrorResponses(124-138)getSwaggerExample(102-108)
src/main/java/com/blockguard/server/domain/auth/api/AuthApi.java (2)
src/main/java/com/blockguard/server/domain/user/dto/request/FindEmailRequest.java (1)
FindEmailRequest(13-28)src/main/java/com/blockguard/server/domain/user/dto/response/FindEmailResponse.java (1)
FindEmailResponse(8-14)
🔇 Additional comments (10)
src/main/java/com/blockguard/server/global/config/swagger/SwaggerResponseDescription.java (2)
61-61: 문법 수정이 올바르게 적용되었습니다.새로운 enum 상수 추가를 위해 기존 마지막 상수에 쉼표를 추가한 것은 적절합니다.
63-65: 이메일 중복 체크 실패 케이스 추가가 적절합니다.새로운 API 엔드포인트에 대한 Swagger 문서화가 올바르게 구현되었습니다.
INVALID_EMAIL_TYPE에러코드를 사용하여 이메일 형식 검증 실패를 명확히 나타내고 있습니다.src/main/java/com/blockguard/server/global/common/codes/SuccessCode.java (1)
24-25: 새로운 성공 코드 추가가 적절하게 구현되었습니다.기존 enum 구조와 일관성을 유지하며, 적절한 HTTP 상태 코드(OK)와 순차적인 코드 번호(2014)를 사용했습니다. 메시지도 명확하고 일관된 한국어 표현을 사용하고 있습니다.
src/main/java/com/blockguard/server/domain/auth/application/AuthService.java (1)
6-8: Import 구조 개선이 적절합니다.wildcard import 사용으로 코드가 간결해졌고, 새로운 response DTO import도 올바르게 추가되었습니다.
src/main/java/com/blockguard/server/domain/user/dto/response/CheckEmailDuplicatedResponse.java (1)
1-14: DTO 구조가 적절하게 설계되었습니다.단일 boolean 필드로 이메일 중복 여부를 명확하게 표현하고 있으며, 다른 response DTO들과 동일한 Lombok 어노테이션 패턴을 사용하여 일관성을 유지하고 있습니다.
src/main/java/com/blockguard/server/domain/user/dto/request/CheckEmailDuplicatedRequest.java (1)
1-14: 요청 DTO가 올바르게 설계되었습니다.이메일 검증을 위한 적절한 어노테이션(
@NotBlank)을 사용하고 있으며, 필요한 Lombok 어노테이션도 적절히 적용되어 있습니다. 구조가 간단하고 목적에 부합합니다.src/main/java/com/blockguard/server/global/common/codes/ErrorCode.java (2)
11-11: 새로운 에러 코드 추가가 적절합니다.일반적인 유효성 검사 실패를 위한
INVALID_REQUEST에러 코드가 적절하게 추가되었습니다. 4000번 코드 사용과 명확한 한국어 메시지가 좋습니다.
28-28: 이메일 형식 전용 에러 코드 추가가 좋습니다.이메일 형식 검증을 위한 전용 에러 코드
INVALID_EMAIL_TYPE이 추가되어 더 세분화된 에러 처리가 가능해졌습니다. 코드 번호 4025도 적절히 할당되었습니다.src/main/java/com/blockguard/server/global/exception/GlobalExceptionHandler.java (1)
6-8: 필요한 import가 적절히 추가되었습니다.
DefaultMessageSourceResolvable과MethodArgumentNotValidExceptionimport가 새로운 기능에 필요한 것들로 적절히 추가되었습니다.src/main/java/com/blockguard/server/domain/auth/api/AuthApi.java (1)
58-58: 코드 포맷팅 개선이 좋습니다.기존 메서드들의 포맷팅이 일관성 있게 개선되었습니다.
Also applies to: 66-66
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
💻 Related Issue
closed #23
🚀 Work Description
🙇🏻♀️ To Reviewer
GlobalExceptionHandler에 유효성 검사 확인관련 코드 추가하였습니다. 유효성 로직에 어긋나는 경우 5000 에러가 아닌 4000 에러가 뜨도록 하였습니다.➕ Next
Summary by CodeRabbit
신규 기능
버그 수정
문서화