[Refactor] #238 - Discord 회원가입 로직 이벤트 Pub-Sub 구조로 리팩토링

[its-sky]

🚀PullRequest🚀

📟 관련 이슈

• Resolved: [Refactor] Discord 회원가입 로직 이벤트 Pub-Sub 구조로 리팩토링 #238

💻 작업 내용

• UserService에서 DiscordAppender의 의존성이 직접적으로 사용되는 것을 고쳤습니다.
• 유저가 회원가입 시 DiscordAppender.signInAppend() 메소드를 직접 호출하는 방식을 제거하고 SignUpEvent를 만들었습니다.

흐름은 다음과 같습니다.

• UserService에서 ApplicationEventPublisher(ApplicationContext 가 구현해줘서 주입받아 그냥 사용하면 됨)를 주입받아 eventPublisher.publishEvent(new SignUpEvent()); 를 호출합니다.
• moonshot-external 모듈에 있는 discord 패키지에 SignUpEventListener를 만들고 해당 리스너에서 이벤트를 구독합니다.
• 해당 이벤트를 구독하고 의존성으로 주입되어 있는 DiscordAppender.signInAppend() 메소드를 호출합니다.

이로서 얻는 이점

• UserService와 DiscordAppender(알림 서비스) 와의 강한 결합도를 느슨하게 만들었습니다.

단점

• 로직의 흐름을 직접적으로 파악하기 어려울 수 있으니 코드리뷰하시고 이해가 안되는 부분은 리뷰 달아주세요!~~

📝 리뷰 노트

[0lynny]

아주 좋군요 ! 작업 수고많으셨습니다 ~!!!