Question

为什么不要直接在Object.prototype上定义方法

Answer 1

总结出来以下几个问题：

容易造成全局污染，和其他库冲突
出了Bug不太好定位问题

有可能出现代码向上不兼容的情况，比如定义了一个Object.prototype.clone。万一ES7、ES8也定义了这个函数，那旧代码不就会出现问题了嘛！

虽然我一般会避免在
Object.prototype 或者其他类似的上添加自定义的方法, 但其实我也觉得是有点 over-defensive 的. 一般情况下,
优质的类库应该都有注意到相关问题, 所以其实即使自定义也不会有太大的风险. for in 的时候记得 hasOwnProperty 就可以了.

不过这个地方我通常喜欢 var hop = Object.prototype.hasOwnProperty; 再用 hop.call(object, propertyName); 避免正好有个叫 "hasOwnProperty" 的 key.

NodeList 相关的问题值得注意下, 不要犯类似的错误.

如果你要做的是让老浏览器兼容新方法, 那么判断一下是否已存在即可, 如果是添加自定义的方法怕与 JS 往后的版本重名, 那不判断是否存在直接加上 (覆盖) 即可. 不用担心, 即使后来真的出现了重名的方法, 也会被你的覆盖掉.

不过刚刚看到题主要造轮子, 如果这个轮子是给其他人用的话, 那么最好要提醒其他人你通过这种方式添加了相关方法. 因为有的人可能没有 hop 的习惯.

哦还有就是如果不用考虑老浏览器的话,,, defineProperty/ies 也好. (enumerable 默认 false.)

for (var k in xxx) {
// 出来了你加在 prototype 上的东西
}

如果你不需要兼容old ie； 那么覆盖 toJSON 也比新定义一个clone好吧。