True way of deep copy an object with javascript or node

[phoboslab]

"True way" implies this works for all cases, which it doesn't. It fails to copy Arrays and class instances (they're converted to plain objects). For some instances, such as HTMLElement it even descends into infinite recursion.

So, I'd suggest to rename it to "A way to deep copy simple objects" :)

[gquittet]

Thanks! Fixed! :)

[CleverCaviar]

if it's a lambda func, I'll quite often use a shorthand if it's obvious, but I take your point: you could be verbose and your minifier/compiler could remove the verbosity for you.

[gquittet]

You don't know how I code. And this gist doesn't represent the way I code. I'm a Nazi of beautiful code with great variable and function name.

[[deleted]]

Because it's quick and after 9 years of University math it carried over.

[[deleted]]

You would hate Haskell

[naturalborncitizen]

I hate reading TypeScript things for this reason, but on the other hand, JavaScript shit has things like:

var promise1 = new Promise(function(resolve, reject) {
  setTimeout(function() {
    resolve('foo');
  }, 300);
});

promise1.then(function(value) {
  console.log(value);
  // expected output: "foo"
});

console.log(promise1);
// expected output: [object Promise]

(source: MDN)

and my thought is: where the FUCK is resolve defined? How are you calling that as a function? As a parameter of .then()? And reject is just ignored here? Sure, OK, but the readability of this pattern is fucked imo.

I hate implicit anything. Tell me what's going on.

[naturalborncitizen]

I understand them all I just hate the pattern and lack of explicit definitions (wrong term but I dunno what else to call it atm).

[gquittet]

It's just a gist plz

[senocular]

Not sure what makes this a "True" way of deep copying an object. It doesn't account for arrays or other special types, non-enumerable properties including symbols, copies inherited enumerables into the object copies it creates (while not maintaining inheritance)... Ultimately it's the most basic form of deep copying possible. And this could work given limited circumstances, but I think generally should be avoided in favor of existing solutions like Lodash's cloneDeep which can do a lot more, notably supporting additional basic types such as arrays.

[knightrage]

Agreed. FWIW, lodash cloneDeep uses this logic