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

次の指摘のうち、マージ前に必ず直すべきものは？

正答

1. インデントが 2 スペースになっている。自分は 4 スペースのほうが読みやすい
2. コメントの読点が「、」ではなく「，」になっている
3. ループ条件が i <= arr.length で、配列の範囲外を読む
4. 変数名をもう少し短くしたほうが好み

解説

i <= arr.length だと最後に arr[arr.length] を読み、存在しない要素にアクセスする。これは動作に関わる不具合で、マージ前に直す必要がある。インデント幅や名前の長さの好みは、チームの規約やフォーマッタで決まっていない限り、書いた人の判断に任せてよい点で、必須の修正として出すものではない。

参考: google.github.io

reviewbugnit