С циклом - два, три активных таких страницы одновременно и у вас за пару секунд будет дохренише запросов в базу. С неправильно настроеной базой это будет огромной проблемой. Но даже с правильно настроенной, 10-20 юзеров и нагрузка охренеть какая. Не надо так делать.
Если интересно, маленький фидбек по коду ) Мне кажется, вам может быть полезно.
Если вы вызываете метод только для проверки условия - вы можете спокойно его вызвать прям в самом условии, как тут кто-то отвечал уже. Это нормальная практика.
Вы можете смело убрать === true из условия, не изменится вообще ничего.
Очень странно видеть переменную sliderInit внутри класса вместо this
Вы привязываете класс к определённому экземпляру класса... зачем? По идее, классы должны быть изолированы от остального кода. А так вы переименуете переменную - придётся лезть в код класса, захотите ещё слайдер - придётся лезть в код класса, ...
А разве функцию checkMinMax нельзя привести к одной строчке, вместо цикла и сравнений: return [this._slider.length - 1, 0];
Номер последнего слайда всегда будет равен slider.length - 1 , номер первого слайда всегда будет 0
Опять же, ради масштабируемости есть смысл передавать переменную aria при вызове слайдера
Это оч странно. Ну, вы можете поставить в старый проект старую версию gulp-а