Добавил Array.prototype.count, но появилась проблема...

[khusamov]

Здравствуйте!

Добавил метод подсчета числа элементов:

Array.prototype.count = function() {
	var result = 0;
	for(var i in this) {
		if (!this.hasOwnProperty(i)) continue;
		result++;
	}
	return result;
};

Но он выдает на один больше число.

Выяснил что оказывается среди подсчитываемых индексов i попадается и добавленный метод count.

Скорректировал так:

Array.prototype.count = function() {
	var result = 0;
	for(var i in this) {
		if (!this.hasOwnProperty(i) || i == "count") continue;
		result++;
	}
	return result;
};

Вопрос, почему не попадаются прочие методы (push, pop и пр.)?

И что делать, когда я добавлю еще методы в Array.prototype?

[B@rmaley.e><e]

Вопрос, почему не попадаются прочие методы (push, pop и пр.)?

Потому что у них есть атрибут DontEnum

И что делать, когда я добавлю еще методы в Array.prototype?

Не добавлять их туда.

К слову, ES5 позволит наконец указывать атрибуты вроде DontEnum для своих свойств, так что можете подождать повсеместной имплементации нового стандарта.

UPD: как заметил Sweet, в данном случае такой проблемы не должно быть, т.к. hasOwnProperty('count') вернет false.

[Sweet]

Что за гонево?! Есть проверка hasOwnProperty, так что метод нормально считает.

[Kolyaj]

Не надо перебирать массивы циклом for-in.

[khusamov]

Что за гонево?!

Дык проверял же. Действительно на единичку больше считает.

Не надо перебирать массивы циклом for-in.

У меня попадаются массивы с разрывом индексов. На этом сайте есть статья об этом. Потому и for-in использую.

подождать повсеместной имплементации нового стандарта.

Это конечно хорошо, но хочется решение для текущей ситуации выработать)))

UPD: как заметил Sweet, в данном случае такой проблемы не должно быть, т.к. hasOwnProperty('count') вернет false.

Странно. Но у меня точно считает на единицу больше. Сейчас попробую локализовать траблу в одном файле.

[Kolyaj]

У меня попадаются массивы с разрывом индексов.

Ну и что? http://alljs.ru/articles/array/iteration.html

[khusamov]

Действительно гонево, хуже некуда...

Код:

<script type="text/javascript">
	Array.prototype.count = function() {
		var result = 0;
		for(var i in this) {
			if (!this.hasOwnProperty(i)) continue;
			result++;
		}
		return result;
	};
</script>

<script type="text/javascript">
	var arr = ["один", "два", "три"];
	alert(arr.count());
</script>

...отрабатывает верно. Выдает 3 на выходе.

Получается у меня где-то в массив попадает элемент с именем этого метода. Причем если имя метода поменять, то новое имя будет попадать... Придется искать где собака зарыта.

Спасибо за отклики!