長期運用アプリのリファクタリングを考える

by shinmiy

Slide 1

Slide 1 text

長期運用アプリのリファクタリングを考える @shinmiy Oct 5, 2022 / DroidKaigi 2022 Day 1 1

Slide 2

Slide 2 text

2 @shinmiy Android Engineer @ Merpay

Slide 3

Slide 3 text

この物語はフィクションです。  実在の人物・団体・事件とは  一切関係ありません。 

Slide 4

Slide 4 text

4 〜2016

Slide 5

Slide 5 text

5 2017/2018 What's new in Kotlin 1.3 https://kotlinlang.org/docs/whatsnew13.html Android and Architecture https://android-developers.googleblog.com/2017/05/android-and-architecture.html

Slide 6

Slide 6 text

6 2021 Jetpack Compose is now 1.0: announcing Android’s modern toolkit for building native UI https://android-developers.googleblog.com/2021/07/jetpack-compose-announcement.html Rebuilding our guide to app architecture https://android-developers.googleblog.com/2021/12/rebuilding-our-guide-to-app-architecture. html

Slide 7

Slide 7 text

Slide 8

Slide 8 text

8 改善が必要なアプリとは？

Slide 9

Slide 9 text

9 今後の変更に対応することが難しくなってきたアプリ

Slide 10

Slide 10 text

10 サービスやチームにも様々な変化： ● 担当者が変化 ○ 人の出入りがある ● 仕様が変化 ○ リリース当初の想定から大きく変化

Slide 11

Slide 11 text

Slide 12

Slide 12 text

Slide 13

Slide 13 text

13 今後の変更に対応することが難しくなってきたアプリ機能開発しづらい😩 コードの学習コストが高い😩 開発者のモチベーション😩

Slide 14

Slide 14 text

14 アーキテクチャがしっかりしていて、忠実に守られている仕様が明確で変化が少ない長期運用 != 改修が必要

Slide 15

Slide 15 text

Slide 16

Slide 16 text

16 今後の変更がしやすいアプリを目指す

Slide 17

Slide 17 text

17 アプリアーキテクチャガイド https://developer.android.com/topic/architecture

Slide 18

Slide 18 text

18 メリット ● ベストプラクティスとして無理がないコミュニティとの対話の末できたもの。今の環境を反映していて、最終的には開発者が判断 ● 共通言語ができた同じ前提を共有できて関連する知見が溜まりやすくなった。オンボーディングもしやすい

Slide 19

Slide 19 text

19 メリット ● ベストプラクティスとして無理がないコミュニティとの対話の末できたもの。今の環境を反映していて、最終的には開発者が判断 ● 共通言語ができた同じ前提を共有できて関連する知見が溜まりやすくなった。オンボーディングもしやすい

Slide 20

Slide 20 text

20 UI アプリ開発ツールキット Jetpack Compose https://developer.android.com/jetpack/compose

Slide 21

Slide 21 text

21 Navigating with Compose / Navigate with arguments https://developer.android.com/jetpack/compose/navigation#nav-with-args

Slide 22

Slide 22 text

22 アプリアーキテクチャガイドに沿った設計 + Jetpack Compose

Slide 23

Slide 23 text

Slide 24

Slide 24 text

Slide 25

Slide 25 text

Slide 26

Slide 26 text

26 1回目理解のため 2回目整理のため 3回目コミットのため

Slide 27

Slide 27 text

Slide 28

Slide 28 text

Slide 29

Slide 29 text

Slide 30

Slide 30 text

Slide 31

Slide 31 text

Slide 32

Slide 32 text

Slide 33

Slide 33 text

Slide 34

Slide 34 text

Slide 35

Slide 35 text

35 git checkout .

Slide 36

Slide 36 text

36 1回目理解のため 2回目整理のため 3回目コミットのため

Slide 37

Slide 37 text

37 git checkout . 🤔

Slide 38

Slide 38 text

38 1回目理解のため 2回目整理のため 3回目コミットのため

Slide 39

Slide 39 text

39 xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx class FooActivity: Activity { override fun onCreate() { doHoge() doFuga() } override fun onDestroy() { doHuga() } private fun doHoge() { why.are = you.reading the.fine.print() if (theres.nothing) { here.to(see) } } private fun doFuga() { db.hoge { result -> if (result.isSuccessful) { val builder = AlertDialog.Builder(this) builder.setTitle(R.string.submit_dialog_title) builder.setMessage(R.string.submit_dialog_message) builder.setPositiveButton(R.string.ok) { p0, p1 -> Logger.log(SubmitLog()) val message = R.string.submit_complete Toast.makeText(this, message, Toast.LENGTH_LONG) .show() finish() } } } } } xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx xx class FooActivity: Activity { override fun onCreate() { doHoge() doFuga() } override fun onDestroy() { doHuga() } private fun doHoge() { why.are = you.reading the.fine.print() if (theres.nothing) { here.to(see) } } private fun doFuga() { db.hoge { result -> if (result.isSuccessful) { this.showDialog() } else { finish() } } } }

Slide 40

Slide 40 text

Slide 41

Slide 41 text

41 集中して一度のセッションで終わらせる

Slide 42

Slide 42 text

42 🍵

Slide 43

Slide 43 text

Slide 44

Slide 44 text

Slide 45

Slide 45 text

Slide 46

Slide 46 text

Slide 47

Slide 47 text

Slide 48

Slide 48 text

Slide 49

Slide 49 text

Slide 50

Slide 50 text

Slide 51

Slide 51 text

Slide 52

Slide 52 text

Slide 53

Slide 53 text

Slide 54

Slide 54 text

Slide 55

Slide 55 text

Slide 56

Slide 56 text

56 Navigating with Compose / Navigate with arguments https://developer.android.com/jetpack/compose/navigation#nav-with-args