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
Yudai Tanaka
June 07, 2022
Programming
0
280
コードレビューやめた
富士もくもく会にて発表した際の資料です。
https://fujimokumokukai.connpass.com/event/244280/
Yudai Tanaka
June 07, 2022
Tweet
Share
More Decks by Yudai Tanaka
See All by Yudai Tanaka
Refactoring Kataを使ってRubyのリファクタリングを練習する会 - 説明資料
ytnk531
0
100
リモートで生産性を上げるスクラム開発事例
ytnk531
0
430
Other Decks in Programming
See All in Programming
TYPO3 v13 – The road to LTS: What's new and new APIs
luisasofie_xoxo
0
200
Behind VS Code Extensions for JavaScript / TypeScript Linnting and Formatting
unvalley
5
900
Prepare for Jakarta EE 11 - Performance and Developer Productivity
ivargrimstad
0
730
1BRC--Nerd Sniping the Java Community
gunnarmorling
0
340
Rubyでたのしむクリエイティブコーディング/Enjoy Creative coding with Ruby
chobishiba
1
180
ScalarDBを用いたマイクロサービスにおけるデータ管理 (Database Engineering Meetup #2)
scalar
0
110
単体テストを書かない技術 #phpcon_odawara
o0h
PRO
26
8.2k
Milestoner
bkuhlmann
1
410
ONE WEDGE_company_guide
1wedge_one
0
470
Goのmultiple errorsについて (2024年4月版)
syumai
3
610
Random\Randomizer クラスで日常のあれこれを解決しよう! / Random\Randomizer class solves familiar trouble
cocoeyes02
0
220
try!Swift Tokyo 2024 参加報告 LT
akidon0000
1
220
Featured
See All Featured
Building Adaptive Systems
keathley
31
1.9k
The Success of Rails: Ensuring Growth for the Next 100 Years
eileencodes
30
6k
Sharpening the Axe: The Primacy of Toolmaking
bcantrill
17
1.4k
Rails Girls Zürich Keynote
gr2m
91
13k
I Don’t Have Time: Getting Over the Fear to Launch Your Podcast
jcasabona
21
1.6k
JavaScript: Past, Present, and Future - NDC Porto 2020
reverentgeek
40
4.4k
Done Done
chrislema
178
15k
Infographics Made Easy
chrislema
238
18k
Adopting Sorbet at Scale
ufuk
68
8.6k
Bash Introduction
62gerente
604
210k
Six Lessons from altMBA
skipperchong
21
3k
Become a Pro
speakerdeck
PRO
11
4.5k
Transcript
田中悠大 (ytnk531) 1
田中悠大 サーバーサイドエンジニア6年目 • 小さいチームでリードエンジニア • Ruby On Rails 2/15
コードレビューしてますか? 3
コードレビュー楽しいですか? 4
つらい人もいるみたい 5
6
関連キーワードもつらい 7
コードレビューのつらさ レビュイーとレビュワーの対立構造 コードを直すのに手間がかかる • 議論で合意して初めてコードが変更される 8 締め切りに間に合わせたい レビュイー 品質を上げたい レビュワー
コードレビューの効果 品質向上 教育 [参考文献] GMO INTERNET GROUP 次世代システム研究室. ソースコードレビュー理由をまとめてみた. 2015.02.10.
https://recruit.gmo.jp/engineer/jisedai/blog/%e3%82%bd%e3%83%bc%e3%82%b9%e3%82%b3%e3%83%bc%e3%83%89%e3%83%ac%e3%83%93%e3%8 3%a5%e3%83%bc%e7%90%86%e7%94%b1%e3%82%92%e3%81%be%e3%81%a8%e3%82%81%e3%81%a6%e3%81%bf%e3%81%9f/ 9
[考察] コードレビューがつらくない環境 OSS 業務 完了義務 なし あり 締め切り 無いことが多い あることが多い
著者性 重要 組織による 10/15
ほかの方法は無いか? ペア/モブプログラミング • コードが生まれるときから複数人で考える ➢ すべてにやるには時間がかかりすぎる [事例] デフォルトでコードレビューは不要 • https://itnews.org/news_contents/no-code-reviews-by-default
• 全員がメインブランチに直接push ➢ ノールックはやっぱり心配 11
[やっていること] コードリレー やること • 心配なところのテスト追加、リファクタリングをレビュワーが行う いいところ • 締め切りのコントロールがしやすい • 協力、伴走の構図が作りやすい
悪いところ • 教育の効果は薄い • 著者性が損なわれると感じるかも 12
比較 コードレビュー ペア/モブプログラ ミング コードリレー 同期/非同期 非同期 同期 非同期 教育効果
中 高 低 品質向上効果 中 高 高 所要時間 長 長 中 13
効果 依頼されてから完了までの時間が半分くらいになった もらったコードの理解が深まった 14
まとめ コードレビューをやめた コードリレーで教育を犠牲にスピードと品質を向上 • 教育は別途行うようにする 15