Immutable data holder 2019/02/28 potatotips #59

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

MVVMによる実装 ● ユーザIDを引数としてViewModelがRepository からfetch ● pro leはDataBindingを通じてViewに表示 fun load(userId: String) = profileRepository.fetchProfile(userId) { profile.value = it } ProfileViewModel.kt

MVVMによる実装 ● ユーザIDを引数としてViewModelがRepository からfetch ● pro leはDataBindingを通じてViewに表示 fun load(userId: String) = profileRepository.fetchProfile(userId) { profile.value = it } ProfileViewModel.kt

MVVMによる実装 ● ユーザIDを引数としてViewModelがRepository からfetch ● pro leはDataBindingを通じてViewに表示 fun load(userId: String) = profileRepository.fetchProfile(userId) { profile.value = it } ProfileViewModel.kt var profile = MutableLiveData()

MVVMによる実装 ● ユーザIDを引数としてViewModelがRepository からfetch ● pro leはDataBindingを通じてViewに表示 fun load(userId: String) = profileRepository.fetchProfile(userId) { profile.value = it } ProfileViewModel.kt var profile = MutableLiveData()

MVVMによる実装 ● ユーザIDを引数としてViewModelがRepository からfetch ● pro leはDataBindingを通じてViewに表示 fun load(userId: String) = profileRepository.fetchProfile(userId) { profile.value = it } ProfileViewModel.kt var profile = MutableLiveData() ͳΜͱ͔͠ͳ͍ͱɾɾɾɾ

Lv1: mutableな変数宣言をやめる 知らないうちにobserveできなくなってしまう val に書き換えでimmutableに var data = MutableLiveData() data.observe(this, Observer { print(it) }) data.value = "hello" data = MutableLiveData() data.value = "world" // ͜͜͸observe͞Εͳ͍ var profile = MutableLiveData() val profile = MutableLiveData()

Lv2: mutableなデータを公開しない 変数をvalにしてもオブジェクト自体はまだmutable 　 ViewがViewModelのデータを変更できてしまう
 外からはread onlyなデータだけが見える形にする val data = MutableLiveData() data.value = "hello" data.value = "world" // いくらでも更新できる private val _profile: MutableLiveData() val profile: LiveData get() = _profile val profile = MutableLiveData()

良さそう？ まあよさそう。だが、、
 
 
 
 
 privateとはいえViewModel内なら書き換え可能 ● ViewModelが肥大化したら多分事故る ● そもそも2つも変数を宣言するのはくどい private val _profile: MutableLiveData() val profile: LiveData get() = _profile ProfileViewModel.kt

Lv3: そもそもmutableにしない switchMapを使いimmutableな定義に 
private val _userId = MutableLiveData() … val profile: LiveData = 
 Transformations.switchMap(_userId){ profileRepository.getProfile(it) }

Lv3: そもそもmutableにしない switchMapを使いimmutableな定義に 
private val _userId = MutableLiveData() … val profile: LiveData = 
 Transformations.switchMap(_userId){ profileRepository.getProfile(it) } userIdの変更を監視

Lv3: そもそもmutableにしない switchMapを使いimmutableな定義に 
private val _userId = MutableLiveData() … val profile: LiveData = 
 Transformations.switchMap(_userId){ profileRepository.getProfile(it) } IDに変更があればfetch

Lv3: そもそもmutableにしない switchMapを使いimmutableな定義に 
private val _userId = MutableLiveData() … val profile: LiveData = 
 Transformations.switchMap(_userId){ profileRepository.getProfile(it) }

Lv3: そもそもmutableにしない switchMapを使いimmutableな定義に 
 ● 「pro leはIDの更新以外を理由に変更できない」
 という制約をコード上で表現することができる ● 「pro leはIDをトリガーとしてfetchした値」
 というデータのフローが明確になる private val _userId = MutableLiveData() … val profile: LiveData = 
 Transformations.switchMap(_userId){ profileRepository.getProfile(it) }

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

Bonus: より簡潔に 拡張関数を活用すると良さそう（ktxにある？）
 
 
 Transformations.switchMap(_userId) { profileRepository.getProfile(it) } _userId.switchMap { profileRepository.getProfile(it) } fun LiveData.switchMap(body: (X) -> LiveData): 
 LiveData { return Transformations.switchMap(this, body) } 長い シンプル https://github.com/google/iosched ͷ Extensions.kt ΑΓ