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
200
1
Share
良いコードレビューとは
t179a
September 02, 2023
More Decks by t179a
See All by t179a
マルチモジュールアプリでJacocoのReportをまとめる方法
t179a
0
830
Other Decks in Programming
See All in Programming
Surviving Black Friday: 329 billion requests with Falcon!
ioquatix
0
830
アクセシビリティ試験の"その後"を仕組み化する
yuuumiravy
1
170
Angular Signal Forms
debug_mode
0
120
レガシーPHP転生 〜父がドメインエキスパートだったのでDDD+Claude Codeでチート開発します〜
panda_program
0
1.1k
Vibe하게 만드는 Flutter GenUI App With ADK , 박제창, BWAI Incheon 2026
itsmedreamwalker
0
550
JAWS-UG横浜 #100 祝・第100回スペシャルAWS は VPC レスの時代へ
maroon1st
0
180
Oxlintとeslint-plugin-react-hooks 明日から始められそう?
t6adev
0
280
Server-Side Kotlin LT大会 vol.18 [Kotlin-lspの最新情報と Neovimのlsp設定例]
yasunori0418
1
190
PicoRuby for IoT: Connecting to the Cloud with MQTT
yuuu
2
660
CDK Deployのための ”反響定位”
watany
5
860
GoogleCloudとterraform完全に理解した
terisuke
1
160
属人化しないコード品質の作り方_2026.04.07.pdf
muraaano
0
240
Featured
See All Featured
I Don’t Have Time: Getting Over the Fear to Launch Your Podcast
jcasabona
34
2.7k
Practical Orchestrator
shlominoach
191
11k
[RailsConf 2023 Opening Keynote] The Magic of Rails
eileencodes
31
10k
Heart Work Chapter 1 - Part 1
lfama
PRO
6
35k
Easily Structure & Communicate Ideas using Wireframe
afnizarnur
194
17k
個人開発の失敗を避けるイケてる考え方 / tips for indie hackers
panda_program
122
21k
Chasing Engaging Ingredients in Design
codingconduct
0
170
Utilizing Notion as your number one productivity tool
mfonobong
4
290
Future Trends and Review - Lecture 12 - Web Technologies (1019888BNR)
signer
PRO
0
3.5k
What the history of the web can teach us about the future of AI
inesmontani
PRO
1
530
Exploring anti-patterns in Rails
aemeredith
3
330
Side Projects
sachag
455
43k
Transcript
良いコードレビューとは 2023-09-05 社内 LT
このスライドは https://testing.googleblog.com/2023/08/testing-on-toil et-presents.html を意訳したものです。
テーマを選んだ理由 - 最近、レビューをする機会をいただいている - 一度レビューについて勉強しようと思ったため。
このようなコードレビューを 受け取ったことがあるかも
None
もしそうなら、おそらく頭を悩ませたことでしょう。 レビュワーはなぜこの質問をしているのでしょうか? 彼らはあなたを試しているのでしょうか? 大きな間違いを犯しましたか? 問題は、コメントが曖昧であることです(そして、おそらく少し失礼ですらあります)。 また、レビュワーが何を考えているかという重要な文脈が省略されているため、返答が 難しくなります。
レビュー コメントが効果的で、有 益で、明確なものであることを確 認するための簡単な方法をいくつ か紹介します。
親切に あなたが敬意を持って接すると、人々はフィードバックをより受け 入れやすくなります。
人ではなくコードに集中 コードではなく人を判断しているという印象を与える可能性がある 「あなた」という言葉を含む発言は避けてください。
コメントをする理由を説明 コードを書いた人には明らかではない代替案をあなたが知っているかも しれませんし、追加の制約を知っているかもしれません。
テスト採点者としてではなく、ガイ ドとして自分の役割に取り組む。 直接的なガイダンスを提供することと、作成者にある程度の自由 度を残すことのバランスをとります。
優先度の低いコメントに Nit、 Optional、FYI などの重大度を付 ける レビューを受ける側が重要なコメントに優先順位を付けられるよう になります
コメントによっては、簡潔に表現するのが難しい場合があります。 微妙な API の使用法 や構造の変更 (たとえば、この大きな変更を小さな変更に分割する) など、書き換えを念 頭に置いている場合は、例を提供することを検討してください。 フィードバックを 2
回以上繰り返しても行き詰まった場合は、チャットやライブ通話など のより直接的なコミュニケーション チャネルに移行して、次のステップについて話し合う ことを検討してください。
ここまでを参考に最初のレ ビューを修正すると
None
ご清聴ありがとうございました