配列もconstで宣言すべきか

Question

JavaScriptで、最近は再代入しない変数はすべてconstで宣言するべきだという風潮があります。
constで宣言した配列の中身を上書きすることはできますが、そうする配列もconstで宣言したほうが良いのでしょうか。
よろしくお願いします。

Answer 1

プロジェクトの規約や人によるのかなぁという気がします。私は「const にする」派です。
明確にオブジェクト（配列含む）の中身も固定してしまいたい場合は Object.freeze を使ってしまえばコードを読む側にもそれを伝えることができますので、それで大体のケースはカバーできるかと思います。

例えば「この配列の中身は後から書き換えるんだ」という意図を持ってあえて let にする、という考えもあるかと思いますし、それは否定できるようなものでもありません。

Comments

[umaru]

すみません、コメントを書いている途中に回答いただいたようで????
letを使う派も少なからずいるのですね。参考になります。
しかし、Object.freezeを使うという手段がある以上は一貫してconstを使う方が合理的な感じもしますね！

[htsign]

そうですね、その方がいいと私も思います！

上では「例えば」と書いた通り「そういう人もいるだろうな」という想定で書いたもので、本当にそう思いながら書く人がいるかは、私には「たぶんいると思います」としか言えないです…。
const はあくまで「参照の固定」に過ぎない、という点で知らない人にはハマりポイントな気がします。

Answer 2

違和感があるのは、わかります。私も違和感ありました。

---

再代入はできないので、「再代入をしてないよ」という意味でconstで宣言した方がいいと思います。
コードを読む人が楽になるかと思います。

Comments

[umaru]

なるほど、確かに再代入をしていないという一貫した基準でconstを使えば読む人も楽ですね！
少し違和感はありますが、constを使っていこうと思います。

Answer 3

constで宣言した配列の中身を上書きすることはできますが、そうする配列もconstで宣言したほうが良いのでしょうか。

はい、配列のインスタンスを置き換えることがないならconstが適切です。

Comments

[umaru]

ありがとうございます☺
なるほど、基準はあくまでインスタンスまるごと再代入するかということなんですね。