2016-10-30 23 views
0

私は現在、forループをforループと置き換えようとしています。私はいつもundefinedを得る。これはjavascriptオブジェクトでも可能ですか?for for for regular forループをオブジェクトに置き換えます。

以上のサンプル・オブジェクトのIループ:

var el = { 
    1: {type: "fish", commonName: "clownfish", scientificName: "sdasd", gender: "m", price: 1.99}, 
    2: {type: "fish", commonName: "dragonfish", scientificName: "dada", gender: "f", price: 2.99} 
}; 

は、アプローチでのための私の作業:

for (var element in el) { 
    if (el[element].type === type && el.hasOwnProperty(element)) { 
     elementNum++; 
    } 
} 

いつも私にCannot read property 'type' of undefinedを取得するループアプローチのためのシンプル:

for(var i = 0, x = Object.keys(el).length; i < x; i++) { 
    if (el[i].type === type && el.hasOwnProperty(i)) { 
     elementNum++; 
    } 
} 
+1

@Matias正しい答えを持っていますが、あなたは、配列を使用していない理由を私は思ったんだけど?いくつかのインデックスはスキップされるのでしょうか?また、 'el [i] .type === type && el.hasOwnProperty(i))'の順序を 'el [i]'が存在しない場合に '.type'をチェックしないように(JavaScriptが 'false 'のものを見つけたら' if AND'ステートメントを調べるのを止めるので – Howzieky

答えて

6

あなたは」 0からループを開始してください。一方、オブジェクト全体の最初のキーs 1

var el = { 
 
    1: { 
 
    type: "fish", 
 
    commonName: "clownfish", 
 
    scientificName: "sdasd", 
 
    gender: "m", 
 
    price: 1.99 
 
    }, 
 
    2: { 
 
    type: "fish", 
 
    commonName: "dragonfish", 
 
    scientificName: "dada", 
 
    gender: "f", 
 
    price: 2.99 
 
    } 
 
}; 
 

 
var keyLength = Object.keys(el).length; 
 

 
// I've simplified the for loop 
 
for (var i = 1; i <= keyLength; i++) { 
 
    console.log(el[i].commonName); 
 
}

+0

答えが短くなることはありませんよね! –

+0

私はこれを見ることができませんでした..ありがとうございました – Sebsemillia

+0

@Sebsemillia (JSHint、JSLint ...)ll –

0

私は、オブジェクトのキーの上に直接反復することをお勧め。

function getCount(type) { 
 
    return Object.keys(el).reduce(function (r, k) { 
 
     return r + +(el[k].type === type); 
 
    }, 0); 
 
} 
 

 
var el = { 1: { type: "fish", commonName: "clownfish", scientificName: "sdasd", gender: "m", price: 1.99 }, 2: { type: "fish", commonName: "dragonfish", scientificName: "dada", gender: "f", price: 2.99 } }; 
 

 
console.log(getCount('fish'));

+0

はい、私はこのアプローチも知っています。私は単純なforループでそれをやろうと思っていました..;)あなたの答えをありがとう! – Sebsemillia

+0

オブジェクトがインデックスのようなキーの固定表記法の場合にのみ機能します。 –