爲什麼不是這個對象循環工作？

Question

我想遍歷這個對象，並將'loc'值添加到數組，如果他們的邊= 2。我做錯了什麼？

2025是房間的對象，整個房間都是房間。

//Object 
    { 
     "2025": { 
     "tom": { 
      "side": 1, 
      "loc": 111 
     }, 
     "billy": { 
      "side": 2, 
      "loc": 222 
     }, 
     "joe": { 
      "side": 2, 
      "loc": 333 
     }, 
     "bob": { 
      "side": 1, 
      "loc": 444 
     } 
     } 
    } 

//Code 
    var side2 = []; 
    for (var key in rooms[room]) { 
     if (rooms[room].hasOwnProperty(key)) { 
      var obj = rooms[room][key]; 
      for (var prop in obj) { 
      if (obj.hasOwnProperty(prop)) { 

        if(prop == 'loc') { 
         if(obj[prop] == 2) { 
          side2.push(key); 
         } 

       } 
      } 
      } 
     } 
    } 
    console.log(side2); 

Answer 1

你要推時side值2，所以要檢查side而不是loc。然後，你可以簡單地推obj.loc

... 
if (obj.hasOwnProperty(prop)) { 
    if(prop == 'side') { 
     if(obj[prop] == 2) { 
      side2.push(obj.loc); 
     } 
    } 
} 
... 

Fiddle Example

---

這就是說，你可以縮短這個代碼不少，刪除不需要的循環和工作，你可以縮短所有的代碼簡單：

for (var key in rooms[room]) { 
    var item = rooms[room][key]; 
    if(item.side == 2) 
     side2.push(item.loc) 
} 

Fiddle Example

Answer 2

從你的說法，你要loc財產的價值推到陣列side2如果side屬性的值爲2

但在你的代碼

if(prop == 'loc') { 
    if(obj[prop] == 2) { 
     side2.push(key); 
    } 
} 

你是比較loc屬性的值爲2，而不是side屬性的值。你可能需要像

if(prop == 'side') { 
    if(obj[prop] == 2) { 
     side2.push(obj['loc']); 
    } 
} 

Answer 3

正如在評論和其他答案中所述，您正在尋找 2不存在的財產。所以眼前的問題可以通過用side代替loc來解決（假設這就是你想要的）。

但是你的代碼可以簡化。在頂層循環很好。然而，你的代碼的整個循環嵌套部分：

for (var prop in obj) { 
    if (obj.hasOwnProperty(prop)) { 
    if(prop == 'loc') { 
     if(obj[prop] == 2) { 
     side2.push(key); 
     } 
    } 
    } 
} 

可以用

if (obj.side == 2) side2.push(key); 

換句話說取代你，你也不需要遍歷一個對象的屬性來找到特定的一個（在這種情況下爲side）。您只需使用obj.side就可以訪問它。

你也可以設想這個問題，因爲想要過濾的房間列表下降到那些人用loc 2，在這種情況下，你可以寫：

Object.keys(rooms) . filter(hasPeopleWithLoc(2)) 

其中

function hasPeopleWithLoc(loc) { 
    return function(roomKey) { 
    var room = rooms[roomKey]; 
    return Object.keys(room) . some(function(personKey) { 
     return room[personKey].loc === loc; 
    }); 
    }; 
} 

請注意，如果該房間中的任何人的loc爲2，則此代碼只會爲結果中的房間輸入一個條目。您的原始代碼行爲稍有不同;它在房間中輸入房間的結果爲每個人在該房間中的所需loc。