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.

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

refactor: 활동 멤버 및 지원서 조회 정렬 로직 변경 완료 #578

Merged
merged 8 commits into from
Oct 3, 2024

Conversation

SongJaeHoonn
Copy link
Contributor

@SongJaeHoonn SongJaeHoonn commented Sep 29, 2024

Summary

#577

활동 멤버 및 지원서 조회 api에서, 페이지에서만 정렬되도록 하는 것이 아닌, 전체 데이터를 대상으로 정렬을 한 후 페이지가 나눠지도록 변경했습니다.
또한, 정렬 시 리더가 최상단에 위치할 수 있도록 PagedResponseDto에 새로운 생성자를 만들고, 그에 따른 변경점을 수정했습니다.

작업 도중, 내가 지원한 활동 목록 조회, 나의 활동 목록 조회에서도 같은 문제를 발견해 수정했습니다.
해당 변경들은 프론트엔드 및 운영진분과 함께 의논하여 문제를 발견하고, 수정했습니다.

Tasks

  • 전체 데이터를 대상으로 정렬하도록 변경
  • 리더가 최상단에 위치할 수 있도록 변경

Results

{
  "success": true,
  "data": {
    "currentPage": 0,
    "hasPrevious": false,
    "hasNext": false,
    "totalPages": 1,
    "totalItems": 4,
    "take": 4,
    "items": [
      {
        "memberId": "leader",
        "memberName": "리더",
        "role": "LEADER",
        "status": "ACCEPTED",
        "applyReason": ""
      },
      {
        "memberId": "202500001",
        "memberName": "설윤",
        "role": "NONE",
        "status": "WAITING",
        "applyReason": "백엔드에 관심이 있어서"
      },
      {
        "memberId": "202500003",
        "memberName": "김민지",
        "role": "MEMBER",
        "status": "ACCEPTED",
        "applyReason": "백엔드에 관심이 있어서"
      },
      {
        "memberId": "202500002",
        "memberName": "오해원",
        "role": "NONE",
        "status": "REJECTED",
        "applyReason": "백엔드에 관심이 있어서"
      }
    ]
  }
}

@SongJaeHoonn SongJaeHoonn added the 🔨 Refactor 코드 수정 및 개선 label Sep 29, 2024
@SongJaeHoonn SongJaeHoonn self-assigned this Sep 29, 2024
@SongJaeHoonn SongJaeHoonn linked an issue Sep 29, 2024 that may be closed by this pull request
2 tasks
@limehee
Copy link
Collaborator

limehee commented Oct 2, 2024

PaginationUtils 내에 정의된 메소드들을 static으로 전환하는 것을 제안드립니다.
Util 클래스는 일반적으로 객체 상태와 무관하게 반복적으로 사용되는 공통 기능을 제공하기 때문에, 메소드를 static으로 선언하여 인스턴스 생성 없이 호출할 수 있도록 하는 것이 좋다고 생각해요. 특히, 정렬이나 슬라이싱과 같은 기능은 상태를 가지지 않으므로 static으로 처리하는 것이 적절하다고 생각해요.

@SongJaeHoonn
Copy link
Contributor Author

PaginationUtils 내에 정의된 메소드들을 static으로 전환하는 것을 제안드립니다.

좋은 말씀 감사합니다..!
유틸리티 클래스이므로, 어디서든 사용될 수 있는 공통 기능이기에 객체 생성 없이 간편하게 메서드를 사용할 수 있어 코드의 효율성과 간결성을 높일 수 있겠네요.
많이 배워갑니다!!

@limehee limehee merged commit 37e4474 into develop Oct 3, 2024
3 checks passed
@limehee limehee deleted the refactor/#577 branch October 3, 2024 12:19
@limehee limehee mentioned this pull request Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 Refactor 코드 수정 및 개선
Projects
None yet
Development

Successfully merging this pull request may close these issues.

활동 멤버 및 지원서 조회 정렬 변경
3 participants