fix: NotificationEventListener 비즈니스 로직 삭제

[Soundbar91]

🔍 개요

• close [공통] NotificationEventListener 비즈니스 로직 제거 #2256

---

🚀 주요 변경 내용

• NotificationEventListener 클래스 내부에 있는 비즈니스 로직을 삭제

---

💬 참고 사항

---

✅ Checklist (완료 조건)

• 코드 스타일 가이드 준수
• 테스트 코드 포함됨
• Reviewers / Assignees / Labels 지정 완료
• 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

[coderabbitai]

Warning

Review limit reached

@Soundbar91, we couldn't start this review because you've used your available PR reviews for now.

Your plan currently allows 1 review/hour. Refill in 39 minutes and 33 seconds.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more review capacity refills, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than trial, open-source, and free plans. In all cases, review capacity refills continuously over time.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f415edc9-088a-44d8-a021-cf3c680d4134

📥 Commits

Reviewing files that changed from the base of the PR and between 9c43277 and bd7ca61.

📒 Files selected for processing (4)

• src/main/java/in/koreatech/koin/common/event/UserMarketingAgreementEvent.java
• src/main/java/in/koreatech/koin/domain/notification/eventlistener/NotificationEventListener.java
• src/main/java/in/koreatech/koin/domain/student/service/StudentService.java
• src/main/java/in/koreatech/koin/domain/user/service/UserService.java

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/2256-notification-eventlistener

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

Unit Test Results

672 tests   669 ✔️  1m 20s ⏱️
166 suites      3 💤
166 files        0 ❌

Results for commit bd7ca61.

♻️ This comment has been updated with latest results.

[BaeJinho4028]

다 좋은데, 요즘 와서 생각드는건 클래스를 꽤 과분리하지않았나도 싶어요.

당시 코드가 방대해져서 분리를 했지만, 클래스마다 메서드 하나씩이라서 좀더 모듈화를 잘 시키는 것이 낫지 않았나 싶은? 그래도 이렇게 로직 정리해주셔서 다음에 리팩터링하기는 편해질 것 같습니다.

[Soundbar91]

해당 작업을 진행하면서 클래스를 과하게 분리하고 있지 않나라는 생각이 들었습니다. 말씀하신 모듈화라던가 응집성을 가져가면서 클래스를 적게 가져갈 수 있는 방법에 대해서 고민을 했는데, 마땅히 좋은 방법이 떠오르지 않더라구여..

알림 비즈니스 로직별로 상위에 서비스 클래스를 두고 하위에 FCMClient와 통신하는 Notification 서비스를 두는 것을 생각하고 해당 작업을 수행했습니다. 향후에 더 좋은 방법이 나온다면 적용해보는 것도 좋을 거 같아요 !