初期値をundefinedにするのは誤りですか？

Question

const defaultState = {
    isUserAuthenticated: false,
    error: {
        message: '',
        code: undefined, // or null???
    }
};

例えば、エラーコードを番号にしていて、number型で扱いたい場合なのですが、初期値に0とかは使いたくありません。そういった場合、undefinedにするのはOKなのでしょうか？

もしくはnullにすべきですか？

codeをそもそも'123'みたいに文字列にすべきな気もしますが、他のケースの場合でも、undefinedって初期値にして良いのかなって思ったことがあったので、ご回答いただけると嬉しいです！

Comments

[coco_bauer]

初期値である事を検出する必要があるのでしょうか？　また、codeに値を入れた後に初期値(undefined?)に戻すというような使い方は想定されてるのでしょうか？

[退会済みユーザー]

codeに値を入れた後は初期値に戻すことは考えてませんでしたが、言われてみれば、いろんなところから参照される可能性があるアプリケーションのスタートなので、認証エラーがない場合はエラーコードを初期値に戻すかもしれません！

[退会済みユーザー]

あ、やっぱり戻さないかもしれません。。。すみません、まとまってなくて。react、reduxなので、初期値にスタート戻してしまうと再レンダリングのサイクルが始まってしまいますね。

Answer 1

undefinedでもOKと思いますが、個人的にはnullをセットしています。
理由としては、undefinedは予約語ではないため、自分のあずかり知らないところでvar undefined = 0;のような超イレギュラーな処理をされ、ハマった経験があるためです。
https://jsfiddle.net/takmatz/vcmug9nd/

Comments

[退会済みユーザー]

わー！予約語ではないのですね！知らなかったです。undefinedを変数名にされるとか想定外過ぎますね。。。
参考になる御回答ありがとうございます！

[miyabi-sun]

まぁNullをセットするのが主流かと思いますが、
undefindが欲しいなら`var hoge = void 0`がベストプラクティスですね。

[退会済みユーザー]

void 0なんて初めてみました！こちらのコメント見た後にたまたまbabelのコンパイルされたやつみてたら、void 0が早速ソースコード内にありました（笑）

Answer 2

コードの見た感じからして、redux用の初期stateを作っているところでしょうか。
その場合だと、「error/codeは文字列またはundefinedである」と自分の中で定義してコード中でもその条件を保てるなら、初期stateをこう書いてしまうこともできます。

const defaultState = {
    isUserAuthenticated: false,
    error: {
        message: ''
    }
};

要は、undefinedであるならばキーごと書かない、と。
でもこれだとstateオブジェクトの構造がdefaultStateオブジェクトを一目見ても把握できないじゃないか、と思われるなら、「error/codeは文字列またはnullである」と定義してnullを初期値にするという方向へも行けます。

正直、undefinedはコードに持ち込みたくない存在です。
おそらくundefinedは、配列や連想配列のキーが見つからなかったときにも実行を止めずにできる限り処理が続けられるようにと用意されたオブジェクトです。しかしオブジェクトとして用意されてしまったためおかしな動きもします。

var a = {key: undefined}

と書くと、aの内容は{}でしょうか、{key: undefined}でしょうか？
正解は後者の{key: undefined}の方なのですが、これだめでしょ、undefinedが格納できてしまっては、配列アクセスしてundefinedが得られたとしても、最初からそのキーがなかったのかそのキーでundefinedが格納されていたのか区別つきません。
こんなおかしいundefinedを自分のコードの中には登場させたくないと考えると、せめてnullを使っていきたいなとなります。

Comments

[退会済みユーザー]

まさにreduxです！初期stateでcodeは書かないというやり方が良い気がしました！ありがとうございます！

Answer 3

NaN なんてどうでしょう？
わかればナンでもいいとは思いますが

Comments

[退会済みユーザー]

確かにナンか良さそうな気がしますね！

[miyabi-sun]

ナンにせよ斬新な発想ですね、面白い。
個人的には数値を文字列で割っちゃった的な駄目な計算結果という先入観があると思うのでミスリードにならないかが心配です。
あと`NaN === NaN`がflaseになるのが気持ち悪いですね。
https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/NaN

[x_x]

ナンだってー！

[退会済みユーザー]

ナンというかもはや！