개편 사항

• 하단 버튼 크기 조절 및 통일화
  -> UX 개선 및 코드 유지 보수에 용이하게 하기 위함

• 메인뷰 UI 수정
  -> 음식 사진을 최대한 많이 보이게함. (유저에게 필요한 정보를 최대한 많이 보여줘 UX를 개선)

Problem

버킷리스트를 통해 음식뷰로 들어가면 즐겨찾기 star가 활성화되지 않았다는 문제가 발생함 왜 일까?

Description

배경

Bug

다음과 같이 버튼을 반환하는 함수는 버튼을 클릭했을때 최종 필터링된 음식 배열에서 random으로 한개를 뽑아내서 Navigation의 destination View를 init하는데 사용된다.

문제는 서버에서 데이터를 get 하는 과정에 시간이 걸려 UI가 나중에 그려진다는 것이다... 당장에 떠오르는것은 else로 분기처리해 껍데 버튼을 놓는 것인데 이것보다 좋은 방법이 있을까?

private func getRecommendationButton() -> some View {
    VStack {
        if !modelData.foodData.isEmpty {
            NavigationLink(destination: FoodInfoView(foodInfo: filteredGroupFinal.randomElement() ?? modelData.foodData.first!)) {
                Button {
                    filteredGroupFinal = self.filteredGroupA.filter{filteredGroupB.contains($0)}.filter{filteredGroupC.contains($0)}.filter{filteredGroupD.contains($0)}
                } label: {
                    Text("맛집. 추천받기!")
                        .customButtonFormat()
                        .padding(.bottom, 20)
                }
            }
        }
    }
}

Discussion

배경

Young과의 멘토링을 통해 받은 피드백과 제 생각을 바탕으로 리드미 및 코드 개선 부분을 리스트업합니다.

To-do List

ReadMe 설명 개선

• !!! 왜 UI 작업에 SwiftUI를 사용하게 되었는가? UIkit과 비교해서 어떤 메리트가 있어서 이를 쓰기로 판단했는가? 적어주기
• !!! NukeUI에 대한 링크를 넣고 SwiftUI의 어떤 부분때문에 이를 사용하게 되었는지 자세하게 기술하기
• !!! async await 말고 다른 선택지가 있는데 왜 이것을 써야했는지 설명하기
• !!! FileManager가 어떤 것인지 설명 적어주기 (영구 저장이라는 키워드 다른말로 바꾸기, 맥락상 케싱을 위해 썼다고는 등 )
• !! 데이터를 왜 제이슨 구조로 저장하는지 설명적어주기
• !!! 왜 MVVM 구조를 채택하게 되었는지 다른 구조랑 비교해서 설명하기
• !! 데이터 플로우 차트 수정하기 (FileManager에 저장되는 것과 NotionDB는 결국 같은 데이터라는 것을 보여줘야함)
• ! Design Skill에 HIG Component를 잘 적어주기 (원론적인 디자인 이야기는 지양)

Code 개선

• !!! ViewModel은 하나의 뷰에 대한 책임을 가진다. 따라서 각 뷰별 ViewModel을 가지는 구조로 바꿔야함
• ! 반복적인 구조의 코드를 최대한 없애보자
• !! server에서 데이터를 가져오는 것과 local에서 가져오는 함수는 따로 관리되어야 한다.

느낀점 작성

• !!! 지저분한 코드를 받고 이를 개선하면서 했던 삽질과 배운점들을 리스트업하기
• !!! 한계점 적기, 시간이 없어서 프로젝트를 종료하지만 나중에 이런 부분들은 반영해서 코드를 짜야겠다 하는 부분 등을 발견해서 적기

Tips

• 자신의 코드에도 리뷰를 한다고 생각하자
• 혼자 작업하더라도 PR을 작성하는 연습을 하자
• PR에는 스크린샷보다는 코드 부분을 링크로 걸자 (보는 사람이 클릭해서 페이지 열어보고 바로 어떤 부분인지 찾을 수 있게!)

문제1

FoodListView와 FoodRecommendView에서 데이터를 두번 fetch해야함

문제2

기능 개선

• URL 이미지 업데이트 속도 저하 문제 개선하기

버전 업데이트 사항

• 로고 교체
• 아쉽게도 원하는 맛집이 없어요... UI 업데이트
• 음식 클릭시, 세부 사항 뷰 UI 업데이트

포트폴리오 업데이트 사항

• 데이터 플로우 정리(Visualization)
• 노션 DB 데이터 관리 이유 정리 (음식점 업데이트 및 추후 카테고리 추가시 용이)
• 즐겨 찾기 데이터 저장 이유 정리 (UserDefault, CoreData 등)