javascript - この三項演算子の何が問題になっていますか?

Question

menuNamesメニュー項目のリストを維持するオブジェクトがあります。menuNamesがすでにある場合slugは、値を増やします。含まれていない場合はslug、値をに等しく設定します1。一意の名前を追跡するためにこれを行っています。私は次のようなものになりたい：

menuNames: {
    home: 1,
    products: 10,
    contact: 1
}

これは機能しません (これは各スラッグを通過するループに含まれます):

menuNames[slug] = (menuNames.hasOwnProperty(slug) ? menuNames[slug]++ : 1);
//this sets every value to 1

しかし、これは機能します (これは各スラッグを通過するループに含まれます):

if(menuNames.hasOwnProperty(slug)) {
    menuNames[slug]++;
} else {
    menuNames[slug] = 1;
}

score 8 · Accepted Answer

menuNames[slug]++値を増やしますが、元の値も返します。

を行っているmenuNames[slug] =ため、値はインクリメントされた後に元の値に戻されます。

それを修正するには、単に次のようにします。

menuNames[slug] = (menuNames.hasOwnProperty(slug) ? menuNames[slug]+1 : 1);

または：

(menuNames.hasOwnProperty(slug) ? menuNames[slug]++ : menuNames[slug] = 1);

score 3 · Accepted Answer

次のように機能すると思います。

menuNames[slug] = (menuNames.hasOwnProperty(slug) ? ++menuNames[slug] : 1);

score 2 · Accepted Answer

他の回答が言うように、問題はポストインクリメントにあります。

別の書き方は次のとおりです。

menuNames[slug] += (some_bool ? 1 : 0);

++バグに非常に敏感です。+=文として書いてみてください。

menuNames[slug]未定義の可能性がある場合は、次のように記述します。

menuNames[slug] = 0;
if (some_bool) {
    menuNames[slug] += 1;
}

これは (私の意見では) 初期化/カウンターループを記述する最も明確な方法です。

ワンライナーが好きならうんざりしますが、バグのないコードが好きなら、これを見て喜んでいるでしょう。

3 に答える 3