コードレビューについて : 小野和俊のブログ

伊藤直也さんが「些末なコードレビュー」というエントリを書いて話題になっている。このエントリで伊藤さんはコードレビューの話と、はてなのJavaScriptの話と2つの話題に触れている。前者のコードレビューについてはアプレッソでは8年ほど前から「コードレビューを通っていないコードはコミット不可」というルールですべてのソースコードに対してコードレビューを必須にしてきた関係で私も思うところがあるので、エントリを書いてみようと思う。 伊藤さんが例示しているように、インデントやreturnの省略などの話は好みの問題であり、議論してもソフトウェアの改善につながらない。なのでコードレビューでこうした宗教論争が起こるようなら、コーディング規約を見直すべきだ。「無駄に悩んだり議論したりすることを減らす」ことはコーディング規約の主たる効果のひとつだと言える。 コードレビューに慣れないチームが、何の考えもナシにコ

[lalha]

ブログを書いた。

[rryu]

「ひよコード」のことを私は「若さあふれるコード」と呼んでいる気がする。

[trashtoy]

ひよコード、良い

[kumokaji]

ひよコード良い

[lxyuma]

レビュー観点

[yamadar]

「やってよかった」コードレビューがとても参考になる。

[qaz76]

_φ(･_･

[yoshiyoshifujii]

なるほど。こーやってまとめて見せていただくと、自分のやり方は悪くなかったことが、分かって良かった。 "コードレビューについて : 小野和俊のブログ"

[haishi]

コードレビューについてはアプレッソでは8年ほど前から「コードレビューを通っていないコードはコミット不可」というルールですべてのソースコードに対してコードレビューを必須にしてきた関係で私も思うところがあ

[manhole]

「ひよコード」

[motsuni2]

ひよコード

[ledsun]

とても参考になる。

[yosuke_furukawa]

ここがピヨピヨしてる、かわいいな。

[t-wada]

コードレビューの視点、およびペアプログラミングとの併用について。「ひよコード」いいな

[h5y1m141]

ひよコードという表現いいなぁ。ここがピヨピヨしてるっていうのは、未熟だけど伸びしろあるというのを的確に表している感じ

[kattton]

なるほど「ひよコード」/ 俺も「ひよコーダー」なのでコードを書くとピヨピヨしていると突っ込まれることが多い。

[joker1007]

確かにこういう内容をレビュー時に指摘している。分類してくれると分かりやすくて良い感じ。

[kozy4324]

今度「ひよコード」使ってみようかな / if-then-elseの話はメンタルモデルと一致するかどうかが一番重要なのではと最近考えてる

[estragon]

「ひよコード」、「ここがぴよぴよしている」

[Vaduz]

「ここがピヨピヨしている」

[devorgachem]

参考になーる

[watarun7]

いい分類だなー

[osawagiboy]

プログラミング

[koemu]

「コードレビューでは「ダメ出しを受けた」という印象になるところが、ペアプロでは「ちょっと案があるから一緒にやってみようか」という雰囲気になることが多々ある。」

[likk]

「ひよコード」「ここがピヨピヨしてる」なんて言われたら「ひよったコード書いてんじゃねえクソが」って言われてると受け取る人もいそう

[saken]

ぴよぴよ

[yoshidashingo]

さ、ささ...

[dolmen777]

🐥 あまり美しくないコードを「ひよコード」、美しくない箇所は「ここがピヨピヨしてる」と表現するようにしています。 🐥

[tpircs]

糞コードってかなり悪質なコードに対しては使うけど、普通のレビューじゃそもそもそんな言葉使わないような。レビューって「俺だったらこうする」って伝えるだけで「ダメだから修正しろ」って言わないしなぁ。

[zanasta]

ピヨピヨ(･ω･ )

[zorio]

人間同士なんで、どう受け取られるかを考慮するのは大事。面と向かってお前のコードはクソだとか言ってたら一緒に仕事出来ない。必要に応じて対面で話すのも重要。文章だけだと喧嘩になりがち。

[gikazigo]

コードレビューについて : 小野和俊のブログ