feat: 팔로우 추가 / 취소 기능

[kdomo]

🌱 관련 이슈

• close ✨ 팔로우 추가 / 취소 기능 #186

📌 작업 내용 및 특이사항

• 팔로우 추가 / 취소 기능 구현하였습니다.

팔로우 추가할 때 예외상황

• targetId가 null일 때
• 로그인 된 회원이 존재하지 않을 때
• target 회원이 존재하지 않을 때
• 이미 팔로우 중일 때

팔로우를 취소할 때 예외상황

• targetId가 null일 때
• 로그인 된 회원이 존재하지 않을 때
• target 회원이 존재하지 않을 때
• 팔로우 중이지 않을 때

[uwoobeat]

중복 로직 따로 추출해두면 좋을 것 같습니다~

[kdomo]

중복로직이라면 어떤 부분일까용?

[uwoobeat]

targetMember 쿼리하는 부분이 겹치는 것 같아서요~

[kdomo]

확인했슴니당

[uwoobeat]

existBy 쓰는 건 어떨까요?

[kdomo]

        MemberRelation memberRelation =
                memberRelationRepository
                        .findByFollowerIdAndFollowingId(currentMember.getId(), targetMember.getId())
                        .orElseThrow(() -> new CustomException(ErrorCode.FOLLOW_NOT_EXIST));

        memberRelationRepository.delete(memberRelation);

현재 코드가 MemberRelation이 존재하지 않을 때 예외를 발생시키는데, 존재할때 해당 객체로 바로 삭제를 진행하는데

existBy로 값이 없다면 예외를 터트리고, 있을 때 MemberRelation을 조회하는 쿼리가 한번 더 나가게되는데
요거에 대해서는 어떻게 생각하시나용?

[uwoobeat]

말씀하신대로 delete는 그렇고, save는 exist 써도 문제 없지 않나요?

[sonarqubecloud]

Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

38 New issues
0 Security Hotspots
53.2% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud