Feature/save weather - Githubissues

minjae-L / ShowWeather

날씨 알려주는 앱

0 stars 0 forks source link

Feature/save weather #9

Closed minjae-L closed 5 months ago

minjae-L commented 5 months ago

선택된 지역의 날씨정보를 저장하는 기능 구현

지역 검색 -> 지역 선택 -> 선택된 지역 추가시 메인화면의 CollectionViewCell에 저장됨
추가 버튼클릭시 주소와 nx,ny가 저장된 데이터 구조 LocationWeatherDataModel이 ViewController로 전송
전송된 데이터모델을 SavedWeatherDataModel로 변환하여 CollectionViewCell에 UI그리기
그려진 Cell 선택시 해당 지역의 날씨 상세정보를 나타내는 WeatherViewController가 나타남

발생한 버그?

메모리 누수 SearchResultTableViewCell이 검색하지 않는경우에도 사라지지않음
WeatherViewController 에서 추가버튼 클릭시 ViewController로 이동해야하는데 SearchResultViewController를 거치고 감

의문점

Delegate 패턴을 통해 WeatherVC -> SearchResultVC -> VC 로 데이터를 전달하고 전달하는 방법을 이용했는데 이게 이상적인 방법일까?
VC에서 self.navigationItem.searchController = searchController로 지정하여 VC가 로드되면 searchController도 메모리에 올라온다. searchController를 dismiss하여도 메모리에서 없어지지 않는다. 위 메모리누수? 버그를 해결하기 위해서는 searchController를 지정하는 시점을 다르게 해줘야할까?
공공데이터포털의 개인키는 유출되면 안되는 키인데 어떻게 관리를 할까? (지금은 그냥 query에 넣어서 사용하고있지만)

Key5771 commented 5 months ago

공공데이터포털의 개인키는 유출되면 안되는 키인데 어떻게 관리를 할까? (지금은 그냥 query에 넣어서 사용하고있지만)

이거 때문에라도 일단은 private 레포로 변경해야겠다..! 그리고 나중에 키 관련 처리하고 public으로 바꿔야겠어