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

[REFACTOR] Member 도메인 리팩토링 #131

Merged
merged 7 commits into from
Jul 31, 2024
Merged

Conversation

paragon0107
Copy link
Contributor

@paragon0107 paragon0107 commented Jul 29, 2024

✨ Related Issue

📝 기능 구현 명세

🐥 추가적인 언급 사항

  • 오류 발견되었습니다

  • 우선 멤버 서비스에서 사용되던 멤버가 존재하는지 존재하지 않는지 확인하는 메서드들을 MemberFinder로 옮겼습니다. 이에 대한 메서드 네이밍을 현재 한것처럼 할 지 두개의 메서드 네이밍을 반대로 할 지 고민했는데 안에서 사용되고 있는 ifPresent라는 메서드의 이름을 따라서 네이밍을 했습니다.
  • Member 도메인 안에서 checkMailFormat과 checkPassword 같은 메서드들을 static으로 할 지 그냥 일반 메서드로 할 지 고민하다가 객체 중심적으로 생각했을때 그 객체가 생성되고 객체가 어떤 일을 하는 것이니 static 사용하지 않고 메서드를 생성했습니다.
  • 고민되는게 있거나 이런 방향으로 하는게 확실하지 않은것 같으면 수요일에 가져가서 이런식으로 바꾸면 되는 것 인지 확인해봐도 좋을 것 같습니다

@paragon0107 paragon0107 requested a review from Chan531 July 29, 2024 06:06
@paragon0107 paragon0107 self-assigned this Jul 29, 2024
@paragon0107 paragon0107 linked an issue Jul 29, 2024 that may be closed by this pull request
5 tasks
@paragon0107 paragon0107 changed the title #130 member refactor [REFACTOR] Member 도메인 리팩토링 Jul 29, 2024
Copy link
Contributor

@Chan531 Chan531 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~! 멤버의 책임에 맞는 코드인 거 같아요!

@paragon0107 paragon0107 merged commit 6d8d005 into develop Jul 31, 2024
1 check passed
@paragon0107 paragon0107 deleted the #130-member-refactor branch July 31, 2024 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[REFACTOR] 객체지향적으로 변경
2 participants