Discussion :

[nims]

Bonjour,

Dans la doc, push() retourne la taille du tableau et n'est pas censé modifier le tableau "parent".

Voici le code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
var obj = {                 
    data : [{x : 100}]
};
 
var result = {                   
    data : []
}; 
 
 
for(var i = 0;i < obj.data.length;i++){ 
 
    result.data.push(obj.data[i]);
 
    console.log(obj.data[i].x+' '+result.data[i].x);
 
 
    result.data[i].x = 200;
 
    console.log(obj.data[i].x+' '+result.data[i].x);
 
}

Mon premier console.log() affiche : 100 100
Le second console.log() affiche : 200 200

Pourquoi le tableau obj.data est modifié ?

Merci d'avance pour votre aide !

[SpaceFrog]

push est supposé ajouter un element dans le tableau ... pas retourner sa taille

[nims]

Il ajoute un élément, modifie la taille du tableau et retourne la nouvelle taille (je viens de retester) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

console.log(obj.data.push('test'));

==> Retourne 2...

Mais c'est pas le problème

Mon code est inutile mais c'est pour l'exemple!

Je copie obj.data dans result.data.
Je modifie la valeur de result.data[i].
Ce qui entraine une modification sur obj.data[i] ..

:s

[nims]

Le problème vient de la structure de mes objets !

Le push() doit copier l'adresse de data[i] et non le contenu...

Si je passe tout en tableau, ca se comporte normalement...

Comment faire ca du coup :s
Et ne proposez pas de tout passer en tableau mdr

[Willpower]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

result.data.push({x:obj.data[i].x});

ou plus globalement :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
var newObj = {};
for(var k in (obj.data[i])
	newObj[k] = obj.data[i][k];
result.data.push(newObj);

[nims]

Merci pour ta réponse.

Sauf que

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

  result.data.push({x:obj.data[i].x});

m'oblige à connaitre la structure de data[i]...

Voici ma proposition:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

result.data.push(JSON.parse(JSON.stringify(obj.data[i])));