前任者から引き継いだ Eight Android アプリ内部の改善

前任者から引き継いだ Eight
Android アプリ内部の改善
2017/02/16
Sansan tech meetup #1 モバイルアプリ編
Sansan 株式会社 Eight 事業部辰濱健一

Copyright © 2014 Sansan, Inc. All rights reserved.
#Sansan_tech_meetup
1

【公開自粛】

> 自己紹介
4
• 辰濱健一 @tatsuhama50
• 徳島生まれ、奈良育ち、徳島県在住
• 2回転職するも、いずれも徳島県内勤務
• Tokushima.app 主催
（徳島でのスマホアプリの勉強会）
• Sansan株式会社所属
• 古民家を改修した神山ラボ勤務
• 名刺アプリ Eight Android 版の開発
• Kotlin, RxJava, Realm, … を使ってます
• リモートワーク
• 趣味は楽器演奏

> Sansan神山ラボ
5

> 趣味
6
• 楽器演奏（エレクトーン・トランペット）
• 徳島県内のイベントで、ちらほら演奏する機会を頂い
てます

Eight Android アプリ関係者の推移
7
#1 #2
#3 #4
#5 #6

8
#1 #2
#3 #4
#5 #6
前任者
社員社員
業務委託
社員
内定者

9
#1 #2
#3 #4
#5 #6
前任者
社員社員
業務委託
社員
内定者
・#5前任者
・#1社員
・#3業託
・#1社員
・#3業託
・#2社員
・#1社員
・#3業託
・#2社員
・#6社員

時代の推移
10
• 前任者時代
• Android エンジニア同士のレビューができない
• どうしても個性が出る
• マッチョな Activity
• バトンタッチ後
• Android エンジニアの複数人体制
• レビューによる議論も盛んに
• 技術顧問の招聘
• モダンなライブラリの紹介＆ハンズオン
• レビューへの参加

バトンタッチ後に導入したライブラリ等
11
• Realm
• Dagger2
• RxJava
• Kotlin
• Retrofit
• など…

バトンタッチ後に導入した CI 環境（Jenkins）
12
• 自動ビルド
• リリースバイナリ作成
• Fabric beta へのアップロード
• Verification
• Lint
• UnitTest
• 関数の UnitTest
• AndroidTest
• Realm のマイグレーションテスト

本題へ…
13

内部設計の改善
14
• レイヤーの整理・分割
• データの流れ、View の更新方法を整理

レイヤーの整理・分割
15
• Before
• マッチョな Activity
• View の構築
• 通信処理を呼び出す準備のコード
• 通信処理のコールバックで DB 書き込み、View 更新
• After
• 表示：Presentation Layer に分割
• 操作：UseCase に切り出して呼び出すだけ

Layers
16
Presentation Layer
Application Layer
Domain Layer
Infrastructure Layer
View Presenter
Activity
Entity UseCase
DB Network

Layers
17
Presentation Layer
Application Layer
Domain Layer
Infrastructure Layer
View Presenter
Activity
Entity UseCase
DB Network
マッチョ
Activity

データの流れ、View の更新方法を整理
18
• モデルの変更は別スレッドで行う
• View はモデル（Store）からの変更通知を受けて
表示を更新する
（Action が View を直接編集しない）

簡単なアプリで考える
19
• 仕様
• サーバ上のデータを TextView に表示する
• 処理
• ローカルにキャッシュがあれば、先にそれを表示する
• サーバからデータが取得できたら
• TextView に反映する
• 値をキャッシュしておく

例）
20

例）
21
→ Presenter に切り出せる

例）
22
→ UseCase に切り出せる

例）
23
色んな場所で
View を更新している！！

例）
24
Action（UseCase）
が表示を更新している！

例）
25
色んな場所で

例）
26
色んな場所で
モデルの変更通知を
受けて更新すべき

モデルの変更通知を受けて View を更新する…
27
• SharedPreference
• OnSharedPreferenceChangeListener
• BehaviorSubject
• onNext → subscribe
• Realm
• Finegrained notifications

28
• BehaviorSubject
• Realm
データの保存場所に依存せず、
View に Entity を提供したい

29
• BehaviorSubject
• Realm
データの保存場所に依存せず、
View に Entity を提供したい
Store
の導入

Store
30
• View は Store からの更新を受けて更新するだけ

Store
31
• View は Store からの更新を受けて更新するだけ

ObjectStore を定義
32
• 値を取得する
• 値の変更を Observable で受け取る
• これらの実装クラスを作成し、
View はこの実装クラスだけを監視して、
更新があったら反映するだけ

例）
33

例）
34
Action（UseCase）
×
不要

例）
35
Action（UseCase）
×
不要

Before / After
36
TextView
Shared
Pref.
初期化時に取得・反映
UseCase
※サーバからの取得処理
反映
反映

Before / After
37
TextViewStoreUseCase
TextView
Shared
Pref.
初期化時に取得・反映
UseCase
※サーバからの取得処理
反映
反映
Shared
Pref.
反映
反映
監視
反映
監視

スッキリ！

補足
39
• 更新通知には rx.Observable を使ってます
• Kotlin 使うと Rx 周りはとてもシンプルに書けます
• 1つの値を保持する ObjectStore だけでなく、集合を
保持する ArrayStore も用意しています
• デグレを支えてくれるテストチームさまさまm(__)m

Fin.

前任者から引き継いだ Eight Android アプリ内部の改善

Recommended

Recommended

More Related Content

What's hot

What's hot (20)

Viewers also liked

Viewers also liked (20)

Similar to 前任者から引き継いだ Eight Android アプリ内部の改善

Similar to 前任者から引き継いだ Eight Android アプリ内部の改善 (20)

More from 健一辰濱

More from 健一辰濱 (20)