指摘・レビュー4 択難易度 応用

指摘「このループは map に書き換えてください」。途中で抜ける必要がある。返信として適切なのは？

正答

1. 今のままでも動いていて、テストも全部通っているので、書き換える必要はないと思います。
2. 前回のレビューでは同じ書き方でも何も言われなかったので、このままにします。
3. わかりました、ご指摘どおり map に直します。（途中で抜けられなくなるが直す）
4. 途中で抜けたいので for にしています。map だと全要素を処理してしまうためです。

解説

指摘に同意できない場合は、なぜ今の書き方にしたのかを技術的な理由で説明する。理由があれば、レビュアーは納得するか、より良い代案を出せる。「動いているから」は理由にならず、過去のレビューを持ち出すのは論点のすり替えになる。納得していないまま言われたとおりに直すと、不具合を持ち込むおそれがあるうえ、相手も誤解に気づけない。

参考: google.github.io

reviewreplydisagree