Skip to content

[Refactor] - 핀하우스 공고리스트 / 공고 필터 / 역할 책임 분리 #375

@kyungchan3007

Description

@kyungchan3007

📌 코드 리펙토링 내용

• 관심사 분리 성과: ListingFilterPartialSheetHooks를 total 계산, 스크롤 상태, URL/전역 상태 동기화로 분리해서 책임이 명확해짐
• 유지보수성 개선 이유: 한 훅에 UI/라우팅/데이터 로직이 섞여 있던 구조를 쪼개서 변경 영향 범위를 줄임
• 안정성 강화 이유: handleScroll만 useCallback으로 고정해 useEffect가 불필요하게 재실행되는 상황을 방지
• 가독성 개선 성과: 주요 함수 동작 이유를 한글 주석으로 명시해서 팀 내 이해 비용 감소
• 구조 정리 성과: 컴포넌트에서는 훅 결과만 소비하도록 단순화하고, 훅 import를 로컬로 정리해 모듈 경계를 명확히 함

✅ 리펙토링 결과

• 책임 단위 분리로 로직 추적이 쉬워짐
• 스크롤/탭 동기화 동작 확인 및 향후 수정 범위 축소

  • 필터 시트 열기/닫기 동작 정상
  • 스크롤 하단 그라데이션 표시 정상
  • 탭 쿼리 파라미터 정리 정상
  • 하단 네비 노출/숨김 로직 영향 없음
  • listings 페이지 필터 상태 유지 정상

Metadata

Metadata

Assignees

Labels

Refactor코드 리팩토링 관련

Type

No type

Projects

Status

Done

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions