minjae-L / ShowWeather

날씨 알려주는 앱
0 stars 0 forks source link

Refactoring #17

Closed minjae-L closed 3 months ago

minjae-L commented 4 months ago

느낀점) 예제로 보던 RxSwift 패턴은 코드의 가독성을 늘려주고 길이도 확 단축된다는데.. 어떻게 코드를 바꿔봐도 크게 달라지는게 없어보임.. 아직 이론이 미숙해서 어디서부터 어디까지 반응형으로 구현해야하는지 감이 안잡힌다. 이전에는 CollectionView의 DataSource 프로토콜을 채택하여 Custom cell을 만들어 줬다면, CollectionView를 Subject와 바인딩해서 셀의 구성을 커스텀해준다. 그러면 Delegate 프로토콜을 채택하여 CollectionView의 flowlayout이나 이런 전체적인 모양은 RxCocoa로 리팩토링 할 수 없을까? Subject는 많은 Observable과 연결이 된다하면 Delegate패턴을 사용하여 뷰컨간의 데이터전달을 리팩토링할 수 없을까?

minjae-L commented 4 months ago
Key5771 commented 4 months ago

그러면 Delegate 프로토콜을 채택하여 CollectionView의 flowlayout이나 이런 전체적인 모양은 RxCocoa로 리팩토링 할 수 없을까?

RxDataSources 이런게 있긴한데, 나도 이건 안써보기도 했고 UI 코드들은 오히려 디버깅이 힘들 수도 있는 것 같아 오히려 RxAlamofire 이런걸 사용하는걸 고려하는게 나을 수도 있을 것 같아.

그리고 Rx에 조금 익숙해지면 Rx를 더 깊이 보는 것보다는 Combine을 보는 것을 추천..!