コードレビュー研修

2020/07/21 に弊社新卒向けに実施したコードレビュー研修の資料です。

[ducktoon]

コードレビューでマウンティングしてくるやつが不幸になりますように

[turanukimaru]

うーん？私とは随分違うな。私はレビュー時には必ず想定するコードを提示してる。例えば f(x,a) と f(y,a) があったら x.f(a) を書いて見せて y.f(a) はレビュイーに書いてもらう。まぁこれは新人への指導だから目的が違うか。

[yujiorama]

" 1. 単純にプロダクトの品質担保のため。 2. プロダクトやコード上の知識を共有するため。 3. （新人さん向け）他人のコードを読む練習をするため。"

[diveintounlimit]

衒学的な現場に居たのでとてもよく分かる

[dltlt]

トヨタ社内でのコードレビューの話か？……と早とちりした。（デンソーが納入したコードとかの）

[tjmschk]

コードレビューだけでなくレビューしやすいPRのつくりかたも

[surume000]

レビューは客観的であり反駁可能性を持たせるべきというのは同意。ただめっちゃめんどくさいよね

[wwolf]

コードレビューではわきまえ重要

[gcyn]

（（無理な話と知りつつ）実際のコメント見てみたいですね…！！)

[morimarii]

レビュー観点・方針・ルールが事前にチーム内で合意されてないの？？

[atico]

実際に動かしてみる。不具合を見つけて報告する。を重ねてレビュアーとして信頼してもらう。フラグか多いコードは、フラグの組み合わせが何通りあって、テストしきれませんのように何が不都合かを伝える。

[odakaho]

“げんがく‐てき 学問・知識を自慢し、見せびらかすさま。ペダンティック。” “PREP法は主にビジネスシーンで用いられる文章構成方法であり、簡潔かつ説得力のある文章を作成する際に用いられる。”