javaScript关于call的基础问题

[cocotsau]

是这样的，我想给json对象扩展一个forEach方法，于是有了下面这段代码，功能是实现了，但是最后有个问题比较诡异，我想应该是出在call上面（代码第11行），在控制台打印，把函数体给输出来了。请大神帮忙看看，谢谢！

if (!Object.prototype.forEach) {
	Object.prototype.forEach = function(fn) {
		if ({}.toString.call(this) != "[object Object]") {
			throw new TypeError();
		}
		if (typeof fn != "function") {
			throw new TypeError();
		}
		for (var key in this) {
			if(key in this){
				fn.call(this, this[key], key, this);
				//执行回调函数，并传入参数：值、索引、对象
			}
		}
	}
}

var obj = {"a": 1,"b": 2};

obj.forEach(function(a,b,c){
	console.log(a,b,c);
	/*
	1 "a" Object {a: 1, b: 2}
	2 "b" Object {a: 1, b: 2}
-----------------------------------------------------------
	//为什么会打印出函数体来？
	function (fn) {
	if ({}.toString.call(this) != "[object Object]") {
	throw new TypeError();
	}
	if (typeof fn != "function") {
	throw new TypeError();
	}
	for (var key in this) {
	if(key in… "forEach" Object {a: 1, b: 2}
	*/
})

[cocotsau]

@天际的海浪 大神请指点

[0拓0]

第9行，遍历对象属性的时候，会将添加的forEach函数也遍历出来。 所以会打印函数体

[天际的海浪]

for...in 循环除了遍历对象本身的所有可枚举属性。还会遍历从它的构造函数的 prototype 继承而来的所有可枚举属性。 你在Object.prototype对象上自定义了forEach属性，这个自定义的forEach属性是可枚举的，所有 Object 的实例对象用for...in 循环都会枚举出这个属性。这等于是对js系统的一种污染。 如果你只要考虑对象本身的属性，而不是它的原型，那么使用 hasOwnProperty() 来确定某属性是否是对象本身的属性

		for (var key in this) {
			if(this.hasOwnProperty(key)){
				fn.call(this, this[key], key, this);
				//执行回调函数，并传入参数：值、索引、对象
			}
		}

[cocotsau]

引用 3 楼 jslang 的回复:

for...in 循环除了遍历对象本身的所有可枚举属性。还会遍历从它的构造函数的 prototype 继承而来的所有可枚举属性。 你在Object.prototype对象上自定义了forEach属性，这个自定义的forEach属性是可枚举的，所有 Object 的实例对象用for...in 循环都会枚举出这个属性。这等于是对js系统的一种污染。 如果你只要考虑对象本身的属性，而不是它的原型，那么使用 hasOwnProperty() 来确定某属性是否是对象本身的属性

		for (var key in this) {
			if(this.hasOwnProperty(key)){
				fn.call(this, this[key], key, this);
				//执行回调函数，并传入参数：值、索引、对象
			}
		}

非常感谢，明白了。改成这个hasOwnProperty()就好使了，可以支持到IE6

[天际的海浪]

更好的办法是把这个自定义的forEach属性设置为不可枚举的。不过要IE9才支持 Object.defineProperty(Object.prototype, "forEach", {enumerable:false});

[cocotsau]

引用 5楼天际的海浪 的回复:

更好的办法是把这个自定义的forEach属性设置为不可枚举的。不过要IE9才支持 Object.defineProperty(Object.prototype, "forEach", {enumerable:false});

嗯嗯好的，我再去试试，这应该是ECMA5的新方法吧？