>>238704895 (OP) Сетевой инжинегр в телекоме, 24 лвл. Зп 85к, с этого года решил жить поэкономнее поэтому по 35к каждый месяц буду скидывать на черный день
>>238706355 Что за блять [0] ? А если его нет? Скрипт упадёт и потянет за собой все последющие скрипты (без замыкания). Если обращаешься к свойству элемента, добавь проверку что она существует. >> $hui == "brightness(0.5)" Выглядит как вырвиглазный пиздец. Но уж коли так то, можно в константу строку выделить. А вообще яб доставал число из строки и потом обратно число в строку подключал. И для этого отдельные функции пишешь, потом могут пригодиться (brightness даже как параметр функции можно запихнуть, т.к. там всякие scale(0.3) и прочее есть) >> for (i=0, i<5 ... ) Что блять за 5 ? А если я удалю одну картинку, то скрипт упадёт. Если добавлю, последняя не будет работать. Делай сразу универсально и гибко
>>238714324 >Если обращаешься к свойству элемента, добавь проверку что она существует. и этого не понял Как свойство у элемента не будет существовать? Я же конкретно имейдж беру, у него стиль и фильтр.
>>238714400 Ты жестко связываешь js и html Представь что ты в какой-то момент класс 'dark' поменял на 'janna-dark'. И теперь ответь что станет с твоим скриптом? Если всё еще не понял, открой прямо сейчас консоль и введи document.querySelectorAll('body.mavrody')[0].filter
Или ты хочешь добавить две галереи на одну страницу. И при клике на второй - изменяется первая.
Конкретно здесь - это работает, но это не правильно, в будущем. И придётся переучиваться, поэтому учись делать правильно сразу.
>>238714238 Я могу и хочу удалять html. Html это язык разметки. Если твой скрипт предполагает наличие какого-то элемента, то пусть проверит что он существует и если его нет, выдаст адекватную ошибку.
Обычно html билдится, а потом расскидывается куда-то в файлики бекерами, и хорошо если в темплейтах, а css и js лежат в стороне. Не нужно намертво связывать html и js (по крайней мере там, где это возможно)
К чему я вообще. Всегда легче сделать так, чтобы работало, но куда важнее сделать так, чтобы не сломалось и было легче использовать. По этой же причине большинство галерей требуют максимально простой html, вроде <ul class="ewr"> <li><img src="cp.jpg"></li> <li><img src="cp.jpg"></li> </ul>
а остальное генерят через js. Так мы практически всегда можем гарантировать что элемент основной картинки существует, доп кнопки и прочее тоже добавились. Ясен хрен это всё можно через консоль сломать или из другого скрипта, но это уже должна быть целенаправленная диверсия (или тотальное распиздяйство), а не случайность.