.lengthはJavaScript配列の適切な呼び出しですか？

Question

なんらかの理由で、配列の長さを取得できません（配列が不完全な場合は未定義です）。

define(function() { 
    'use strict'; 

    var self = {}, 
    events = {}; 

    self.publish = function publish(eventName, data) { 
    var subscribers, x, length; 

    if (events[eventName]) { 
     return false; 
    } 

    subscribers = events[eventName]; 

    for (x = 0, length = subscribers.length || 0; x < length; x += 1) { 
     subscribers[x](data); 
    } 

    return true; 
    }; 

    self.subscribe = function subscribe(eventName, func) { 
    if (!events[eventName]) { 
     events[eventName] = []; 
    } 

    events[eventName].push(func); 
    }; 
    return self; 
}); 

JSLintは、「期待している」、代わりに鋸 '、'。 Jasmineは "TypeError：プロパティ '長さ'が未定義です。 '

なぜ長さは0に設定されていませんか？私はここでの操作の構文を誤解していますか？

Answer 1

私はこの問題は、この行であると思う：

for (x = 0, subscribers.length || 0; x < length; x += 1) { 

あなたは、変数「長さ」を定義し、それにデータを割り当て/初期化されません。具体的には

：

subscribers.length || 0 

これを行うことになっていただきましたか？

x < length 

x < length or x < subscribers.length? 

Answer 2

ドンBhrayanシンが

あなたのコードは、あなたの前のコードは、あなたがpublish機能に否定を欠けているこの

// x = 0,   0   ; x < undefined; x += 1 
for (x = 0, subscribers.length || 0; x < length; x += 1)  { 
    subscribers[x](data); 
} 

Answer 3

のように実行します

for (var x = 0, length = subscribers.length || 0; x < length; x += 1) { 
    subscribers[x](data); 
} 

する必要があります言ったように：

self.publish = function publish(eventName, data) { 
    var subscribers, x, length; 

    if (events[eventName]) { // <= Here you exit if there are subscribers, and continue if there arent 
     return false; 
    } 

    subscribers = events[eventName]; // So subscribers is guaranteed to be undefined 

    // can't get length (or any property) of undefined 
    for (x = 0, length = subscribers.length || 0; x < length; x += 1) { 
... 

これにより、テストが失敗します。これは、移動するための方法である：その空の場合

self.publish = function publish(eventName, data) { 
    // you can use const and let 
    // you are storing subscribers in a variable anyway, so why not doing it 
    // before the check? 
    const subscribers = events[eventName]; 

    if (!subscribers) { 
     return false; 
    } 

    // you don't need to cache length in a variable, iirc it only improves performance slightly on IE browsers 
    // neither need you `|| 0` 
    // there other ways iterating through an array, see for..of and Array.prototype.forEach 
    for (let x = 0; x < subscribers.length; x++) { 
     subscribers[x](data); 
    } 

    return true; 
}; 

it is undefined if it's unpopulated, right?

いいえ、配列の長さはゼロです。オブジェクトの不足しているプロパティーを読むときは未定義ですが、配列の長さは常に数値です。

Answer 4

私が誤字を修正したとき、コードは正常に機能しました。

if (events[eventName]) { 

は、その補正後の意図したとおりにすべてが働いていた

if (!events[eventName]) { 

ことになっています。 私のデバッグの問題は常に馬鹿げたものです。