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.

Sign up for GitHub

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

Jump to bottom

[Feat] 회원 탈퇴 구현 #74

Open
wants to merge 12 commits into
base: develop
Choose a base branch
from
Open

[Feat] 회원 탈퇴 구현 #74

wants to merge 12 commits into from

Conversation

nahyeon99
Copy link
Member

❗️ 이슈 번호
open #72

📝 구현 내용
일부 로직을 제외하고, 회원 탈퇴 로직을 구현했습니다. 일부 기능은 송현 님 파트와 직결된 부분이라 제 임의로 구현하기에 애매해서 논의 후 이어서 진행할 예정입니다.

🙇🏻‍♂️ 리뷰어에게 부탁합니다!

  1. 회원 탈퇴 용도가 아닌, 기존의 TeamCalendar를 삭제하는 기능도 Participant가 예상한 쿼리로 나가서 정상적으로 삭제가 됐나요? 코드상으로 애매해 보여서 확인이 필요할 것 같습니다..!

  2. 회원 탈퇴 시 Team, TeamCalendar, Participant에 대해 고려해야 하는 시나리오는 다음과 같습니다. 이에 대해 쿼리를 어떻게 나눠서 보내서 삭제 할 지 정해야 합니다.

Team 내 Member 제거 시나리오

1. 팀이 존재하지 않는 경우
		 바로 반환
2. 팀이 존재하는 경우
		 a) 탈퇴 회원이 마지막 회원인 경우 (Team.team_status == ALONE)
		 		 0. [a, b, c 공통] SleepoverCalendar : 모든 외박캘린더 삭제
		 		 1. TeamCalendar : 모든 TeamCalendar 삭제
		          		- Participant : 각 TeamCalendar를 가진 모든 Participant 삭제
		 		 2. Team : TeamStatus == DELETED 변경

		 b) 탈퇴 회원 삭제 시 한 명만 팀에 남는 경우
		 		 0. [a, b, c 공통] SleepoverCalendar : 모든 외박캘린더 삭제
		 		 1. [b, c 공통] Participant : 탈퇴 회원을 가진 모든 Participant 삭제
		 		 2. [b, c 공통] TeamCalendar : Participant가 하나도 없는 경우 삭제
		 		 3. Team : TeamStatus == ALONE 변경

		 c) 탈퇴 회원 삭제 후 두 명 이상 팀에 있는 경우
		 		 0. [a, b, c 공통] SleepoverCalendar : 모든 외박캘린더 삭제
		 		 1. [b, c 공통] Participant : 탈퇴 회원을 가진 Participant 삭제
		 		 2. [b, c 공통] TeamCalendar : Participant가 하나도 없는 경우 삭제
		 		 3. Team : TeamStatus == ACTIVE 유지

💡 참고 자료

@nahyeon99 nahyeon99 requested a review from songhyeon99 May 5, 2024 20:35
@nahyeon99 nahyeon99 self-assigned this May 5, 2024
@idorm idorm deleted a comment from songhyeon99 May 6, 2024
@songhyeon99
Copy link
Contributor

songhyeon99 commented May 7, 2024 •
edited
Loading

상세 내용

기존 코드에는 해당 기능이 구현되어 있지 않았으며, 멤버의 탈퇴와 팀에서의 제거는 동일한 요구사항을 가졌습니다. 이에 대한 결정을 하기 위해 회의를 진행한 결과, 팀 서비스단에서 구현하는 것과 포트를 통해 구현하는 방법 중 포트를 통해 어댑터에서 처리하는 것으로 결정하여 구현하였습니다. 포트를 선택한 이유는 서비스단에서 구현하게 되면 멤버가 팀 서비스에 의존성을 가지게 되어 의존성이 높아지고, 이렇게 되면 추후 유지보수와 확장성 측면에서 제한될 수 있다고 판단하여 후자가 채택되었습니다!

문제 발견

  • teamCalendar 도메인의 participant가 @ElementCollection 타입으로 엔티티가 아니여서 벌크 연산을 할 수 없는 문제가 발견되었습니다..! 이부분에 대해 논의해봐야 할 것 같습니다!

nahyeon99
nahyeon99 commented May 12, 2024
View reviewed changes
src/main/java/idorm/idormServer/calendar/adapter/out/persistence/DeleteTeamMemberAdapter.java Outdated
Comment on lines 33 to 46
final Team team = loadTeamPort.findByMemberId(member.getId());
final List<TeamCalendar> teamCalendars = loadTeamCalendarPort.findByMemberId(member.getId());

team.deleteMember(member);
if (team.getMembers().isEmpty()) {
deleteTeamPort.delete(team);
} else {
deleteSleepoverCalendarPort.deleteByMemberId(memberId);
deleteSleepoverCalendarPort.deleteByMemberId(member.getId());
teamCalendars
.forEach(teamCalendar -> teamCalendar.deletePariticipant(member.getId()));
.forEach(teamCalendar -> {
teamCalendar.deletePariticipant(member.getId());
if (teamCalendar.getParticipants().isEmpty())
deleteTeamCalendarPort.delete(teamCalendar);
});
Copy link
Member Author

Choose a reason for hiding this comment

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

팀에 회원이 한 명 남았을 때 teamStatus가 변경되어야 합니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

팀 도메인의 deleteMember() 내에 처리되어 있습니다!

nahyeon99
nahyeon99 commented May 12, 2024
View reviewed changes
src/main/java/idorm/idormServer/calendar/adapter/out/persistence/DeleteTeamMemberAdapter.java Outdated
Comment on lines 34 to 47
final Member member = loadMemberPort.loadMember(memberId);
public void deleteTeamMember(Member member) {
final Team team = loadTeamPort.findByMemberId(member.getId());
final List<TeamCalendar> teamCalendars = loadTeamCalendarPort.findByMemberId(member.getId());

team.deleteMember(member);
if (team.getMembers().isEmpty()) {
deleteTeamPort.delete(team);
} else {
deleteSleepoverCalendarPort.deleteByMemberId(memberId);
deleteSleepoverCalendarPort.deleteByMemberId(member.getId());
teamCalendars
.forEach(teamCalendar -> teamCalendar.deletePariticipant(member.getId()));
.forEach(teamCalendar -> {
teamCalendar.deletePariticipant(member.getId());
if (teamCalendar.getParticipants().isEmpty())
deleteTeamCalendarPort.delete(teamCalendar);
});
}
Copy link
Member Author

Choose a reason for hiding this comment

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

해당 메서드에서 NOT_FOUND_MEMBER 에러가 터집니다. 로그 속 제거되는 회원은 팀이 있는 상태입니다.

스크린샷 2024-05-12 오후 4 58 19

Copy link
Contributor

Choose a reason for hiding this comment

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

해당 부분은 제 쪽에서는 예외가 터지지 않아 문제 원인을 제대로 파악하지 못 하고 있습니다..

그러나 해당 부분의 예외는 teamCalendar의 participant에 memberId가 없을 경우에 예외가 발생할 수 있는데, 팀캘린더를 participant와 조인하여 조회하기 때문에 예외가 발생할 가능성이 거의 없다고 생각됩니다..

그렇지만 예외가 발생한 원인을 명확하게 파악하기위해 계속해서 분석하겠습니다! 혹시 원인 파악에 도움이 될만한 정보가 있다면 공유 해주시면 감사하겠습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@songhyeon99 songhyeon99 Awaiting requested review from songhyeon99

Assignees

@nahyeon99 nahyeon99

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nahyeon99 @songhyeon99