NN
я переписал на циклы, потому что у них вроде производительность выше
Ухахатбл
Size: a a a
2020 December 02
Д
изначально было так вообще
Д
No Name
Ухахатбл
ну разницы никакой по итогу, да)
SS
я переписал на циклы, потому что у них вроде производительность выше
у тебя там аглоритм квадратной сложности, а ты доли процента решил съэкономить 🤣
E
было хотя бы более читаемо)
SS
и какой нафиг push! там return надо
NN
ну разницы никакой по итогу, да)
Во первых читаемость, во вторых, как написал товарищ выше, у тебч время выполнения О²
Д
Sergiy Shatunov
и какой нафиг push! там return надо
unsortedList — массив
SS
unsortedList — массив
вот и присвой этому полю свежесформированный массив
E
это все экономия на спичках, надо избавляться от вложенных циклов в первую очередь.
по коду видно, что можно свести к одному проходу по
но оно изначально криво написано, конечно
не удивлюсь, если и в других местах тоже адок.
по коду видно, что можно свести к одному проходу по
this.tariffsно оно изначально криво написано, конечно
не удивлюсь, если и в других местах тоже адок.
SS
а если надо массив дополнить, то достаточно одного вызова push со всеми новыми элементами массива
ОМ
Подскажите плиз, почему не срабатывает addEventListener, вот код
ОМ
let elemBlock = document.querySelector('.side-category-accordion a'),
elemClick = document.querySelector('.side-category-accordion a i')
elemBlock.classList.add('active');
elemClick.textContent = '-';
elemBlock.addEventListener('click', (e) => {
console.log('click');});
elemClick = document.querySelector('.side-category-accordion a i')
elemBlock.classList.add('active');
elemClick.textContent = '-';
elemBlock.addEventListener('click', (e) => {
console.log('click');});
ОМ
при том, что перед кликом, срабатывает все
ОМ
очень странно, и если я имитирую клик, то срабатывает, а на физ. клик — нет(
ab
очень странно, и если я имитирую клик, то срабатывает, а на физ. клик — нет(
Имитируешь через elemBlock.click() ?
AP
let elemBlock = document.querySelector('.side-category-accordion a'),
elemClick = document.querySelector('.side-category-accordion a i')
elemBlock.classList.add('active');
elemClick.textContent = '-';
elemBlock.addEventListener('click', (e) => {
console.log('click');});
elemClick = document.querySelector('.side-category-accordion a i')
elemBlock.classList.add('active');
elemClick.textContent = '-';
elemBlock.addEventListener('click', (e) => {
console.log('click');});
у тебя только одна ссылка?
ab
Может быть элемент какой перекрывает этот