Style/#135 빈 화면 구현

[wotjs020708]

🔗 연결된 이슈

• Closed: [style] 빈 화면 구현 #135

📄 작업 내용

• 어쩌구저쩌구

구현 내용	IPhone 16 pro	IPhone 13 mini
GIF

[coderabbitai]

📝 Walkthrough

Walkthrough

TreatmentFilterView에 빈 상태 UI를 추가했습니다. treatments 목록이 비어있을 때 이미지와 안내 텍스트를 표시하고, 화면 로드 시 초기 데이터 조회를 트리거하는 .task 수정자를 추가했으며, 새로운 asset을 추가했습니다.

Changes

Cohort / File(s)	변경 사항
TreatmentFilterView 빈 상태 UI 구현 Cherrish-iOS/.../TreatmentFilterView.swift	treatments 목록 조건부 렌더링 추가: 빈 목록일 시 filterEmptyView 표시. .task 수정자로 화면 로드 시 fetchTreatments 호출. 빈 상태 UI를 위한 private extension 추가 (이미지, spacer, 안내 텍스트 포함). 기존 ForEach에 수평 padding 추가.
새로운 Asset 파일 추가 Cherrish-iOS/.../treatment_icon/Contents.json, Cherrish-iOS/.../treatment_icon/illustration_nosearch.imageset/Contents.json	빈 상태 화면용 asset catalog 구조 추가. illustration_nosearch.svg 이미지 리소스 정의.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• Feat/#114 시술 필터 API 연결 , 다운 타임 설정 구현 #122: 동일한 TreatmentFilterView에 .task 추가 및 treatments 렌더링 방식 변경
• Refactor/#103 코드 리뷰 반영 #107: TreatmentFilterView.swift의 UI 변경 사항
• Style/#76 시술 플로우 화면 구현 #83: TreatmentFilterView 구현 추가 및 빈 상태 UI 개선

Suggested reviewers

• sum130
• y-eonee
• soseoyo12

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목 'Style/#135 빈 화면 구현'은 빈 화면(empty state) 구현이라는 주요 변경 사항을 명확하게 전달하며, 연결된 이슈 #135와 일치한다.
Description check	✅ Passed	PR 설명에서 연결된 이슈 #135을 명시하고 구현 내용과 스크린샷을 포함하여 변경사항과 관련된 내용을 담고 있다.
Linked Issues check	✅ Passed	PR의 코드 변경 사항(TreatmentFilterView에 빈 상태 UI 추가, filterEmptyView 구현, 이미지 자산 추가)이 이슈 #135의 요구사항인 '빈 화면 추가'를 충족한다.
Out of Scope Changes check	✅ Passed	모든 변경 사항이 빈 화면(empty state) 구현과 관련되어 있으며, 연결된 이슈 #135의 범위 내에 있다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ 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 style/#135-emptyview

---

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.}

[soseoyo12]

수고하셨습니다.

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

Cherrish-iOS/Cherrish-iOS/Presentation/Feature/Calendar/Treatment/View/Treatment/TreatmentFilterView.swift (1)

62-65: .task 안에서 중첩 Task 생성은 피하세요

중첩 Task는 뷰 소멸 시 취소가 전파되지 않아 불필요한 fetch가 남을 수 있고, 에러도 버려집니다. .task 본문에서 직접 await 하고 에러를 처리하는 방식으로 바꾸는 게 안전합니다.

✅ 수정 제안

-        .task {
-            Task {
-                try await viewModel.fetchTreatments()
-            }
-        }
+        .task {
+            do {
+                try await viewModel.fetchTreatments()
+            } catch {
+                // TODO: 에러 처리(토스트/알럿/로그 등)
+            }
+        }

[sum130]

어푸푸 굿입니당