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
Sponsored
·
Ship Features Fearlessly
Turn features on and off without deploys. Used by thousands of Ruby developers.
→
t179a
September 02, 2023
Programming
1
190
良いコードレビューとは
t179a
September 02, 2023
Tweet
Share
More Decks by t179a
See All by t179a
マルチモジュールアプリでJacocoのReportをまとめる方法
t179a
0
820
Other Decks in Programming
See All in Programming
AIに仕事を丸投げしたら、本当に楽になれるのか
dip_tech
PRO
0
180
15年目のiOSアプリを1から作り直す技術
teakun
1
600
モジュラモノリスにおける境界をGoのinternalパッケージで守る
magavel
0
3.4k
ふつうの Rubyist、ちいさなデバイス、大きな一年
bash0c7
0
110
オブザーバビリティ駆動開発って実際どうなの?
yohfee
3
680
今更考える「単一責任原則」 / Thinking about the Single Responsibility Principle
tooppoo
3
1.4k
Claude Code、ちょっとした工夫で開発体験が変わる
tigertora7571
0
200
The Ralph Wiggum Loop: First Principles of Autonomous Development
sembayui
0
3.7k
米国のサイバーセキュリティタイムラインと見る Goの暗号パッケージの進化
tomtwinkle
2
420
AWS Infrastructure as Code の新機能 2025 総まとめ 〜SA 4人による怒涛のデモ祭り〜
konokenj
10
3.1k
浮動小数の比較について
kishikawakatsumi
0
370
Swift ConcurrencyでよりSwiftyに
yuukiw00w
0
240
Featured
See All Featured
Visualizing Your Data: Incorporating Mongo into Loggly Infrastructure
mongodb
49
9.9k
Building the Perfect Custom Keyboard
takai
2
710
DBのスキルで生き残る技術 - AI時代におけるテーブル設計の勘所
soudai
PRO
62
51k
Primal Persuasion: How to Engage the Brain for Learning That Lasts
tmiket
0
280
YesSQL, Process and Tooling at Scale
rocio
174
15k
Avoiding the “Bad Training, Faster” Trap in the Age of AI
tmiket
0
96
Music & Morning Musume
bryan
47
7.1k
The Cost Of JavaScript in 2023
addyosmani
55
9.7k
We Are The Robots
honzajavorek
0
190
Public Speaking Without Barfing On Your Shoes - THAT 2023
reverentgeek
1
330
Bioeconomy Workshop: Dr. Julius Ecuru, Opportunities for a Bioeconomy in West Africa
akademiya2063
PRO
1
68
Claude Code のすすめ
schroneko
67
220k
Transcript
良いコードレビューとは 2023-09-05 社内 LT
このスライドは https://testing.googleblog.com/2023/08/testing-on-toil et-presents.html を意訳したものです。
テーマを選んだ理由 - 最近、レビューをする機会をいただいている - 一度レビューについて勉強しようと思ったため。
このようなコードレビューを 受け取ったことがあるかも
None
もしそうなら、おそらく頭を悩ませたことでしょう。 レビュワーはなぜこの質問をしているのでしょうか? 彼らはあなたを試しているのでしょうか? 大きな間違いを犯しましたか? 問題は、コメントが曖昧であることです(そして、おそらく少し失礼ですらあります)。 また、レビュワーが何を考えているかという重要な文脈が省略されているため、返答が 難しくなります。
レビュー コメントが効果的で、有 益で、明確なものであることを確 認するための簡単な方法をいくつ か紹介します。
親切に あなたが敬意を持って接すると、人々はフィードバックをより受け 入れやすくなります。
人ではなくコードに集中 コードではなく人を判断しているという印象を与える可能性がある 「あなた」という言葉を含む発言は避けてください。
コメントをする理由を説明 コードを書いた人には明らかではない代替案をあなたが知っているかも しれませんし、追加の制約を知っているかもしれません。
テスト採点者としてではなく、ガイ ドとして自分の役割に取り組む。 直接的なガイダンスを提供することと、作成者にある程度の自由 度を残すことのバランスをとります。
優先度の低いコメントに Nit、 Optional、FYI などの重大度を付 ける レビューを受ける側が重要なコメントに優先順位を付けられるよう になります
コメントによっては、簡潔に表現するのが難しい場合があります。 微妙な API の使用法 や構造の変更 (たとえば、この大きな変更を小さな変更に分割する) など、書き換えを念 頭に置いている場合は、例を提供することを検討してください。 フィードバックを 2
回以上繰り返しても行き詰まった場合は、チャットやライブ通話など のより直接的なコミュニケーション チャネルに移行して、次のステップについて話し合う ことを検討してください。
ここまでを参考に最初のレ ビューを修正すると
None
ご清聴ありがとうございました