MVVM 에서 코드의 분리가 어떻게 이뤄져야 할지 v0.1

[seongmin221]

EasyVel PR 에서
@meltsplit 의 코드리뷰 + 내 개인적인 생각으로 느끼게 된 V(VC) / VM 에서의 특징들(?)에 대해 기록해두려고 한닷

별거 아님 주의

---

View

1. (당연히) 화면을 띄우는 것

2. 개인 취향 차이가 있겠지만, VC 에서 self.view.mainButton.rx.tap 식으로 선언하는것보다
   View 에서 아래와 같이 선언한걸 가져다가 쓰는게 좋다고 생각했음

var mainButtonTapTrigger: Driver<Void> {
    return self.mainButton.rx.tap.asDriver()
}

• VC 쪽 코드가 훨씬 더 깔끔해짐
• 캡슐화가 더 잘 되어 있음 (thanks to @meltsplit)

이런 이유들 때문 !

3. BottomSheet 를 보여주는 코드는 (따로 전달할 데이터나 갖고 있는게 없다면) View 쪽에 두기

---

ViewController

1. VC 는 View 에서의 유저 input 을 VM 으로 전달하는 중간다리 역할만을 해야 한다(고 생각한다) ( thanks to @meltsplit)

이 때문에 아래와 같은 코드 대신

let deleteFolderTrigger = self.storageView.deleteFolderTrigger
    .map { _ in return self.folderName }
    .asDriver()

folderName 을 애초에 VM 의 init 단계에서 넣어주고 (데이터 관련이기 때문)
아래처럼 코드 바꾸기

let deleteFolderTrigger = self.storageView.deleteFolderTrigger
    .asDriver()

즉, View 에서의 Input 에 의해, VC 에 존재하는 값을 VM 으로 전달하지 않기

2. output 을 다루는 과정에서 ViewModel 호출하지 않기

아래와 같은 코드는 ViewModel 에서 output 을 내보내기 전에 충분히 처리 가능한 과정
따라서 정말 필요하지 않은 이상 요 안에서 ViewModel 을 호출하지 않기

self.rx.methodInvoked(#selector(self.viewWillDisappear(_:)))
    .subscribe(with: self) { owner, _ in
        owner.viewModel.deleteRealmStoragePosts()
    }.disposed(by: self.disposeBag)

요로코롬 바뀜

let fetchPostTrigger = self.rx.viewWillAppear
    .asDriver()

---

ViewModel

1. (당연히) 모든 로직은 ViewModel 에서 -> UI 관련 로직은 없어야 함 !

그래서 원래는 UIKit 를 import 하지 않으려고 했는데,
그래도 생각해보니 만약 UIImage 를 cache 에 넣어둔다든 ,,,
쨋든 View 나 VC 에서 다루기 좀 그런 로직들은 다뤄야 할 필요가 있긴 할듯

결론적으로는 꼭 필요하지 않은 이상 import UIKit 자체를 안해도 되도록 하기

[dangsal]

이미지 cache 가 어떻게 구현되어있는지 몰라도 ViewModel 에서는 UIKit 을 몰라야 하는게 맞습니다.
만약 UIkit 을 import 해야한다면 로직이 잘못되었다고 봐도 됩니다.

꼭 필요하지 않은 이상이 아니라 꼭 import UIkit 을 하면 안된다고 생각함당

[seongmin221]

헤헤ㅔ 첫 댓글이당

그러면 저희 이번에 진행할 때에는 철저히 모르게 만들어보죠 !!!