少人数規模の組織でレガシーコードと向き合う

2023/04/22

Transcript

1. 少人数規模の組織で レガシーコードと向き合う Kota Esaki
2. 自己紹介
3. @kke1229 SIer(銀行システム) → Start-up Engineer
4. レガシーコード改善に割く時間がないひと 聴いてほしい人
5. 自分含めエンジニア3人 JOINしたときのチームの状況
6. サービスのリニューアルに追われている JOINしたときのチームの状況
7. 入って半年、事件は起きた...
8. 一行のコードの書き忘れで ◦百万円の損失を産みかける なんとか回収できた....
9. なぜ起きたかをチームで話し合った
10. なぜ起きたかをチームで話し合った 技術的な負債が 原因だった • 誰もそのコードについてオーナーシップを持てな い状態だった • 設計時やPRでも間違いに気づくことができな かった
11. 今ある技術的な負債を洗い出してみよう
12. テストコードが実装されていない 技術的負債その1 自動テストが実装されていない または、実装されていてもテストの保守がされていない What?
13. テストコードが実装されていない 技術的負債その1 既存コード: テストするべき箇所をMUST, BETTER, OTHERに振り分けた 運用方法
14. テストコードが実装されていない 技術的負債その1 既存コード: テストするべき箇所をMUST, BETTER, OTHERに振り分けた MUST: あらゆるタスクの中で最優先で実装 運用方法
15. テストコードが実装されていない 技術的負債その1 既存コード: テストするべき箇所をMUST, BETTER, OTHERに振り分けた MUST: あらゆるタスクの中で最優先で実装 BETTER: Test dayをチームとして定期的に設けた 運用方法
16. テストコードが実装されていない 技術的負債その1 既存コード: テストするべき箇所をMUST, BETTER, OTHERに振り分けた MUST: あらゆるタスクの中で最優先で実装 BETTER: Test dayをチームとして定期的に設けた OTHER: その箇所が修正されるタイミングでテストコードを記述 運用方法
17. テストコードが実装されていない 技術的負債その1 新規コード テスト駆動開発の導入 運用方法
18. テストコードが実装されていない 技術的負債その1 運用を続けてどうなったか テストカバレッジ 5% → 64%
19. ライブラリの保守がされていない 技術的負債その2 • RubyのGemやTypeScriptのnpm, 言語バージョンが追いついていない • ライブラリの更新を怠ると依存しているコードが古くなる • 便利な機能の恩恵を受けられなくなる What?
20. ライブラリの保守がされていない 技術的負債その2 隔週チーム全員でUpdate DAYを行った 運用方法
21. ライブラリの保守がされていない 技術的負債その2 Ruby 2.2 → 3.0 Rails 4.0 → 7.0 Node 12.x → 16.x MariaDBなどその他いろいろ 運用を続けてどうなったか
22. 少人数規模で大切なこと 小さく、はやく始めることを 心がける
23. まとめ 技術的負債を減らす順序 1. まずチームとして負債を認知する 2. 負債に対しての(心理的、物理的)障壁と優先順位を考える 3. 障壁を取り除く(組織としての理解、ツールのリニューアルなど) 4. 決めた優先順位をもとに仕組みとして運用する