Immutable data holder

Transcript

1. Immutable data holder 2019/02/28 potatotips #59

2. よくある話 タップしてユーザのプロフィールを表示 λοϓ

3. MVVMによる実装 • ユーザIDを引数としてViewModelがRepository からfetch • pro leはDataBindingを通じてViewに表示 fun load(userId: String)

   = profileRepository.fetchProfile(userId) { profile.value = it } ProfileViewModel.kt

4. MVVMによる実装 • ユーザIDを引数としてViewModelがRepository からfetch • pro leはDataBindingを通じてViewに表示 fun load(userId: String)

   = profileRepository.fetchProfile(userId) { profile.value = it } ProfileViewModel.kt

5. MVVMによる実装 • ユーザIDを引数としてViewModelがRepository からfetch • pro leはDataBindingを通じてViewに表示 fun load(userId: String)

   = profileRepository.fetchProfile(userId) { profile.value = it } ProfileViewModel.kt var profile = MutableLiveData<Profile>()

6. MVVMによる実装 • ユーザIDを引数としてViewModelがRepository からfetch • pro leはDataBindingを通じてViewに表示 fun load(userId: String)

   = profileRepository.fetchProfile(userId) { profile.value = it } ProfileViewModel.kt var profile = MutableLiveData<Profile>()

7. MVVMによる実装 • ユーザIDを引数としてViewModelがRepository からfetch • pro leはDataBindingを通じてViewに表示 fun load(userId: String)

   = profileRepository.fetchProfile(userId) { profile.value = it } ProfileViewModel.kt var profile = MutableLiveData<Profile>() ͳΜͱ͔͠ͳ͍ͱɾɾɾɾ

8. Lv1: mutableな変数宣言をやめる 知らないうちにobserveできなくなってしまう val に書き換えでimmutableに var data = MutableLiveData<String>() data.observe(this,

   Observer { print(it) }) data.value = "hello" data = MutableLiveData() data.value = "world" // ͜͜͸observe͞Εͳ͍ var profile = MutableLiveData<Profile>() val profile = MutableLiveData<Profile>()

9. Lv2: mutableなデータを公開しない 変数をvalにしてもオブジェクト自体はまだmutable 　 ViewがViewModelのデータを変更できてしまう
 外からはread onlyなデータだけが見える形にする val data =

   MutableLiveData<String>() data.value = "hello" data.value = "world" // いくらでも更新できる private val _profile: MutableLiveData<Profile>() val profile: LiveData<Profile> get() = _profile val profile = MutableLiveData<Profile>()

10. 良さそう？ まあよさそう。だが、、
 
 
 
 
 privateとはいえViewModel内なら書き換え可能 • ViewModelが肥大化したら多分事故る •

    そもそも2つも変数を宣言するのはくどい private val _profile: MutableLiveData<Profile>() val profile: LiveData<Profile> get() = _profile ProfileViewModel.kt

11. Lv3: そもそもmutableにしない switchMapを使いimmutableな定義に 
private val _userId = MutableLiveData<String>() … val

    profile: LiveData<Profile> = 
 Transformations.switchMap(_userId){ profileRepository.getProfile(it) }

12. Lv3: そもそもmutableにしない switchMapを使いimmutableな定義に 
private val _userId = MutableLiveData<String>() … val

    profile: LiveData<Profile> = 
 Transformations.switchMap(_userId){ profileRepository.getProfile(it) } userIdの変更を監視

13. Lv3: そもそもmutableにしない switchMapを使いimmutableな定義に 
private val _userId = MutableLiveData<String>() … val

    profile: LiveData<Profile> = 
 Transformations.switchMap(_userId){ profileRepository.getProfile(it) } IDに変更があればfetch

14. Lv3: そもそもmutableにしない switchMapを使いimmutableな定義に 
private val _userId = MutableLiveData<String>() … val

    profile: LiveData<Profile> = 
 Transformations.switchMap(_userId){ profileRepository.getProfile(it) }

15. Lv3: そもそもmutableにしない switchMapを使いimmutableな定義に 
 • 「pro leはIDの更新以外を理由に変更できない」
 という制約をコード上で表現することができる • 「pro

    leはIDをトリガーとしてfetchした値」
 というデータのフローが明確になる private val _userId = MutableLiveData<String>() … val profile: LiveData<Profile> = 
 Transformations.switchMap(_userId){ profileRepository.getProfile(it) }

16. まとめ mutable、やめよう • 変数宣言 • オブジェクトが抱えるデータ 
 LiveDataならswitchMap(Transformations)
 を使うといい感じに

17. Bonus: より簡潔に 拡張関数を活用すると良さそう（ktxにある？）
 
 
 Transformations.switchMap(_userId) { profileRepository.getProfile(it) } _userId.switchMap

    { profileRepository.getProfile(it) } fun <X, Y> LiveData<X>.switchMap(body: (X) -> LiveData<Y>): 
 LiveData<Y> { return Transformations.switchMap(this, body) } 長い シンプル https://github.com/google/iosched ͷ Extensions.kt ΑΓ