Почему не работает условие в javascript?

Question

[Леонид Роженцев]

Здравствуйте. На сайте есть скрипт (в файле script.js), который показывает модальное окно при попытке пользователя закрыть страницу:

// функция возвращает cookie с именем name, если есть, если нет, то undefined    
function getCookie(name) {
    var matches = document.cookie.match(new RegExp(
    "(?:^|; )" + name.replace(/([\.$?*|{}\(\)\[\]\\\/\+^])/g, '\\$1') + "=([^;]*)"
    ));
    return matches ? decodeURIComponent(matches[1]) : undefined;
}
// проверяем, есть ли у нас cookie, с которой мы не показываем окно и если нет, запускаем показ
var alertwin = getCookie("alertwin");
if (alertwin != "no") { 
    $(document).mouseleave(function(e){
        if (e.clientY < 0) {
            $(".exitblock").fadeIn("fast");    
            // записываем cookie на 1 день, с которой мы не показываем окно
            var date = new Date;
            date.setDate(date.getDate() + 1);    
            document.cookie = "alertwin=no; path=/; expires=" + date.toUTCString();       
        }    
    });
    $(document).click(function(e) {
        if (($(".exitblock").is(':visible')) && (!$(e.target).closest(".exitblock .modaltext").length)) {
            $(".exitblock").remove();
        }
    });  
}

Чтобы окно не выскакивало часто, устанавливаются куки. Но проблема в том, что модальное окно показывается постоянно при попытке закрыть страницу. При этом сами куки устанавливаются. В чём может быть проблема? Сайт натянут на WordPress, скрипты подключены нормально

Comments

[Сергей delphinpro]

var alertwin = getCookie("alertwin");
console.log(alertwin); // Что в консоли видите?

Answer 1

[Владимир Брумер]

В чём может быть проблема?

в том, что Вы выполняете проверку(получаете куку) один раз после(в момент) загрузки страницы, а Вам надо при событиях.(при первой загрузки там будет пусто - undefined, согласно Вашему коду - самая первая строка)

Но проблема в том, что модальное окно показывается постоянно при попытке закрыть страницу. При этом сами куки устанавливаются.

а если страницу перезагрузить после установки куки?

з.ы

...
var alertwin = getCookie("alertwin");//при отсутствии куки тут пусто.
//значение легло при загрузке страницы
//значит после установки куки надо перезагрузить страницу, чтоб там что-то было
//либо сделать повторный запрос getCookie(name)
if (alertwin != "no") { 
    $(document).mouseleave(function(e){
        if (e.clientY < 0) {
...
_________________________________________________________________________
...
$(document).mouseleave(function(e){
	var alertwin = getCookie("alertwin");//получаем куку при каждом событии mouseleave
//при первом событии там пусто и код ниже отработает
//при повторном наведении там не пусто и код ниже не отработает
	if (alertwin != "no") {
...

Comments

[Леонид Роженцев]

Спасибо!

[Леонид Роженцев]

Всё равно не помогает, окошко выскакивает постоянно, хотя куки записываются

[Леонид Роженцев]

Вот можете посмотреть на сайте, если не сложно. Просто не могу разобраться, в чём причина странного поведения cookie https://melnica-muka.ru

[Владимир Брумер]

Леонид Роженцев,

Вот исправленный скрипт, но и он не работает

строки 60-70, зачем 2 раза $(document).mouseleave(function(e){?

spoiler

[Владимир Брумер]

строки 60-70 лишние, они работают по старой схеме

[Леонид Роженцев]

Владимир Брумер, А что тогда оставить?
Вот так:

var alertwin = getCookie("alertwin");
if (alertwin != "no") { 
        if (e.clientY < 0) {
            $(".overlay_stop").fadeIn("fast");    
            // записываем cookie на 1 день, с которой мы не показываем окно
            var date = new Date;
            date.setDate(date.getDate() + 1);    
            document.cookie = "alertwin=no; path=/; expires=" + date.toUTCString();       
        } 
}
$(document).mouseleave(function(e){
    var alertwin = getCookie("alertwin");
    if(alertwin != "no") {
        if (e.clientY < 0) {
            $(".overlay_stop").fadeIn("fast");    
            // записываем cookie на 1 день, с которой мы не показываем окно
            var date = new Date;
            date.setDate(date.getDate() + 1);    
            document.cookie = "alertwin=no; path=/; expires=" + date.toUTCString();       
        } 
    }
});

[Владимир Брумер]

Леонид Роженцев,

// функция возвращает cookie с именем name, если есть, если нет, то undefined    
function getCookie(name) {
    var matches = document.cookie.match(new RegExp(
    "(?:^|; )" + name.replace(/([\.$?*|{}\(\)\[\]\\\/\+^])/g, '\\$1') + "=([^;]*)"
    ));
    return matches ? decodeURIComponent(matches[1]) : undefined;
}
// проверяем, есть ли у нас cookie, с которой мы не показываем окно и если нет, запускаем показ
$(document).mouseleave(function(e){
    var alertwin = getCookie("alertwin");
    if(alertwin != "no") {
        if (e.clientY < 0) {
            $(".overlay_stop").fadeIn("fast");    
            // записываем cookie на 1 день, с которой мы не показываем окно
            var date = new Date;
            date.setDate(date.getDate() + 1);    
            document.cookie = "alertwin=no; path=/; expires=" + date.toUTCString();       
        } 
    }
});
      
    jQuery('.stop__close').on('click', function () {
        jQuery('.overlay_stop').fadeOut(500);
    });

});

[Владимир Брумер]

при на ведении срабатывает событие $(document).mouseleave(function(e){
в котором запрашивается кука
var alertwin = getCookie("alertwin");
и если куки нету
if(alertwin != "no") {
вылезет окно и кука установится

$(".overlay_stop").fadeIn("fast");    
            // записываем cookie на 1 день, с которой мы не показываем окно
            var date = new Date;
            date.setDate(date.getDate() + 1);    
            document.cookie = "alertwin=no; path=/; expires=" + date.toUTCString();

в противном случае этот код не сработает так как не выполнится условие if(alertwin != "no") {

[Леонид Роженцев]

Владимир Брумер, Спасибо большое!

[Владимир Брумер]

Леонид Роженцев, на здоровье

Answer 2

[Леонид Роженцев]

Вот исправленный скрипт, но и он не работает

function getCookie(name) {
    var matches = document.cookie.match(new RegExp(
    "(?:^|; )" + name.replace(/([\.$?*|{}\(\)\[\]\\\/\+^])/g, '\\$1') + "=([^;]*)"
    ));
    return matches ? decodeURIComponent(matches[1]) : undefined;
}
// проверяем, есть ли у нас cookie, с которой мы не показываем окно и если нет, запускаем показ
var alertwin = getCookie("alertwin");
if (alertwin != "no") { 
    $(document).mouseleave(function(e){
        if (e.clientY < 0) {
            $(".overlay_stop").fadeIn("fast");    
            // записываем cookie на 1 день, с которой мы не показываем окно
            var date = new Date;
            date.setDate(date.getDate() + 1);    
            document.cookie = "alertwin=no; path=/; expires=" + date.toUTCString();       
        } 
    });
}
$(document).mouseleave(function(e){
    var alertwin = getCookie("alertwin");
    if(alertwin != "no") {
        if (e.clientY < 0) {
            $(".overlay_stop").fadeIn("fast");    
            // записываем cookie на 1 день, с которой мы не показываем окно
            var date = new Date;
            date.setDate(date.getDate() + 1);    
            document.cookie = "alertwin=no; path=/; expires=" + date.toUTCString();       
        } 
    }
});