Discussion :

[SpaceFrog]

Kim je crois justement que l'objet de ce post est la copie en profondeur... donc clonage par valuer de chacun des éléments et non par référence

Quel interet de faire une "sauvegarde" si en modifiant un element (sous ou element) de l'original cela modifie la sauvegarde...
sinon le slice suffit amplement


Kern toSource n'est pas reconnu par :

Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 5.1; Trident/4.0; Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1) ; .NET CLR 1.1.4322; .NET CLR 2.0.50727; .NET CLR 3.0.04506.30; .NET CLR 3.0.04506.648; .NET CLR 3.0.4506.2152; .NET CLR 3.5.30729)

[kernelfailure]

une fonction pour tout cloner ....
...

var ar = new Array(new String("toto"), "toto");
var ar2 = deepClone(ar);

alert(ar[0] +' '+ ar[1]) //toto toto
alert(ar2[0] +' '+ ar2[1]) //[object Object] toto

[kernelfailure]

Kern toSource n'est pas reconnu par :

Ok, merci.
Lunascape, non ?

Qu'on parle bien de la même chose : Tu parles de la méthode toSource native ou de mon Object.prototype.toSource ?

[le_chomeur]

Yop Bovino :

( plusieurs div sur ta page )

Code :

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

1
2
3
4
5
6
7
 
var listElements = document.getElementsByTagName('div');					
alert(listElements instanceof Array);
var tabElem = [];
		for(var i = 0, j = listElements.length; i < j; i++){
			tabElem.push(listElements[i]);
		}

Une collection d'objet n'est pas un array , mais est typé object et a une propriété length

le mic mac xD

[SpaceFrog]

kern => oui toSource est reconnu par lunascape 5 ...

[kernelfailure]

Yop Bovino :

( plusieurs div sur ta page )

Code :

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

1
2
3
4
5
6
7
 
var listElements = document.getElementsByTagName('div');					
alert(listElements instanceof Array);
var tabElem = [];
		for(var i = 0, j = listElements.length; i < j; i++){
			tabElem.push(listElements[i]);
		}

Une collection d'objet n'est pas un array , mais est typé object et a une propriété length

le mic mac xD

Il s'agit d'un NodeList.

Imitation d'un NodeList :
http://www.developpez.net/forums/d84...v/#post4855053

[kimjoa]

suffit d'ajouter à Object.isObjectLike, si l'obet n'a pas comme constructeur String

Code :

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

1
2
3
Object.isObjectLike = function(obj){
         return obj  && typeof obj == 'object'  && obj.constructor != Array && obj.constructor != String obj.constructor != Date && obj.constructor != RegExp && !Object.isCollection(obj);
    };

bizarre quand même

[kernelfailure]

suffit d'ajouter à Object.isObjectLike, si l'obet n'a pas comme constructeur String

Code :

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

1
2
3
Object.isObjectLike = function(obj){
         return obj  && typeof obj == 'object'  && obj.constructor != Array && obj.constructor != String obj.constructor != Date && obj.constructor != RegExp && !Object.isCollection(obj);
    };

bizarre quand même

Manque '&&'
et Number

Et ça plante avec null :
Erreur*: obj is null

EDIT:
Par contre, ton isCollection m'a fait penser à quelque chose. Merci à toi (voir cloneNode) :
http://www.developpez.net/forums/d85...e/#post4861868

[le_chomeur]

ça peut paraitre bizare mais je n'ai pas vu de fonction récursive ici pour la copie :-/ ou alors j'ai raté un truc ( oui même ton poste spaffy ;-) )

[kernelfailure]

ça peut paraitre bizare mais je n'ai pas vu de fonction récursive ici pour la copie :-/ ou alors j'ai raté un truc ( oui même ton poste spaffy ;-) )

Ou c'est moi. Je n'ai vu que des solutions qui emploient la récursivité.

[kernelfailure]

Je suis tombé sur une page où il est dit que le clonage n'est pas une copie.

L'argument par de la méthode clone() de java qui n'est qu'une copie (comme le stipule la doc java) et nom pas un clonage.

La fonction présentée, a des propriétés censée évoluer tout en donnant la possibilité de les ré-écrire :

Code :

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

1
2
3
4
5
6
7
8
 
function clone(obj) {
    function Clone() { } 
    Clone.prototype = obj;
    var c = new Clone();
    c.constructor = Clone;
    return c;
}

Sur le papier ça a l'air intéressant, mais je n'ai pas encore réussi à en faire quoi que ce soit

Perso, je trouve curieux que ce qui est appelé "clone" bénéficie des altérations de sa souche.

Un autre exemple est donné ici :
https://developer.mozilla.org/en/Dif..._in_JavaScript

On a plus à faire à de l'héritage dynamique que du clonage, non ?

[danielhagnoul]

Bonsoir.

Personnellement j'utilise cette méthode :

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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
 
/*
 * Deep Copy in JavaScript
 * by Santoshkumar July 21, 2008
 * http://blog.imaginea.com/deep-copy-in-javascript/
*/
 
Array.prototype.__isArray = true;
 
var ObjectHandler = {
	getCloneOfObject: function(oldObject) {
		// public method
 
		var tempClone = {};
 
		if (typeof(oldObject) == "object"){
			for (var prop in oldObject) {
				// ajout du cas null par Daniel Hagnoul
				if (oldObject[prop] == null){
					tempClone[prop] = null;
				} else if ((typeof(oldObject[prop]) == "object") && (oldObject[prop]).__isArray){
					// for array use private method getCloneOfArray
					tempClone[prop] = this.getCloneOfArray(oldObject[prop]);
				} else if (typeof(oldObject[prop]) == "object"){
					// for object make recursive call to getCloneOfObject
					tempClone[prop] = this.getCloneOfObject(oldObject[prop]);
				} else {
					// normal (non-object type) members
					tempClone[prop] = oldObject[prop];
				}
			}
		}
 
		return tempClone;
	},
	getCloneOfArray: function(oldArray) {
		// private method (to copy array of objects) - getCloneOfObject will use this internally
 
		var oldArrayLength = oldArray.length;
		var tempClone = [];
 
		for (var arrIndex = 0; arrIndex <= oldArrayLength; arrIndex++){
			if (typeof(oldArray[arrIndex]) == "object"){
				tempClone.push(this.getCloneOfObject(oldArray[arrIndex]));
			} else {
				tempClone.push(oldArray[arrIndex]);
			}
		}
 
		return tempClone;
	}
};

Il me semble qu'elle fonctionne correctement.

[kernelfailure]

Bonsoir.
...
Il me semble qu'elle fonctionne correctement.

Bonsoir,

Il lui manque encore des choses :

Code :

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

1
2
3
4
5
6
7
8
9
var a = new Array(new Function("","return rien;"),new Boolean(true),new Array(3),true,"toto",new String("kill'em all"), null, undefined, new Date(),new RegExp("^(.*)$","g"));

alert(a.toSource());
//[(function anonymous() {return rien;}), (new Boolean(true)), [, , ,], true, "toto", (new String("kill'em all")), null, (void 0), (new Date(1261009446804)), /^(.*)$/g]

alert(ObjectHandler.getCloneOfArray(a).toSource());
// [(function anonymous() {return rien;}), {}, {__isArray:true}, true, "toto", {0:"k", 1:"i", 2:"l", 3:"l", 4:"'", 5:"e", 6:"m", 7:" ", 8:"a", 9:"l", 10:"l"}, {}, (void 0), {}, {}, (void 0)]

- Boolean non reconnu transformé en {};
- Array(3) non reconnue ;
- Problème avec String (C'est récurent);
- null transformé en {} ;
- Date transformé en {} ;
- RegExp transformé en {} ;
- Une cellule de Array ajoutée (arrIndex <= oldArrayLength) devrait être (arrIndex < oldArrayLength) ;



Une autre méthode (accentuée sur le polymorphisme) qui permet d'étendre ses objets, où il serait utile de mutualiser l'itération de Array et Object :

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
23
24
25
26
27
28
29
30
31
String.prototype.clone = 
Number.prototype.clone = 
Date.prototype.clone = 
Boolean.prototype.clone = 
RegExp.prototype.clone = function () { return new this.constructor(this); };
Function.prototype.clone = function () { return this; };
 
Array.prototype.clone = function () {
  var a = new Array(this.length);
  for (var k in this) {
    if (!this.hasOwnProperty(k)) continue;
    a[k] = this[k] === null ? null
           : (this[k] === undefined ? undefined
             : (typeof(this[k]) == 'object' || typeof(this[k]) == 'function' ? this[k].clone() : this[k])
             );
  }
  return a;
}
 
Object.prototype.clone = function () {
  if (this.cloneNode) return this.cloneNode(true);
  var a = new this.constructor(this);
  for (var k in this) {
    if (!this.hasOwnProperty(k)) continue;
    a[k] = this[k] === null ? null
           : (this[k] === undefined ? undefined
             : (typeof(this[k]) == 'object' || typeof(this[k]) == 'function' ? this[k].clone() : this[k])
             );
  }
  return a;
}

[danielhagnoul]

Bonne nuit !

Elle m'avait pourtant rendu des services.

Mais on peut améliorer le programme facilement.

Attention, il faut appeler

Code :

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

ObjectHandler.getCloneOfObject(a);

car

Code :

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

ObjectHandler.getCloneOfArray()

est destiné à un usage interne comme il est dit dans les commentaires.

[EDIT 25/12/2009 RegExp OK]

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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
/*
 * Deep Copy in JavaScript
 * by SantoshkumarJuly 21, 2008
 * http://blog.imaginea.com/deep-copy-in-javascript/
 *
 * Modifié par Daniel Hagnoul
 * Version 1.0.4 du 25/12/2009 (2009-12-25)
*/
Array.prototype.__isArray = true;
 
var ObjectHandler = {
	getCloneOfObject: function(oldObject) {
		// public method
 
		var tempClone = {};
 
		if (typeof(oldObject) == "object"){
			for (var prop in oldObject) {
				if (oldObject[prop] === null){
					tempClone[prop] = null;
				} else if (oldObject[prop] === undefined){
					tempClone[prop] = undefined;
				} else if ((typeof(oldObject[prop]) == "object") && (oldObject[prop]).__isArray){
					// for array use private method getCloneOfArray
					tempClone[prop] = this.getCloneOfArray(oldObject[prop]);
				} else if (typeof(oldObject[prop]) == "object"){
					if (oldObject[prop] instanceof Boolean){
						tempClone[prop] = new Boolean(oldObject[prop].toString());
					} else if (oldObject[prop] instanceof String){
						tempClone[prop] = new String(oldObject[prop].toString());
					} else if (oldObject[prop] instanceof Date){
						tempClone[prop] = new Date(oldObject[prop].toString());
					} else if (oldObject[prop] instanceof RegExp){
						var str = oldObject[prop].toString();
						var flags = str.match(/[gmi]+$/g);
						var pattern = str.replace(/^\//g,"").replace(/[\/gmi]+$/g,"");
 
						tempClone[prop] = new RegExp(pattern, flags[0]);
					} else {
						// for object make recursive call to getCloneOfObject
						tempClone[prop] = this.getCloneOfObject(oldObject[prop]);
					}
				} else {
					// normal (non-object type) members
					tempClone[prop] = oldObject[prop];
				}
			}
		}
 
		return tempClone;
	},
	getCloneOfArray: function(oldArray) {
		// private method (to copy array of objects) - getCloneOfObject will use this internally
 
		var oldArrayLength = oldArray.length;
		var tempClone = [];
 
		for (var arrIndex = 0; arrIndex <= oldArrayLength; arrIndex++){
			if (typeof(oldArray[arrIndex]) == "object"){
				tempClone.push(this.getCloneOfObject(oldArray[arrIndex]));
			} else {
				tempClone.push(oldArray[arrIndex]);
			}
		}
 
		return tempClone;
	}
};
 
 
	var a = new Array(
		new Function("","return rien;"),
		new Boolean(true),
		new Array(3),
		true,
		"toto",
		new String("kill'em all"),
		null,
		undefined,
		new Date(),
		new RegExp("^(.*)$","g")
	);
 
	var b = ObjectHandler.getCloneOfObject(a);