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
t179a
September 02, 2023
Programming
1
130
良いコードレビューとは
t179a
September 02, 2023
Tweet
Share
More Decks by t179a
See All by t179a
Flutter3.13で気になった点
t179a
0
69
マルチモジュールアプリでJacocoのReportをまとめる方法
t179a
0
670
Other Decks in Programming
See All in Programming
iOS開発におけるCopilot For XcodeとCode Completion / copilot for xcode
fuyan777
1
720
Jakarta EE meets AI
ivargrimstad
0
340
暇に任せてProxmoxコンソール 作ってみました
karugamo
2
760
今年一番支援させていただいたのは認証系サービスでした
satoshi256kbyte
1
270
AWSのLambdaで PHPを動かす選択肢
rinchoku
2
330
CQRS+ES の力を使って効果を感じる / Feel the effects of using the power of CQRS+ES
seike460
PRO
0
210
Findy Team+ Awardを受賞したかった!ベストプラクティス応募内容をふりかえり、開発生産性向上もふりかえる / Findy Team Plus Award BestPractice and DPE Retrospective 2024
honyanya
0
120
KubeCon NA 2024の全DB関連セッションを紹介
nnaka2992
0
100
create_tableをしただけなのに〜囚われのuuid編〜
daisukeshinoku
0
310
Kaigi on Railsに初参加したら、その日にLT登壇が決定した件について
tama50505
0
120
PHPカンファレンス 2024|共創を加速するための若手の技術挑戦
weddingpark
0
100
Cloudflare MCP ServerでClaude Desktop からWeb APIを構築
kutakutat
1
600
Featured
See All Featured
Understanding Cognitive Biases in Performance Measurement
bluesmoon
27
1.5k
The Cost Of JavaScript in 2023
addyosmani
46
7k
Helping Users Find Their Own Way: Creating Modern Search Experiences
danielanewman
29
2.3k
Testing 201, or: Great Expectations
jmmastey
41
7.2k
Evolution of real-time – Irina Nazarova, EuRuKo, 2024
irinanazarova
6
470
Building Better People: How to give real-time feedback that sticks.
wjessup
366
19k
Responsive Adventures: Dirty Tricks From The Dark Corners of Front-End
smashingmag
251
21k
Product Roadmaps are Hard
iamctodd
PRO
50
11k
The Illustrated Children's Guide to Kubernetes
chrisshort
48
49k
Music & Morning Musume
bryan
46
6.2k
CoffeeScript is Beautiful & I Never Want to Write Plain JavaScript Again
sstephenson
159
15k
BBQ
matthewcrist
85
9.4k
Transcript
良いコードレビューとは 2023-09-05 社内 LT
このスライドは https://testing.googleblog.com/2023/08/testing-on-toil et-presents.html を意訳したものです。
テーマを選んだ理由 - 最近、レビューをする機会をいただいている - 一度レビューについて勉強しようと思ったため。
このようなコードレビューを 受け取ったことがあるかも
None
もしそうなら、おそらく頭を悩ませたことでしょう。 レビュワーはなぜこの質問をしているのでしょうか? 彼らはあなたを試しているのでしょうか? 大きな間違いを犯しましたか? 問題は、コメントが曖昧であることです(そして、おそらく少し失礼ですらあります)。 また、レビュワーが何を考えているかという重要な文脈が省略されているため、返答が 難しくなります。
レビュー コメントが効果的で、有 益で、明確なものであることを確 認するための簡単な方法をいくつ か紹介します。
親切に あなたが敬意を持って接すると、人々はフィードバックをより受け 入れやすくなります。
人ではなくコードに集中 コードではなく人を判断しているという印象を与える可能性がある 「あなた」という言葉を含む発言は避けてください。
コメントをする理由を説明 コードを書いた人には明らかではない代替案をあなたが知っているかも しれませんし、追加の制約を知っているかもしれません。
テスト採点者としてではなく、ガイ ドとして自分の役割に取り組む。 直接的なガイダンスを提供することと、作成者にある程度の自由 度を残すことのバランスをとります。
優先度の低いコメントに Nit、 Optional、FYI などの重大度を付 ける レビューを受ける側が重要なコメントに優先順位を付けられるよう になります
コメントによっては、簡潔に表現するのが難しい場合があります。 微妙な API の使用法 や構造の変更 (たとえば、この大きな変更を小さな変更に分割する) など、書き換えを念 頭に置いている場合は、例を提供することを検討してください。 フィードバックを 2
回以上繰り返しても行き詰まった場合は、チャットやライブ通話など のより直接的なコミュニケーション チャネルに移行して、次のステップについて話し合う ことを検討してください。
ここまでを参考に最初のレ ビューを修正すると
None
ご清聴ありがとうございました