コードレビューで気をつけていること 5 選

こんにちはnasaちゃんです。 コードレビューの記事を見かけたので僕がコードレビュー時に考えていること、行っていることを書いてみようと思いました。 この記事ではレビューを受ける側、行なう側それぞれの話がありましたが、ここではレビューを行なう側のことを書いていきます。(洗い出してみるとすべてがレビューコメントに関するものでした。) Whyを書く コードの変更をリクエストする際になぜ変更したほうが良いのかを書くようにしています。 レビュイーが変更を取り込むか判断する材料になるのでちゃんとなぜこっちのコードのほうが望ましいのかを書くようにしています。あと、言語化することで自分の理解も深まるので良いですね。 このとき、レビュー中のコードを批判しないことを心がけています。 「今のコードは〇〇というデメリットがあるので変えたほうが良い」と伝えるよりも「このような書き方をすることで〇〇がよくなると思いま

[sbrtnpg]

この業界に限らずマトモなフィードバックについて無理解な人間の多いこと。基本的でも大事なことなので、こうやって各々アウトプットするのはとてもいいと思う。

[otchy210]

この辺はよく分かる。これにもう一つ付け加えるなら「良かったところを積極的に褒める」があるかな。「リファクタありがと！」「へぇ知らなかった！」「(以前の指摘を受けて) 今度はバッチリだね！」とか。

[odakaho]

“リンクを無言で投げつける行為を「リンクで殴る」と呼んでいます”

[yamami78651]

何を指摘してるのか、わかるのが第一でその上でそれぞれの心遣いがあればいいなと、確かに思う。やりとりは少なくできればいいと心がけてる。

[remonoil]

無言〇〇はブチギレの場合にやると面白いぞ

[diveintounlimit]

こういうのをアウトプットしていくのは良いね。ただ個人の好みは争いしか生まない。suggestだけが駄目っていうのは手間かかるだけのケースも多いのでやらないことが多い。絵文字を毎度付けるのは個人的にはウザい。