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

2017/07/15 09:51:04

xbs2r

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

2017/07/15 11:16:36

jiminko

ファンタジーなレビューになりそう

2017/07/15 13:16:26

yyanma

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

2017/07/15 14:55:19

koichi99

せやな。

2017/07/15 15:08:37

nacika_inscatolare

そして誰もやらなかった

2017/07/15 15:24:12

garage-kid

53

2017/07/15 15:54:00

megazalrock

コードレビューって1つのコードに対する目の数を増やす以外にもまさに他山の石効果があるんだな。やっぱりプログラミングに限らず自分が書いたものに対してって頭のなかに入っちゃってるから盲目になりがち。

2017/07/15 16:15:49

civitaspo

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

2017/07/15 16:26:29

cosmocleaner

匿名にしたらよいと思う

2017/07/15 16:48:11

aceraceae

そのとおりだと思う。コードレビューだけじゃなくて業務における評価なども似たようなものだと思う。

2017/07/15 18:22:45

verda

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

2017/07/15 20:25:37

l-liroki

よい

2017/07/15 21:00:07

cc000777

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

2017/07/15 21:42:08

Shalie

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

2017/07/15 22:11:42

otihateten3510

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

2017/07/15 22:30:10

macority

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

2017/07/15 23:26:00

koogawa

正に心理的安全性

2017/07/16 00:17:58

yoshiko_pg

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

2017/07/16 00:23:42

ene0kcal

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

2017/07/16 01:51:56

umai_bow

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

2017/07/16 08:50:52

uturi

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

2017/07/16 08:52:04

raitu

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

2017/07/16 10:22:15

kibarashi9

相手は棚に上がってることなんて知らんし、一方的にムカついてますけどね。

2017/07/16 10:35:40

ngsw

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

2017/07/16 11:25:59

hatehateyahoo

棚に上げ続けてる人は嫌われる

2017/07/16 11:58:05

wordi

スパゲティコード書いた他人のバグ修正したら、その人が手抜きしたであろう所を色々修正点出してきた時は流石に自分で直せと思った事ある

2017/07/16 18:51:31

torinky

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

2017/07/16 20:53:49

mr_mayama

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

2017/07/17 03:19:36

l-_-ll

心に棚を作れ

2017/07/17 21:18:02

naga_sawa

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

2017/07/19 11:27:16

s99e209

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

2017/07/19 12:06:42

moccos_info

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

2017/07/19 12:45:41

aroma_black

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

2017/07/19 20:26:55

animist

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