Форум

danik.js · #41 (**permalink**) 17.11.2014, 23:12

Сообщение от Pechen

примеры выше не работают в старых браузерах. я стараюсь избегать таких методов

Чет не похоже. indexOf не работает даже в IE8. Тогда как querySelector, вместо которого у тебя велосипед - работает в нем.

danik.js · #42 (**permalink**) 17.11.2014, 23:15

Сообщение от рони

что не так?

Да даже код Pechen и тот лучше. Может он не правильный, но когда я смотрю на него - без напряга понимаю что он делает.

У тебя же раздутые строки и неявный if. Еще непонятно что на 18 строке.

danik.js · #43 (**permalink**) 17.11.2014, 23:17

Неявный if еще можно использовать в примерно таких случаях:

hasAttr || addAttr();

dirtyFlag && updateSmth();

Тут это наоборот упрощает код и его чтение.

Pechen · #44 (**permalink**) 18.11.2014, 00:43

Сообщение от danik.js

indexOf не работает даже в IE8. Тогда как querySelector, вместо которого у тебя велосипед - работает в нем.

ага не заметил -- смотрел у стринга)) так лучше? как обойтись без call, чтоб работало как с indexOf? крутил по-всякому)

<!DOCTYPE HTML>
<html>
  <head></head>
  <body>
    <div class="a">Div 1</div>
    <div class="a">Div 2</div>
    <div class="a">Div 3</div>
    <div class="a">Div 4</div>
    <div class="a">Div 5</div>

    <script>
	var elem = document.getElementsByTagName('div');
	var elements = [];
	for(var i=0; i<elem.length;i++){
		var option = elem[i].className;
		if(option=='a'){
			elements.push(elem[i]);
		}
	}
	for(var j=0;j<elements.length;j++){
		elements[j].onclick  = function () {
			alert(myIndexOf.call(elements, this));
			}
	}
	function myIndexOf(elemento){
		for(var i=0;i<this.length;i++){
			if(elemento == this[i])
				return i;
		}
	}
    </script>

  </body>
</html>

danik.js · #45 (**permalink**) 18.11.2014, 05:35

Pechen, половина кода лишняя. Используй querySelectorAll.
Если в IE нет indexOf, то проще обойтись без него. Используй замыкания либо пиши индекс в data-атрибут.
Если же хочешь indexOf, то к чему пляски с call? Делай вызов напрямую. Можешь кстати на прототипе объявить, только делай тогда нормальную реализацию (возьми из ES5-shim)

kostyanet · #46 (**permalink**) 18.11.2014, 06:28

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

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

kostyanet · #47 (**permalink**) 18.11.2014, 06:32

Сообщение от danik.js

либо пиши индекс в data-атрибут.

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

Например типовая задача произвольного доступа.

щелчок на N

если N+1 < L то следующий N+1 иначе следующий 0
если N-1 > 0 то предыдущий N-1 иначе предыдущий L-1

Где взять N? Разумеется из data-index

Pechen · #48 (**permalink**) 18.11.2014, 09:45

Сообщение от danik.js

Если же хочешь indexOf

не хочу) у меня ж своя функция там была myIndexOf и я спрашивал как ее запустить методом моего массива без явного call)
с замыканиями мне еще предстоит познакомиться, а на счет дата атрибутов тоже только узнал и мне показалось такое решение менее изящным чем добавление по свойству каждому элементу.

<!DOCTYPE HTML>
<html>
  <head></head>
  <body>
    <div class="a">Div 1</div>
    <div class="a">Div 2</div>
    <div class="a">Div 3</div>
    <div class="a">Div 4</div>
    <div class="a">Div 5</div>

    <script>
	var elements = document.querySelectorAll('div.a');
	for(var j=0;j<elements.length;j++){
        elements[j].index = j;
		elements[j].onclick  = function () {
			alert(this.index );
		}
	}
    </script>

  </body>
</html>

danik.js · #49 (**permalink**) 18.11.2014, 09:51

Сообщение от Pechen

я спрашивал как ее запустить методом моего массива без явного call

Я ж говорю - объяви на прототипе массива. он находится в Array.prototype.
Но за это могут и п*зды дать.

Форум

Справочник

Discord чат

Курсы javascript