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
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
KagglerがMixSeekを触ってみた
morim
0
370
Xdebug と IDE による デバッグ実行の仕組みを見る / Exploring-How-Debugging-Works-with-Xdebug-and-an-IDE
shin1x1
0
340
L’IA au service des devs : Anatomie d'un assistant de Code Review
toham
0
200
Linux Kernelの1文字のミスで 権限昇格ができた話
rqda
0
2.3k
GoのDB アクセスにおける 「型安全」と「柔軟性」の両立 - Bob という選択肢
tak848
0
310
モックわからないマン卒業記 ~振る舞いを起点に見直した、フロントエンドテストにおけるモックの使いどころ~
tasukuwatanabe
3
440
Goの型安全性で実現する複数プロダクトの権限管理
ishikawa_pro
2
1.4k
見せてもらおうか、 OpenSearchの性能とやらを!
shunta27
1
170
存在論的プログラミング: 時間と存在を記述する
koriym
5
770
車輪の再発明をしよう!PHP で実装して学ぶ、Web サーバーの仕組みと HTTP の正体
h1r0
3
500
AI活用のコスパを最大化する方法
ochtum
0
370
テレメトリーシグナルが導くパフォーマンス最適化 / Performance Optimization Driven by Telemetry Signals
seike460
PRO
2
220
Featured
See All Featured
Crafting Experiences
bethany
1
110
A Modern Web Designer's Workflow
chriscoyier
698
190k
Large-scale JavaScript Application Architecture
addyosmani
515
110k
Building a A Zero-Code AI SEO Workflow
portentint
PRO
0
430
Reflections from 52 weeks, 52 projects
jeffersonlam
356
21k
DevOps and Value Stream Thinking: Enabling flow, efficiency and business value
helenjbeal
1
160
From π to Pie charts
rasagy
0
160
Visualization
eitanlees
150
17k
Optimizing for Happiness
mojombo
378
71k
First, design no harm
axbom
PRO
2
1.2k
Reality Check: Gamification 10 Years Later
codingconduct
0
2.1k
Building Better People: How to give real-time feedback that sticks.
wjessup
370
20k
Transcript
良いコードレビューとは 2023-09-05 社内 LT
このスライドは https://testing.googleblog.com/2023/08/testing-on-toil et-presents.html を意訳したものです。
テーマを選んだ理由 - 最近、レビューをする機会をいただいている - 一度レビューについて勉強しようと思ったため。
このようなコードレビューを 受け取ったことがあるかも
None
もしそうなら、おそらく頭を悩ませたことでしょう。 レビュワーはなぜこの質問をしているのでしょうか? 彼らはあなたを試しているのでしょうか? 大きな間違いを犯しましたか? 問題は、コメントが曖昧であることです(そして、おそらく少し失礼ですらあります)。 また、レビュワーが何を考えているかという重要な文脈が省略されているため、返答が 難しくなります。
レビュー コメントが効果的で、有 益で、明確なものであることを確 認するための簡単な方法をいくつ か紹介します。
親切に あなたが敬意を持って接すると、人々はフィードバックをより受け 入れやすくなります。
人ではなくコードに集中 コードではなく人を判断しているという印象を与える可能性がある 「あなた」という言葉を含む発言は避けてください。
コメントをする理由を説明 コードを書いた人には明らかではない代替案をあなたが知っているかも しれませんし、追加の制約を知っているかもしれません。
テスト採点者としてではなく、ガイ ドとして自分の役割に取り組む。 直接的なガイダンスを提供することと、作成者にある程度の自由 度を残すことのバランスをとります。
優先度の低いコメントに Nit、 Optional、FYI などの重大度を付 ける レビューを受ける側が重要なコメントに優先順位を付けられるよう になります
コメントによっては、簡潔に表現するのが難しい場合があります。 微妙な API の使用法 や構造の変更 (たとえば、この大きな変更を小さな変更に分割する) など、書き換えを念 頭に置いている場合は、例を提供することを検討してください。 フィードバックを 2
回以上繰り返しても行き詰まった場合は、チャットやライブ通話など のより直接的なコミュニケーション チャネルに移行して、次のステップについて話し合う ことを検討してください。
ここまでを参考に最初のレ ビューを修正すると
None
ご清聴ありがとうございました