Skip to content

Conversation

@bbbbooo
Copy link
Member

@bbbbooo bbbbooo commented Jan 3, 2025

#️⃣연관된 이슈

ex) #이슈번호, #이슈번호

  • close #이슈번호

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • 팔로우 존재 확인하는 쿼리문에 전달할 파라미터 순서 변경했습니다.
  • 쿼리문 순서를 변경하는게 맞겠지만 소희님 코드와도 연관돼있어서 우선 제가 전달하는 파라미터 순서를 변경했습니다.

스크린샷 (선택)

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메소드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

그 쿼리문쪽 수정사항을 제가 잘 못 전달한거 같습니다.

    @Query(
            "SELECT COUNT(f) > 0 " +
                    "FROM Follow f " +
                    "WHERE f.follower.userId = :targetUserId " +
                    "AND f.followee.userId = :userId "
    )
    boolean existsByFollower_FollowerIdAndFollowee_FolloweeId(@Param("userId")int userId, @Param("targetUserId")int targetUserId);

이 부분 메소드명에 의하면 followerId, followeeId 순으로 찾게끔 되어있으나 실제로는 followeeId 파라미터를 씁니다. 그래서 변경이 된다면

    @Query(
            "SELECT COUNT(f) > 0 " +
                    "FROM Follow f " +
                    "WHERE f.follower.userId = :userId" +
                    "AND f.followee.userId = :targetUserId"
    )
    boolean existsByFollower_FollowerIdAndFollowee_FolloweeId(@Param("userId")int userId, @Param("targetUserId")int targetUserId);

이렇게 쓰는게 메소드 의도와 실제 쿼리가 일치해요. 혹은

    @Query(
            "SELECT COUNT(f) > 0 " +
                    "FROM Follow f " +
                    "WHERE f.follower.userId = :follwerId" +
                    "AND f.followee.userId = :follweeId"
    )
    boolean existsByFollower_FollowerIdAndFollowee_FolloweeId(@Param("follwerId")int follwerId, @Param("follweeId")int follweeId);

이런식으로 헷갈리지 않게 필드명을 일치시키는 것도 방법인 것 같습니다.

@bbbbooo bbbbooo requested a review from soheeparklee January 3, 2025 02:32
@bbbbooo bbbbooo self-assigned this Jan 3, 2025
@bbbbooo bbbbooo merged commit 042cef6 into develop Jan 3, 2025
1 check passed
@bbbbooo bbbbooo deleted the feature/follow branch January 3, 2025 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants