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
akkiee76
August 23, 2022
0
360
明日からできるコードレビューの心構え
akkiee76
August 23, 2022
Tweet
Share
More Decks by akkiee76
See All by akkiee76
Android Target SDK 35 (Android 15) 対応の概要
akkie76
0
240
コードレビューを支援するAI技術の応用
akkie76
5
480
オブジェクト指向コードレビューの新しいアプローチ
akkie76
3
6.5k
Jetpack Compose で Adaptive Layout に対応しよう
akkie76
0
140
Observationではじめる値監視
akkie76
3
2k
TextField 表示スタイル変更の 有効活用例 5 選
akkie76
0
320
rememberUpdatedState の使いどころを考える
akkie76
0
210
M3 NavigationBar をマスターする
akkie76
0
510
アーキテクチャを明文化して臨んだ新規アプリ開発戦略
akkie76
0
140
Featured
See All Featured
Building Applications with DynamoDB
mza
88
5.7k
jQuery: Nuts, Bolts and Bling
dougneiner
60
7.2k
Robots, Beer and Maslow
schacon
PRO
155
8k
Principles of Awesome APIs and How to Build Them.
keavy
121
16k
Pencils Down: Stop Designing & Start Developing
hursman
117
11k
Large-scale JavaScript Application Architecture
addyosmani
504
110k
Building a Scalable Design System with Sketch
lauravandoore
457
32k
Building Adaptive Systems
keathley
32
1.9k
ParisWeb 2013: Learning to Love: Crash Course in Emotional UX Design
dotmariusz
104
6.7k
Build The Right Thing And Hit Your Dates
maggiecrowley
25
2k
It's Worth the Effort
3n
180
27k
The World Runs on Bad Software
bkeepers
PRO
61
6.8k
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
原文もタメになるので、ぜひご覧ください!
まとめ レビュアーの心構えとして大切なこと ① レビュイーに対して敬意と思いやりを持つ ② 理由を聞く・説明する ③ 直接話してみる(リモートもあり!) 明日からのコードレビューでぜひ実践してみよう!
ご清聴ありがとうございました