Как правильно прописать условие?

Question

[goozer]

Всем привет!
Необходимо добавить цели метрики на успешно отправленную форму. Сделать нужно только на некоторых страницах сайта. Вот сам код:

$.ajax({
			type: "POST", url: "/include/form_handler.php",
			data: form.serialize(),
			datatype: "html",
			cache: false,
			success: function(answer){
if (window.location.href.indexOf("gisteroskopiya") > -1) {
ym(67782700,'reachGoal','gisteroskopiya');};
if (window.location.href.indexOf("organizma-posle-covid") > -1) {
ym(67782700,'reachGoal','check-up-diagnostika-organizma-posle-covid-19');};
if (window.location.href.indexOf("nevrologiya") > -1) {
ym(67782700,'reachGoal','nevrologiya');};
if (window.location.href.indexOf("travmpunkt") > -1) {
ym(67782700,'reachGoal','travmpunkt');};
				$(".popup").removeClass("active");
				$(".popup-sent").addClass("active");
				$(".popup-sent__info").html(answer);				
				form.trigger("reset");
console.log("success");
			},
			error: function( xhr, textStatus ) {
				console.log([ xhr.status, textStatus ]);
			}
		});
	}

Добавлял просто через window.location.href. С JS я не очень, добавлял методом тыка. Есть ли какой-то способ добавить эти условия получше?

Comments

[Алексей Дубровин]

Для кода есть отдельный тег, при редактировании справа 2й

[goozer]

Алексей Дубровин, поправил

[Алексей Дубровин]

goozer,
Нууу само решение вызывает вопросы, ну допустим
Для начала нам нужно сделать его читаемым, для этого мы выносим
window.location.href - в отдельную переменную

Далее мы вместо проверки на > - что не правильно по смыслу
Проверяем через includes
в результате код как минимум станет проще раза в 3

{
success: function(answer){
const href = window.location.href;
if (href.includes("gisteroskopiya")){
    ym(67782700,'reachGoal','gisteroskopiya');
}
if (href.includes("organizma-posle-covid")){
    ym(67782700,'reachGoal','check-up-diagnostika-organizma-posle-covid-19');
};
if (href.includes("nevrologiya")) {
    ym(67782700,'reachGoal','nevrologiya');
};
if (href.includes("travmpunkt")) {
    ym(67782700,'reachGoal','travmpunkt');
};
        $(".popup").removeClass("active");
        $(".popup-sent").addClass("active");
        $(".popup-sent__info").html(answer);				
        form.trigger("reset");
console.log("success");
      },}

Тк как это по логике отдельный логический блок(Проверка на какой мы странице находимся и отправка соответствующей метрики) можно вынести это в соответствующую функцию.

Как бы я сделал
Собрал масив с нужными страницами, проходил по нему, и смотрел текущая ли эта страница, если да отправлял метрику

const arrHref = ['gisteroskopiya','aasdads','home'];
const href = window.location.href;
arrHref.forEach(item=>{
  if(href.includes(item)) {
    ym(67782700,'reachGoal',item);
  }
});

[goozer]

Алексей Дубровин, Спасибо, попробую