分析用コードをアプリから 切り離す設計の実現

#pixiv_app_night

Transcript

1. 分析用コードをアプリから 切り離す設計の実現 pixiv Inc. makun 2019.11.22
2. 2 自己紹介 • 18年 新卒入社 • Androidアプリ開発を担当 • エンジニア採用を担当 • 好きなのは設計やアイデアだし • 苦手なのは収束させること makun アプリエンジニア
3. 3 • マンガサービス • 講談社と協業 • 女性がターゲット • 女性誌特集なども • Android、iOS
4. アーキテクチャ 4
5. 5 Dispatcher Store (ViewModel) ActionCreator (ViewModel) View (Activity, Fragment) Server Repository Database Entity Action Action Item Entity Remote Model Local Model
6. 6 Database Entity Core Feature Production App Repository Development App Feature Feature Repository Repository Database Database Remote Model Local Model Presentation Domain Data Resources
7. 分析コードとは 7
8. ユーザーの行動を把握しアプリのマーケティン グやパフォーマンス改善に関する 意思決定を行うためのデータを得ることのでき るコード、もしくは得るためのコード 8
9. 分析コードを 実装するとは 9
10. 意思決定を行うためのデータを得ることのでき るコード、もしくは得るためのコードが動作する よう記述する 10
11. パルシィのコードに 分析コードを実装してみる 11
13. class ComicActionCreator : ViewModel() { fun tapFollowComic(comic: Comic) { launch { … } } } 13
14. class ComicActionCreator : ViewModel() { fun tapFollowComic(context: Context, comic: Comic) { FirebaseAnalytics .getInstance(context.applicationContext) .logEvent(“follow_comic”, bundleOf( “comicId” to comic.id, “comicTitle” to comic.title )) launch { … } } } 14
15. class ComicActionCreator : ViewModel() { fun tapFollowComic(context: Context, comic: Comic) { FirebaseAnalytics… Repro.track(“【フォロ】コミック”, mapOf( “comicId” to comic.id, “comicTitle” to comic.title )) launch { … } } } 15
16. class ComicActionCreator : ViewModel() { fun tapFollowComic(context: Context, comic: Comic) { FirebaseAnalytics… Repro.track(…) AppsFlyerLib.getInstance().trackEvent( context.applicationContext, “key_follow_comic”, mapOf( “comicId” to comic.id, “comicTitle” to comic.title )) launch { … } } } 16
17. 本当にこの実装でいいんだっけ？ 17
18. Context受け取るようになったけど テストのことちゃんと考慮してる？ 18
19. 同じ分析コードを埋め込む箇所が 増えたときはどうする？ 19
20. 機能モジュールの依存増えてない？ ビルド時間とかも大丈夫？ 20
21. 分析ツールが 増えたり減ったりしたときは どうする？ 21
22. そもそもなんでこんなに 分析ツールあんの？ 22
24. 24 ツール 目的 Firebase ピクシブ全体で積極的に利用している BigQueryにデータをあげて全体的な分析に利用 Repro 協業先に利用実績があり 特定のアクションに対してアプリ内メッセージをだしたりに利用 AppsFlyer 協業先に利用実績があり アプリの流入や経路別のコンバージョンをみるために利用 分析用ツールを複数使う理由
25. 分析コードを実装する場 合の設計を考える 25
26. 26 それぞれの視点で考える
 設計時にやること
27. 27 機能を開発する人の視点
 

28. 28 分析コードを追加する人の視点
 

29. 29 今回は触れない視点 • アプリを利用するユーザーの視点 • テストの視点 • リファクタリングの視点
30. 30 設計次以外のメリット • コードを書くときにも有効 • 視点ごとにプルリクをわけられる • 実装時の思考コストが減る • レビュー時の思考コストが減る • 意味のあるまとまりで開発をすすめられる
31. 機能を開発する人の視点でみてみる 31
32. class ComicActionCreator : ViewModel() { fun tapFollowComic(context: Context, comic: Comic) { FirebaseAnalytics… Repro… AppsFlyerLib… launch { … } } } 32
33. class ComicActionCreator : ViewModel() { fun tapFollowComic(context: Context, comic: Comic) { FirebaseAnalytics… Repro… AppsFlyerLib… launch { … } } } 33 ユーザーがボタンをタップし たときの処理をここにつくる
34. class ComicActionCreator : ViewModel() { fun tapFollowComic(context: Context, comic: Comic) { FirebaseAnalytics… Repro… AppsFlyerLib… launch { … } } } 34 要求だとFirebase、Repro、 AppsFlyerにイベントを送信す る必要があるぞ
35. class ComicActionCreator : ViewModel() { fun tapFollowComic(context: Context, comic: Comic) { FirebaseAnalytics… Repro… AppsFlyerLib… launch { … } } } 35 分析コードの実装のために Contextを受け取らないと いけないぞ
36. class ComicActionCreator : ViewModel() { fun tapFollowComic(context: Context, comic: Comic) { FirebaseAnalytics… Repro… AppsFlyerLib… launch { … } } } 36 今回の要求だとComicの Entityだけでデータはおくれ そうだぞ
37. class ComicActionCreator : ViewModel() { fun tapFollowComic(context: Context, comic: Comic) { FirebaseAnalytics… Repro… AppsFlyerLib… launch { … } } } 37 Context追加しちゃったから メソッドを実行してる部分に もContextわたさないといけ ないぞ
38. class ComicActionCreator : ViewModel() { fun tapFollowComic(context: Context, comic: Comic) { FirebaseAnalytics… Repro… AppsFlyerLib… launch { … } } } 38 あとテストも書き直さないと いけないなぁ
39. 39 Entity Core Feature Production App Feature Feature Firebase Repro AppsFlyer
40. 分析コードを実装する人の視点でみてみる 40
41. class ComicActionCreator : ViewModel() { fun tapFollowComic(context: Context, comic: Comic) { FirebaseAnalytics… Repro… AppsFlyerLib… launch { … } } } 41 まずはこのコードまでたどりつく必要がある
42. class ComicActionCreator : ViewModel() { fun tapFollowComic(context: Context, comic: Comic) { FirebaseAnalytics… Repro… AppsFlyerLib… launch { … } } } 42 フォローボタンをタップした ときで、すでにフォローして いたかどうかが判定されて いない
43. class ComicActionCreator : ViewModel() { fun tapFollowComic(context: Context, comic: Comic) { if (comic.isFollowed.not()) { FirebaseAnalytics… Repro… AppsFlyerLib… } launch { … } } } 43
44. 44 分析コードはロジックをもつ

45. 45 ユーザー体験とは関係ないはずなのに・・ • ユーザー体験にかかわるところ (Featureモジュール) にコードをかく • 既存の処理を変更したりしないといけないことがある • 変更をくわえちゃうから実機のテストもちゃんとしておきたくなる • テストもちゃんとかいておきたくなる • 考えないといけないところが増えた気がする
46. パルシィでの設計 46
47. Flux + Tracker 47
48. 48 Dispatcher Store (ViewModel) ActionCreator (ViewModel) View (Activity, Fragment) Server Repository Database Entity Action Action Item Entity Remote Model Local Model
49. 49 Dispatcher Store (ViewModel) ActionCreator (ViewModel) View (Activity, Fragment) Server Repository Database Tracker Entity Action Action Item Action Entity Remote Model Local Model
50. // Actionを受け取れる概念をinterfaceにする interface ActionReceiver { fun receive(action: Action) } 50
51. // Dispatcherを継承する本番用Dispatcher class MainDispatcher( // ActionReceiverを実装したクラスを受け取る private vararg val receivers: ActionReceiver ) : Dispatcher() { override fun dispatch(action: Action) { // 全てのActionReceiverにActionを送信する receivers.forEach { it.receive(action) } super.dispatch(action) } } 51
52. // Dispatcherを継承する本番用Dispatcher class MainDispatcher( // ActionReceiverを実装したクラスを受け取る private vararg val receivers: ActionReceiver ) : Dispatcher() { override fun dispatch(action: Action) { // 全てのActionReceiverにActionを送信する receivers.forEach { it.receive(action) } super.dispatch(action) } } 52
53. // Firebase用にActionReceiverを実装したクラス // このクラスのインスタンスをMainDispatcherにわたす // パルシィではKoinでBeanをつくる class FirebaseTracker( private val app: Application ) : ActionReceiver { override fun receive(action: Action) { FirebaseAnalytics… } } 53
54. class ComicActionCreator : ViewModel() { fun tapFollowComic(comic: Comic) { launch { dispatch(TapFollowComicAction(comic)) // 以下に実際のフォロー処理をかく … } } } 54
55. 55 Entity Core Feature Production App Feature Feature Firebase Repro AppsFlyer
56. 56 Entity Core Feature Production App Feature Feature Tracker
57. 結果と考察 57
58. class ComicActionCreator : ViewModel() { fun tapFollowComic(context: Context, comic: Comic) { FirebaseAnalytics… Repro… AppsFlyerLib… launch { … } } } 58
59. class ComicActionCreator : ViewModel() { fun tapFollowComic(comic: Comic) { launch { dispatch(TapFollowComicAction(comic)) // 以下に実際のフォロー処理をかく … } } } 59 ・分析コードをかく必要がない ・機能開発だけに集中できる ・レビューコストが減る
60. 60 Entity Core Feature Production App Feature Feature Firebase Repro AppsFlyer
61. 61 Entity Core Feature Production App Feature Feature Tracker 機能開発がモジュール内だけで完結
62. 62 Entity Core Feature Production App Feature Feature Tracker 機能開発がモジュール内だけで完結 分析コードのロジックが Trackerモジュールにまとまる
63. 63 Firebase Tracker Repro Tracker AppsFlyer Tracker Tracker ツールそれぞれのロジックが 各モジュールにまとまる Production App
64. 64 Feature Tracker Production App Firebase Tracker 分析コード実装時の考える依存が少ない
65. 65 Production App Firebase Tracker 分析コード実装時の手を加えたり、考えるス クープはさらに小さい
66. 快適な実装 66
67. ミスの少ない実装 67
68. 最高 68