[Feat] #191 - 위치선택 지도에서 선택 기능 추가 #205

513sojin · 2023-11-13T14:53:27Z

🌱 작업한 내용

(tmap) 경도 위도 기반 주소 가져오는 api 연결
- 처음에 카카오 주소 api 연결했었는데 혹~시라도 나중에 쓰일 수 있을까봐 코드는 삭제해두지 않았습니다
지도에서 선택 | 현재 위치에서 선택 하는 기능 추가

자잘한 수정 사항

view에서 tap gesture를 쉽게 사용하기 위해 GesturePublisher, GestureSubscription 추가했습니다 (구글링해서 긁어옴ㅎㅎ)
BaseView를 추가했습니다. view 만들 때 상속해서 사용하면 편한데 앞으로 view를 추가로 만들 수 있을 것 같아서 추가했어요

🌱 PR Point

기존의 코스 그리기 화면은 위치 검색 -> 해당 위치에 마커 찍음 -> 코스 그리기 화면 의 순서로 진행되는데
지도에서 선택한 경우 위치 검색 -> 해당 위치에 마커 찍기가 아니라
위에는 마커를 띄워만 두고, 실제로 코스를 그리는 화면에서 마커를 찍게 됩니다

결론 : 지도에서 선택한 경우는 분기처리가 필요했다 !! 입니다.

따라서 크게 두 가지 경우로 나누어 생각했습니다

현재 위치에서 시작하기에서 진입 | 검색으로 진입
지도에서 선택하기로 진입

고민했던 부분

enum SelectedType {
    case other
    case map
}

이 코드를 검색 화면에 넣어두고 3개의 파일에서 사용했는데
여러 개의 파일에서 사용하기 때문에 더 상단에 넣어야할 것 같다는 고민을 마주했답니다 ..
결론적으로는 코드 업로드와 관련된 네트워크 파일에 넣으려고 하는데 어떤가요 ?!

현재는 SelectedType case 를 변수를 넘겨주는데 싱글톤 코드로 리팩토링 하겠습니다 ~

📸 스크린샷

구현 내용	스크린샷
검색 화면
지도에서 선택 화면	RPReplay_Final1699878718.2.mov

📮 관련 이슈

Resolved: [Feat] #191 - 현재 위치 / 지도에서 선택하기 기능 추가 #191

xcode 14.3 버전에서 생기는 에러 수정을 위한 코드

thingineeer

고생 많으셨습니다~~ 🔥🔥🔥🔥

CombineCocoa는 처음 보네요
UI 되게 깔끔하게 짜신 거 대박 멋지십니다 👍

중간에 보였던 오타와, 궁금했던 점 위주로 달았고, 고민하셨던 부분에 살짝 이해가 부족하여 추가 질문 남겼습니다.

얼른 코드 리뷰 반영하고, 머지 돼서 직접 코드 돌려서 보고 싶네요 ㅎㅎ
또한 긁어오셨던 부분은 조금 어려워서, 대면 또는 회의할 때 설명해 주시면 좋을 것 같아요!! 😅

...Runnect-iOS/Network/Dto/CourseDrawingDto/ResponseDto/KakakoAddressSearchingResponseDto.swift

Runnect-iOS/Runnect-iOS/Presentation/CourseDrawing/Views/LocationSelectView.swift

Runnect-iOS/Runnect-iOS/Global/Extension/UIKit+/UIGestureRecognizer+.swift

Runnect-iOS/Runnect-iOS/Presentation/CourseDrawing/VC/DepartureSearchVC.swift

thingineeer · 2023-11-13T17:43:45Z

Runnect-iOS/Runnect-iOS/Presentation/CourseDrawing/VC/CourseDrawingVC.swift

+                        do {
+                            let responseDto = try result.map(TmapAddressSearchingResponseDto.self)
+                            self.updateData(model: responseDto.toDepartureLocationModel(latitude: latitude, longitude: longitude))
+                        } catch {
+                            print(error.localizedDescription)
+                        }
+                    }


이 부분에 do~catch 인 것 처럼 다른 부분도 do-catch 사용 하셨는데
어떠한 에러를 잡아서 처리하기 위해 사용 하셨는지 궁금합니다 ! 🤔

또는 어떠한 에러가 발생하는 경우가 생길까요? 🙋‍♂️

파싱하는 부분에서 에러가 생길 수 있어서
여태 모든 서버 통신 함수는 do~catch 를 사용한 것 같고
저 또한 그대로(죄송합니다😇) 사용했는데요

다른 VC에서도 확인해보니 파싱 실패시 print(error.localizedDescription) 로 print를 찍어주는걸로만 예외처리를 해온 것 같네요
근데 이렇게 되면 사실상 예외처리를 한게 의미가 없어진다고는 생각합니다

이번 기능 연결하고 네트워크 쪽 리팩토링 진행해보고 싶은데 그때 또 코드리뷰 갈겨주십쇼..
예리한 질문 덕분에 저도 생각을 다시 해봤네요 꾸벅꾸벅

없는 곳도 있지만, 동시에 do-catch사용하는 부분이 있었는데요!

"그냥 코드들을 읽어보다가 여기는 없는데, 여기는 있네?" 라고 과거에 생각해 보다가 마침 소진님이 사용하시는 거 보고 궁금하여 질문을 해본 거였습니다 ㅎㅎ

저도 중복 되는 부분도 리팩토링 할 생각이 있는데, 2차 업데이트 끝나고 여유있으면 같이 작업해보아요~ 💪💪

thingineeer · 2023-11-13T17:51:32Z

Runnect-iOS/Runnect-iOS/Global/Base/BaseView.swift

+//
+//  BaseView.swift
+//  Runnect-iOS
+//
+//  Created by Sojin Lee on 2023/09/25.
+//
+
+import UIKit
+import SnapKit
+
+protocol BaseViewProtocol {
+    func setDelegate()
+    func setUI()
+    func setLayout()
+}
+
+class BaseView: UIView, BaseViewProtocol {
+    // MARK: - Initializer
+
+    override init(frame: CGRect) {
+        super.init(frame: frame)
+
+        setDelegate()
+        setUI()
+        setLayout()
+    }
+
+    @available(*, unavailable)
+    required init?(coder _: NSCoder) {
+        fatalError("init(coder:) has not been implemented")
+    }
+
+    // MARK: - Setup Methods
+
+    func setDelegate() { }
+
+    func setUI() { }
+
+    func setLayout() { }
+}
+


오.. 리팩터링할때 정말 유용할 것 같습니다 👍👍👍

Runnect-iOS/Runnect-iOS/Presentation/CourseDrawing/VC/DepartureSearchVC.swift

지도 선택 | 검색 | 현재 위치 설정 하는 경우는 세 개의 파일에서 사용되기 때문에 싱글톤으로 수정하였습니다

thingineeer

싱글톤 굿 잡 👍

thingineeer · 2023-11-14T12:44:29Z

Runnect-iOS/Runnect-iOS/Network/Router/CourseDrawingRouter/DepartureSearchingRouter.swift

+final class SelectedInfo {
+    static let shared = SelectedInfo()
+    var type: SelectedType = .other
+
+    private init() {}


thingineeer · 2023-11-14T12:48:44Z

Runnect-iOS/Runnect-iOS/Presentation/CourseDrawing/VC/CourseDrawingVC.swift

+                        do {
+                            let responseDto = try result.map(TmapAddressSearchingResponseDto.self)
+                            self.updateData(model: responseDto.toDepartureLocationModel(latitude: latitude, longitude: longitude))
+                        } catch {
+                            print(error.localizedDescription)
+                        }
+                    }


없는 곳도 있지만, 동시에 do-catch사용하는 부분이 있었는데요!

"그냥 코드들을 읽어보다가 여기는 없는데, 여기는 있네?" 라고 과거에 생각해 보다가 마침 소진님이 사용하시는 거 보고 궁금하여 질문을 해본 거였습니다 ㅎㅎ

저도 중복 되는 부분도 리팩토링 할 생각이 있는데, 2차 업데이트 끝나고 여유있으면 같이 작업해보아요~ 💪💪

thingineeer · 2023-11-14T12:51:51Z

Runnect-iOS/Runnect-iOS/Presentation/CourseDrawing/VC/DepartureSearchVC.swift

-/// 현재 위치 | 검색 | 지도에서 선택 중 분기처리를 해주기 위함
-enum SelectedType {
-    case other
-    case map
-}
-


확실히 깔끔쓰 하네요

thingineeer · 2024-02-03T08:00:53Z

.../Runnect-iOS/Network/Dto/CourseDrawingDto/ResponseDto/KakaoAddressSearchingResponseDto.swift

+    let documents: [Document]
+
+    func toDepartureLocationModel(latitude: Double, longitude: Double) -> DepartureLocationModel {
+        let roadName = self.documents[0].roadAddress.buildingName ?? "내가 설정한 출발지"


@513sojin
여기는 코드를 보면 중간에 옵셔널일 경우가 없어서 그냥 없애거나 중간에 옵셔널 타입을 넣어야겠습니다!

생각해보면 아무 것도 안온다고 해도 nil이 아니라 빈 문자열이 오겠네요 ... 조언 감사합니당 !! 근데 요 친구 어차피 사용 안해서 없애줄게요 ㅎ.ㅎ

네네 제가 없앨게요~~

513sojin added 23 commits September 25, 2023 20:50

[Add] Runnect#191 - 아이콘 추가

ffd8ada

[Feat] Runnect#191 - 현재 위치 혹은 지도 선택 UI 구현

074eba6

[Feat] Runnect#191 - BaseView 추가

58b23f5

[Feat] Runnect#191 - combineCocoa 설치

442c834

[Feat] Runnect#191 - view에 tapgesture 추가

bb600b2

[Feat] Runnect#191 - 위도 경도에 따른 주소 가져오는 api 연결

1a8cb86

[Feat] Runnect#191 - tmap api 연결

f261df1

[Feat] Runnect#191 - 지도에서 선택시 마커 추가

b641edf

[Feat] Runnect#191 - 지도 위치에 따라 위치 정보 변경

0e733d9

[Feat] Runnect#191 - 지도에서 선택한 경우 분기처리

90a10bc

[Feat] Runnect#191 - 코스 이름 바텀시트 UI 구현

1f77f0f

[Feat] Runnect#191 - textfield 이벤트 추가

c457fcf

Merge branch 'develop' into Runnect#191-위치선택

ed1874c

[Feat] Runnect#191 - 주석 추가

e77759e

[Del] Runnect#191 - 바텀시트VC 삭제

2b98db7

[Fix] Runnect#191 - 카카오 주소 찾기 api 삭제

e7b6a3a

[Feat] Runnect#191 - 지도에서 선택된 경우 네비바 문구 수정

79aa673

[Feat] Runnect#191 - 지도에서 선택시 안내 문구 뷰 추가

0e21db2

[Feat] Runnect#191 - 라이브러리 업데이트

0b5232a

Merge branch 'develop' into Runnect#191-위치선택

02c6365

Merge branch 'develop' into Runnect#191-위치선택

78c4a15

[Fix] podfile 수정

63bb063

xcode 14.3 버전에서 생기는 에러 수정을 위한 코드

[Fix] Runnect#191 - project 파일 변경 및 접근제한자 수정

2ba2da7

513sojin requested review from thingineeer and lee-yeonwoo November 13, 2023 14:54

513sojin self-assigned this Nov 13, 2023

513sojin added the 소진🍊 label Nov 13, 2023

513sojin changed the title ~~#191 위치선택~~ [Feat] #191 - 위치선택 지도에서 선택 기능 추가 Nov 13, 2023

thingineeer reviewed Nov 13, 2023

View reviewed changes

[Fix] Runnect#191 - 코드리뷰 반영

1367f1f

[Fix] Runnect#191 - 싱글톤으로 수정

4c5966f

지도 선택 | 검색 | 현재 위치 설정 하는 경우는 세 개의 파일에서 사용되기 때문에 싱글톤으로 수정하였습니다

thingineeer approved these changes Nov 14, 2023

View reviewed changes

thingineeer reviewed Nov 14, 2023

View reviewed changes

513sojin merged commit c7a8d73 into Runnect:develop Nov 14, 2023
1 check passed

thingineeer reviewed Feb 3, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Feat] #191 - 위치선택 지도에서 선택 기능 추가 #205

[Feat] #191 - 위치선택 지도에서 선택 기능 추가 #205

513sojin commented Nov 13, 2023

thingineeer left a comment •

edited

Loading

thingineeer Nov 13, 2023

513sojin Nov 14, 2023

thingineeer Nov 14, 2023

thingineeer Nov 13, 2023

thingineeer left a comment

thingineeer Nov 14, 2023

thingineeer Nov 14, 2023

thingineeer Nov 14, 2023

thingineeer Feb 3, 2024

513sojin Feb 3, 2024

thingineeer Feb 3, 2024

[Feat] #191 - 위치선택 지도에서 선택 기능 추가 #205

[Feat] #191 - 위치선택 지도에서 선택 기능 추가 #205

Conversation

513sojin commented Nov 13, 2023

🌱 작업한 내용

🌱 PR Point

고민했던 부분

📸 스크린샷

📮 관련 이슈

thingineeer left a comment • 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

thingineeer 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

thingineeer left a comment •

edited

Loading