Skip to content

[칸] Chapter 8. Spring Security - Security 구조, 폼 로그인#109

Open
kannikii wants to merge 1 commit into
khan/mainfrom
khan/#108
Open

[칸] Chapter 8. Spring Security - Security 구조, 폼 로그인#109
kannikii wants to merge 1 commit into
khan/mainfrom
khan/#108

Conversation

@kannikii
Copy link
Copy Markdown

@kannikii kannikii commented May 21, 2026

✏️ 작업 내용

#️⃣ 연관된 이슈

closes #108


💡 함께 공유하고 싶은 부분

해당 주차를 공부하면서 함께 이야기하고 싶은 주제를 남겨주세요.

(어려웠던 부분과 해결 과정, 핵심 코드, 참고한 자료 등)


🤔 질문

해당 주차 워크북을 공부하면서 궁금했던 질문들을 남겨주세요.


✅ 워크북 체크리스트

  • 모든 핵심 키워드 정리를 마쳤나요?
  • 핵심 키워드에 대해 완벽히 이해하셨나요?
  • 이론 학습 이후 직접 실습을 해보는 시간을 가졌나요?
  • 미션을 수행하셨나요?
  • 미션을 기록하셨나요?

✅ 컨벤션 체크리스트

  • 디렉토리 구조 컨벤션을 잘 지켰나요?
  • pr 제목을 컨벤션에 맞게 작성하였나요?
  • pr에 해당되는 이슈를 연결하였나요?
  • 적절한 라벨을 설정하였나요?
  • 스터디원들에게 code review를 요청하기 위해 reviewer를 등록하였나요?
  • 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?

📌 주안점

@kannikii kannikii requested a review from a team May 21, 2026 14:46
@kannikii kannikii self-assigned this May 21, 2026
@kannikii kannikii added the 🚀Week 8 8주차 워크북 미션 label May 21, 2026
Comment on lines +31 to +32
// 성별 문자열을 enum으로 변환 (DTO에선 String으로 받았지만 엔티티는 enum)
private static Gender parseGender(String value) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

문자열 값을 Gender의 enum 형식으로 바꾸는 메서드를 따로 두신 거 정말 좋은 것 같아요! 저는 enum 값을 만들어야할 때마다 .valueOf를 반복적으로 작성하면서 일일이 enum값으로 바꿨는데, 저도 enum값을 인자로 넣을 일이 많아지면 이렇게 메서드를 따로 만들어서 해야겠네요!

private String address;

@Column(name = "email", nullable = false, length = 50)
@Column(name = "email", nullable = false, length = 50, unique = true)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

이메일 칼럼에 unique = true을 적용하신 거 정말 좋은 것 같아요! 저는 회원가입 서비스 로직에서 memberRepository.existsByEmail(email)을 사용해 이메일 중복을 따로 검증했는데, 이메일 칼럼에 unique = true을 적용하면 회원가입 서비스 로직에서 이메일 중복 로직을 제거할 수 있어 더 간단해지겠네요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚀Week 8 8주차 워크북 미션

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants