Как повесить несколько событий

[Сергей Ракипов]

buttonNavigMobile.onclick = function(){
	bodyOverflow.classList.toggle(overflow);
}

buttonNavigMobile.onclick = function(){
	if (pageNavig.className === "page_navig"){
		pageNavig.classList.remove("page_navig_hide");
		pageNavig.classList.add("page_navig_show");
	}
	else if (pageNavig.className === "page_navig page_navig_show"){
		pageNavig.classList.remove("page_navig_show");
		pageNavig.classList.add("page_navig_hide");
	}
	else if (pageNavig.className === "page_navig page_navig_hide"){
		pageNavig.classList.remove("page_navig_hide");
		pageNavig.classList.add("page_navig_show");
	}
}

А что нельзя что ли на один элемент вот так повестить события ?
И можно или а главное нужно ли закончить вторую функцию не else if а просто else

[рони]

Сергей Ракипов,
addEventListener

[Сергей Ракипов]

Сергей Ракипов,
addEventListener

это я понимаю, просто хотел узнать что такое написание это ошибка?

и обязательно ли заканчивать else

[рони]

обязательно ли заканчивать else

нет

[Сергей Ракипов]

нет

Спасибо

[Сергей Ракипов]

А как более правильнее писать

function overflow(){
bodyOverflow.classList.toggle("overflow_mobile");
}
buttonNavigMobile.addEventListener("click", overflow);

function navigHideShow(){
	if (pageNavig.className === "page_navig"){
		pageNavig.classList.remove("page_navig_hide");
		pageNavig.classList.add("page_navig_show");
	}
	else if (pageNavig.className === "page_navig page_navig_show"){
		pageNavig.classList.remove("page_navig_show");
		pageNavig.classList.add("page_navig_hide");
	}
	else if (pageNavig.className === "page_navig page_navig_hide"){
		pageNavig.classList.remove("page_navig_hide");
		pageNavig.classList.add("page_navig_show");
	}
}
buttonNavigMobile.addEventListener("click", navigHideShow);

создать две функции или

function navigHideShow(){
	if (pageNavig.className === "page_navig"){
		pageNavig.classList.remove("page_navig_hide");
		pageNavig.classList.add("page_navig_show");
	}
	else if (pageNavig.className === "page_navig page_navig_show"){
		pageNavig.classList.remove("page_navig_show");
		pageNavig.classList.add("page_navig_hide");
	}
	else if (pageNavig.className === "page_navig page_navig_hide"){
		pageNavig.classList.remove("page_navig_hide");
		pageNavig.classList.add("page_navig_show");
	}
	bodyOverflow.classList.toggle("overflow_mobile");

}
buttonNavigMobile.addEventListener("click", navigHideShow);

в одну функции засунуть.

[рони]

как более правильнее

правильнее не использовать className в условии(есть classList.contains) и никогда не назначать click в click!!!

[рони]

Сергей Ракипов,

function overflow(){
bodyOverflow.classList.toggle("overflow_mobile");
pageNavig.classList.toggle("page_navig_show");
}
buttonNavigMobile.addEventListener("click", overflow);

остальное лишнее.

[Сергей Ракипов]

правильнее не использовать className в условии(есть classList.contains)

classList.contains вроде как не может проверить наличие сразу двух классов, а className может или я не знаю синтаксис.
у меня скрипт проверяет наличие двух классов в элементе

[Сергей Ракипов]

и никогда не назначать click в click!!!

я там не правильно скопировал, сейчас поправил, и есть две функции и есть одна.