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
480
コードレビューやめた
富士もくもく会にて発表した際の資料です。
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
CI_CD「健康診断」のススメ。現場でのボトルネック特定から、健康診断を通じた組織的な改善手法
teamlab
PRO
0
210
明日から始めるリファクタリング
ryounasso
0
140
dynamic!
moro
10
7.4k
なぜGoのジェネリクスはこの形なのか? Featherweight Goが明かす設計の核心
ryotaros
7
1.1k
タスクの特性や不確実性に応じた最適な作業スタイルの選択(ペアプロ・モブプロ・ソロプロ)と実践 / Optimal Work Style Selection: Pair, Mob, or Solo Programming.
honyanya
3
160
Go Conference 2025: Goで体感するMultipath TCP ― Go 1.24 時代の MPTCP Listener を理解する
takehaya
9
1.7k
Foundation Modelsを実装日本語学習アプリを作ってみた!
hypebeans
0
110
コードとあなたと私の距離 / The Distance Between Code, You, and I
hiro_y
0
150
kiroとCodexで最高のSpec駆動開発を!!数時間で web3ネイティブなミニゲームを作ってみたよ!
mashharuki
0
150
Django Ninja による API 開発効率化とリプレースの実践
kashewnuts
0
1.3k
Building, Deploying, and Monitoring Ruby Web Applications with Falcon (Kaigi on Rails 2025)
ioquatix
4
2k
Software Architecture
hschwentner
6
2.3k
Featured
See All Featured
Java REST API Framework Comparison - PWX 2021
mraible
33
8.9k
Cheating the UX When There Is Nothing More to Optimize - PixelPioneers
stephaniewalter
285
14k
Rebuilding a faster, lazier Slack
samanthasiow
84
9.2k
The Invisible Side of Design
smashingmag
302
51k
Optimizing for Happiness
mojombo
379
70k
Why You Should Never Use an ORM
jnunemaker
PRO
59
9.6k
Distributed Sagas: A Protocol for Coordinating Microservices
caitiem20
333
22k
Docker and Python
trallard
46
3.6k
Measuring & Analyzing Core Web Vitals
bluesmoon
9
620
Raft: Consensus for Rubyists
vanstee
139
7.1k
Making Projects Easy
brettharned
119
6.4k
[RailsConf 2023 Opening Keynote] The Magic of Rails
eileencodes
31
9.7k
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