Validator에 대하여 질문
155
작성한 질문수 25
@Data
@Builder
@NoArgsConstructor
@AllArgsConstructor
public class LoginDto {
@NotBlank
private String email;
@NotBlank
private String password;
}
-------------------------------------
@Component
public class LoginValidator implements Validator {
private final UserRepository userRepository;
public LoginValidator(UserRepository userRepository) {
this.userRepository = userRepository;
}
@Override
public boolean supports(Class<?> clazz) {
return LoginDto.class.isAssignableFrom(clazz);
}
@Override
public void validate(Object target, Errors errors) {
LoginDto loginDto = (LoginDto) target;
User user = userRepository.findByEmail(loginDto.getEmail());
if (user == null) {
throw new LoginFailedException("User not found");
}
if (!user.getPassword().equals(loginDto.getPassword())) {
throw new LoginFailedException("Invalid password");
}
}
}위 코드처럼 DTO 클래스에서 간단한 검증은 bean validation으로 시행하고, 복잡한 검증로직은 LoginValidator에서 관리 및 커스텀 예외를 리턴하는 방식은 안좋은 방식인가요?
제가 알기로는 errors는 bindingresult처럼 복잡한 검증이 아니라 단순한 검증결과를 담는 것으로 알고 있는데, validate를 override하여 복잡한 검증 실행 및 검증 결과를 LoginFailedException(커스텀 예외)으로 반환하려고 하니 Errors를 리턴해야 한다는 강제성 때문에 혼란이 생겼습니다.
따라서 제가 궁금한 점은, 간단한 검증작업(ex. 정규식) 과 복잡한 검증작업(ex.이메일,비밀번호가 DB에 있는지 검증)은 어떻게 처리하는게 이상적인가? 입니다. 저는 각각 Bean Validation, Validator(커스텀 예외 반환)로 해보려했는데 아닌 것 같아서 질문드립니다.
감사합니다.
답변 1
0
안녕하세요. liltjay님
이 부분은 정답이 없습니다.
다만 보통 간단한 검증 작업은 Bean Validation을 사용하고, 복잡한 경우 별도의 Validator를 사용하거나 또는 별도의 비즈니스 로직으로 처리하는 방법을 주로 사용합니다.
예를 들어서 다음과 같이 둘을 합쳐서 사용하는 방법도 있습니다.
@RestController
public class LoginController {
private final LoginValidator loginValidator;
@Autowired
public LoginController(LoginValidator loginValidator) {
this.loginValidator = loginValidator;
}
@PostMapping("/login")
public ResponseEntity<?> login(@Valid @RequestBody LoginDto loginDto, BindingResult result) {
loginValidator.validate(loginDto, result);
if (result.hasErrors()) {
// 오류 메시지를 반환하거나 적절히 처리
return ResponseEntity.badRequest().body(result.getAllErrors());
}
// 로그인 성공 로직
return ResponseEntity.ok().build();
}
}감사합니다.
0
그렇다면 복잡한 검증 시 Validator를 사용하는 경우, bean validation 실패 시 Errors와 bindingresult에 예외가 담겨서 복잡한 검증은 Errors에 담으면 안되는 줄 알았는데, 딱히 상관없을까요?
LoginValidator에서 복잡한 검증 발생 시 Errors가 아닌 커스텀 예외(ex.LoginFailedException)를 반환하기 위해, 스프링에서 제공하는 Validator를 implement 하지 않아도 상관 없나요?
@Component
@RequiredArgsConstructor
public class LoginValidator {
private final UserRepository userRepository;
public void validate(LoginDto loginDto) {
Optional<User> userOptional = userRepository.findByEmail(loginDto.getEmail());
if (userOptional.isEmpty()) {
throw new LoginFailedException("이메일 또는 비밀번호가 맞지 않습니다.");
}
User user = userOptional.get();
if (!BCrypt.checkpw(loginDto.getPassword(), user.getPassword())) {
throw new LoginFailedException("비밀번호가 맞지 않습니다.");
}
}
}
이미지 업로드와 db 트랜잭션 묶는법
0
40
1
Could not resolve org.springframework.boot:spring-boot-starter-validation:2.4.4
0
45
2
MessageSourceTest 코드
0
46
1
인터셉터 에러 설정
0
47
1
resolveArgument()메서드 질문
0
54
1
43강 검증1 에서 실패 로직 관련 질문있습니다.
0
52
2
타임리프 3.X 버전 rendering, serializer 에러 해결 방법
2
130
3
스프링 빈에 등록이 안되는거 같은데 어떻게 하면 좋을까요?ㅠㅠ
0
86
3
pdf 오타 문의
0
54
1
ItemUpdateForm 검증 관련 질문입니다.
0
46
1
22page 링크 주소 변경
0
55
2
특정 데이터와 파일을 함께 저장 시, 테이블 구조 질문
0
52
1
섹션3번 수업에 대한 질문입니다.
0
78
2
@Autowired 보다 더 좋은 방법이 어떤 걸까요?
0
84
2
타입컨버터 가 람다랑 비슷해 보이는데 저의 생각이 맞는지?.
0
63
1
자바스크립트 인라인에서 객체 직렬화 시 오류가 납니다
0
140
3
스프링부트 - 오류페이지2 에서 500.html 에서 쓰인 객체 질문
0
62
1
톰캣 에러 페이지가 안보입니다.
0
98
2
apiEceptionController에서 센드 에러 호출하면 안되는지?
0
80
1
세션 타임아웃시 쿠키 삭제 방법이 없나요?
0
116
2
ApiExceptionController 질문드립니다.
0
62
1
셀렉박스 챕터에서 option value에 ==배송 방식 선택== 이것을 넣은 이유가 궁금함, 이렇게 구상해도 되는지?
0
64
1
MemberRepository 필드의 fianl 선언 유무
0
83
2
혹시 index.html 에서는 fragment 사용이 안되는건가요
0
57
1





