push() : modifie le tableau d'origine ?! [Résolu]

[nims]

Bonjour,

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

Voici le code :

Code :

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 :

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 :

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

ou plus globalement :

Code :

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 :

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

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

Voici ma proposition:

Code :

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

[Willpower]

Je pense que ma seconde solution est meilleure alors, bien que je ne connaisse pas le code natif de JSON.stringify, je suposse qu'il fait une boucle et que pour chaque élément il l'ajoute à un string.

Ensuite JSON.parse fait probablement l'inverse, tu as donc 2 boucle plus des convertion en string, alors que ma solution effectue une simple boucle.

Après évidement si tu as une structure encore plus complexe avec des objets contenus dans des objets, tu peux faire une fonction de parcours/copie qui vérifiera pour chaque élément s'il s'agit d'un objet et dans ce cas faire un appel récursif.

[nims]

Oui, tu a raison.

J'utilise ceci pour un plugin jQuery.
Du coup c'est pas l'aspect principale de la chose mais ça pourra faire l'objet d'une amélioration...

Merci pour ton aide