コードレビューは「コードの欠点を指摘する行為」ではない - id:anatooのブログ

コードレビューを「コードの欠点を指摘する行為」だと無意識に思っている人を見かけるけども、そういうふうに認識しないほうがチームにとって良いですよ、という話。理由は以下。 レビュワーの方がレビュイーよりも実力が無いといけない、という認識と結びつきがち チームの若いメンバーがレビュワーになりづらくなる 古株のメンバーやリーダーの書いたコードがレビューされなくなる レビューで指摘された項目がない = (指摘された欠点が無いということなので)良いコードという図式になりやすい レビュワーが欠点を指摘するあまり攻撃的なレビューをしてしまうことがある 逆にレビュワーがレビュイーに遠慮してあまりレビューしなかったりする レビュワーが誰でもわかる間違いしか指摘できなくなり、建設的な議論が起こらなくなる コードレビューが機能不全に陥る原因の一つが、コードレビューに対する基本的な認識がずれていることだと思う。 じ

[anatoo]

書いた

[nunota]

人間性を否定する行為でもないよと、前の上司に言いたい。

[crexist]

“じゃあどういう感じでコードレビューしていけばいいのかは、あとで書く” 今書いてくれ〜(期待)

[aukusoe]

はてな引用スターは「ブックマークコメントの欠点を指摘する行為」ではない

[NOV1975]

「ではない」ではなく「だけではない」かな。

[Beluga]

コードレビューやるよと言うと「コードの間違いとか見つけてくれるんですか？」とか言ってくるやついるわな。違うと言うと「それあんまり意味なさそうですね」と。そんなもん日頃からLintツールでチェックしとけや。

[xlc]

私のコードレビューの方法はコードを書いたものがそれを説明するというやり方。指摘するまでもなく自分で間違いに気づく。

[kknsd]

レビューはもちろんコードの欠点を指摘する事も含まれると思うんだけど、何をレビュー観点とするのか、優先度はどうするのかを予め皆で合意しておかないとグダグダになりがち

[programmablekinoko]

ブログのレイアウトが増田めいていてよい・・・

[chiroruxx]

おお、本題が書かれてないw

[vanbraam]

冒頭3項目は特に重要だと思う.若い/不慣れなレビュワーがベテランのコードをレビューすることは,もっと一般的になっていい

[ardarim]

クソコードを憎んでレビュイーを憎まず

[tettekete37564]

リーダビリティーと最低限のリファクタリングだけでおｋ可読性が低く意図不明なコードほど保守コストが高いコードはない。

[hdampty7]

自分の改修範囲でない規約違反をひたすら指摘されて直させられる行為でもないよねと言いたい。

[Yoshiya]

仕様書のレビューはするけど、コードのレビューなんかした事ありません。仕様書がきちんとできていたら、ソースコードはそれを具体化するだけ。コードレビューをするって事は、仕様がキッチリしてないって事？

[fock]

この記事は「「コードの欠点を指摘する行為」の欠点を指摘している」

[junpeso]

vimmerに言いたいのだけどtypoやらlint違反多すぎ。IDE並の環境整えてからvimで書いてください。

[perl-o-pal]

codeReviewImplが実装されてません！

[lyiase]

コードレビューは「レビュイーが成長した将来、自分のコードを見たときに悶絶しないようにすること」を念頭に入れて「ルールに従っていて、処理が容易に理解出来て、謎の部分は無いか？」を念頭にレビューしてるね。

[fai_fx]

コードには書いた人の人間性が表れているので、コードの欠陥を指摘することは、その人の人間性の欠陥を指摘することになる…気がしてならない。

[Derabon]

わかる。

[Palantir]

あとで書いたのも読みたい

[agricola]

もちろん人格攻撃になっては駄目だが、少なくともバグや脆弱性は見つけたら指摘しなきゃイカンだろ。雑なレビューで承認されたバグ入りコードの尻拭いをする身にもなってくれ。

[tick2tack]

なるほど。/　ブコメ "ある日自分が引き継ぐことになっても困らないものになっているか"

[ksk_uchimura]

コードレビューはコミュニケーションだと思ってるから、何のコメントや議論もないってのは寂しいなぁ。

[ockeghem]

予告編だった

[kazumi_wakatsu]

“古株のメンバーやリーダーの書いたコードがレビューされなくなる”外部のレビューに攻撃的に噛み付いたりして恥かくやつだ

[donchan922]

すごく共感します。続きが気になる。。

[roirrawedoc]

良い指摘だなー

[koogawa]

続きが気になっちゃう！

[degucho]

説明できないコードがないかどうか