[✨ feat] 그룹 정보 보기 API 구현

[jsoonworld]

🚀 What’s this PR about?

• 작업 내용 요약: 그룹 정보 보기 API를 구현하였습니다.
• 핵심 작업: QueryDSL 도입 및 조회 기능 최적화

🛠️ What’s been done?

• 주요 변경 사항:
  • 그룹 정보 보기 API 구현
  • Response DTO가 많아짐에 따라 API별 DTO를 구분하기 위해 패키지를 분리
  • QueryDSL 도입 (다음과 같은 이점 확보)
    • 쿼리 최적화 및 성능 개선
    • 동적 쿼리 작성의 편의성
    • 가독성과 유지보수성 향상

🧪 Testing Details

• 테스트 코드 및 결과:
  • 그룹 정보 조회 API의 동작을 검증하는 테스트 코드를 작성하였습니다.
  • 멤버, 멤버그룹, 그룹 간의 연관 관계를 설정하는 코드를 가독성 있게 개선하였습니다.
  • 기존 테스트와 새로 추가된 테스트가 모두 정상적으로 동작함을 확인하였습니다.

👀 Checkpoints for Reviewers

• 리뷰 시 확인할 사항:
  • DTO 네이밍에 신경을 많이 썼는데, 적절한지 확인 후 피드백 부탁드립니다.
  • 그룹 정보를 반환하는 과정에서 groupInfo가 API 네이밍과 중복될 가능성이 있어 groupSummary로 변경하였습니다.
  • QueryDSL 적용이 적절한지 검토해 주시면 감사하겠습니다.

🎯 Related Issues

• [feat] 그룹 정보 보기 API 구현 #54

[0-tae]

고생 많으셨어요. 요청하신 부분과 함께 궁금한 점 남기고 마치겠습니다.

네이밍은 적절하게 잘 변형된 것 같습니다. GroupInfo처럼, ~Info 접미사를 통해서 반환하고자 하는 도메인 정보의 최상단으로 명시하고, Summary는 바꾸신 것 처럼 중간 요소를 의미하는 것으로 규칙을 고정하면 될 것 같습니다.

QueryDSL은 쿼리 최적화에 뛰어난 것 같습니다. CRUD 기능 개발을 자주 하게 된다면 유지보수가 좋다는 장점도 부각될 것 같습니다☺️

[0-tae]

findMembersByGroupId -> findMemberByGroupIdAndMemberId처럼 WHERE 조건 절에 MemberId와 GroupId가 일치하는 확인하는 방법으로 해결할 수 없을까요?

조건부 연산을 DBMS에서 수행할 수 있으면, DBMS에서 넘어오는 데이터에 대한 Bandwidth의 부담을 줄일 수 있습니다!

[jsoonworld]

피드백 감사합니다! 😊

현재 findMembersByGroupId(groupId)를 호출한 후 Java 스트림을 이용해 memberId를 필터링하고 있는데, QueryDSL으로 구현된 로직을 개선하면 해당 필터링을 DB에서 직접 수행할 수 있을 것 같습니다.

제안해주신 대로 findMemberByGroupIdAndMemberId(groupId, memberId)와 같은 형태로 WHERE 조건을 적용하면, 불필요한 데이터를 애플리케이션으로 가져오지 않고 DBMS가 직접 필터링을 수행하여 네트워크 및 메모리 사용량을 줄일 수 있을 것 같습니다.

예를 들어, 현재 findMembersByGroupId(groupId)는 다음과 같이 모든 멤버를 가져옵니다:

public List<Member> findMembersByGroupId(Long groupId) {
    return queryFactory
        .select(member)
        .from(memberGroup)
        .join(memberGroup.member, member)
        .where(memberGroup.group.groupId.eq(groupId))
        .fetch();
}

이후 Java 스트림을 사용하여 필터링하고 있는데, QueryDSL에서 직접 처리하는 방식으로 변경하면 더 효율적일 것입니다:

public Optional<Member> findMemberByGroupIdAndMemberId(Long groupId, Long memberId) {
    return Optional.ofNullable(
        queryFactory
            .select(member)
            .from(memberGroup)
            .join(memberGroup.member, member)
            .where(memberGroup.group.groupId.eq(groupId)
                .and(member.memberId.eq(memberId)))
            .fetchOne()
    );
}

이렇게 하면 DB에서 직접 groupId와 memberId 조건을 적용해 검색하므로 불필요한 데이터 전송을 방지할 수 있을 것 같아요!

따라서 findMemberInGroup 메서드를 다음과 같이 변경하는 것이 더 적절할 것 같습니다:

private Member findMemberInGroup(Long memberId, Long groupId) {
    return memberGroupRepository.findMemberByGroupIdAndMemberId(groupId, memberId)
            .orElseThrow(() -> new GroupException(GroupErrorCode.MEMBER_NOT_FOUND));
}

이렇게 하면 전체 데이터를 불러온 후 필터링하는 것이 아니라, DB에서 직접 원하는 데이터를 가져오기 때문에 성능상 더 유리겠어요!

이 부분은 이슈로 등록하고 개선하겠습니다!

좋은 피드백 감사합니다! 🙌 🚀

[0-tae]

👍👍👍