指摘・レビュー4 択難易度 標準

指摘「ここ、もう少し何とかなりませんか」の意図が分からない。返信として適切なのは？

正答

1. 了解です。（何を直すか分からないまま、返信だけしてコードはそのままにする）
2. 読みやすさと性能、どちらの観点でしょうか。関数に分ける案で合っていますか？
3. 意図が分からなかったので、とりあえず関数全体を自分なりに書き直してみました。
4. どこをどう直せばよいのか、具体的に書いていただかないとこちらでは対応できません。

解説

意図が分からない指摘は、推測で直すより確かめたほうが手戻りが少ない。「どの観点か」と「自分はこう理解したが合っているか」を具体的に書くと、相手は一言で答えられる。分からないまま全体を書き直すと、別の問題を持ち込んだり、レビューをやり直す負担を増やしたりする。突き放す返し方は、確認の意図があっても対立を生みやすい。

参考: google.github.io

reviewreplyclarify