Почему после вызова функции меняется объект?

Question

[Николай Маринкин]

Есть вот такой, упрощенный код:

var Game = function (players) {
	this.game = {
		players: []
	};

	console.log(this.getStateWithMe(players[0]).players); //тут все нормально
	console.log(this.getStateWithMe(players[1]).players); //а тут this.game уже изменен
}

Game.prototype = {
	getStateWithMe: function (player) {
		//функция должна возвращать модифицированную копию this.game
		var g = {};
		for (prop in this.game) if (this.game.hasOwnProperty(prop)) {
			g[prop] = this.game[prop];
		}

		g.players.push(player);

		return g;
	}
}

new Game(['a', 'b']);

Я пытаюсь реализовать функцию получения дочернего объекта, в зависимости от параметра. Но проблема в том, что прототипная функция getStateWithMe() изменяет сам объект.
Как я понял это из-за того, что var создает ссылку на объект, а не его копию.
Как это поправить, оставив функцию в прототипе?

Comments

[Евгений Петров]

g[prop] = this.game[prop];
Вы записали в свойство объекта значение ссылки на массив. Чего же удивляться, что он изменяется?)

Answer 1

[Константин Китманов]

Нет, это потому что this.game.players является массивом, и, соответственно, ссылочным типом. То есть g.players и this.game.players ссылается на один и тот же объект.

Вам нужна функция, которая делает deep clone. Самый лаконичный способ, который я знаю — JSON.parse(JSON.stringify(this.game)), если у вас там нет хитрых данных типа дат и регекспов, то все должно быть нормально.

Answer 2

[Виталий]

var g = jQuery.extend(new Game([]), this.game);

Comments

[Николай Маринкин]

jQuery нету...

[Виталий]

Ну тогда просто создавай объект игры заново и копируй туда руками все свойства

[Виталий]

можно здесь почитать еще: stackoverflow.com/questions/728360/most-elegant-wa...

[Николай Маринкин]

Виталий Дятлов: не сработало...

[Евгений Петров]

А как это отменит то, что в свойствах разных объектов хранится значение ссылки на один и тот массив?)

[Константин Китманов]

Евгений Петров: +1.

Answer 3

[keslo]

var Game = function (players) {
  this.game = {
    players: []
  };

  console.log(this.getStateWithMe(players[0]).players); // тут все нормально
  console.log(this.getStateWithMe(players[1]).players); // и тут вроде тоже
}

Game.prototype = {
  getStateWithMe: function (player) {

    var g = {};
    g.players = this.game.players.slice(0); // копируем массив-значение из this.game.players
    g.players.push(player);

    return g;
  }
}

new Game(['a', 'b']);