コードレビューはコードの責任をレビュアに転嫁する作業 - まなめはうす

言いたいことはタイトルの通りです。 c5meru.hatenablog.jp コードレビューをつらいと思う人がいるなんて自分には考えもしてなかったので、この記事には驚かされました。指摘を人格否定と思いがちな人がいるのか、人格否定かのような指摘をする人がいるのか、その両方かは分かりませんが、つらいですね。 プログラマである自分にとって、結果という変わらないものではなく、創意工夫を見せることができるコードを見てもらえる場所って一番の見せ場じゃないですか。プロのスポーツ選手だって成績という数字も大事だけれど、いかに試合でファンを魅了できるかも大事だと思うんですよ。だったら、プロのグラマーである我々も品質とか効率とかの数字だけでなく、レビューという最高のステージでレビュアを魅了させてこそだと思うんです。特に、難しいプログラムを任されるようになってからは、いかに難しい仕様を分かりやすくコードに落とし

[netcraft3]

おそらく指摘の仕方の問題でコードレビュー時に「はぁ？」「そう書くの…」「全然ダメじゃんw（嘲笑）」とかプレッシャー掛けてくる人が過去の職場にいた。

[turanukimaru]

言ってることは正しいのだが難しすぎるというか、「コードレビューのやり方教育を受けた」人っている？教育されてないってことはうまくやれるとは限らないってことだよ。今必要なのはきっとレビューの仕方の具体例。

[hiro_curry]

人類にコードレビューは早すぎたのだ…。

[nokogiring]

レビューの人格否定は結構あると思う。 昔の現場だと設計書レビューで怒号が飛び交い泣く人とかいたし。 ただ淡々と製品を良くしたいと思ってやれるといいんだけど、ついついマウティングがはじまることもある

[nharuki]

責任を一人で追わせないことによる心理的負担の軽減。バグが発覚しても自分だけが苛まれることを避けられるのが最大の効能だと思う

[ducky19999]

人間性に帰着してしまう

[blockchain-bitcoin]

この記事の筆者はきっと良い同僚に恵まれているのだろうが、元記事の意味はコードレビューを気に入らない人間を攻撃する格好のチャンスと捉えている奴がいるということでは？

[sin20xx]

難しいのは優秀なプログラマーが必ずしも優秀なレビュワーではないのですよ。技術がある人ではなく適切な目と技術を持った人をレビュワーにしなければならないのだけど、そういう環境では技術のみで人を選んでしまう

[koogawa]

自分がレビューしたところでバグが出ると「正直すまんかった」って気持ちになるもんな😩

[tofu-kun]

コードと人格を切り離すの、誰でもできることではないなと思うので、そういう気持ちでやって文化を作って行く必要がある。あと人間以外が指摘できるところを増やす。

[junpeso]

自分はいつでも引き継げるように知識を並列化する場だと思ってる。

[Error401]

UnitTestって何ですかという組織の1回1,000行のコードレビュー。地獄。

[you0328]

まともにレビューできてなくてすみません…

[MasaoBlue]

チームならば「誰が作ったバグでもチームの責任」と胸張って言いたい。つまりコードレビューの実施有無を含めた開発プロセス自体、チームに責任があり、その全体のリスクを下げるためにレビューをする。と思っている

[pmint]

「コードレビューでバグを取る」という発想は… それならテストケースのレビューをしたほうがいい。/「レビューは成長のため」というのも間違い。何でも成長のためにはなるので。

[ghostbass]

sonar,fxcop,stylecopである程度抑えられる気もしている

[puhitaku]

レビュアーもレビュイーもやるけど、とにかく人格否定的な文面や煽りにだけはならないように相当気をつけているし、多分それができない人がいるんだろうな

[jitojito]

デザインやコードはその人の思考も入るので人格批判と受け取られやすいのかもね。

[masayoshinym]

転嫁とまではいかないけど責任の分散くらいに思ってるのでそんなにつらくない。（つらくないとは言っていない）

[xlc]

教育者として有能な人だけが有効なコードレビューができるのだが、そういう人材は少なくなったね。

[nandemoya_future]

責任を回避したいレビュワー(兼上司)が多いこと多いこと。

[inaba629]

コードレビューでバグを潰すのは無理がある。普通にテストコード書いて検証に回すべき。

[cocoasynn]

仕事とプライベートを割り切れなきゃだめなんだよな、結局。（ここに書いてある内容はとてもためになりました）

[tbpg]

ここに書いてある良い側面を失うための土台が整っているのが多重請負構造かなと思う。人間関係の希薄さ、チームメイトの長期的な成長に対する動機の薄さ。ところで我々プログラマーは「プロのグラマー」だったのか

[letsspeak]

コードレビューは極力コードでレビューすると捗る

[baronhorse]

転嫁まではいかない気がするな。

[otihateten3510]

コードレビューって外注に出したほうがよくね？　あとはsonor入れるとか。同僚のレベルなんて自分と大差ないでしょ。　感想会はいいと思うけど

[aceraceae]

いってることはもっともなんだけど、世の中そんなにうまくいかないと思う。匿名化されたコードを匿名化されたレビュアがレビュウするとかいうシステムがあったらいいのに。

[onefootinthegrave]

転嫁とまで言わないけど責任をシェアする感はある。 キモっぽいとこ以外も隅々までみるのは無理だもん

[honeybe]

「良いコードレビューで、良い人生を」 / 良くないコードレビューはわるいぶんめい。

[kiri3]

レビュアの技量次第なのが辛い

[hachibeechan]

最近いろいろと考えた結果、lintとテストがあればコードレビューそこまで必要なくねという気分になりつつある。PR単位のレビュー、おおむねdiffみながらnitpickするだけの作業に陥りがち

[umai_bow]

正しい