エンジニア1年目で複雑なコードの改善に取り組んだ話

by mtnmr

Slide 1

Slide 1 text

Slide 2

Slide 2 text

Slide 3

Slide 3 text

Slide 4

Slide 4 text

Slide 5

Slide 5 text

AbemaTV, Inc. All Rights Reserved  5 エンジニアになって思ったこと ● 新しい技術は移り変わるので、リリースから数年経ったアプリにはレガシーコードが必ずある ● レガシーコードがアプリの根幹となる機能を今も支えていたりして簡単には変えられない ● 今のアプリ開発のスタンダードを知っていることはもちろん大事だがそれだけでは業務できない... レガシーコードに向き合う力が必要

Slide 6

Slide 6 text

Slide 7

Slide 7 text

Slide 8

Slide 8 text

Slide 9

Slide 9 text

Slide 10

Slide 10 text

Slide 11

Slide 11 text

AbemaTV, Inc. All Rights Reserved  11 ABEMA モバイルアプリについて ● ABEMAは発足から10年目 ○ Androidのリポジトリも2015年7月が最初のコミット ○ 2016年2月29日にAndroidモバイルアプリがリリース ● 提供機能 ○ 24時間365日番組型の放送 ○ ビデオ・ライブ配信などの動画視聴 ○ ABEMAプレミアム ○ パートナーサービス new!

Slide 12

Slide 12 text

Slide 13

Slide 13 text

AbemaTV, Inc. All Rights Reserved  13 リアーキテクチャの取り組み Clean Architectureベースのアーキテクチャ + KMPの採用『ABEMA モバイルアプリにおけるリアーキテクチャの取り組みと展望』（CA BASE NEXT 2021） https://ca-base-next.cyberagent.co.jp/2021/sessions/abema-mobile-rearchitecture/

Slide 14

Slide 14 text

Slide 15

Slide 15 text

Slide 16

Slide 16 text

Slide 17

Slide 17 text

Slide 18

Slide 18 text

Slide 19

Slide 19 text

Slide 20

Slide 20 text

Slide 21

Slide 21 text

Slide 22

Slide 22 text

Slide 23

Slide 23 text

Slide 24

Slide 24 text

Slide 25

Slide 25 text

Slide 26

Slide 26 text

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

Slide 36

Slide 36 text

Slide 37

Slide 37 text

Slide 38

Slide 38 text

Slide 39

Slide 39 text

AbemaTV, Inc. All Rights Reserved  設計 39 Clean Architecture ● レイヤーごとに関心の分離 ● 依存は外側から内側にだけ向かう Robert C. Martin『The Clean Architecture』 https://blog.cleancoder.com/uncle-bob/2012/08/13/the-clean-architecture.html

Slide 40

Slide 40 text

Slide 41

Slide 41 text

Slide 42

Slide 42 text

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

Slide 57

Slide 57 text

Slide 58

Slide 58 text

Slide 59

Slide 59 text

Slide 60

Slide 60 text

Slide 61

Slide 61 text

Slide 62

Slide 62 text

AbemaTV, Inc. All Rights Reserved  課金基盤のFlux解体プロジェクトまとめ 62 キャッチアップ設計実装 QA テストリリース・既存実装書き起こし・分からないままにせず　インプット・リアーキテクチャ・インターフェース分離・定期的なレビュー・設計をもとに分担・トランクベース・機能の担保（網羅性）・理解のしやすさ（可読性）・Feature Flag ・旧実装の削除まで

Slide 63

Slide 63 text

Slide 64

Slide 64 text

AbemaTV, Inc. All Rights Reserved  復習・今回のプロジェクトの目的 64 ● 課金処理を拡張していきたかったが既存のまま拡張するコストが高く、先にレガシーコードを作り替えるのが良いという判断 ● コード改善に時間をかけられない問題 ○ 継続的な成長・開発速度の向上のためにやった方が良い場合がある ○ 新規機能の追加・ユーザー体験の改善と比較してメリットが見えにくい、コストと効果が見合わない ○ 今回は目的が明確にあり人的コストをかけられた

Slide 65

Slide 65 text

Slide 66

Slide 66 text

Slide 67

Slide 67 text

Slide 68

Slide 68 text

Slide 69

Slide 69 text

Slide 70

Slide 70 text

Slide 71

Slide 71 text

Slide 72

Slide 72 text

Slide 73

Slide 73 text

Slide 74

Slide 74 text

Slide 75

Slide 75 text

AbemaTV, Inc. All Rights Reserved  読みやすいコードを書く 75 ● コード行数が短いほど良いわけではない ● メソッドの責務分離 ● クラス・メソッドの命名 ● 適切なコードコメントなど Dustin Boswell, Trevor Foucher (2012) 『リーダブルコード』オライリージャパン Chris Zimmerman (2023) 『ルールズ・オブ・プログラミングより良いコードを書くための21のルール』オライリージャパン

Slide 76

Slide 76 text

Slide 77

Slide 77 text

AbemaTV, Inc. All Rights Reserved  まとめ 77 ● 自分が経験したレガシーコード改善の詳細プロセス ○ コミュニケーションをしっかり取りながらプロジェクト完遂し、その後の効果が大きい改善になった ○ これが常に最適解というわけではなくその時の状況やレベルに合わせて選択する ● 1年目でレガシーコードに向き合って多くの学びが得られた ○ 経験と自信 ○ 自分が残すコードやドキュメントへの考え方

Slide 78

Slide 78 text

AbemaTV, Inc. All Rights Reserved  参考 78 過去発表資料 ● 『ABEMA モバイルアプリにおけるリアーキテクチャの取り組みと展望』（CA BASE NEXT 2021） https://ca-base-next.cyberagent.co.jp/2021/sessions/abema-mobile-rearchitecture/ 書籍 ● マイケル•C•フェザーズ（2009）『レガシーコード改善ガイド』（ウルシステムズ株式会社監訳, 平澤章, 越智典子, 稲葉信之, 田村友彦, 小堀真義訳）翔泳社 ● David Scott Bernstein （2019）『レガシーコードからの脱却 ―ソフトウェアの寿命を延ばし価値を高める9つのプラクティス』（吉羽龍太郎, 永瀬美穂, 原田騎郎, 有野雅士訳）オライリージャパン ● 日高正博, 小西裕介, 藤原聖, 吉岡毅, 今井智章（2018）『Androidアプリ設計パターン入門』PEAKS出版 ● Robert C.Martin（2018）『Clean Architecture 達人に学ぶソフトウェアの構造と設計』（角征典, 高木正弘訳）アスキードワンゴ ● Dustin Boswell, Trevor Foucher（2012）『リーダブルコード』（角征典訳）オライリージャパン ● Chris Zimmerman（2023）『ルールズ・オブ・プログラミングより良いコードを書くための21のルール』（久富木隆一訳）オライリージャパンサイト ● Flux https://facebookarchive.github.io/ﬂux/docs/in-depth-overview ● Robert C. Martin『The Clean Architecture』 https://blog.cleancoder.com/uncle-bob/2012/08/13/the-clean-architecture.html

Slide 79

Slide 79 text

Slide 80

Slide 80 text