コードレビューの極意。それは「自分のことは棚に上げる」こと！！ - Qiita

はじめに：コードを良くするためなら遠慮は不要 昨日Twitterに投稿した内容が思った以上に拡散されていたので、タイムラインに流れてしまわないようにQiitaにも書いておきます。 内容は上に書いてあるとおりです。 コードレビューはコードの問題点を指摘し、そのコードを良くすることが第一の目的です。 そのため、少しでもおかしいと思った部分は遠慮せずにどんどんツッコむ必要があります。 しかし、レビューする側が「これ、自分もあまりできてないんだよなあ」「お前もできてないじゃん！って言われたら返す言葉もないし・・・」などと思って遠慮してしまうと、コードを改善できるせっかくのチャンスが失われてしまいます。 「自分ができているかどうか」と「そのコードを改善すること」は、それぞれ別の問題です。 なので、レビューする人は自分のことを棚に上げてでも、コードの問題点を指摘する必要があります。 また、レビューされ

[xbs2r]

棚に上げるというか、指摘が自分もできてないと思うのであれば、それは次に直せばいいのであって、他人のコードレビューの時に心配するこっちゃない

[yyanma]

言っただけに次からやらなきゃ感も生まれるような気もするので、だいじですね

[umai_bow]

他者への指摘はすべてこれだと思ってるなあ

[nacika_inscatolare]

そして誰もやらなかった

[otihateten3510]

未だにコードレビューというものが分からない。そんなに言うことないか、設計レベルの話になるから「今更言っても遅すぎるなあ」になる。コード議論とか、コードTIPS共有なら分かるんだが。

[koogawa]

正に心理的安全性

[macority]

コードレビューという、レビューは全般的にそうだと思う。そして、自分が設計、実装する時には同じ指摘をされないように気をつける。

[mr_mayama]

ギスギス感がビシビシ伝わる

[animist]

コードに限らず自分の事を棚上げしないと批評なんてできない

[ducktoon]

意識して自分のことは棚に上げているならいいけど意識せずに棚に上げてる奴はぶっ飛ばしたくなる

[hiroomi]

“「自分のことは棚に上げてコードレビューしろ」という話は、僕がソニックガーデンに入社して間もない頃に言われた言葉です。”

[mas-higa]

心に棚を作れ!

[teckl]

> 「自分ができているかどうか」と「そのコードを改善すること」は、それぞれ別の問題です。遠慮してしまうと、コードを改善できるせっかくのチャンスが失われてしまいます。

[nihonbuson]

分かる。 というか、コードレビューに限らず、レビュー全般に言えることだと思うけど。 第三者の目線って大切だし、自分で作りつつ第三者の目線になるのは相当難しい。

[Nyoho]

良い文化っぽい

[aroma_black]

“遠慮してしまうと、コードを改善できるせっかくのチャンスが失われてしまいます。 「自分ができているかどうか」と「そのコードを改善すること」は、それぞれ別の問題です。”

[moccos_info]

正しい指摘されたと仮定して、そこでお前はどうなんだと人間批判を始めるのであれば、それは技術者の議論における最低限を満たしていないしなあ

[s99e209]

コードレビューでは自分のことは棚に上げて問題点を指摘することでチーム内のコード品質はもっと向上する。

[naga_sawa]

『自分のことは棚に上げる』なるほど/ある程度技量が揃ってないと難しいかもしれない

[l-_-ll]

心に棚を作れ

[torinky]

コードレビューの極意。それは「自分のことは棚に上げる」こと！！

[ngsw]

とてもよい。「書く」と「レビューする」の視点観点は違うことが多いのだから、レビュアー自身が「書く」側のレイヤにまで下りてしまうとレビューが機能しないという事だと受けとめた。

[raitu]

職場の心理的安全性が担保されてると安心して問題点を指摘できるという話

[uturi]

レビューって基本的にそういうもんだよな、とは思う。慣れてないとそういうのに抵抗があるんだけども。

[ene0kcal]

「棚上げ」は一見良いのだが、「このハーゲー！」等言う人外者がいない前提。人外者がいた場合はコードレビューが荒れるだけでなく心を病むけど。結局、レビューは精神の落ち着いたグループでしかやれない。

[yoshiko_pg]

“「自分ができているかどうか」と「そのコードを改善すること」は、それぞれ別の問題です。なので、レビューする人は自分のことを棚に上げてでも、コードの問題点を指摘する必要があります。”

[Shalie]

これ、いろんな仕事に言えそう。"「お前もできてないじゃん！って言われたら返す言葉もないし・・・」などと思って遠慮してしまうと、コードを改善できるせっかくのチャンスが失われてしまいます"

[cc000777]

これとても大事だと感じてる

[l-liroki]

よい

[verda]

コードレビューに関わらず お前は出来んのか？というのは言わないのがお約束さ

[aceraceae]

これはそのとおり。批評家は創作者である必要はない。でもブーメランを期待するような環境はけっこう多いんじゃないかな。

[cosmocleaner]

匿名にしたらよいと思う

[civitaspo]

コードレビューの極意。それは「自分のことは棚に上げる」こと！！ - Qiita