Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
DMMプラットフォームにおけるコード品質を改善する取り組みの理想と現実
Search
pospome
November 21, 2023
Programming
3
2.3k
DMMプラットフォームにおけるコード品質を改善する取り組みの理想と現実
"コード品質向上のいろは - 先達に学ぶ実践例 Lunch LT" の資料です。
https://findy.connpass.com/event/300912/
pospome
November 21, 2023
Tweet
Share
More Decks by pospome
See All by pospome
DMMプラットフォームがTiDB Cloudを採用した背景
pospome
9
4.5k
DDDはなぜ難しいのか / 良いコードの定義と設計能力の壁
pospome
29
11k
マイクロサービス環境におけるDB戦略 in DMMプラットフォーム
pospome
12
3.4k
組織全体で開発生産性に取り組むために 専門チームを作った話
pospome
2
1.5k
DMMプラットフォームにおける GKE を利用した プラットフォームエンジニアリングへの 取り組み
pospome
1
450
(再アップロード)Microservices & APIs
pospome
0
77
(再アップロード)Datastore/Go のデータ設計と struct の振る舞いについて
pospome
0
75
マイクロサービス環境におけるToilを削減するTerraformの活用 in DMMプラットフォーム
pospome
3
1.3k
組織のコード品質を向上させる “レビューシステム”の取り組み
pospome
14
7.7k
Other Decks in Programming
See All in Programming
Exploring the Implementation of “t.Run”, “t.Parallel”, and “t.Cleanup”
akarin
1
160
Good first issues of TypeProf
mame
4
560
Documentation testsの恩恵 / Documentation testing benefits
ssssota
1
560
RaaP
ksss
0
160
The Cutting Edge Of Versioning (LambdaConf 2024)
chriskrycho
0
250
Jetpack Composeとデザインシステム
rmakiyama
0
230
Long journey of Ruby standard library RubyKaigi 2024
andpad
2
210
TypeScript Custom GitHub Action Development Tips
peaceiris
1
140
Native Federation: The Future of Micro Frontends in Angular
manfredsteyer
PRO
0
170
Open AI APIを使う前に知っておきたいアカウントTier の話
akki_megane
0
130
ソースコードを美しくたもつために ~コードレビューの認知限界を突破し、年間400リリースを達成する~
kotauchisunsun
1
740
FoodGram
iseruuuuu
0
230
Featured
See All Featured
Docker and Python
trallard
35
2.7k
Code Reviewing Like a Champion
maltzj
515
39k
4 Signs Your Business is Dying
shpigford
176
21k
Done Done
chrislema
178
15k
jQuery: Nuts, Bolts and Bling
dougneiner
60
7.2k
What the flash - Photography Introduction
edds
64
11k
Writing Fast Ruby
sferik
622
60k
The Illustrated Children's Guide to Kubernetes
chrisshort
32
47k
Easily Structure & Communicate Ideas using Wireframe
afnizarnur
188
16k
Fantastic passwords and where to find them - at NoRuKo
philnash
39
2.5k
Creatively Recalculating Your Daily Design Routine
revolveconf
211
11k
Why Our Code Smells
bkeepers
PRO
331
56k
Transcript
DMMプラットフォームにおける コード品質を改善する取り組み 理想と現実 @pospome
登壇者 名前:pospome(ぽすぽめ) 所属:DMMプラットフォーム Twitter:@pospome
今回の発表内容について DMMプラットフォーム x コード品質向上 x 理想と現実
DMMプラットフォームについて 扱う領域:DMM会員、決済、DMMポイント、不正対策など エンジニア数:120名以上 開発チーム数:16チーム マイクロサービス数:約40サービス ピーク時のリクエスト:19,000RPS
レガシーシステムのリプレイスプロジェクト • DMMプラットフォーム内のレガシーシステムをリプレイスする。 • コード品質を高く保つ体制が必要になる。
pospomeの持論 コード品質を高く保つには以下の要素が必要である。 1. 定期的なリファクタリング 2. 静的解析 3. 設計スキルの高いエンジニア←これが重要
レビューシステムの導入 • 各チームのコードを設計スキルの高いエンジニアがレビューする。 擬似的に「強いエンジニアがいる環境」を作った。 • 自チームの数人が業務時間を割いて取り組んだ。 • 正直な話、結構成功させる自信があった。
レビューシステムの導入
そんなことはなかった・・・ チームによって様々な課題を抱えていた。 • リファクタリング時間の確保が難しい。 • コードの良し悪しを精度高く判断できる人がいない。 • テストコードを書いていない & カバレッジ可視化してない。
• マネージャーやテックリードがコード品質に対するリテラシーを持っていな い。
Developer Productivity Teamの設立 • 専門のチームを設立 コード品質に対して組織的に取り組んでいく
Developer Productivity Teamの取り組み • レビューシステムによるコードレビューは継続 ◦ 問題のあるコードを記録 ◦ 解消率によってリファクタリング時間の確保を可視化 •
Sonar Cloudの導入 ◦ テストカバレッジの可視化 ◦ 問題のあるコードの可視化(静的解析) • モノレポの導入 ◦ コードを一元管理
Developer Productivity Teamの取り組み • 各チームのコード品質を可視化し、比較できるようにする。 ◦ 各チームのマネージャーやTLに危機感を持ってもらう。 ◦ 各チームに介入する妥当性を保証できる。
まとめ • 大きな組織になるほどコード品質に向き合うのは難しくなる。 • 組織的に戦略を立てて取り組む必要がある。 ◦ 組織としてコード品質に投資する意思決定が必要になる。
おわり