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])));

[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