Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Sign up for free
Menu
Search
Features
All features
Private URLs
Password Protection
Custom URLS
Scheduled publishing
Remove Branding
Restrict embedding
Deck Collections
Notes
Features
All features
Private URLs
Password Protection
Custom URLS
Scheduled publishing
Remove Branding
Restrict embedding
Deck Collections
Notes
Explore
Featured decks
Featured speakers
Programming
Technology
Storyboards
Explore
Featured decks
Featured speakers
Programming
Technology
Storyboards
Pricing
Search
Sign in
Sign up for free
良いコードレビューとは
Search
t179a
September 02, 2023
Programming
210
1
Share
Embed
Copy iframe code
Copy JS code
Copy link
Start on current slide
良いコードレビューとは
t179a
September 02, 2023
More Decks by t179a
See All by t179a
マルチモジュールアプリでJacocoのReportをまとめる方法
t179a
0
890
Other Decks in Programming
See All in Programming
C#の現在地 進化の歴史と、AI時代の.NET Everywhere
neuecc
5
5k
Androidだけじゃない、DroidKaigiで広がった私の技術スタック
akkeylab
0
120
[2026-09-26]空論ジェネリックプロセス~テスト資産とAIで紡ぐ、再現可能なパフォーマンスチューニングの話~
tosite
0
260
仕様駆動開発による爆速プロダクト開発 / Bakusoku Spec Driven Development
kobakei
0
210
Embedded Swiftで作る自作USBデバイス によるiOSデバイスの自動テスト / iOSDC Japan 2026 glassfiber
glassfiber
0
310
パズルゲームの作り方 / how to make puzzle games
kaityo256
PRO
2
270
速習iPhone Duo対応
yuukiw00w
2
970
mrbgem 三角測量 開発
ogom
0
220
大喜利で理解するLLM as a Judge / Understanding LLM-as-a-Judge through Ogiri
rockname
0
200
Augmenting AI with the Power of Jakarta EE
ivargrimstad
0
500
動作中のプログラムの中身をリアルタイムに覗く / Realtime Debugger for CSharp with Roslyn
prota
1
1.8k
技術的負債の返済は、AI時代の複利で効く投資 — 経営としての意思決定とその遂行
curekoshimizu
1
2.3k
Featured
See All Featured
ReactJS: Keep Simple. Everything can be a component!
pedronauck
666
130k
Rails Girls Zürich Keynote
gr2m
96
14k
Game over? The fight for quality and originality in the time of robots
wayneb77
1
300
How to Create Impact in a Changing Tech Landscape [PerfNow 2023]
tammyeverts
56
3.5k
Fireside Chat
paigeccino
43
4.1k
RailsConf 2023
tenderlove
30
1.6k
How GitHub (no longer) Works
holman
316
150k
SEO Brein meetup: CTRL+C is not how to scale international SEO
lindahogenes
2
2.9k
A Soul's Torment
seathinner
8
3.7k
Exploring the Power of Turbo Streams & Action Cable | RailsConf2023
kevinliebholz
37
6.6k
Reality Check: Gamification 10 Years Later
codingconduct
0
2.3k
Designing Experiences People Love
moore
143
24k
Transcript
良いコードレビューとは 2023-09-05 社内 LT
このスライドは https://testing.googleblog.com/2023/08/testing-on-toil et-presents.html を意訳したものです。
テーマを選んだ理由 - 最近、レビューをする機会をいただいている - 一度レビューについて勉強しようと思ったため。
このようなコードレビューを 受け取ったことがあるかも
None
もしそうなら、おそらく頭を悩ませたことでしょう。 レビュワーはなぜこの質問をしているのでしょうか? 彼らはあなたを試しているのでしょうか? 大きな間違いを犯しましたか? 問題は、コメントが曖昧であることです(そして、おそらく少し失礼ですらあります)。 また、レビュワーが何を考えているかという重要な文脈が省略されているため、返答が 難しくなります。
レビュー コメントが効果的で、有 益で、明確なものであることを確 認するための簡単な方法をいくつ か紹介します。
親切に あなたが敬意を持って接すると、人々はフィードバックをより受け 入れやすくなります。
人ではなくコードに集中 コードではなく人を判断しているという印象を与える可能性がある 「あなた」という言葉を含む発言は避けてください。
コメントをする理由を説明 コードを書いた人には明らかではない代替案をあなたが知っているかも しれませんし、追加の制約を知っているかもしれません。
テスト採点者としてではなく、ガイ ドとして自分の役割に取り組む。 直接的なガイダンスを提供することと、作成者にある程度の自由 度を残すことのバランスをとります。
優先度の低いコメントに Nit、 Optional、FYI などの重大度を付 ける レビューを受ける側が重要なコメントに優先順位を付けられるよう になります
コメントによっては、簡潔に表現するのが難しい場合があります。 微妙な API の使用法 や構造の変更 (たとえば、この大きな変更を小さな変更に分割する) など、書き換えを念 頭に置いている場合は、例を提供することを検討してください。 フィードバックを 2
回以上繰り返しても行き詰まった場合は、チャットやライブ通話など のより直接的なコミュニケーション チャネルに移行して、次のステップについて話し合う ことを検討してください。
ここまでを参考に最初のレ ビューを修正すると
None
ご清聴ありがとうございました