周末赋闲在家，因为太冷了，不想出门，索性宅一天好了。但是闲着没事做总是很无聊的，正好新的一年想抓一下童鞋同学的代码质量，就随便打开了几个童鞋写的代码。于是故事就展开了。
团队大了之后，如何统一团队代码风格其实是一个蛮重要的问题，目前我们团队使用lint的方式进行了限制，这次的review可以说是初见成效，除了不少同学偷偷摸摸的通过noverify的方式提交代码以外。不过没看多久就发现了一段有趣的代码：
function deepClone(obj, res = {}) {
const _res = res;
for(let key in obj) {
if (obj[key] == obj) {
continue;
}
if (typeof obj[key] === 'object') {
if(Array.isArray(obj[key])){
_res[key] = obj[key].slice();
} else {
_res[key] = deepClone(obj[key], _res[key]);
}
} else {
_res[key] = obj[key];
}
}
return _res;
}
初看上去就会好奇，为什么不使用lodash现成的深拷贝呢？一看是个h5的项目，推测可能是为了整体包的大小做了取舍，也无可厚非吧。不过仔细看代码，乍一看好像还挺好，还细心的考虑的数组的情况，但是再仔细看得时候又觉得好像有什么地方不对，如果入参是个字符串感情你给别人返回一个空对象么。。跑了一个case发现果然有点问题：
var c = { a:1 };
var d = new Map();
d.set('a', 1);
var a = {
a: 1,
b: true,
c: ()=>{console.log(123)},
d: [1,2],
e: d,
f: c,
g: {}
}
var b = deepClone(a);
1、虽然正常的处理好像都没有什么问题，但是遇到新的数据结构如Map的时候，这种拷贝就会出问题；
2、另外，它只处理了单层循环引用的情况，多层的时候情况会更复杂；
3、而且这样递归，层级一深还会有爆栈的隐患，相当的不安全...
4、虽然它对单独处理的数组的拷贝，但如果数组的某项的值是一个对象，它这样的处理依然有问题...
所以深拷贝究竟应该怎么写呢？
本着能google不手写的原则，查了下网络，好的写法没发现几个，倒是几个误区经有的文章经常会提到且一笔略过：
1、JSON.parse(JSON.stringify(obj)) 的实现究竟算不算深拷贝？
当然算，但是这种实现有几个潜在风险：
1）它的原理是将能够JSON化的值JSON化，再重新生成一个新的JSON对象。所以它能够实现的基础是这个值是能够被JSON化的，像诸如function、map、set全是不能JSON化的，一转就没了。
2) 它还有一个风险是在处理循环引用时是会报错的。这点很多童鞋在实操的时候特别容易忽略，特别是在node端进行端端通信的时候，曾经一个报错查半天，真的是血的教训。
所以，如果是纯JSON的数据的深拷贝且不包含循环引用，是可以使用这个方法的
2、递归在js中是有风险的
常见的实现都是基于递归的，但是递归本身在js的runtime，很容易因为层级过深而导致爆栈。
而通常的方式则是通过“拍平”树级结构的对象成一个数组，来进行拷贝。
3、深拷贝的情况因业务场景的定义会有不同
有些业务场景需要保持拷贝对象中的值的引用关系不变，而有些却要改变。另外，js的数据结构发展到今天，需要在拷贝时处理的边界情况已经很多了，你需要好好考虑清楚哪些情况需要怎么处理。
其实话说回来，仔细看得话，你会发现第一种写法和jQuery中extend的方式其实是很像的，通常的情况也基本能覆盖了，不过在现在这个语境下，相比比较“安全”的实现深拷贝，还是建议使用lodash的cloneDeep(相比jQuery和underscore深拷贝大概60行左右的代码，lodash使用了近几百行代码，考虑了各种边界情况，也可谓是业界楷模了)