Что за аномалии с массивами в JS?

Question

[Q Q]

Привет.

Уже пол дня ломаю голову с такой аномалией в JS.

Есть исходный массив:

arr[arr.length] = {
	"id": 22,
  "category": 34,
	"name": "Jel",
  "movie": "Avatar"
};
arr[arr.length] = {
	"id": 34,
  "category": 12,
	"name": "Sali",
  "movie": "Ted 2"
};
arr[arr.length] = {
	"id": 33,
  "category": 34,
	"name": "Chesko",
  "movie": "Avatar"
};
arr[arr.length] = {
	"id": 22,
  "category": 34,
	"name": "Donny",
  "movie": "Avatar"
};

У него уникальные значения, это id + movie.
Нужно из него сделать вот такой:

[{
    "id": 22,
    "movie": "Avatar",
    "actors":  [{
        "category": 34,
        "name": "Jet"
    }, {
         "category": 34,
        "name": "Donny"
    }]
},
{
    "id": 34,
    "movie": "Ted 2",
    "actors":  [{
        "category": 12,
        "name": "Sali"
    }]
},
{
    "id": 33,
    "movie": "Avatar",
    "actors":  [{
        "category": 34,
        "name": "Chesko"
    }]
}]

Написал код:

var arr = [],
		movies = [];
    
arr[arr.length] = {
	"id": 22,
  "category": 34,
	"name": "Jel",
  "movie": "Avatar"
};
arr[arr.length] = {
	"id": 34,
  "category": 12,
	"name": "Sali",
  "movie": "Ted 2"
};
arr[arr.length] = {
	"id": 33,
  "category": 34,
	"name": "Chesko",
  "movie": "Avatar"
};
arr[arr.length] = {
	"id": 22,
  "category": 34,
	"name": "Donny",
  "movie": "Avatar"
};

for (var i = 0; i < arr.length; i++) {

	var id = arr[i].id + '_' + arr[i].movie;

	movies[id] = {
  	"id": arr[i].id,
    "movie": arr[i].movie
  }
  
  if (!movies[id].actors) movies[id].actors = []
  
  movies[id].actors[movies[id].actors.length] = {
  	"category": arr[i].category,
    "name": arr[i].name,
  }

}

console.log(movies);

https://jsfiddle.net/52ubh0tg/

Но он как-то аномально работает, он не увеличивает movies[id].actors[movies[id].actors.length], и все время считает что movies[id].actors пустой массив. В итоге у Avatar с id 22 только один актер, а не два.

Answer 1

[HoHsi]

1) Не добавляйте элемент в массив через index, используйте #push
2) Проверяйте на существование переменной, вы каждый раз затираете movies[id];
3) Если вы используете конструкцию for (var i = 0; i < arr.length; i++) { }, то не вызывайте каждый раз length. Делайте так:

for (var i = 0, len = arr.length; i < len; i++) { }

, а еще лучше for (var item in arr) { }

var arr = [],
    movies = {};
    
arr.push({
  "id": 22,
  "category": 34,
  "name": "Jel",
  "movie": "Avatar"
});
arr.push({
  "id": 34,
  "category": 12,
  "name": "Sali",
  "movie": "Ted 2"
});
arr.push({
  "id": 33,
  "category": 34,
  "name": "Chesko",
  "movie": "Avatar"
});
arr.push({
  "id": 22,
  "category": 34,
  "name": "Donny",
  "movie": "Avatar"
});

movies = arr.reduce(function(res, item){
  var id = item.id + "_" + item.movie;
  
  res[ id ] = res[ id ] || {
    "id":     item.id,
    "movie":  item.movie,
  };

  res[id].actors = res[id].actors || [];

  res[id].actors.push({
    "category": item.category,
    "name":     item.name,
  });
  
  return res;
}, movies);

console.log(movies);

Comments

[Q Q]

Век живи - век учись. Спасибо!

[timfcsm]

вот только фор-ином по массиву ходить не комильфо

[HoHsi]

Тимур: почему?

[timfcsm]

HoHsi: ну хотя бы поэтому:
var arr = [1,2,3]
arr.tro = 'lolo'
for (var i = 0, len = arr.length; i < len; i++) console.log('%s = %s', i, arr[i]) // => 0 = 1,1 = 2, 2 = 3
for (key in arr) console.log('%s = %s', key, arr[key]) // => 0 = 1,1 = 2,2 = 3, tro = lolo
да и вообще, есть же forEach для массивов)

[HoHsi]

Тимур: И часто вы добавляете атрибуты массивам не через прототип?

[timfcsm]

HoHsi: ну, во-первых, добавлять свои выдуманные атрибуты в прототип встроенных объектов языка ещё большее зло. во-вторых, имелось ввиду, что вдруг кому-то понадобится хранить какую-то информацию для конкретного массива в его свойстве, не обязательно этим кем-то буду я... я если честно, вообще не помню, чтоб я так делал :) суть в том, что такой вариант есть, а также есть специальный метод для обхода в цикле массива

[HoHsi]

Тимур: недавно бенчил ES5, так у него все медленее чем lodash аналог, на 20% - 80%. Если уже начать придираться то самый Ъ это обходить массив через while.

Answer 2

[Stalker_RED]

Так?

var newArr = arr.map(function(a){
  a.actors = [{
    category: a.category,
    name: a.name
  }];
  delete a.category;
  delete a.name;
  return a;
})

Демо: https://jsfiddle.net/upbmosj0/