コードレビューで嫌われる人の特徴7選 - Qiita

Deleted articles cannot be recovered. Draft of this article would be also deleted. Are you sure you want to delete this article? 「コードレビュー・・・うっ頭が」となっているそこのアナタへ。 先週弊社キカガクで人生初の実務コードレビュー体験をしました。 控えめに言って最高すぎました。 お互いが「気持ちよく・効率的に」学びを深められるように組まれた一級品のレビュー構成。 細部に渡る心遣いとテクニックの為せる技だと思いました。 そこで私は考えた ー。 真逆のことをしたらどうなるんだろう？ 想像してみたらなかなかブラックな開発環境が脳内で出来上がりました （大学時代のコードレビュー現場そっくりだなと思ったのは内緒）。 自分がコードレビューに参加する時こうはなるまいぞいう

[bopperjp]

ただのロジカルな指摘でも、「決定権がレビュイにない」「締め切りが差し迫ってる」ってだけで十分攻撃と誤認されるよ。それを解決するのは結構難しい。

[w1234567]

レビュー指摘内容をコーティングガイドラインに柔軟に反映していく作業が一番重要な気がする。レビュアーの頭の中にしかないすっげー狭い常識を押し付けられるのが一番ストレス溜まるからな

[odakaho]

指摘に対して「(俺)さんはこだわりが強いんですね^^; リリース予定が近いので今回は修正しません^^;」みたいな反応されると次からキツめの言い方になるってことはある。

[toromoti]

最近思う、コードレビューという文化は日本人に向いていないのではないか。

[Error401]

英語でレビューするというハックを思いついた

[daichirata]

出来てない所に指摘するからドンドン褒めなくなるので、意識して良い点は良いですねって書くようにしたい。

[ducktoon]

卑屈になってはいけないとはいえ偉くなったと勘違いしてマウントのようなレビューを繰り返す奴がいるんだよなあ

[xlc]

設計レビューだとなぜかこうはならない。日本におけるプログラマの地位の低さ、自尊心の低さが人格攻撃合戦を生んでいるのではないかな。

[arket789]

なぜ直すべきかの説明と道筋展開は気をつけたいところ… ちょっとでも良いと感じた箇所は少しくらいオーバーに褒め＆感謝するように心がけている。

[hiroomi]

コードレビュー

[ene0kcal]

熟練度が違い、学習能力、教授力、忍耐性、性格も何もかも違うので、むしろレビューアーの能力が一番高くないと駄目だと最近考えている。別の視点では、レビューアーの評価をレビューイーが後でするとか。

[nodoca_engine]

「なぜ直すべきか説明しない」とか「どう直せばいいのか示さない」とかは実装者のレベルとか修正難度とかにもよりそう。親切にしすぎると一生成長してくれないので…

[bitebitebite]

コードレビューに限らず成果物のレビューにおける重要なマインド

[fukken]

理由を説明するのは自分の勉強だと思って、出典や「どのような時に困るか」を指摘するようにしてる。単に好みの問題の場合や些事の場合はそのように書く（採否は委ねる）。

[d14a]

日本語がレビューという行為に向いてないってコメントで納得（英語でやるかはさておき……）レビュールールをチームで共有・更新してきたいですね。

[doscoy_t]

「褒める」…サンドイッチ話法は本質を相手に伝わりづらくさせるのでレビューの場で使うべきではない。TeamGeekにも書いてある。何者でもない一人のQiitaの１意見でしかないものがまるで正解かのように広がる現象も嫌い

[saiyu99sp]

レビューでクラスの責務と違うこと指摘したら「お前は設計意図を理解してない！！」とキレられたよ。そこの元の設計俺作ったのに。

[remonoil]

感情入れない＆些細なことでも理由を絶対に書くようにしている。例文で書くなら"axiosを使ってください。理由: 全体で同一のライブラリを使うことでライブラリ移行や脆弱性対応などが楽になるため" なので重いとか言わ

[kiyo_hiko]

“褒めない” 俺が…いる。例えば「さすが」を1回は言うとか、心がければいいのかな

[ghostbass]

櫻井ボイスで「ﾅﾝﾃﾞｽｶｺﾚ」って言ってしまいそう