[Feat] #317 - 식당 정보 다를 시 제보 화면 구현

[EunsuSeo01]

🔥 Pull requests

👷 작업한 내용

• [3차 스프린트] 식당 상세 정보가 다를 시 제보하는 화면을 구현하였습니다.
• 이 PR로서 맡은 뷰는 완료고... [Feat] 식당 상세 지도뷰에 데이터 연동하기 #319 에서 지도에 데이터 연결, 주소 불러오는 것만 하면 끝입니다!

🚨 참고 사항

• @mcrkgus 엄마 제가 HomeVM에 코드를 추가했습니도 (주요 코드 설명 참고)
• 그리고 원래 홈 맵뷰 클릭할 때 실행되던 코드들을 showHankkiListBottomSheet 함수로 묶어서 관리하도록 했습니다.

📸 스크린샷

구현 내용	SE	13 mini	15 pro
GIF

🖥️ 주요 코드 설명

• 식당 삭제가 성공하면 Alert가 뜸 -> 확인 버튼 누르면 이전 화면으로 돌아가야 함
• dismiss 1번에 pop을 2번 해서 나가야 하는데 이건 popToThreeStepsBack 함수 만들어서 해결함
• 그런데 홈에서 상세로 들어왔다가 나가는 경우에는 나가도 그 삭제된 식당의 썸네일(포커싱 되어 있다는 뜻)이 뜨는 게 문제 -> 썸네일 없애고 전체 식당 리스트 리로드 하고 바텀 시트를 위로 올려야 됨
• 그래서 HomeVM을 건드렸습니다. contains 함수를 통해 포커싱 하고 있는 썸네일이 삭제된 식당인지 확인하고, 맞으면showHankkiListBottomSheet 클로저를 호출하여 HomeVC에서 전체 식당 리스트 바텀 시트를 올려줍니다!

HomeViewModel

// 포커싱하고 있는 식당이 삭제된 식당인지 아닌지 확인
func checkThumbnailHankkiValidation() {
        let isValid = hankkiLists.contains { $0.id == hankkiThumbnail?.id ?? 0 }
        if !isValid {
            showHankkiListBottomSheet?()
        }
}

HomeViewController

viewModel.hankkiListsDidChange = { [weak self] data in
            guard let self = self else { return }

            self.viewModel.checkThumbnailHankkiValidation() // 한끼 리스트 변경될 때 썸네일 validation 확인
...

viewModel.showHankkiListBottomSheet = {
            self.showHankkiListBottomSheet()
}

func showHankkiListBottomSheet() { // 맵뷰 클릭할 때마다 실행되던 코드를 함수화 함 -> 이거로 바텀 시트가 올라옴
        self.rootView.bottomSheetView.viewLayoutIfNeededWithDownAnimation()
        self.hideMarkerInfoCard()
}

RemoveHankkiViewController

// 전체 식당 리스트가 떠있는 홈으로 이동
    func popToThreeStepsBack() {
        if let viewControllers = self.navigationController?.viewControllers {
            if viewControllers.count > 2 {
                self.navigationController?.popToViewController(viewControllers[viewControllers.count - 3], animated: true)
            }
        }
    }

• popToViewController로 쓰리 스텝 전의 화면으로 돌아갈 수 있게 했습니다.

✅ Check List

• Merge 대상 브랜치가 올바른가?
• 최종 코드가 에러 없이 잘 동작하는가?
• 전체 변경사항이 500줄을 넘지 않는가?

📟 관련 이슈

• Resolved: [Feat] 식당 상세 화면 UI 변경 - 식당 정보 다를 시 제보 화면 #317

[shimseohyun]

그녀의 값진 크리스마스
고생했어요
크리스마스 선물로 멋진 트리를 드릴게요

        *
       ***
      *****
     *******
    *********
   ***********
  *************
       |||

[shimseohyun]

이런 함수도 뷰모델로 빼눈거군요 참고하겠습니다

[EunsuSeo01]

저 checkThumbnailHankkiValidation 함수 내에서 다루는 데이터들(hankkiLists, hankkiThumbnail)이 VM에서 관리되고 있었기 때문에 요렇게 진행했습니다 비즈니스 로직이기도 하고...

[shimseohyun]

DAMM~~~~~~~~
LGTM

[shimseohyun]

이전의 이전의 이전

[shimseohyun]

여긴 . 왜 스크롤이 불가한가요? 그냥 궁금

[EunsuSeo01]

아 원래 컬뷰 높이 설정을 해두었을 때 추가했던 코드인데 지금은 그렇게 해두지 않아서 레전드 무쓸모네요
고맙소 처단하겠소