Почему в цикле всего 1 итерация?

Question

[vetsmen]

Есть код:

let items = {}; // в данном массиве 2 объекта, идет запрос в бд
	let results = {}
	results.type,
	results.special,
	results.wea,
	results.simple,
	results.coll = [];
	
	items.forEach((item) => {
		switch(item.gang) {
			case 1:
				results.type.push(item);
				break;
			case 2:
				results.special.push(item);
				break;
			case 3:
				results.wea.push(item);
				break;
			case 4:
				results.simple.push(item);
				break;
			case 5:
				results.coll.push(item);
				break;
		}
	});

Однако итерация продолжается только 1 раз, заходит в switch, там обрабатывается и все.
Если switch убрать, то все нормально.
В чем проблема?

Comments

[Егор Живагин]

items - Object ? Как вы по нему forEach-ем проходите-то?)

[vetsmen]

Егор Живагин, тьфу, массив. Проблема была в том, что если определять переменные results.type и остальные через запятую, то они не определяются, а если по отдельности, то все нормально. Только я не понимаю почему.

[Andrey Perov]

vetsmen results.type и все остальные без указания будут undefined

[Егор Живагин]

vetsmen, при записи

results.type,
  results.special,
  results.wea,
  results.simple,
  results.coll = [];

Массивом будет только results.coll , остальные свойства просто создадутся, без явного указания типа. Поэтому метода push() у них не будет

Answer 1

[Andrey Perov]

let items = [] тогда уж присвойте массив, forEach это метод массива array.prototype, а туда пусть падает коллекция объектов или что там с бд приходит

Answer 2

[Виталий]

Если вам нужно перебрать именно объект {} используйте цикл for..in, но как сказали выше, лучше сделать массив [] из объектов {} и перебирать его.