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
420
コードレビューやめた
富士もくもく会にて発表した際の資料です。
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.5k
Refactoring Kataを使ってRubyのリファクタリングを練習する会 - 説明資料
ytnk531
0
160
リモートで生産性を上げるスクラム開発事例
ytnk531
0
530
Other Decks in Programming
See All in Programming
Go1.24 go vetとtestsアナライザ
kuro_kurorrr
2
610
令和トラベルにおけるコンテンツ生成AIアプリケーション開発の実践
ippo012
1
270
PHPでお金を扱う時、終わりのない 謎の1円調査の旅にでなくて済む方法
nakka
4
1.4k
私の愛したLaravel 〜レールを超えたその先へ〜
kentaroutakeda
12
3.6k
S3静的ホスティング+Next.js静的エクスポート で格安webアプリ構築
iharuoru
0
200
RCPと宣言型ポリシーについてのお話し
kokitamura
2
150
SLI/SLOの設定を進めるその前に アラート品質の改善に取り組んだ話
tanden
2
740
The Evolution of Enterprise Java with Jakarta EE 11 and Beyond
ivargrimstad
0
1.1k
20250326_生成AIによる_レビュー承認システムの実現.pdf
takahiromatsui
17
5.7k
リアルタイムレイトレーシング + ニューラルレンダリング簡単紹介 / Real-Time Ray Tracing & Neural Rendering: A Quick Introduction (2025)
shocker_0x15
1
150
PHPによる"非"構造化プログラミング入門 -本当に熱いスパゲティコードを求めて- #phperkaigi
o0h
PRO
0
1.1k
보일러플레이트 코드가 진짜 나쁜 건가요?
gaeun5744
0
380
Featured
See All Featured
Building Your Own Lightsaber
phodgson
104
6.3k
XXLCSS - How to scale CSS and keep your sanity
sugarenia
248
1.3M
A better future with KSS
kneath
238
17k
Typedesign – Prime Four
hannesfritz
41
2.6k
実際に使うSQLの書き方 徹底解説 / pgcon21j-tutorial
soudai
177
52k
Speed Design
sergeychernyshev
28
870
Faster Mobile Websites
deanohume
306
31k
[RailsConf 2023] Rails as a piece of cake
palkan
53
5.4k
The Art of Delivering Value - GDevCon NA Keynote
reverentgeek
12
1.4k
The Art of Programming - Codeland 2020
erikaheidi
53
13k
Rebuilding a faster, lazier Slack
samanthasiow
80
8.9k
ピンチをチャンスに:未来をつくるプロダクトロードマップ #pmconf2020
aki_iinuma
118
51k
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