На сколько мой подход правильный?

Question

[Даниил Сухих]

У меня есть файл со всеми функциями которые мне нужны, в этом файле я их и вызываю. Но на некоторых страницах сайта некоторые функции мне не нужно вызывать, по этому я решил вызывать каждую фунцию на определённом url. На сколько это правильн?

var url = document.URL;
url = url.split('/');
url = url[url.length - 1];
url = url.replace(/\?.*/i, '');
url = url.replace(/#.*/i, '');

switch(url) {
	case '':
	fullPage();
	mobileMenu();
	menuScroll();
	ymaps.ready(init);
	infoCookie();
	headerScroll();
	promoPopup();
	menuAutoScroll();
	formAjax();
	popUp('.call-form__wrapper', '.main-section__slogan__button', '.call-form__close');
	popUp('.promo-popup__wrapper', '.promo-info', '.promo-popup__close');
	break;

	case 'shop':
	mobileMenu();
	menuScroll();
	infoCookie();
	headerScroll()
	promoPopup();
	shopFade();
	popUp('.call-form__wrapper', '.call-form__open', '.call-form__close');
	popUp('.promo-popup__wrapper', '.promo-info', '.promo-popup__close');
	formAjax()
	break;
}
// Функции для всех страниц
preloader();

Answer 1

[Александр]

А в чём проблема, для каждой страницы свой файл скрипта собирать тогда? А такой подход так себе, начнет подводить при увеличении количества страниц/логики.

Comments

[Даниил Сухих]

А как поступить с функциями которые и там и там используються?

[Александр]

danchikraw, Если в ТЗ нету IE, то можно использовать <script type="module" src="..."></script>, модульную структуру и собирать можно при помощи webpack/rollup. Если есть IE - тогда babel'ем склеивать их в один файл.

[Robur]

Александр, какой смысл делать для каждой страницы свой файл, если их все в один потом склеивать?

[Александр]

Robur, Ну, если нету IE, то причина есть. Код разносится по модулям. Rollup после собирает файлы так, что модули, используемые только в 1 файле будут сразу вставлены в этот файл, а те, которые используются в некоторых файлах останутся. Можно также babel прикрутить. Такой подход оставит модули по разным файлам и они будут загружаться и кэшироваться. То есть на другой странице, где этот модуль будет использоваться, он будет загружаться из кэша. То есть не надо будет тянуть весь файл с логикой для каждой страницы.

[Robur]

это совсем не то же самое что "склеить в один файл". К подходу "раскидать разные модули/чанки по разным страницам" у меня нет вопросов