Modeling UiEvent

モバチキ 〜Mobile Tips 共有会〜 #5
https://karabiner.connpass.com/event/323020/

Transcript

1. Modeling UiEvent モバチキ#5 mikan( 一瀬喜弘)
2. 自己紹介 object Mikan { val name = " 一瀬喜弘" val company = "karabiner.tech" val work = Engineer.Android val hobby = listOf( " 漫画", " アニメ", " ゲーム", " 折り紙", "OSS 開発・コントリビュート", ) }
3. Compose で複雑なUI を作る際に 直面している課題
4. 関数の引数が無限に増える ( 特にFunction Type)
5. 実務のコードだと最大で 72 個 のパラメータを持っており、今後も増えていく予定 @Composable fun カート画面( slot: @Composable () -> Unit, aUiModel: AUiModel, contents: List<Content>, onClickA: (id: ContentId) -> Unit, onClickA2: (id: ContentId, index: Int) -> Unit, bannerUiModel: BannerUiModel, onClickBanner: () -> Unit, notifications: List<Notification>, onClickNotification: (messageId: String, url: String) -> Unit, // ... )
6. 問題: 新しいユーザーインタラクションが増えたとき 1. ViewModel にイベントハンドラーを追加 2. 親コンポーネントの引数を増やす 3. 親コンポーネントを呼び出している箇所を修正 4. パラメータをバケツリレーしていく… 階層が深いほどめんどくささが増す
7. 問題: 既存のユーザーインタラクションを消すとき 1. ユーザーインタラクションを発火している箇所を消す 2. それを呼び出しているコードを修正 3. それを呼び出しているコードを修正 4. それを(ry 5. ViewModel 側の処理が不要になったのならそれも削除する ユーザーインタラクションを消すという単純なことなのに バケツリレーのすべてが影響を受け、変更容易性がだんだん下がる
8. ユーザーインタラクション数 = UI 要素数 × n (n: 自然数) ユーザーインタラクションの​ 数は​ UI 要素数に​ たいして​ n 倍程度​ 増える​ もの​
9. UI は本質的に複雑なもの
10. パラメータ無限増幅問題を​ 実装の​ 仕方に​ よって​ 多少改善させられないかと​ 思って​ 実践している​ ことを​ 共有します
11. 解決案: ユーザーインタラクションを​ モデリングする​ ことで​ イベントハンドラーを​ 集約させる​
12. ユーザーインタラクションをモデリング ユーザーインタラクションを HogeUiEvent という sealed interface として直積的に実装する UiEvent の種類 クリック おそらくこれがほとんど キーボード入力 sealed interface CartUiEvent { data class ClickA(id: ContentId) : CartUiEvent data object ClickBanner : CartUiEvent // ... }
13. イベントハンドラーを集約させる Composable 関数に渡す関数型のパラメータは原則1 つに制限する @Composable fun カート画面( onUiEvent: (CartUiEvent) -> Unit, aUiModel: AUiModel, bUiModel: BUiModel, // ... ) { }
14. UiEvent の発火やハンドリングはどう実装するのか 1. ViewModel にイベントハンドラーを定義する 2. onUiEvent をComposable 関数に渡す 3. UiEvent を発火する
15. 1. ViewModel にイベントハンドラーを定義する class CartViewModel() : ViewModel() { // エントリーポイントだけ外部に公開する fun onUiEvent(event: CartUiEvent) { // UiEvent と具体的な処理をマッピングする when (event) { is CartUiEvent.ClickA -> onClickA(event.id) CartUiEvent.ClickBanner -> onClickBanner() // ... } } // 具体的な処理はすべてprivate で隠蔽する private fun onClickA(id: ContentId) { // ... } private fun onClickBanner() { // } // ... }
16. 2. onUiEvent を渡す CartScreen( // ... onUiEvent = viewModel::onUiEvent )
17. 3. UiEvent を発火する onClick = { onUiEvent(CartUiEvent.ClickA(aUiModel.id)) }
18. UiEvent を​ 導入した​ ことで​ 変更は​ しやすくなったのか
19. UiEvent sealed interface CartUiEvent { data class ChangeQuantity(itemId: String, quantity: Int): CartUiEven data class DeleteItem(itemId: String) : CartUiEvent data class ClickAd(url: String) : CartUiEvent }
20. ViewModel class CartViewModel() : ViewModel() { fun onUiEvent(event: CartUiEvent) { when (event) { is ChangeQuantity -> changeQuantity(event.itemId, event.quantity) is DeleteItem -> deleteItem(event.itemId) is ClickAd -> transitionToWebView(event.url) } } private fun changeQuantity(itemId: String, quantity: Int) { // ... } private fun deleteItem(itemId: String) { // ... } private fun transitionToWebView(url: String) { // ... } }
21. Activity class CartActivity() : ComponentActivity() { pruivate val viewModel: CartViewModel by viewModels() override fun onCreate(savedInstanceState: Bundle?) { setContent { MyAppTheme { Surface { CartScreen( // ... onUiEvent = viewModel::onUiEvent ) } } } } }
22. Parent Component @Composable fun CartScreen( // ... onUiEvent: (CartUiEvent) -> Unit, ) { LazyColumn { items(uiModel.items, { it.id }) { item -> CartItem( // ... onUiEvent = onUiEvent ) } item { AdSection( // ... onUiEvent = onUiEvent ) } } }
23. Child Component @Composable fun CartItem( // ... onUiEvent: (CartUiEvent) -> Unit ) { // ... Spinner( onQuantityChange = { quantity -> onUiEvent(ChangeQuantity(uiModel.itemId, quantity)) } ) DeleteButton( onClick = { onUiEvent(DeleteItem(uiModel.id)) } ) }
24. シナリオ: お気に入りボタン追加
25. UiEvent イベント追加 sealed interface CartUiEvent { data class ChangeQuantity(itemId: String, quantity: Int): CartUiEven data class DeleteItem(itemId: String) : CartUiEvent data class ClickAd(url: String) : CartUiEvent data class AddToFavorite(itemId: String) : CartUiEvent }
26. ViewModel イベントハンドラー追加 UiEvent とイベントハンドラーのマッピング追加 class CartViewModel() : ViewModel() { fun onUiEvent(event: CartUiEvent) { when (event) { is ChangeQuantity -> changeQuantity(event.itemId, event.quantity) is DeleteItem -> deleteItem(event.itemId) is ClickAd -> transitionToWebView(event.url) is AddToFavorite -> addToFavorite(event.itemId) } } // ... private fun addToFavorite(itemId: String) { // ... } }
27. Activity class CartActivity() : ComponentActivity() { pruivate val viewModel: CartViewModel by viewModels() override fun onCreate(savedInstanceState: Bundle?) { setContent { MyAppTheme { Surface { CartScreen( // ... onUiEvent = viewModel::onUiEvent ) } } } } }
28. Parent Component @Composable fun CartScreen( // ... onUiEvent: (CartUiEvent) -> Unit, ) { LazyColumn { items(uiModel.items, { it.id }) { item -> CartItem( // ... onUiEvent = onUiEvent ) } item { AdSection( // ... onUiEvent = onUiEvent ) } } }
29. Child Component コンポーネント追加 イベント発火 @Composable fun CartItem( // ... onUiEvent: (CartUiEvent) -> Unit ) { // ... FavoriteButton( onClick = { onUiEvent(AddToFavorite(uiModel.id)) } ) }
30. 変更した箇所 UiEvent ViewModel: イベントの到達点 CartItem: イベントの発火点 → Activity や親コンポーネントなどの途中の要素への変更をしな くてよくなった
31. メリット 変更箇所が減る( 複雑なUI ほど恩恵が大きい) Composable 関数内でのバケツリレーが単純になる イベントが型になるので引数が同じイベントであっても区別しやすい デメリット 単純な画面、変更がほとんど行われない画面に導入すると複雑さを無闇に増 やすだけに繋がりかねない イベント発火点を消しただけではUiEvent の参照はなくならないので実装が中 途半端に残って技術的負債化する恐れがある
32. UiEvent を構造化する: 複雑化するUI への対処 UI 要素が増えてUiEvent がどんどん複雑化していったとき ClickMoreA とClickMoreB は名前が非常に似ているので取り違える可能性が高い → 実行できるUiEvent に制限を設けて間違いを減らす sealed interface CartUiEvent { data class ChangeQuantity(val itemId: String, val quantity: Int) : C data class DeleteItem(val itemId: String) : CartUiEvent data class ClickAd(val url: String) : CartUiEvent data class AddToFavorite(itemId: String) : CartUiEvent data object ClickMoreA : CartUiEvent data object ClickMoreB : CartUiEvent }
33. UiEvent を構造化する: 複雑化するUI への対処 UiEvent を継承する sealed interface CartUiEvent sealed interface ASectionUiEvent : CartUiEvent { data object ClickMore : ASectionUiEvent } sealed interface BSectionUiEvent : CartUiEvent { data object ClickMore : BSectionUiEvent }
34. UiEvent を構造化する: 複雑化するUI への対処 Section レベルのComposable 関数に渡すUiEvent を制限する @Composable fun ASection( onUiEvent: (ASectionUiEvent) -> Unit, // ... ) { // ASectionUiEvent を実装したイベントしか発行できない } @Composable fun BSection( onUiEvent: (BSectionUiEvent) -> Unit, // ... ) { // BSectionUiEvent を実装したイベントしか発行できない }
35. UiEvent を構造化する: 複雑化するUI への対処 CartUiEvent を継承しているのでonUiEvent をそのまま渡せる 🎉🎉 @Composable fun CartScreen( onUiEvent: (CartUiEvent) -> Unit, // ... ) { // CartUiEvent は継承元なのでそのまま渡すことができる ASection( // ... onUiEvent = onUiEvent ) BSection( // ... onUiEvent = onUiEvent ) }
36. 共通パーツのUiEvent 複数の​ 画面で​ 使い回す共通の​ UI は​ どうしよう? このような​ UI には​ 共通の​ UiEvent は​ 定義せずに​ 利用する​ 側で​ それぞれUiEvent を​ 定義していく​ ほうが​ いい​ 気が​ している​ → 機能要件は​ 同じでも、​ 非機能要件​ （ログとか）は​ 画面ごとに​ やりたいことが​ 若干違ったりする​
37. UiEvent がオーバーエンジニアリングしないように注意する イベントと状態の関係はステートマシンとして表現可能なので、 一貫したステー トマシンを作るための仕組みを導入したくなる 1. 自作のDSL を作る 2. 既存のMVI フレームワークを使う
38. UiEvent がオーバーエンジニアリングしないように注意する このようなMVI フレームワークでは 状態、イベント、副作用を型 引数としたインターフェースなり抽象クラス（Screen<State, Event, Effect> ）を作っていたり、 Redux アーキテクチャよろ しくReducer という概念を導入してくるが、 これらはいまのとこ ろ個人的にはやりすぎだと思っている 一貫性をもたせるために導入するには複雑すぎる Google が方針転換したり新しい実装パターンを提示した瞬間に 技術負債と化す
39. 参考 https://medium.com/@hunterfreas/handling-ui-events-in-jetpack-compose-a-clean-approach-c8fd1bfc6231 iOSDC Japan 2020: SwiftUI 時代の Functional iOS Architecture / 稲見 泰宏 https://www.youtube.com/watch?v=g_hq3qfn-O8 圏論( モナド) にまで手をだしているので上級者向け → 関数型の考えは参考になる