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
470
コードレビューやめた
富士もくもく会にて発表した際の資料です。
https://fujimokumokukai.connpass.com/event/244280/
Yudai Tanaka
June 07, 2022
Tweet
Share
More Decks by Yudai Tanaka
See All by Yudai Tanaka
ふつうのWebサービス開発者がRubyKaigiを楽しむためのRubyの知識
ytnk531
4
3.9k
Refactoring Kataを使ってRubyのリファクタリングを練習する会 - 説明資料
ytnk531
0
190
リモートで生産性を上げるスクラム開発事例
ytnk531
0
570
Other Decks in Programming
See All in Programming
デザイナーが Androidエンジニアに 挑戦してみた
874wokiite
0
310
時間軸から考えるTerraformを使う理由と留意点
fufuhu
15
4.7k
詳解!defer panic recover のしくみ / Understanding defer, panic, and recover
convto
0
230
RDoc meets YARD
okuramasafumi
4
170
ProxyによるWindow間RPC機構の構築
syumai
3
1.2k
Deep Dive into Kotlin Flow
jmatsu
1
310
250830 IaCの選定~AWS SAMのLambdaをECSに乗り換えたときの備忘録~
east_takumi
0
390
Testing Trophyは叫ばない
toms74209200
0
860
Compose Multiplatform × AI で作る、次世代アプリ開発支援ツールの設計と実装
thagikura
0
140
HTMLの品質ってなんだっけ? “HTMLクライテリア”の設計と実践
unachang113
4
2.8k
もうちょっといいRubyプロファイラを作りたい (2025)
osyoyu
1
420
Processing Gem ベースの、2D レトロゲームエンジンの開発
tokujiros
2
120
Featured
See All Featured
"I'm Feeling Lucky" - Building Great Search Experiences for Today's Users (#IAC19)
danielanewman
229
22k
The Cost Of JavaScript in 2023
addyosmani
53
8.9k
It's Worth the Effort
3n
187
28k
Speed Design
sergeychernyshev
32
1.1k
Fashionably flexible responsive web design (full day workshop)
malarkey
407
66k
Principles of Awesome APIs and How to Build Them.
keavy
126
17k
Mobile First: as difficult as doing things right
swwweet
224
9.9k
Templates, Plugins, & Blocks: Oh My! Creating the theme that thinks of everything
marktimemedia
31
2.5k
Cheating the UX When There Is Nothing More to Optimize - PixelPioneers
stephaniewalter
285
13k
ピンチをチャンスに:未来をつくるプロダクトロードマップ #pmconf2020
aki_iinuma
126
53k
The Pragmatic Product Professional
lauravandoore
36
6.9k
Being A Developer After 40
akosma
90
590k
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