Feat/#172 검색로직 수정 #177

hooni0918 · 2025-01-31T06:25:32Z

🔗 연결된 이슈

close: [feat] 검색후 포커싱 #172

📄 작업 내용

검색시 해당 지역 포커싱
검색시 해당지역 맛집들만 리스트업

구현 내용	IPhone 15 pro	IPhone SE
GIF

💻 주요 코드 설명

�검색 화면에서 특정 위치 상세 화면으로 이동할 때, 검색 화면은 스택에서 제거하고 위치 상세 화면을 새로 추가하는 로직
검색 pop 하자마자 push하는 로직입니다. 이게 없으면 mapPath에서 경로를 못찾아서 fatalError가 납니다

func navigateToSearchLocation(locationId: Int, locationTitle: String) {
        if let lastView = mapPath.last,
           case .searchView = lastView {
            pop(1)
        }

        push(.searchLocationView(
            locationId: locationId,
            locationTitle: locationTitle
        ))
    }

HomeViewModel

주석으로 로직 대체

👀 기타 더 이야기해볼 점

API 추가 + 기존 한 파일에 있던 코드들을 각 구조체별로 분리 + 동일한 UI를 가진 바텀시트 추가 가 되어 코드 뻥튀기가 심함니다. 천천히 슥슥 넘기면서 보셔도 무방합니다.
슥슥 넘기고 네비게이션매니저 파일부터 보셔도 됩니다

Actor클래스는 불필요해서 일부로 다시 전부 제외햇습니다. 추후 리팩토링때 반영할게요!

카메라 시점 해제시 그위치 그대로

thingineeer · 2025-02-01T13:24:44Z

Spoony-iOS/Spoony-iOS/Network/Service/HomeService/HomeService.swift

+                        do {
+                            let responseDto = try response.map(BaseResponse<ResturantpickListResponse>.self)
+                            guard let data = responseDto.data else {
+                                throw NSError(domain: "HomeService", code: -1, userInfo: [NSLocalizedDescriptionKey: "No data available"])


이건 무슨 에러 인가요??

fetchLocation 에 대한 기본 에러타입 지정을 안해서 기본 NSError로 전부 빼놓았습니다 (fatalerror같은종류,,)
해당 코드 에러케이스 추가해서 넣어두겟습니다

thingineeer · 2025-02-01T13:27:23Z

Spoony-iOS/Spoony-iOS/Resource/Components/BottomSheet/Plain/BottomSheetItem.swift

+                    defaultPlaceholder
+                case .empty:
+                    defaultPlaceholder
+                @unknown default:


여기도 그렇고, 다른 곳도 그렇고 AsyncImage 에 @unknown 키워드를 꼭 붙여 줘야하나요?

저번에 말햇듯 당시 린트에러인줄 모르고 붙여놧던것들입니다

thingineeer · 2025-02-01T13:34:15Z

Spoony-iOS/Spoony-iOS/Resource/Components/BottomSheet/Plain/BottomSheetList.swift

+                        Color.clear.frame(height: 90.adjusted)
+                    }
+                }
+                .coordinateSpace(name: "scrollView")


제가 봤을때는, coordinateSpace 활용하는 곳이 없는 것 처럼 보이는데, 어디에서 활용하나요??

원래는 Geometry 를 사용할떄 같이 부모뷰(여기서는 스크롤뷰) 의 위치를 보고 움직여야할때 사용이 되는데요 기존 코드상 바텀시트 중간너비에서 스크롤이 될때 사용햇던 코드인데 해당 부분이 빠지면서 남아있는 코드입니다 삭제해둘게요

thingineeer · 2025-02-01T13:39:25Z

Spoony-iOS/Spoony-iOS/Source/Feature/Home/HomeViewModel.swift

앱잼때는 안 물어봤는데, 파일 ViewModel로 네이밍을 하신 건 실수인가요? 아니면 의도인가요

어떤의도의 질문인지 모르겟어요

왜 뷰모델을 사용해서 MVVM을 사용한 것인가

ViewModel 이라는 말이 이상한것인가
1번이라면 이전 PR 설명에서 홈에서의 VIew-Only 로직이 너무 복잡해서 일단 간단하게나마 뷰모델에 분리를 햇다 하지만 MVVM은 아니다 라고 말씀드렷고 2번인 경우에 어떤이름으로 바꾸는게 좋을까요?

왜 여기만 ViewModel이 있는 것인가에 대한 질문이었는데, 로직이 너무 복잡해서 분리를 하셨군요

원래 아키텍처라면 저희는 MVI를 따라야 하는데 여기는 MVI가 아니어서 여쭤본 겁니다!

thingineeer · 2025-02-01T13:40:05Z

Spoony-iOS/Spoony-iOS/Source/Feature/Home/HomeViewModel.swift

+    func fetchLocationList(locationId: Int) async {
+            isLoading = true
+            do {
+                clearFocusedPlaces()
+                selectedLocation = nil
+
+                // API 호출하여 새 데이터 받아오기
+                let response = try await service.fetchLocationList(
+                    userId: Config.userId,
+                    locationId: locationId
+                )
+
+                // 데이터가 성공적으로 받아와진 후에만 기존 리스트 교체
+                self.pickList = response.zzimCardResponses
+
+                // 첫 번째 장소가 있다면 지도 중심점 이동
+                if let firstPlace = response.zzimCardResponses.first {
+                    self.selectedLocation = (firstPlace.latitude, firstPlace.longitude)
+                }
+            } catch {
+                self.error = error
+                print("Error in fetchLocationList:", error)
+                // 에러 발생 시 기존 데이터 유지
+            }
+            isLoading = false


여기 함수 블럭에는 Task { ... } 가 왜 없나요?

async 를 사용한 함수이기에 사용할 필요가 없습니다.
Task 를 사용할때에는 비동기 컨텍스트가 아닐때 해당 컨텍스트를 비동기로 받겟다 라는 의미이기에 해당 코드에서는 필요가 없습니다.
단, 이렇게 async 로 지정한다면 하위 컨텍스트가 모두 비동기로 지정되지만 해당 뷰모델 자체가 비동기로 돌아가야해서 큰 상관은 없을것같아 사용햇습니다

thingineeer · 2025-02-01T13:49:09Z

Spoony-iOS/Spoony-iOS/Source/Feature/Search/SearchStore.swift

예를 들어 성수동, 성수 등 을 입력했는데, 서버 호출에 빈값이 와서, 이 부분 서버분들에게 건의해야 할 것 같습니다.!
(서버 호출에 대한 문제는 아님)

해당부분은 서버 구현이 안된게 맞습니다 현재 서버에서 불러올수있는값은 마포 강남 의 경우에만 가능할거에요

ChoiAnYong

고생하셨습니다!

ChoiAnYong · 2025-02-02T07:40:45Z

Spoony-iOS/Spoony-iOS/Network/Model/ResturantpickListResponse.swift

+        return lhs.placeId == rhs.placeId &&
+               lhs.placeName == rhs.placeName &&
+               lhs.placeAddress == rhs.placeAddress &&
+               lhs.postTitle == rhs.postTitle &&
+               lhs.photoUrl == rhs.photoUrl &&
+               lhs.latitude == rhs.latitude &&
+               lhs.longitude == rhs.longitude &&
+               lhs.categoryColorResponse == rhs.categoryColorResponse


placeId가 고유한 값이 아닌가요?
고유한 값이라면 나머지 값들도 비교하는 이유가 있을까요??

지금은 수정이 되었는데 해당 코드 작성시에 해당 placeID 와 식당 정보들이 매핑이 이상하게 내려오고 있었습니다.
현재는 수정되어있으니 placeID가 유일한 코드가 맞아요!

ChoiAnYong · 2025-02-02T07:43:56Z

Spoony-iOS/Spoony-iOS/Resource/Components/BottomSheet/Plain/BottomSheetItem.swift

+                                    .frame(width: 16, height: 16)
+                            default:
+                                Color.clear
+                                    .frame(width: 16, height: 16)


adjusted 넣어주세요!

ChoiAnYong · 2025-02-02T07:44:18Z

Spoony-iOS/Spoony-iOS/Resource/Components/BottomSheet/Plain/BottomSheetItem.swift

+                    .padding(.vertical, 12)
+                    .padding(.horizontal, 12)


.padding(12)로 해도 될 것 같습니다!

ChoiAnYong · 2025-02-02T07:50:55Z

Spoony-iOS/Spoony-iOS/Resource/Components/BottomSheet/SearchResult/SearchLocationCardItem.swift

+                                        .frame(width: 16, height: 16)
+                                default:
+                                    Color.clear
+                                        .frame(width: 16, height: 16)


adjusted 넣어주세요!

ChoiAnYong · 2025-02-02T07:51:44Z

Spoony-iOS/Spoony-iOS/Resource/Components/BottomSheet/SearchResult/SearchLocationCardItem.swift

+                        .padding(.vertical, 12)
+                        .padding(.horizontal, 12)


여기도 padding(12)로 해도 될 것 같습니다!

ChoiAnYong · 2025-02-02T07:52:39Z

Spoony-iOS/Spoony-iOS/Resource/Tab/NavigationManager.swift

@@ -16,7 +16,7 @@ final class NavigationManager: ObservableObject {

    @Published var popup: PopupType?

-    @ViewBuilder
+    @MainActor @ViewBuilder


@mainactor를 추가하신 이유가 뭔가요??

원래는 해당 builder 에서 뷰 전환 로직을 해당 builder에서 모두 가진채로 화면 전환이 필요해서 권한을 넣었을때 줫던 코드입니다.
해당 코드에서는 필요없는게 맞습니다. 지우도록할게요

ChoiAnYong · 2025-02-02T08:02:14Z

Spoony-iOS/Spoony-iOS/Source/Feature/Home/HomeViewModel.swift

@@ -9,7 +9,7 @@ import Foundation

 @MainActor
 final class HomeViewModel: ObservableObject {
-    private let service: HomeServiceType
+    let service: HomeServiceType


외부에서 사용하는 곳이 없는 것 같은데 private를 쓰지 않은 이유가 있나요?

juri123123

고생하셨습니다!

juri123123 · 2025-02-06T05:04:58Z

Spoony-iOS/Spoony-iOS/Resource/Components/BottomSheet/Plain/BottomSheetList.swift

+            .background(.white)
+            .cornerRadius(10, corners: [.topLeft, .topRight])
+            .offset(y: UIScreen.main.bounds.height - currentStyle.height + offset)
+            .gesture(


simultaneousGesture랑 그냥 gesture의 차이가 있나요?

juri123123 · 2025-02-06T05:05:38Z

Spoony-iOS/Spoony-iOS/Resource/Components/BottomSheet/SearchResult/SearchLocationCardItem.swift

+                case .failure:
+                    defaultPlaceholder
+                case .empty:
+                    defaultPlaceholder
+                default:
+                    defaultPlaceholder


하나로 묶어도 좋을 것 같아요 ! 개인취향..

juri123123 · 2025-02-06T05:07:47Z

Spoony-iOS/Spoony-iOS/Resource/Tab/NavigationManager.swift

@@ -72,4 +74,16 @@ final class NavigationManager: ObservableObject {
        }
    }

+    func navigateToSearchLocation(locationId: Int, locationTitle: String) {


변수 currentLocation이랑 locationTitle이 다른 역할을 하나요 ?

hooni0918 added 11 commits January 25, 2025 04:59

feat: #172 검색 API 불러오기 로직 추가

7bc8559

feat: #172 카메라 시점 고정

93d6fef

feat: #172 카메라 시점 고정

2a0a6c8

카메라 시점 해제시 그위치 그대로

feat: #172 마커 되돌리기 수정

00680d7

feat: #172 바텀시트 분리

173a39d

feat: #172 API까지는 불러옴

3f57ab4

feat: #172 actor 분기시 짝수번째에만 호출

3e60757

feat: #172 로직 완성

913844d

feat: 검색 바텀시트 연결

18d49a4

feat: label 수정

15910c5

feat: #172 로직 재수정

1dd96e1

hooni0918 added feat 기능구현시 사용 Jihoon 나는지훈 labels Jan 31, 2025

hooni0918 requested review from juri123123, thingineeer and ChoiAnYong January 31, 2025 06:25

hooni0918 self-assigned this Jan 31, 2025

thingineeer requested changes Feb 1, 2025

View reviewed changes

hooni0918 added 2 commits February 2, 2025 16:20

feat: #172 unknown 키워드 삭제

b65889f

feat: #172 coordinateSpace 삭제

3eb5e2f

ChoiAnYong approved these changes Feb 2, 2025

View reviewed changes

thingineeer approved these changes Feb 2, 2025

View reviewed changes

fix: #172 코리반영

eb6ff6b

juri123123 approved these changes Feb 6, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feat/#172 검색로직 수정 #177

Feat/#172 검색로직 수정 #177

hooni0918 commented Jan 31, 2025

thingineeer Feb 1, 2025 •

edited

Loading

hooni0918 Feb 2, 2025

thingineeer Feb 1, 2025

hooni0918 Feb 2, 2025

thingineeer Feb 1, 2025

hooni0918 Feb 2, 2025

thingineeer Feb 1, 2025

hooni0918 Feb 2, 2025

thingineeer Feb 2, 2025

thingineeer Feb 1, 2025

hooni0918 Feb 2, 2025

thingineeer Feb 1, 2025

hooni0918 Feb 2, 2025

ChoiAnYong left a comment

ChoiAnYong Feb 2, 2025

hooni0918 Feb 5, 2025

ChoiAnYong Feb 2, 2025

ChoiAnYong Feb 2, 2025

ChoiAnYong Feb 2, 2025

ChoiAnYong Feb 2, 2025

ChoiAnYong Feb 2, 2025

hooni0918 Feb 5, 2025

ChoiAnYong Feb 2, 2025

juri123123 left a comment

juri123123 Feb 6, 2025

juri123123 Feb 6, 2025

juri123123 Feb 6, 2025

Feat/#172 검색로직 수정 #177

Are you sure you want to change the base?

Feat/#172 검색로직 수정 #177

Conversation

hooni0918 commented Jan 31, 2025

🔗 연결된 이슈

📄 작업 내용

💻 주요 코드 설명

👀 기타 더 이야기해볼 점

thingineeer Feb 1, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ChoiAnYong left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

juri123123 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

thingineeer Feb 1, 2025 •

edited

Loading