android: Observe

Question

override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
  super.onViewCreated(view, savedInstanceState)

// ViewModelFolder.selectFolder はMutableLiveData<String>
  ViewModelFolder.selectFolder.observe(viewLifecycleOwner, Observer { name ->
      name?.let {
        taskViewModel.getTasksByFolder(it, false)
          .observe(viewLifecycleOwner, Observer { tasks ->
            tasks?.let { taskAdapter.setTasks(it) }
          })
      }
    })
}

TabLayoutのためにFragmentを使ってます。
Folder名が変更またはTaskが変更した場合にtaskViewModel.getTasksByFolder を呼びたいです。
上記のようにObserveの中にObserveを書くと中のObserveが複数回呼ばれてしまいます。
どうすれば正常に動かせますか？

Answer 1

observeをネストして記述するべきではありません。Transformations.switchMapを使いましょう。

また、ご提示のコードではView側にロジックを書いているようですが、これはMVVMの設計思想に反します。ロジックはViewModel以下に閉じ込めてView側は単純に値を受け取るのみにすべきです。

• ViewModel

val tasks = selectFolder
    .nonNull()
    .switchMap { getTasksByFolder(it, false) }
    .nonNull()

// nullの場合は値を流さないLiveData
fun <T> LiveData<T?>.nonNull(): LiveData<T> {
    val mediator = MediatorLiveData<T>()
    mediator.addSource(this) { if (it != null) { mediator.value = it } }
    return mediator
}

• View

viewModel.tasks.observe(viewLifecycleOwner, Observer {
    taskAdapter.setTasks(it)
})

Comments

[Nippun]

observeをネストしてはいけない理由はなんですか？
パーフォマンスの問題ですか？

[kakajika]

ネストしてはいけない、というよりはネストしない方が賢明、ぐらいのニュアンスのつもりでした。（「するべきではない」というのはちょっと表現が強かったかもしれません。）

observeを2重、3重とネストしていくとコードは複雑になり、メンテナンス性が低下します。また、observeを行うのは基本的にView側の役目なのでView側にロジックを露出させないためにも避けるべきです。

[Nippun]

なるほどです。
参考になります。ありがとうございます。

Answer 2

LiveDataは最新の情報を保持してアタッチされているLifecycleOwnerがアクティブになったタイミングで情報を再度通知します。なのでデータが変更されていなければ再度通知しないなどの仕組みを実現するには、作りこみが必要です。

• 一度だけ通知するAndroid Architecture ComponentsのLiveDataを作る
• Android: 拡張関数でLiveDataのSingle Eventを扱う
• SingleLiveEvent

個人的には複雑な実装をしないのであればSingleLiveEventだけで事足りるのでは、と思います。