Open smilejakdu opened 10 months ago
@PostMapping("/signup") public BaseResponse<SignUpRes> createUser(@RequestBody SignUpReq signUpReq){ // 형식적 validation checkUsernameValidation(signUpReq.getName()); checkEmailValidation(signUpReq.getEmail()); checkCodeValidation(signUpReq.getCode()); checkPasswordValidation(signUpReq.getPassword()); checkGreetingValidation(signUpReq.getGreeting()); SignUpRes signUpRes = userService.createUser(signUpReq); return new BaseResponse<>(signUpRes); }
check 코드 보시면 단순 blank or null 값 체크하는 코드가 대부분인데, request dto 로 뺄 수 있으면 빼는게 좋을것 같아요 그리고 생성한뒤에 signUpRes 객체를 바로 넘겨주는데 저기에 value 값 어떤것들이 찍혀요 ?? email 도 checkEmailValidation 보니까 빈값만 체크되어있던데 이메일이 맞는지도 검증하면 좋을것 같아요 정규 표현식을 쓰든 아니면 제공해주는 어노테이션을 사용하든
check 코드 보시면 단순 blank or null 값 체크하는 코드가 대부분인데, request dto 로 뺄 수 있으면 빼는게 좋을것 같아요 그리고 생성한뒤에 signUpRes 객체를 바로 넘겨주는데 저기에 value 값 어떤것들이 찍혀요 ?? email 도 checkEmailValidation 보니까 빈값만 체크되어있던데 이메일이 맞는지도 검증하면 좋을것 같아요 정규 표현식을 쓰든 아니면 제공해주는 어노테이션을 사용하든