Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
Androidアプリをいつまでも楽しく開発し続けるための取り組み
Search
こやまカニ大好き
February 13, 2020
Programming
5
2.4k
Androidアプリをいつまでも楽しく開発し続けるための取り組み
cookpad.apk #4
https://cookpad.connpass.com/event/165136/
こやまカニ大好き
February 13, 2020
Tweet
Share
More Decks by こやまカニ大好き
See All by こやまカニ大好き
マルチモジュールアプリの画面遷移処理実装
nein37
0
6.1k
クックパッド Android アプリのマルチモジュール化とデモアプリの活用
nein37
1
6.6k
2020年代の WebView 実装 / saikou_no_webview_2021
nein37
2
11k
minSdkVersion=21にしてから1年経った話
nein37
8
2.2k
Androidアプリエンジニアの基礎知識
nein37
16
11k
クックパッドアプリのマルチモジュール化への取り組み
nein37
11
10k
Android のセキュリティよくなってきた話
nein37
3
2.4k
Androidアプリのデザイン整理への取り組み
nein37
1
950
Androidアプリのタブレット向けレイアウト
nein37
4
1.1k
Other Decks in Programming
See All in Programming
macOS でできる リアルタイム動画像処理
biacco42
6
1.7k
外部システム連携先が10を超えるシステムでのアーキテクチャ設計・実装事例
kiwasaki
1
220
Importmapを使ったJavaScriptの 読み込みとブラウザアドオンの影響
swamp09
4
1.2k
RailsのPull requestsのレビューの時に私が考えていること
yahonda
5
1.7k
PHP でアセンブリ言語のように書く技術
memory1994
PRO
1
150
CPython 인터프리터 구조 파헤치기 - PyCon Korea 24
kennethanceyer
0
240
EventSourcingの理想と現実
wenas
6
2.1k
CSC509 Lecture 09
javiergs
PRO
0
100
Googleのテストサイズを活用したテスト環境の構築
toms74209200
0
270
リリース8年目のサービスの1800個のERBファイルをViewComponentに移行した方法とその結果
katty0324
5
3.6k
とにかくAWS GameDay!AWSは世界の共通言語! / Anyway, AWS GameDay! AWS is the world's lingua franca!
seike460
PRO
1
540
WEBエンジニア向けAI活用入門
sutetotanuki
0
300
Featured
See All Featured
Git: the NoSQL Database
bkeepers
PRO
425
64k
How to Create Impact in a Changing Tech Landscape [PerfNow 2023]
tammyeverts
46
2.1k
5 minutes of I Can Smell Your CMS
philhawksworth
202
19k
Designing Experiences People Love
moore
138
23k
Producing Creativity
orderedlist
PRO
341
39k
YesSQL, Process and Tooling at Scale
rocio
167
14k
Understanding Cognitive Biases in Performance Measurement
bluesmoon
26
1.4k
Building an army of robots
kneath
302
42k
Imperfection Machines: The Place of Print at Facebook
scottboms
264
13k
Helping Users Find Their Own Way: Creating Modern Search Experiences
danielanewman
29
2.2k
Raft: Consensus for Rubyists
vanstee
136
6.6k
jQuery: Nuts, Bolts and Bling
dougneiner
61
7.5k
Transcript
Androidアプリをいつまでも楽しく開発 し続けるための取り組み Cookpad.apk #4
自己紹介 こやまカニ大好き(@nein37) モバイル基盤部所属 社内全Androidプロジェクトの開発効率化と技術サポート minSdkVersion 上げたりモジュール分割したりしてます 2
android-cookpad アプリのこ れまでの歩み 3
2012年にAndroidアプリをリリース • 一つのチームでアプリを開発 • 当初はほぼWebViewで構成されたアプリ ◦ コンテンツの更新や新機能をすばやくユーザーに提供することを目的としていた ◦ 探すと当時のインタビュー記事が出てくる ◦
https://el.jibun.atmarkit.co.jp/rails/2012/10/html5-d1ba.html • JavascriptInterface を利用 ◦ Javascript からネイティブ実装を呼び出すための仕組み ◦ この時入った仕組みの一部はいまだにクックパッドアプリで利用されている • 課題 : WebViewによるテスト、不具合検知の難しさ 4
2014年にフルリニューアル • ネイティブアプリとしてリニューアル • ネイティブ実装によるユーザー体験の向上、メンテナンス性の向上 • 当時は Android Developers の実装ガイドなどが充実していなかったため、
書いた時期や開発者によって実装スタイルがバラバラ • このころまでは一つのチームで開発を行っていた • 課題 : いつ/どの機能がリリースされるのか周知することが難しい 5
モバイル基盤誕生 • 各部署のメンバーが担当する機能の開発を行う方式へ • CI整備やリリース作業などどの部署にも属さない役割のカバーや全体的な効率 化のためにモバイル基盤グループが誕生 • アプリの状態監視や各機能のバグ修正も行っていた • 課題
: リリースごとに各部署の調整が必要になっていて高コスト 6
フルリニューアル後リリースフローを整備 • アプリの運用ルールを整備 ◦ 2週間毎のリリース ◦ 開発・テスト・リリース後監視フローの整理 ◦ リリーマネージャ・監視の手順化・持ち回り •
課題:各施策・各部署の連携と開発リソース調整 7
モバイル基盤の役割の明確化 • モバイル開発に関する課題を解決する部署へ • バグ調査やリリースまわりの調整から開発しやすさの向上へ • Danger などのコードレビュー負荷軽減 • CI・開発環境の改善
• ビルド時間の計測・改善に向けた努力 • リリースに関する負荷の軽減 8
楽しいAndroid開発#とは 9
Androidアプリの開発 • ユーザーに価値を届けるための開発 • 最小のコストで機能開発/バグ修正を行う • 最小のコストでユーザーの手元に届ける • コスト=時間+手間 10
時間がかかると楽しくない • ビルド時間が長い • CIの待ち時間が長い • 変更がマージされてから リリースまでの時間が長い 11
• コードをどこに書けば良いのかわからない • コードをどう書けば良いのかわからない • 認証処理・ユーザー状態の判定が難しい • 古いJavaコード触りたくない • 意味不明なビルドバリアント
手間がかかると楽しくない 12
モバイル基盤部の役割 • 時間と手間を最小限にするために取り組む • アプリ開発を魅力的なものにする • ユーザーに新しい価値を 素早く届けられるようにする • モバイル基盤部の目標にも「生産性について」
という項目があり、機能開発にかかる時間を 最小化することを明確に目標にしている 13
実際の取り組み(時間) 14
ビルド時間の削減 • ビルドの時間が遅いと全てが遅いので優先度は高い • まずは測定するところから始める ◦ ビルド時間を可視化できると成果がでたときにも嬉しい • 差分ビルド時間の最小化にフォーカスする ◦
開発者の手元の作業によって大きく変動するので細かい数値は追わないほうが良い ◦ 一定期間の傾向としての増減で見ている 15
ビルド時間の削減 • ビルド時間を短くするためにできることは公式に書いてある ◦ https://developer.android.com/studio/build/optimize-your-build • モジュールを分割してキャッシュを有効利用する ◦ Gradleキャッシュへの理解を深める •
Apply Changes がまともに動くようにする ◦ https://developer.android.com/studio/run#apply-changes ◦ 使えない場面のほうが多いが使えたほうが良い場面はもちろんある ◦ debug build variant がまともに動くようにする 16
ビルド時間の削減 • minSdkVersion を上げる ◦ 過去のOSバージョンをサポートするためにやっている処理が不要になるビルド時間を短縮 ◦ 特に minSdkVersion 21
未満の場合に有効 17
ビルド時間の削減 • PCのスペックを上げる ◦ モバイル開発者は全員 MacBook Pro ◦ CPU 2.4GHz
8コア/RAM 32GB ◦ 実際の計測結果などを Gradle build scan で 記録に残しつつ必要スペックを決めている ◦ 検証手順もきちんと残す 18
ビルド時間の削減 • ビルド時間の改善は影響の大きそうなところから崩していく • 後はひたすらモジュール分割とGradle/Gradle Plugin の改善についていく • アプリの仕様やマシンの見直しも必要ならやっていく 19
CI待ち時間の改善 • 最近 CodeBuild に切り替えた ◦ https://techlife.cookpad.com/entry/2020/01/30/100000 • CIビルド時間はまだ改善の余地がありそう •
ジョブの分割によって問題のある箇所がわかりやすくなった 20
リリース待ち時間の改善 • 毎週リリース ◦ これまでは重要な施策がある場合そちらを優先して スケジュールを調整していたため、 実際にリリースされるタイミングは不透明だった ◦ https://techlife.cookpad.com/entry/2018/09/14/090000 •
リリースフローの(半)自動化 • 問題発生時のフロー整備 ◦ 緊急リリースorリリーススキップの判断基準 ◦ 実際の緊急リリース作業のドキュメント化 21
実際の取り組み(手間) 22
アーキテクチャ/コードベースの改善 • 無法状態(+若干のMVP) -> VIPER ◦ 現在画面のおよそ35%がVIPER化されたコード ◦ サンプルコードを含むドキュメントを整備 •
既存画面のVIPER化よりも共有コードの改善を優先 • Kotlin 化は昨年後半でかなり進み現在半分以上が Kotlin 23
レガシーコードの改善 • DIの(再)導入 ◦ 以前 Toothpick を導入していた ◦ Toothpick のコード生成が失敗することがあった
◦ (特に)シングルトンインスタンスの管理が分散 • KOINを導入 ◦ これまでのシングルトンインスタンス管理もすべて DIに寄せる ◦ DIの利用方法・モジュール分割時の記述方法をドキュメント化 ◦ 新規のコードではKOINの利用を推奨(レビュー時の指摘/お手本PR) 24
レガシーコードの改善 • 認証・通信処理の改善 ◦ 最初は認証モジュール・通信モジュールで分離していた ◦ 通信途中での再認証など特殊な処理が多いため、認証・通信モジュールにした ◦ Kotlin の
internal と組み合わせて可視性を最適化 25
レガシーコードの改善 • アプリケーション初期化処理の改善 ◦ ApplicationInitializer というクラスがアプリ内の初期化処理を行っていた ◦ シングルトンインスタンスの初期化など処理順を意識する必要があった ◦ 「depends
on XXX」 という真偽不明のコメントにより触れない処理たち • 初期化処理を KOIN で行えるものは KOIN で行う • 無理なものはきちんとカテゴリごとに整理してこまかくコメント 26
レガシーコードの改善 • その他の取り組みなど ◦ Cookpad.apk や 技術ブログで随時共有しています ◦ https://techlife.cookpad.com/entry/2019/10/21/144205 ◦
https://techlife.cookpad.com/entry/2020/01/16/090000 ◦ https://cookpad.connpass.com/event/137772/ 27
リリース デバッグ ビルドバリアントの整理 • モジュールによる設定解決 ◦ 接続先設定などのモジュール化 ◦ デバッグ機能のモジュール化 ◦
モジュールの依存解決によって アプリの設定・機能の切り替えを行う • 実装方法は2種類 ◦ 同一パッケージ/同一クラスにコードを配置 ◦ インターフェイスを共通モジュールにおいて 実装だけを切り替える 28 App_Release App_Debug Feature_Debug Feature_Release Settings_Release Settings_Debug Settings_Base Libraries Features
まとめ 29
まとめ • アプリ開発にかかる時間と手間を最小化することで 楽しく開発を続けることができる • クックパッドではモバイル基盤が業務として 時間と手間を最小化に取り組んでいる • これからも引き続きユーザーに価値を届けるための開発を サポートしていく
30
おまけ 31
minSdkVersion 23 のお知らせ • クックパッドアプリでは3月からminSdkVersion を 23 にします ◦ AppCompat
v1.1.0 が 5.x 端末でクラッシュする問題 ◦ 5.x で `android:foreground` 使えない問題 ◦ Drawable への tint 適用時の挙動などが 5.x で異なる問題 ◦ Samsung 5.x 端末での kotlinx.coroutines クラッシュ問題 など 32