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

问题原本的描述不太清晰，我重新叙述一下 ：
我半年前自己写了玩物—— zry656565/JustJS · GitHub ，是直接在prototype上定义方法的，只是纯粹想扩展js内置对象(Object/Array/String)，补充一些好用的函数，想同时支持前端和Node，所以无关DOM对象。

但我也知道 直接在prototype上定义方法是有问题的 。我自己总结出来以下几个问题：

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

1. 有可能出现代码向上不兼容的情况，比如定义了一个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好吧。

补充顾老师的一下
Prototype最著名的失误在于 当时实现document.getElementsByClassName，直接使用了Document.prototype并返回了一个数组的实例。但是DOM 后来原生方法里对这个方法返回的是一个Nodelist实例，结果就是之前用了Prototype库的代码，运行在新浏览器的时候，用的是DOM原生方法，返回的对象是无法继承数组的任何方法的，所以就会报错。