Step8 Feedback - Githubissues

somedd / swift-StoreApp

쇼핑 iOS 앱 저장소 / `18.04.26~`18.06.14

4 stars 1 forks source link

Closed somedd closed 6 years ago

somedd commented 6 years ago

피드백 모음

somedd commented 6 years ago

1차 피드백

셀룰러에서 WiFi에 연결되거나 WiFi에서 셀룰러로 바뀌는 경우에도 모두 다시 그려야 좋을까요? 어떤가요?
연결성을 판단하고 섹션을 만드는 것을 ViewController가 해야할까요? Sections 객체 내부에서 판단하는 것과 어떤 차이가 있을까요?

somedd commented 6 years ago

1차 피드백 반영

네트워크 연결이 이전에 없었을 경우에만, Sections를 새로 넣어주도록 개선 (셀룰러 - WiFi간 연결상태 변화에 대한 부분을 고려하지 못했습니다.)
Sections가 생성될 때, 연결성을 판단하여 생성되도록 개선 (연결성 판단과 섹션 생성하는 부분은 VC의 역할이 아닌,Model의 역할이 맞다고 생각합니다.)

somedd commented 6 years ago

2차 피드백

자 이렇게 하고나이 Section 객체가 외부 네트워크 매니저에 의존성이 커졌습니다. 이러면 Section 만 테스트 가능할까요? 어떻게 생각해요? 제 의견보다 개발담당의 기준이 중요합니다.

somedd commented 6 years ago

2차 피드백 반영

이전 피드백을 반영하였을 때, Sections가 init()의 형태로 생성될 때, 자동으로 네트워크상태를 판단하여 생성되도록 의도한 것인데 NetworkManager의 의존성을 고려하지 못했습니다. 이전 피드백에서 ViewController가 연결성을 판단했던 부분을 개선하면서 구현한 NetworkManager의 현재 연결상태(connectionStatus)를 넣어주도록 개선하였습니다.
그에 따라, 오프라인에서 셀룰러 또는 WiFi 전환될 때의 reloadSections 메서드에서도, 네트워크상태를 받아오도록 개선하였습니다.

somedd commented 6 years ago

3차 피드백 : (클린 아키텍쳐 관련) 네트워크를 모니터링하는 객체를 Model이 소유하면 의존성이 커진다.