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
レビューガイドラインで技術力を見える化する
Search
akkiee76
April 06, 2022
Programming
0
1k
レビューガイドラインで技術力を見える化する
・
https://rakus.connpass.com/event/240393/
・
https://rakus.connpass.com/event/251357/
で発表した資料になります。
akkiee76
April 06, 2022
Tweet
Share
More Decks by akkiee76
See All by akkiee76
Graph Art with Charts API – Beyond Data Visualization
akkie76
0
160
Meet the Translation API
akkie76
0
420
コードレビューで開発を加速させるAIコードレビュー
akkie76
1
670
Android Target SDK 35 (Android 15) 対応の概要
akkie76
0
6k
コードレビューを支援するAI技術の応用
akkie76
5
1.2k
オブジェクト指向コードレビューの新しいアプローチ
akkie76
3
9.3k
Jetpack Compose で Adaptive Layout に対応しよう
akkie76
0
1k
Observationではじめる値監視
akkie76
4
4.7k
TextField 表示スタイル変更の 有効活用例 5 選
akkie76
0
730
Other Decks in Programming
See All in Programming
Cursorハンズオン実践!
eltociear
2
1.1k
CSC305 Lecture 06
javiergs
PRO
0
250
デミカツ切り抜きで面倒くさいことはPythonにやらせよう
aokswork3
0
250
AI駆動で0→1をやって見えた光と伸びしろ
passion0102
1
480
What Spring Developers Should Know About Jakarta EE
ivargrimstad
0
260
bootcamp2025_バックエンド研修_WebAPIサーバ作成.pdf
geniee_inc
0
120
理論と実務のギャップを超える
eycjur
0
140
Swift Concurrency - 状態監視の罠
objectiveaudio
2
550
Software Architecture
hschwentner
6
2.3k
(Extension DC 2025) Actor境界を越える技術
teamhimeh
1
260
なぜあの開発者はDevRelに伴走し続けるのか / Why Does That Developer Keep Running Alongside DevRel?
nrslib
3
410
The Past, Present, and Future of Enterprise Java
ivargrimstad
0
260
Featured
See All Featured
[Rails World 2023 - Day 1 Closing Keynote] - The Magic of Rails
eileencodes
37
2.6k
CoffeeScript is Beautiful & I Never Want to Write Plain JavaScript Again
sstephenson
162
15k
Unsuck your backbone
ammeep
671
58k
Building Adaptive Systems
keathley
44
2.8k
Designing for humans not robots
tammielis
254
26k
Java REST API Framework Comparison - PWX 2021
mraible
34
8.9k
The MySQL Ecosystem @ GitHub 2015
samlambert
251
13k
A Modern Web Designer's Workflow
chriscoyier
697
190k
Why You Should Never Use an ORM
jnunemaker
PRO
59
9.6k
Connecting the Dots Between Site Speed, User Experience & Your Business [WebExpo 2025]
tammyeverts
10
600
Stop Working from a Prison Cell
hatefulcrawdad
271
21k
RailsConf 2023
tenderlove
30
1.3k
Transcript
〜 数値で分かるチームの弱点 〜 レビューガイドラインで 技術力を見える化する @akkiee76
自己紹介 Akihiko Sato / 株式会社ラクス Lead Engineer / @akkiee76 SaaS
開発 (Backend, Frontend) / Mobile 開発 (iOS, Android) 上流工程、コードレビュー、チームの課題改善など 読書 / コーヒー / HHKB / 腹筋ローラー
今日伝えたいこと レビューガイドラインで チームの特徴を見える化させることはとても有意義!
私が join した当時のチーム状況 ・実装のセオリーが分からない ・レビューの観点が分からない レビューがしっかり行われていない体制だった モバイル開発経験者が少なく実装のノウハウがない
その結果・・・ ・技術力がなかなか伸びない ・定期的にバグが発生する
レビュー指摘を類型化できないか? チームメンバーの弱点を表せないか? どうやって課題を改善するか ガイドラインを作って、レビューコメントを類型化しよう! ( レビュー指摘は財産 )
ガイドライン作成にあたって 基本的なアウトラインを決めよう! 重要度 x 観点 の基準でレビューコメントを類型化することに
重要度について まずは指摘の重要度を 4 つに分類 ・MUST(修正が必須) ・SHOULD(リリースまでには修正が必要) ・IMO(修正なしも許容) ・NITS(細かい指摘)
観点について ① 指摘の観点を 7 つに分類 ・Design(設計) ・Functionality(機能を充足しているか) ・Simplicity(理解容易性) ・Style(コードスタイル)
観点について ② 指摘の観点を 7 つに分類 ・Naming(クラス、メソッド、変数名などの命名) ・Tests(自動テストが適切である) ・Document(コメント、ドキュメンに関連)
具体的な利用方法 コメントの prefix として利用する。 指摘例 MUST(Design):ドメインロジックがControllerクラスに実装されてます。 domain層の対象packageに新しくクラスを作成して実装を移してください。
prefix を付けたコメントを集計 GitLab API (Note API)を利用してコメント集計します。 具体的な方法は、 ラクス Advent Calendar
2021(12/23) 「GitLab API で Merge Request のコメントを一括取得する方法」 をご覧ください。
集計結果 運用開始 2 ヶ月で、コメント数は約 75 件。
得られた気付き ➔ Tests のコメントが圧倒的に多い ◆ テストコードの実装力の課題 ➔ 次いで Design のコメントが多い
◆ 設計力、オブジェクト指向が課題
苦手克服のため今後のアクションプラン ① オブジェクト指向の輪読会の実施 ② テストの書き方トレーニング会 ③ ペアプロの導入
まとめ レビューガイドラインで技術力を見える化すると ① チームメンバーのスキルが定量化できる ② 苦手分野が明らかになる ③ スキルアップのためのアクションプランが検討しやすい チームでの導入を検討してみてはいかがでしょうか
ご清聴ありがとうございました