指摘・レビュー4 択難易度 基礎

指摘「この関数は 200 行あって読みにくいです」の受け止め方として適切なのは？

正答

1. 自分の能力を否定されたと受け止めて落ち込む
2. コードへの指摘として受け止め、分割の案を考える
3. 相手のコードにも長い関数があると指摘し返す
4. 読みやすさは人それぞれの主観なので、無視してよいと考える

解説

レビューの指摘はコードを良くするためのもので、書いた人の評価ではない。200 行の関数は一度に頭に入りにくく、責務ごとに分けると読みやすく、テストもしやすくなる。指摘し返すのは論点のすり替えで、議論が感情的になる。読みやすさは主観の面もあるが、長さのような具体的な根拠がある指摘を無視する理由にはならない。

参考: google.github.io

reviewmindset