멘토링 피드백 5주차

Q. 깃 플로우 잘 쓰시고 계시나?

release 가 되면 자동으로 배포해주는 전략이 있을 것임. hotfix가그래서 Master에 붙어있는거.

왠만한 플로우는 잘 지키고 있으시다!

Q. 기존의 아키텍쳐에서 지금 현재의 아키텍쳐로 변경되었다. 다만 이러한 변경의 이유는 테스트의 용이성, 느슨한 결합도 등을 위해서였는데 그래서 바꿔서 그러한 결과를 얻었냐? 라고 물어본다면 확실하게 대답 할 수 없을 것 같다.

SOLID를 지키려고 프로젝트에 도입하고 개선하고 생각하는 의도는 충분히 좋다. 그러나 SOLID를 지키기 위한 강박? 적인 느낌이 조금 느껴진다.

또한 좋은 의견이고, 추구해야 할 방향성은 맞으나 설득력이 크게 느껴지지 않는다.

Singleton을 사용한 순간부터 SOLID를 지키기에는 조금 어려운 부분도 있다.

이부분은 팀과 잘 고려하면서 설득력 및 충분한 이해 (팀이 다 같은 생각을 가질 수 있도록)를 바탕으로 개선할 수 있다면 좋을 것 같다.

Q. Rx를 도입하면서 특정 model을 한번만 읽어와서 뷰를 업데이트 하게 되는데 굳이 바인드 시켜서 계속 업데이트를 할 수 있게 유지할 필요가 있는가?

take(1) 하면 한번만 받을 수 있음. 이렇게 하면 바로 dispose 된다.

Q. 화면 이동에서 viewModel을 전달할 때 파라미터가 여러개 있는 이유? → 줄일 수 있는 방법?

다른 화면으로의 이동에서 뷰모델에 무언가 처리(가공)을 해야 할 때 이거를 VC에서 혹은 init의 인자로 받지 말고 viewModel 자체에서 메소드를 만들어서 가공해서 viewModel을 리턴값으로 받아서 viewModel만 전달할 수 있도록 하는게 좋아보인다!