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 - Facade 패턴 적용 및 리팩토링 #169

Merged
merged 38 commits into from
Jun 21, 2024

Conversation

kseysh
Copy link
Member

@kseysh kseysh commented Jun 18, 2024

Related issue 🚀

Work Description 💚

  • facade pattern을 적용하였습니다.
  • @transactional을 필요한 곳에만 구현하도록 수정하였습니다.
  • 일부 부분을 가독성을 위해 수정하였습니다.
  • Controller에서 최대한 두 개의 클래스를 참조하지 않도록 하였습니다.
  • Domain에서 @NotNull 제약조건을 통해 DB ddl에 not null 제약 조건 부여 및 스프링 내부에서 객체 생성시 validation이 되도록 하였습니다.

PR 참고 사항

  • 다만, userService에서는 OnboardingInfo와 Problem Repository를 참조하고 있습니다.
  • 이를 위해 service를 추가하고 facade를 추가하는 것이 더 복잡성을 증가시킬 것이라고 생각하였습니다.
  • authFacade에서 addChallenge 메서드를 호출하기 위해 challengeFacade를 참조하는 것이 불가피한 상황인데, 이를 어떻게 해결하면 좋을지에 대해 고민이 많네용... 이 글을 보면 facade끼리의 참조도 신경만 잘 써주면 큰 상관은 없다고 하는 것 같지만 일부러 만든 facade layer가 깨지는 느낌이라 해결방안을 같이 고민해봤으면 좋겠어요

@kseysh kseysh added 👨🏻‍💻 승환 승환이가 작성한 Label 🔨 Refactor 코드 리펙토링 (기능 변경 없이 코드만 수정할 때) labels Jun 18, 2024
@kseysh kseysh requested a review from jumining June 18, 2024 15:53
@kseysh kseysh self-assigned this Jun 18, 2024
Copy link
Collaborator

@jumining jumining left a comment

Choose a reason for hiding this comment

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

수정사항이 매우 많네요 많은 작업하시느라 수고하셨어요..👍

@kseysh kseysh merged commit 8d432d9 into develop Jun 21, 2024
1 check passed
@jumining jumining deleted the refactor/#122-use-facade-pattern branch November 19, 2024 04:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👨🏻‍💻 승환 승환이가 작성한 Label 🔨 Refactor 코드 리펙토링 (기능 변경 없이 코드만 수정할 때)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

refactor - Facade 패턴 도입 및 리팩토링
2 participants