レビュアーの意見に対処する方法

レビュアーの意見を受けてから適切に対処する方法をいくつか紹介する。

感情的に受け入れない

コードレビューの重要な価値は、コードと製品の品質を高く維持するためだ。レビュアーはあなたと、共同のコードと、Googleを助けるためにあなたのコードを批評するだけで、あなたやあなたの実力を攻撃しているわけではない。

たまにはレビュアーが不満げにコメントを残すこともある。それは確かにレビュアーの過ちだが、作成者もこのような状況に備える必要がある。レビュアーがどんな助けを与えようとしたのかを自問し、相手が実際にそのように言ったと想像してレビューを続ける。

怒った感情状態では、コードレビューに絶対コメントを残さない。あまりにも怒って不気味で親切なコメントができない状態なら、しばらく席を外したり、心が落ち着くまで別の仕事をする。それでもレビュアーが親切ではなく、建設的な方向へと会話をしようとしない場合は、直接会って話し合う。直接会うことができないか、またはビデオ通話もできない場合は、個人的にメールを送る。どんな行動が気に入らず、どのように自分を扱ってほしいということを優しい言い方で伝える。個人的に対話をしても建設的な討論に繋がらなかったり、対話後も是正ができない場合は、上位マネージャを通じて仲裁する。

コードを修正する

レビュアーがコードを理解していない場合は、最初にコードをより明確に修正する。コードをより明確にすることができない場合は、コードがなぜ存在するのかを作成する。コメントも無意味に見える場合は、最後の手段としてコードレビューツールに説明を作成します。すぐにレビュアーがコードを理解できなければ、後でコードを読む人も理解できない可能性が高く、その時にコードレビューツールに書いてある内容は、何の役にも立たないということに留意する。

自分で悩んでみる

あなたはCLを完成させるために多くの努力をした。一生懸命やったのでもう修正することがないと思い、レビューを要請することもある。だからレビュアーが色々と改善点を指摘してくると本能的に抵抗感が生じ、相手の意見が間違っていたという気がする。しかし、どんなに完璧だと感じても一歩下がって、レビュアーの提案が共同のコードとGoogleに役立つのかを考えてみる。もしレビュアーの言葉が合っているか判断ができなければ、もう少し説明を要請する。

一歩下がって考えてみても、あなたの方法が正しいという結論になったら、なぜ自分の方法がコードと、ユーザーと、Googleにもっと有益なのか説明するコメントをつけても良い。実際、レビュアーはあなたが最善の選択をするように提案し、助けを与えるだけだ。もしかしたら、あなたがユーザーやコードをよりよく知っているかもしれない。レビュアーにもこの情報を共有してみる。ほとんどの場合、技術的な議論を通じて、あなたとレビュアーは合意に達することができる。

意見の対立を解決する

意見が衝突したら、常にレビュアーとの合意するように努力する。意見の擦り合わせができない場合は、コードレビューの大原則に基づいて解決する。