Androidアプリをリアーキテクチャした話/Android-rearchitecture

Transcript

1. Androidアプリを リアーキテクチャした話 2021.09.16 morizooo Tech Night © 2021 Mirrativ, Inc.
2. •Profile •morizooo(@morizo_999) •ミラティブAndroid担当(2018/2〜) •マイブーム:高級食パン,APEX
3. 99 • リアーキテクチャをした理由 • リアーキテクチャを実施した流れ • 現在取り組んでいること 今日話すこと
4. 目次 99 リアーキテクチャを した理由
5. 99 • 副業メンバーからのフィードバック ◦ 既存のコードのキャッチアップがつらい ◦ ルールがないのでどのように書けばいか迷う • 既存コードが原因で離脱してしまうことも →　チームとしてスケールするために見直しが必要 理由
6. 99 • 厳密なルールがないのでActivityに処理が集中 ◦ 6000行のActivityが存在😇 • 最凶のBaseクラス
 ◦ すべてのActivityが1000行のクラスを継承
 ◦ 状態の変更が意図せず起こってしまう
 • EventBusでたらい回しにされる
 ◦ たらい回しの途中で状態が更新されていることもある
 • 度重なるパッチコード対応で意図がわからない • Android Annotationsの学習コスト 既存コードの問題の例
7. 目次 99 リアーキテクチャを 実施した流れ
8. 99 • Baseクラスを解体 • ライブラリへの依存(Android Annotationsなど)をなくす • パッケージ構造の整理 • Kotlin化する 詳細はTech Blogへ！！ https://tech.mirrativ.stream/entry/android-dx 既存コードの問題を整理(2019/04 - 2019/10)
9. 99 • Fluxアーキテクチャを選定 ◦ 状態管理の問題を解消できそうだった ◦ iOSもFluxを選定していた • Fluxでのリファレンス実装を作成 ◦ 12月時点のPRで新規画面ではFlux実装を必須にした ◦ 現在でも都度都度議論して随時更新している 理想形の作成と周知(2019/10 - 2019/12)
10. 99 FLUXアーキテクチャについて ActionCreator Dispatcher Store View User Interaction
 ActionEvent
 EventBus
 State
 (BindModel)

11. 99 • コードはTech Blogに詳細を書いたので見てください！！ https://tech.mirrativ.stream/entry/2021/06/01/102044 FLUXアーキテクチャについて
12. 99 • 触ったところをFluxに書き換えていくようにした ◦ プロダクト開発が最優先 ◦ 既存コードの変更はnice to have ◦ 追加開発に時間がかかるようになってきたものは、長めに 開発期間をもらう等の交渉はした ◦ QA時に影響がありそうな部分は伝えて見てもらう ▪ targetSDKのバージョンアップはフルチェック →　お祭りとして強気に変更していた FLUXアーキテクチャに置き換え(2020/01 -)
13. 99 • 画面リニューアルの場合はまるっと書き換えた ◦ 既存のコードは引きづられるので見ないほうがいい • 更新がない画面はそのまま残っている ◦ リーアキテクチャするメリットが無い 現在では普段触る部分はFluxになっている FLUXアーキテクチャに置き換え(2020/01 -)
14. 目次 99 現在 取り組んでいること
15. 99 • Multi Module化 ◦ ビルド速度早くしたい ◦ 機能単位のモジュールは作ったので移行中 • Jetpack Composeの導入 ◦ 状態が以前の状態に左右されず処理がわかりやすい ◦ リファレンス実装作って導入済み ▪ 今は各々実装しながら議論しているところ 現在の取り組み
16. 99 • 今QのOKR 現在の取り組み
17. 99 • 段階的にリアーキテクチャをすすめた ◦ まずは既存コードの見通しをよくする ◦ 理想系を決めて周知し、プロダクト開発と一緒に書き換え ていく • リアーキテクチャの効果 ◦ 副業時にコードが辛いと離脱した人も今では社員に！ ◦ 最近Joinしたメンバーからも好評 • 更に一緒に改善していく仲間を探しています！！！ まとめ