🔗 :: [#337] 연도 역순으로 변경

[circle0802]

개요

연도 역순으로 변경

작업사항

• 연도 역순으로 변경

UI

Summary by CodeRabbit

릴리스 노트

• 리팩터

  • 연도 필터 옵션의 정렬 순서를 조정하여 연도 선택 흐름을 역순으로 표시합니다.

• 스타일

  • 검색 입력 필드의 안내 문구를 더 직관적인 설명으로 변경했습니다.

[coderabbitai]

Walkthrough

ReviewFilter에서 연도 목록의 순서를 역순(2026→2020)으로 변경하고, 인덱스→연도 매핑 로직을 이에 맞게 수정했습니다. SearchReview에서는 검색 입력 필드의 플레이스홀더 문구를 교체했습니다.

Changes

Cohort / File(s)	변경 사항 요약
연도 옵션 순서 역순 변경 Projects/Presentation/Sources/ReviewFilter/ReviewFilterViewController.swift	연도 목록 초기화와 컬렉션 뷰 선택 처리에서 연도 배열을 2020...2026 → 2026...2020으로 역순 변경. 인덱스 기반 매핑(선택된 연도) 로직 업데이트.
검색어 플레이스홀더 변경 Projects/Presentation/Sources/SearchReview/SearchReviewViewController.swift	searchTextField의 placeholder 문자열을 "검색어를 입력해주세요"에서 "찾고 싶은 면접 후기를 입력해주세요"로 변경. 제어 흐름은 변경되지 않음.

Sequence Diagram(s)

sequenceDiagram
    participant UI as CollectionView (Year)
    participant VC as ReviewFilterViewController
    participant Relay as selectedYearRelay

    rect rgb(240,248,255)
    UI->>VC: user selects cell(index)
    note right of VC: years = [2026,2025,...,2020]
    end

    VC->>VC: year = years[index]  %% 인덱스→연도 매핑 (역순 반영)
    VC->>Relay: selectedYearRelay.accept(year)
    Relay-->>VC: (binding updates UI/state)

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

• Review focus: 인덱스→연도 매핑이 모든 선택/초기화 시 일관된지 확인
• 검토 권장 파일:
  • ReviewFilterViewController.swift (매핑/바인딩 확인)
  • SearchReviewViewController.swift (문구 변경 영향 검증)

Possibly related PRs

• 🔗 :: (#333) 리뷰 상세페이지 질문 리스트 글씨 잘림 버그 해결 #334 — 동일 파일(ReviewFilterViewController.swift)에 대한 이전 변경사항과 관련(선택 인덱스/초기화 로직).
• 🔗 :: (#323) 후기 리스트 조회 및 검색 #325 — SearchReviewViewController.swift에 대한 이전/유사 변경사항으로 플레이스홀더 변경과 연관.

Suggested reviewers

• leejh08
• cyj513
• juyeong525

Poem

🐰 연도를 뒤집어 놓았더니,
최신부터 차례차례 내려오네.
작은 인덱스 하나 바뀌었을 뿐인데,
선택의 방향이 새로워졌구나. ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	PR 제목 "🔗 :: [#337] 연도 역순으로 변경"은 변경사항의 주요 내용을 정확하게 반영하고 있습니다. ReviewFilterViewController에서 연도 옵션을 2020...2026에서 2026...2020으로 역순으로 변경한 것이 핵심 변경사항이며, 이는 제목에서 명확하게 표현되고 있습니다. 제목은 구체적이고 명확하여 팀원들이 커밋 히스토리를 스캔할 때 주요 변경사항을 쉽게 이해할 수 있습니다. 모든 세부사항을 포함할 필요는 없다는 지침에 따라, SearchReviewViewController의 placeholder 텍스트 변경 같은 부가적인 변경사항이 제목에 포함되지 않은 것은 문제가 되지 않습니다.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feature/(#337)-year_reversed

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4049f98 and d8ef17e.

📒 Files selected for processing (1)

• Projects/Presentation/Sources/SearchReview/SearchReviewViewController.swift (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: JOBIS_DSM_iOS | JOBIS v2 CI | Build - iOS
• GitHub Check: JOBIS_DSM_iOS | JOBIS v2 CI | Test - iOS

🔇 Additional comments (1)

Projects/Presentation/Sources/SearchReview/SearchReviewViewController.swift (1)

24-24: 플레이스홀더 텍스트 개선 확인!

검색 필드의 플레이스홀더를 "검색어를 입력해주세요"에서 "찾고 싶은 면접 후기를 입력해주세요"로 변경하여 사용자에게 더 명확한 안내를 제공합니다. 사용자 경험을 개선하는 좋은 변경입니다.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cyj513]

All File Checked ✅

✅	Auto Pull Request SwiftLint Success!!!!

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (1)

Projects/Presentation/Sources/ReviewFilter/ReviewFilterViewController.swift (1)

246-251: 연도 역순 정렬 로직이 정확하게 구현되었습니다.

최신 연도가 먼저 표시되도록 변경이 올바르게 적용되었습니다. 그러나 동일한 연도 배열 생성 로직이 line 263에도 중복되어 있습니다.

중복을 제거하기 위해 연도 배열을 프로퍼티로 추출하는 것을 권장합니다:

 public final class ReviewFilterViewController: BaseViewController<ReviewFilterViewModel> {
     private let filterApplyButtonDidTap = PublishRelay<Void>()
     private var selectedYear: String = ""
     private let selectedYearRelay = BehaviorRelay<String>(value: "")
     private var selectedJobIndex: Int?
     private var selectedYearIndex: Int?
+    private let years = (2020...2026).reversed().map { String($0) }

그런 다음 두 곳에서 이 프로퍼티를 사용하도록 변경:

     private func handleYearSelection(at indexPath: IndexPath) {
         if let previousIndex = selectedYearIndex,
            let previousCell = yearsCollectionView.cellForItem(at: IndexPath(item: previousIndex, section: 0)) as? ReviewMajorCollectionViewCell {
             previousCell.isCheck = false
         }

         if let cell = yearsCollectionView.cellForItem(at: indexPath) as? ReviewMajorCollectionViewCell {
             cell.isCheck = true
             selectedYearIndex = indexPath.item

-            let years = (2020...2026).reversed().map { String($0) }
             if indexPath.item < years.count {
                 selectedYear = years[indexPath.item]
                 selectedYearRelay.accept(selectedYear)
             }
         }
     }

     public override func configureViewController() {
         viewWillAppearPublisher.asObservable()
             .bind {
                 self.hideTabbar()
                 self.setSmallTitle(title: "필터 설정")
                 self.navigationController?.navigationBar.prefersLargeTitles = false
             }
             .disposed(by: disposeBag)

-        let years = (2020...2026).reversed().map { String($0) }
-
         Observable.just(years)
             .bind(to: yearsCollectionView.rx.items(

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5a3231d and 4049f98.

📒 Files selected for processing (1)

• Projects/Presentation/Sources/ReviewFilter/ReviewFilterViewController.swift (2 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)

• GitHub Check: JOBIS_DSM_iOS | JOBIS v2 CI | Test - iOS
• GitHub Check: JOBIS_DSM_iOS | JOBIS v2 CI | Build - iOS

🔇 Additional comments (1)

Projects/Presentation/Sources/ReviewFilter/ReviewFilterViewController.swift (1)

263-273: 이미 line 246-251에 대한 리뷰 코멘트에서 이 코드 중복에 대해 다루었습니다.

[cyj513]

All File Checked ✅

✅	Auto Pull Request SwiftLint Success!!!!