Skip to content

Instantly share code, notes, and snippets.

@Trindaz
Created March 29, 2012 06:46
Show Gist options
  • Star 2 You must be signed in to star a gist
  • Fork 2 You must be signed in to fork a gist
  • Save Trindaz/2234277 to your computer and use it in GitHub Desktop.
Save Trindaz/2234277 to your computer and use it in GitHub Desktop.
Javascript object deep copy
function processObjWithRef(obj, result){
if(obj==null || typeof obj != 'object'){
//nothing really to do here - you're going to lose the reference to result if you try an assignment
}
if(obj instanceof Array) {
for(var i=0; i<obj.length; i++){
result.push();
processObjWithRef(obj[i], result[i]);
}
}
if(obj instanceof Object){
for(var k in obj){
var count=0;
if(obj[k]==null || typeof obj[k] != 'object'){
result[k] = obj[k];
}else if(obj[k] instanceof Array) {
result[k] = [];
processObjWithRef(obj[k], result[k]);
}else if(obj[k] instanceof Object){
result[k] = {};
for( var attr in obj[k]){
processObjWithRef(obj[k], result[k]);
}
}
}
}
}
@scottoffen
Copy link

What is line 13 suppose to be counting?

@jbmonroe
Copy link

Probably our ability to detect a leftover variable. 😄 Should be using Array.isArray() to detect arrays, probably, and using "===" for comparison against null. The last loop would be a lot faster using function variables rather than if() tests, but one would have to use typeof instead of instanceof in order to make that work.

@mercmobily
Copy link

This fails while cloning a simple case: var b = { a: 10, b: { ba: new Date(), bb: [ 1, 2, { bba: 'pp' } ] } }; var c = {}; processObjWithRef( b, c );. Fails with TypeError: Cannot set property 'bba' of undefined

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment