Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Speaker Deck
PRO
Sign in
Sign up
for free
Android cookpadLiveの開発改善 / cookpad_tech_kitchen#23
star_zero
January 30, 2020
Programming
1
440
Android cookpadLiveの開発改善 / cookpad_tech_kitchen#23
star_zero
January 30, 2020
Tweet
Share
More Decks by star_zero
See All by star_zero
star_zero
4
430
star_zero
3
370
star_zero
2
410
star_zero
0
120
star_zero
0
250
star_zero
8
2k
star_zero
0
230
star_zero
2
680
star_zero
3
5.6k
Other Decks in Programming
See All in Programming
bkuhlmann
4
270
sters
2
120
dqneo
3
290
neripark
3
560
morimorihoge
1
140
mitohato14
0
110
nearmugi
0
110
yoshinoriiiii
0
110
manfredsteyer
PRO
0
180
williln
0
210
hackz_inc
0
310
kgmyshin
1
440
Featured
See All Featured
rocio
155
11k
holman
288
130k
swwweet
206
6.9k
paulrobertlloyd
72
1.4k
mthomps
39
2.3k
samanthasiow
56
6.3k
lara
590
61k
bkeepers
408
58k
vanstee
117
4.8k
sstephenson
145
12k
gr2m
83
11k
tenderlove
53
3.5k
Transcript
Android cookpadLiveの 開発改善 Cookpad Tech Kitchen #23 2020/01/30
About me •Kenji Abe •2018/07入社 •Android cookpadLive •Google Developers Expert
for Android •Twitter: @STAR_ZERO
話すこと •Android cookpadLiveで改善してきた色々なこと ‣ たくさんあるので一部抜粋して •今後やりたいこと
アーキテクチャの変更
VIPER MVVM
VIPER •View / Interactor / Presenter / Entity / Router
•iOS由来のアーキテクチャ •https://cheesecakelabs.com/blog/ios-project- architecture-using-viper/
View Presenter Interactor Entity Router
VIPERの問題点 •Jetpackとの相性 ‣ これらを活かせないため効率と品質を上げられない ‣ Lifecycleの管理が面倒でクラッシュも起きやすい •コード量が多い ‣ Interfaceによるやりとり ‣
あちこちにViewの更新コード
MVVM •Model / View / ViewModel •Google 推奨のアーキテクチャ •https://developer.android.com/jetpack/docs/guide
Activity / Fragment ViewModel Repository Room(DB) Retrofit(API) AppSync LiveData
MVVMの良いとこ •Jetpackが活用しやすい ‣ 特にViewModel, LiveData, DataBinding便利 ‣ これらを活用することでバグが減らせる ‣ 効率よく開発できる
‣ その他Jetpackライブラリも導入が簡単
Activity / Fragment ViewModel Repository Room(DB) Retrofit(API) AppSync LiveData Jetpack
Jetpack Jetpack
MVVMの良いとこ •Google推奨(デファクトスタンダード) ‣ 知見やサンプルがたくさんある ‣ 新しく来た人が理解しやすい(可能性が高い) ‣ 去っていく人も次に活かせる(可能性が高い)
Modularization
Single Module Multi Module
Single Module •スマートフォンアプリのみ ‣ AndroidTV / FireTVとコードが重複 •ビルドスピードの問題 •依存関係の絡まり
Multi Module •スマートフォンアプリ、AndroidTV、FireTV ‣ APIアクセスなどは共有 •ビルドスピードの改善 •関心事の分離
app 最初のSingleModule時代
core app 共通部分をcoreに分ける
core smarttv app androidtv firetv AndroidTV / FireTVをプロジェクト統合
core smarttv app androidtv firetv appsync AppSyncをcoreから分離
core smarttv appcore app androidtv firetv appsync app専用の共通モジュールを作成
core smarttv appcore app androidtv firetv features appsync 機能単位にモジュールを分ける
core smarttv appcore app androidtv firetv features appsync 機能単位にモジュールを分ける まだ途中
コメント表示改善
改善前 •受信したコメントをそのまま表示 ‣ 描画負荷が高い ‣ 場合によってはクラッシュを引き起こす •RecyclerViewのアニメーション ‣ デフォルトがそのまま ‣
大量のコメントで違和感
改善後 •受信したコメントを少しバッファリングして表示 ‣ ある程度まとめ描画させて、描画負荷を軽減 •RecyclerViewのアニメーションは無効に ‣ 無効にしたほうが自然な感じ
Activity Repository ViewModel AppSync LiveData
Activity Repository ViewModel AppSync LiveData RxJavaのObservableに変換
Activity Repository ViewModel AppSync LiveData ここで数msバッファリング
Activity Repository ViewModel AppSync LiveData LiveDataでコメントActivityへ
コード自動生成
コード自動生成 •現状、サーバー側がRails Garageを使ってる ‣ 欲しいレスポンスをGETパラメータで指定する ‣ 例: /api/episodes?fields=hoge,foo •面倒なので、Retrofitの戻り値から自動生成 ‣
Annotation Processing
/api/episodes?fields=hoge,foo interface APIService { @Garage @GET("/api/episodes") fun getEpisodes(): Single<List<Episode>> }
data class Episode( val hoge: String? val foo: String? ) このアノテーションから自動生成
デバッグツールの活用
デバッグツールの活用 •Hyperion ‣ アプリ実行中にデバッグメニューから色々できる ‣ 設定値をクリアしたり、APIのエンドポイントを切り替え たり ‣ https://github.com/willowtreeapps/Hyperion-Android
None
デバッグツールの活用 •Flipper ‣ デスクトップアプリを使ったデバッグツール ‣ Stethoみたいなもの ‣ ログ、Preferences、Layoutなどの確認 ‣ プラグインを作れる
‣ https://fbflipper.com/
None
デバッグツールの活用 •Flipper ‣ プラグインを作成して、ダミーのLive配信 • OkHttp Interceptorでダミーデータを差し込む ‣ 簡単な見た目の確認すぐできる ‣
ステージングの配信を作る必要がない
None
OkHttp API Server Flipper OkHttp Interceptor レスポンス 書き換え ダミーデータ
今後やりたいこと
今後やりたいこと •Coroutinesの導入 •テストを充実 •リソースの整理 •さらなる効率化
ありがとうございました