🐛 Fix: users - club 연관관계 오류 수정

[imjuyongp]

#️⃣ Issue Number

• ✨Feat: 실시간 알림기능 #38

📝 요약(Summary)

문제점

1. leader 중복 관리: Club.leader 필드와 ClubMember.role=LEADER 두 곳에서 동호회장 관리
2. 복합 유니크 제약조건 누락: 같은 사용자가 같은 동호회에 중복 가입 가능
3. User에 Club 관계 누락: 사용자가 가입한 동호회 조회 불편

---

DB 스키마 변경

-- clubs 테이블에서 leader_id 컬럼 제거됨
ALTER TABLE clubs DROP COLUMN leader_id;

-- club_members 테이블에 유니크 제약조건 추가됨
ALTER TABLE club_members ADD UNIQUE (club_id, user_id);

🛠️ PR 유형

어떤 변경 사항이 있나요?

• 버그 수정
• 코드 리팩토링

📸스크린샷 (선택)

💬 공유사항 to 리뷰어

• erd 수정사항 있습니다.

✅ PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

• 커밋 메시지 컨벤션에 맞게 작성했습니다.
• 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

Summary by CodeRabbit

릴리스 노트

• 개선 사항
  • 클럽 리더 참조 방식이 변경되어 리더 확인과 멤버 관리가 더 명확해졌습니다.
  • 사용자의 클럽 가입 내역 추적 기능이 추가되었습니다.
• 신규 기능
  • 클럽 가입 요청을 생성·관리하는 요청 엔티티가 도입되었습니다.
  • 다양한 유형의 알림(가입 요청, 승인/거절, 채팅, 댓글 등)과 읽음 처리 기능이 추가되었습니다.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

Club 엔티티의 리더 관계를 User 직접 참조에서 ClubMember를 통한 간접 접근으로 리팩토링하고, Club 생성 API 시그니처 변경, User에 clubMemberships 추가, JoinClubRequest 및 Notification 엔티티 추가가 포함된 변경입니다.

Changes

Cohort / File(s)	Summary
Club 리더 관계 리팩토링 src/main/java/com/be/sportizebe/domain/club/entity/Club.java	private User leader 필드 제거. getLeaderMember(), getLeader(), isLeader(Long userId) 헬퍼 메서드 추가.
Club 생성 요청 API 변경 src/main/java/com/be/sportizebe/domain/club/dto/request/ClubCreateRequest.java	toEntity(User, String) → toEntity(String)로 시그니처 변경. User import 및 빌더에서 리더 설정 제거.
Club 서비스 적용 src/main/java/com/be/sportizebe/domain/club/service/ClubServiceImpl.java	변경된 toEntity() 호출 반영. 생성 흐름에서 리더를 ClubMember로 추가하는 로직 명시화. 리더 검증을 club.isLeader(userId)로 변경.
User 엔티티 확장 src/main/java/com/be/sportizebe/domain/user/entity/User.java	@OneToMany(mappedBy = "user") clubMemberships 필드 추가 (List<ClubMember>), 빌더 기본값 초기화.
Join 요청 엔티티 추가 src/main/java/com/be/sportizebe/domain/notification/entity/JoinClubRequest.java	JoinClubRequest 엔티티 추가(유니크 제약 user_id+club_id). 상태 enum(PENDING, ACCEPTED, REJECTED) 및 accept(), reject() 메서드 포함.
Notification 엔티티 추가 src/main/java/com/be/sportizebe/domain/notification/entity/Notification.java	Notification 엔티티 추가(수신자, 타입 enum, 메시지, isRead, joinClubRequest, targetId, markAsRead()). 테이블 인덱스 추가.

Sequence Diagram(s)

(생성된 변경은 엔터티 추가 및 리팩토링에 집중되어 명확한 다중 컴포넌트 순차 흐름이 정의되어 있지 않아 시퀀스 다이어그램을 생략합니다.)

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Possibly related PRs

• 🐛 Fix: users - club 연관관계 오류 수정 #48 — 동일한 Club/User/ClubMember 리팩토링(Club.leader 제거, ClubCreateRequest 시그니처 변경, leader 로직 이동)과 직접적으로 연관되어 있음.

Poem

리더는 이제 토끼를 피해 걷지 않아요 🐰
멤버십 길로 슬며시 발자국 남기고,
관계는 가지를 따라 흐르네,
알림이 와서 문을 두드리면,
함께 모여 뛰놀자고 속삭여요. 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 44.44% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목은 사용자-클럽 관계 오류 수정이라는 주요 변경사항을 명확하게 설명합니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/notification

---

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.}