Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Speaker Deck
PRO
Sign in
Sign up for free
明日からできるコードレビューの心構え
akkie76
August 23, 2022
0
300
明日からできるコードレビューの心構え
akkie76
August 23, 2022
Tweet
Share
More Decks by akkie76
See All by akkie76
Crashlytics と Android の連携で トラブルシューティングを効率化する
akkie76
0
140
アプリアーキテクチャを明文化しチームの開発効率をアップ
akkie76
1
420
こんなコードレビューは嫌だ
akkie76
0
420
アプリアーキテクチャを明文化しチームの開発効率をアップ
akkie76
1
870
チームの設計力を高めるためのTIPS
akkie76
0
510
これで失敗しない! JUnit 5 へのマイグレーション方法
akkie76
0
220
オブジェクト指向で類型化するコードレビュー
akkie76
0
230
アーキテクチャを明文化して開発に臨んだ話
akkie76
0
530
レビューガイドラインで技術力を見える化する
akkie76
0
710
Featured
See All Featured
It's Worth the Effort
3n
177
26k
Building Adaptive Systems
keathley
27
1.3k
A Philosophy of Restraint
colly
193
15k
Typedesign – Prime Four
hannesfritz
34
1.5k
Building an army of robots
kneath
302
40k
Support Driven Design
roundedbygravity
88
8.9k
Optimizing for Happiness
mojombo
365
64k
Learning to Love Humans: Emotional Interface Design
aarron
263
38k
How New CSS Is Changing Everything About Graphic Design on the Web
jensimmons
214
12k
Adopting Sorbet at Scale
ufuk
65
7.8k
Building Better People: How to give real-time feedback that sticks.
wjessup
346
17k
Design and Strategy: How to Deal with People Who Don’t "Get" Design
morganepeng
109
16k
Transcript
〜 チームの生産性を高めるコードレビューのTIPS 〜 明日からできるコードレビューの心構え @akkiee76
自己紹介 Akihiko Sato / 株式会社ラクス Lead Engineer / @akkiee76 SaaS
開発 (Backend, Frontend) / Mobile 開発 (iOS, Android) 上流工程、コードレビュー、チームの課題改善など 読書 / コーヒー / HHKB / 腹筋ローラー
今日伝えたいこと コードレビューの心構えを知ることで より気持ちのいいレビューができる! (主にレビュアー向け)
コードレビュー技術について ・オブジェクト指向で類型化するコードレビュー ・レビューガイドラインで技術力を見える化する
実際の現場では・・・ 実際にドライにコードレビューをしていると メンバー同士の関係もギクシャク ということも少なくないのでは・・・。
レビューの現場では、技術的要素が先行してしまい、 攻撃的なコメントをもらった経験がある方も少なくないのでは?? 現場あるある ・脳死横展開はやめてください ・実装の意味が分かりません。頭大丈夫ですか? ・私の期待とは違います(のみ)
コードレビューで最も大事なこと レビューで最も重要なのは、 「レビュイーに対する思いやり」です 技術の有用性だけではなく、チームメンバーに対して 敬意を表すことがコード品質を高めることに繋がります。
Google Engineering Practices Documentation Google が公開しているエンジニアのプラクティス https://github.com/google/eng-practices ・The Code Reviewer's
Guide ・The Change Author's Guide
How to write code review comments Summary ・思いやりを持つこと ・理由を説明すること ・指摘に対する明確な指示を与つつ、開発者の意思決定のバランスを取ること
・複雑性を説明させる代わりに、コードを単純化するかコードコメントを追加するように開発者 に推奨せよ https://github.com/google/eng-practices/blob/master/review/reviewer/comments.md
Chromium Docs - Respectful Code Reviews Do (やるべきこと) ・能力と善意を想定する ・会って話し合う
・理由を説明する ・理由を聞く https://chromium.googlesource.com/chromium/src/+/HEAD/docs/cr_respect.md
Chromium Docs - Respectful Code Reviews Do (やるべきこと) ・終わりを見つける ・適度な時間内に返信する
・ポジティブに述べる
Chromium Docs - Respectful Code Reviews Don’t (やってはいけないこと) ・人を辱めない ・過激な言い方やネガティブな表現は避ける
・ツールの使用を妨げない ・自転車置き場にしない https://chromium.googlesource.com/chromium/src/+/HEAD/docs/cr_respect.md
・Google Engineering Practices Documentation ・Chromium Docs - Respectful Code Reviews
原文もタメになるので、ぜひご覧ください!
まとめ レビュアーの心構えとして大切なこと ① レビュイーに対して敬意と思いやりを持つ ② 理由を聞く・説明する ③ 直接話してみる(リモートもあり!) 明日からのコードレビューでぜひ実践してみよう!
ご清聴ありがとうございました