Navigationを使った業務プロセスのモジュール化

Copyright © 2020 Present ANDPAD Inc. This information is confidential
and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止ラクマ × Voicy × ANDPAD ~ Android開発の裏側 ~ 株式会社アンドパッド工藤厚太郎 Navigationを使った業務プロセスのモジュール化

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止自己紹介工藤厚太郎開発本部 SWE モバイルアプリエンジニア 2020年1月入社 Twitter: derakudo 金融系SIer→toC向け事業会社→アンドパッド KotlinとかFlutterとか使ってます

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止今日話すこと Navigationコンポーネントを既存アプリへ導入した話表っぽいキラキラした話ではなく裏っぽい地味な内容です

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止 ANDPADとは ANDPADは現場の効率化から経営改善まで一元管理できるクラウド型建設プロジェクト管理サービスです

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止建築業界の課題ペインポイント市場規模社会課題・人材不足・低生産性現場課題・長時間残業・低粗利率 •建設産業 50兆円市場 •業者数約50万社特定建設業許可：45,742件一般建設業許可：445,304件 •従業者約500万人 •ANDPADの浸透率

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止マルチアプリの詳しい話はこちら https://speakerdeck.com/andpad/fu-shu-falseapuriwotong-shi-niririsusinagarasabisug urosusaserufang-fa

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止同じ(ように見える)機能はどうする? わかりやすい機能はライブラリ化済み - カメラ - 写真ピッカー建築ドメイン色が強かったりプロダクト固有の事情があると独自に実装されがち

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止ログイン画面 APIエンドポイントが違っていたので別で実装していたセキュリティ改善でアプデが多め - SSO - クライアント証明書 - etc ・・・

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止ライブラリ化した ANDPAD CHAT 短工事検査図面アプリリポジトリからログイン処理を分離 AndpadAuth().startLoginProcess(context)

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止 BaseActivityがある ※補足一番重要なのはアプリが価値を生み出すかなので、 BaseActivityを作った判断を批判する意図はありません

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止 BaseActivityとは画面の基本コンポーネントであるActivityを継承した神クラス。複数の画面で利用する処理やボイラープレートを実装して各画面のActivityで継承して利用する古のデザインパターンです。神クラスなのでそれぞれ関係が無いけど重要な機能が集まって可読性が落ちるといった問題が起こりがち。昔はライフサイクルメソッドに連動した処理を切り出すのが難しかったので止む終えない感じもしましたが LifeCycleObserver等の登場によって過去の遺産となりました(多分)。アンドパッドのBaseActivityも神として崇められた結果†Dagger†Componentの injectや共通エラー監視・特定の種類の画面でしか使わない処理などが大事に保管されています。本当は今すぐ消したい。

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止 Navigationコンポーネントとは Google製の画面遷移を抽象化するライブラリ https://developer.android.com/guide/navigation?hl=ja

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止今までの印象使うモチベーションがわからなかった小規模アプリ →AndroidStudioで参照先見ればなんとかなる大規模アプリ →もっと重要な課題がある

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止ナビゲーショングラフを作る <?xml version="1.0" encoding="utf-8"?> <navigation xmlns:app="http://schemas.android.com/apk/res-auto" xmlns:tools="http://schemas.android.com/tools" xmlns:android="http://schemas.android.com/apk/res/android" app:startDestination="@id/blankFragment"> <fragment android:id="@+id/blankFragment" android:name="com.example.cashdog.cashdog.BlankFragment" android:label="@string/label_blank" tools:layout="@layout/fragment_blank" > <action android:id="@+id/action_blankFragment_to_blankFragment2" app:destination="@id/blankFragment2" /> </fragment> <fragment android:id="@+id/blankFragment2" android:name="com.example.cashdog.cashdog.BlankFragment2" android:label="@string/label_blank_2" tools:layout="@layout/fragment_blank_fragment2" /> </navigation>

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止 Activityを作る val navHostFragment = supportFragmentManager.findFragmentById(R.id.nav_host_fragment) as NavHostFragment val navController = navHostFragment.navController

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止遷移方法を変える override fun onClick(view: View) { val action = SpecifyAmountFragmentDirections .actionSpecifyAmountFragmentToConfirmationFragment() view.findNavController().navigate(action) }

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止移行ステップ Developerサイトを参考にしました https://developer.android.com/guide/navigation/navigation-migrat e?hl=ja

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止 1. BaseActivityをSingleActivityにする 2. 画面ActivityをFragmentに変換 3. ナビゲーショングラフを作る 4. 遷移処理を生成された関数に置き換え

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止 BaseActivity問題と相性が良かった BaseActivity Activity1 Activity2 Fragment1 Fragment2 ViewModel2 HostActivity Fragment1 Fragment2 ViewModel2 親子関係が変わらないので書いた処理と既存処理を比較しやすいリファクタリングの最初のステップとしては難易度が低い

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止作業コストが低め構造を大きく変える必要がない →大掛かりなアーキテクチャ変更はリードタイムが長くなってしまうサービス的に意味のあるまとまった単位で改善できる →画面単位の改善だとミクロな視点になりがち

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止責務の分離ヘッダーがある画面は Activityで実装? メインセクションはFragment にする? UIはFragment! Before After Activityの役割が明確になった = Fragmentの用途もわかりやすくなった

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止デメリット普通にモジュール分割できているなら無理して使わなくても良いかも画面遷移パターンを整理しないと着手しにくい(デメリット?)

and was prepared by ANDPAD Inc. for the use of our client. It is not to be relied on by and 3rd party. Proprietary & Confidential 無断転載・無断複製の禁止 1. Navigationでライブラリを作った 2. BaseActivityとSingleActivityは相性が良かった 3. BaseActivity脱却の糸口が見えた

Navigationを使った業務プロセスのモジュール化

Navigationを使った業務プロセスのモジュール化

More Decks by くどう

Other Decks in Programming

Featured

Transcript