Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3주차] 기본과제 제출 #3

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[3주차] 기본과제 제출 #3

wants to merge 1 commit into from

Conversation

sunseo18
Copy link
Collaborator

@sunseo18 sunseo18 commented May 4, 2023

db 주소랑 비번 안 가리고 푸시해서 큰일날뻔 ~ !

@sunseo18 sunseo18 marked this pull request as ready for review May 4, 2023 01:53
@PgmJun
Copy link
Member

PgmJun commented May 15, 2023

db 주소랑 비번 안 가리고 푸시해서 큰일날뻔 ~ !

해킹하고 IQ 추적한거 이미 PPT 따놨습니다.

Copy link
Member

@PgmJun PgmJun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고했숴!


private final UserService userService;

@PostMapping("/user/signup")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@RestController
@RequestMapping("/user")
@RequiredArgsConstructor
public class UserController {

    private final UserService userService;

    @PostMapping("/signup")

유저 컨트롤러이기 때문에 이렇게 해주는게 어떨까 싶습니다만..?
image

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ㅋㅋㅋㅋㅋ ㅇ..이거또한 마찬가지.. 복붙의 결과입니다만 ... 제가 다시 제 코드로 바꿀게여 ,, ,, ,, ,,, 정말 좋은 지적, ,,감사합니다 노여움 푸시죠 ,,,

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ㅋㅋㅋㅋㅋ ㅇ..이거또한 마찬가지.. 복붙의 결과입니다만 ... 제가 다시 제 코드로 바꿀게여 ,, ,, ,, ,,, 정말 좋은 지적, ,,감사합니다 노여움 푸시죠 ,,,

한번 봐드릴게여!

@PostMapping("/user/signup")
@ResponseStatus(HttpStatus.CREATED)
public ApiResponseDto<UserResponseDto> create(
@RequestBody @Valid final UserRequestDto request) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

final로 불변 처리 아주 좋읍니다.

regexp="(?=.*[0-9])(?=.*[a-zA-Z])(?=.*\\W)(?=\\S+$).{8,20}",
message = "비밀번호는 영문 대,소문자와 숫자, 특수기호가 적어도 1개 이상씩 포함된 8자 ~ 20자의 비밀번호여야 합니다."
)
private final String password;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

검증처리 너무 잘 해놓으셨어요~
image

public class UserResponseDto {

private Long userId;
private String nickname;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

정적 팩터리 메서드로 dto인스턴스를 생성하는 것 같은데, 필드에 final로 불변 처리 해주시는건 어떨까요??
아니면 혹시 final로 처리하지 않은 이유가 있으신가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ㅇ..어.. 세미나 코드 복붙의 결과랄까요.. 제가 생각이 짧았습니다.. 다시 한 번 검토해서 제 코드로 만들어볼게요.. 하하..

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ㅇ..어.. 세미나 코드 복붙의 결과랄까요.. 제가 생각이 짧았습니다.. 다시 한 번 검토해서 제 코드로 만들어볼게요.. 하하..

헤헤 네네^^

Copy link
Member

@05AM 05AM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

너무나 깔끔한 세미나 과제 좋습니다~~ 👏👏👏

Comment on lines +1 to +6
spring:
datasource:
driver-class-name: com.mysql.cj.jdbc.Driver
url: ${db-url}
username: ${db-username}
password: ${db-password}
Copy link
Member

@05AM 05AM May 19, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

준서 코멘트 보고 저도 yaml 파일 삭제했으여 ㅋㅋㅋㅋㅋㅋ

중요정보 처리 너무 좋습니다~

혹시 위 정보는 properties에 따로 저장하시는 걸까요, 아님 다른데인가요?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants