Почему не работает код JavaScript?

Question

[sneeskaa]

Доброго времени суток. Написал код, который должен проверять на нахождению юзера по юрл и наличие куки, если нет - выводить в блок.

После открытия блока и нажатия кнопки - юзер блок видеть больше не должен, до закрытия браузера. Но не работает: блок не всплывает.

Вероятно, где-то накосячил.

var modal = document.getElementById("mymodal");
    var btn = document.getElementById("close");

	function setcookie(){
        setcookie("show", "0");
	    if (document.URL.indexOf("/make/") != -1 & document.cookie == "show=0" ){
            modal.style.display = "block";
  }
}	

  btn.onclick = function() {
         modal.style.display = "none";
         setcookie("show", "1");
  }

Comments

[Steppp]

setcookie("show", "0"); Что это за функция?

Answer 1

[scottparker]

1) логическое и = &&, у тебя &
2) у тебя на сайте только одна кука с ключем show, если нет, то не правильная проверка наличие куки с ключем show
3) функция же setcookie() где-то вызывается?

и убери установку значения куки равной 0 в начале функции, окно ж так всегда показываться будет

Comments

[sneeskaa]

Да, show только здесь использую. Это не правильно?

[scottparker]

sneeskaa, я имел ввиду не только здесь, а только одна кука (show) на сайте?

[sneeskaa]

scottparker, Да, show больше нигде нет

[scottparker]

sneeskaa, моя твоя не понимать

на сайте есть другие куки кроме show?

[sneeskaa]

scottparker, Вроде поправил, всё равн не работает

var modal = document.getElementById("mymodal");
    var btn = document.getElementById("test");


	    if (document.URL.indexOf("/make/") != -1 && document.cookie == "show-modal-a=0" ){
            modal.style.display = "block";
  }


  btn.onclick = function() {
         modal.style.display = "none";
         document.cookie = "show-modal-a=1";
  }

[sneeskaa]

scottparker, аА ты об этом. Да, конечно. Сайт на CMS, тут полно куки.

[scottparker]

sneeskaa, проверка на значение куки не правильная, ты ее не установил, а проверяешь ее значение

[sneeskaa]

scottparker, Кхм.. Ты сказал убрать сначала. Надо было просто вынести за функцию?

[scottparker]

sneeskaa, ну раз полно, то как ты строку с куками, где их полно, сравниваешь с show-modal-a=0

введи в консоли document.cookie, увидишь строку с куками (твоеей там не будет, но все же, даже если бы была, явно бы полная строка с куками не ровнялась show-modal-a=0)

[sneeskaa]

scottparker, Да, её там нет. Как тогда правильно провести сравнение? Я всё перегуглил сравнение куки js и ничего не нашёл, так же и в ютубе какая-то ересь. На лёрн JS тоже толком ничего не сказано.

[sneeskaa]

Может, так ближе к правде будет?

var modal = document.getElementById("mymodal");
var btn = document.getElementById("test");


	    if (document.URL.indexOf("/make/") != -1 ){
                document.cookie = "show-modal-a=0";
                    if (document.cookie == "show-modal-a=0) {
                        modal.style.display = "block";
  }
}


  btn.onclick = function() {
         modal.style.display = "none";
         document.cookie = "show-modal-a=1";
  }

[scottparker]

На лёрн JS тоже толком ничего не сказано

там есть функция для проверки установлена ли кука или нет, посмотри внимательнее, как раз рядом с setCookie, которую ты оттуда же взял )

[sneeskaa]

scottparker, Ты про getcookie, да ?

[scottparker]

sneeskaa, логично. ну точнее не проверка установлена или нет, а получение значения куки. можешь либо переделать под проверку, либо получить значение куки и проверить полученный результат

[sneeskaa]

function getCookie(show-modal-a) {
  let matches = document.cookie.match(new RegExp(
    "(?:^|; )" + name.replace(/([\.$?*|{}\(\)\[\]\\\/\+^])/g, '\\$1') + "=([^;]*)"
  ));
  return matches ? decodeURIComponent(matches[1]) : undefined;
  if (matches == 0) {
      modal.style.display = "block";
}
}

Вот так проверяется?

[scottparker]

sneeskaa, эм.. скопируй оттуда функцию, и вызови ее для своего ключа куки...

[sneeskaa]

scottparker,

function getCookie(show-modal-a) {
  let matches = document.cookie.match(new RegExp(
    "(?:^|; )" + name.replace(/([\.$?*|{}\(\)\[\]\\\/\+^])/g, '\\$1') + "=([^;]*)"
  ));
  return matches ? decodeURIComponent(matches[1]) : undefined;
  if (getcookie(show-modal-a) == 0 {
      modal.style.display = "block";
 }
}

[scottparker]

sneeskaa, без экспериментов

[sneeskaa]

scottparker, Я не понимаю. Я же её вызываю, как в функции и проверяю. Почему не правильно?

Первый день в JS выдаётся насыщенным)

[scottparker]

sneeskaa, у тебя вызов функции внутри функции, еще и после ретурна. забудь про совет переделать под проверку, просто скопируй, вызови для своей куки и проверь результат

[sneeskaa]

scottparker,

function getCookie(show-modal-a) {
  let matches = document.cookie.match(new RegExp(
    "(?:^|; )" + name.replace(/([\.$?*|{}\(\)\[\]\\\/\+^])/g, '\\$1') + "=([^;]*)"
  ));
  return matches ? decodeURIComponent(matches[1]) : undefined;
}
getcookie(show-modal-a);

Просто скопировал и просто вызвал. Оно?

или

function getCookie(show-modal-a) {
  let matches = document.cookie.match(new RegExp(
    "(?:^|; )" + name.replace(/([\.$?*|{}\(\)\[\]\\\/\+^])/g, '\\$1') + "=([^;]*)"
  ));
  return matches ? decodeURIComponent(matches[1]) : undefined;
}
if (getcookie(show-modal-a) == 0) {
modal.style.display = "block";
}

[scottparker]

sneeskaa, плохо скопировал функцию
if (getcookie(show-modal-a) == 0) что здесь show-modal-a?

[sneeskaa]

scottparker, name кука

[scottparker]

sneeskaa, так может это строка должна быть?

[sneeskaa]

scottparker, В каком смысле? show-modal-a и есть строка, на число же не похоже) Разве что ты имеешь в виду:
if (getcookie() == 0)

[scottparker]

sneeskaa, ну да, ну да
только ты не в обычном мире сейчас, изучай, что есть строка

[scottparker]

sneeskaa,

Разве что ты имеешь в виду:
if (getcookie() == 0)

нет уж, если параметр в описании функции есть (и у него не задано значение по умолчанию) то и в вызове он должен быть

[sneeskaa]

scottparker, Понял, с этим я разобрался. Получается, проверка выглядит так?

var modal = document.getElementById("mymodal");
 var btn = document.getElementById("test");

if (document.URL.indexOf("/make/") != -1 ){  // Заходим на нужный урл
    document.cookie = "show-modal-a=0";    // Регистрируем куки со значением 0
function getCookie(show-modal-a) {  // Функция, чтобы потом проверить значение куки
                    let matches = document.cookie.match(new RegExp(
                     "(?:^|; )" + name.replace(/([\.$?*|{}\(\)\[\]\\\/\+^])/g, '\\$1') + "=([^;]*)"
                       ));
                    return matches ? decodeURIComponent(matches[1]) : undefined;
}

if (getcookie() == 0) {  // Проверяем значение и выводим блок
      modal.style.display = "block";
 }
}
 btn.onclick = function() { // Нажимая на кнопку закрываем блок и меняем значение куки, чтобы окно не выводилось до следующего раза
         modal.style.display = "none";
         document.cookie = "show-modal-a=1";
  }

Всё равно не работает. Подозреваю, что я не правильно завожу куки

[sneeskaa]

scottparker, строка, если я правильно понимаю - string. modal-show-a и есть переменна типа string, разве не так ?

[scottparker]

Получается, проверка выглядит так?

нет

Подозреваю, что я не правильно завожу куки

в 3ий раз пишу, скопируй нормально функцию. и вызов не правильный, смотри предыдущее сообщение

[sneeskaa]

scottparker, Ну вот, теперь я окончательно запутался

[scottparker]

modal-show-a и есть переменна

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

[sneeskaa]

scottparker,

function getCookie(name) {
  let matches = document.cookie.match(new RegExp(
    "(?:^|; )" + name.replace(/([\.$?*|{}\(\)\[\]\\\/\+^])/g, '\\$1') + "=([^;]*)"
  ));
  return matches ? decodeURIComponent(matches[1]) : undefined;
}

Функция getCookie(name) возвращает куки с указанным name:

Написано на лёрн JS. Тут прямым текстом сказано: замените нейм на нейм вашей куки

[scottparker]

sneeskaa, ну, удачи тогда. хз зачем тебе куки эти, если ты не знаешь как функцию вызвать, и как ты вообще работаешь с этим сайтом...

[scottparker]

sneeskaa, успех. ну теперь вызови эту функцию для с именем своей куки

[sneeskaa]

getcookie(name) {
}

Так ?

[scottparker]

sneeskaa, что такое name?

[sneeskaa]

scottparker, Я не знаю. В прошлый раз я поставил modal-show-a и ты спросил тоже самое. Если подставляю имя куки, неправильно. Не подставляю - тоже не правильно. Как правильно?

[scottparker]

sneeskaa,

Как правильно?

передать строку с именем куки

[sneeskaa]

scottparker, ну тогда остаётся только

if (getcookie("modal-show-a") == 0) {
    modal.style.display = "block";
}

[scottparker]

sneeskaa, ну вот, теперь посмотри что функция возвращает

[sneeskaa]

она вернула show-modal-a=0 , но блок по прежнему не появился

[scottparker]

sneeskaa, может просто 0?
и да, тебе не нужно устанавливать значение куки равной 0 каждый раз при загрузке страницы.

в условии тебе нужно проверить, существует ли в принципе кука с таким именем

[sneeskaa]

Сделал вот так, но всё равно не работает. Хотя, казалось бы: условие соблюдается. Кука не равна accept, а значит блок должен выводиться. Но он не выводиться(

document.cookie = "show-modal-a=decline";
    var modal = document.getElementById("mymodal");
    var btn = document.getElementById("test");


      if (document.URL.indexOf("/make/") != -1 ){


                function getCookie(name) {
                    let matches = document.cookie.match(new RegExp(
                     "(?:^|; )" + name.replace(/([\.$?*|{}\(\)\[\]\\\/\+^])/g, '\\$1') + "=([^;]*)"
                       ));
                    return matches ? decodeURIComponent(matches[1]) : undefined;
}
                     if (getcookie("show-modal-a") != "accept") {
                         modal.style.display = "block";
                         
}

  



                       btn.onclick = function() {
                       modal.style.display = "none";
                       document.cookie = "show-modal-a=accept";
  }
}

[scottparker]

почему в описании функции C большое, а в вызове - маленькое? ну ты бы хоть консоль смотрел, ошибки же выводит

[sneeskaa]

scottparker, Заработало! Спасибо тебе огромное!

[sneeskaa]

scottparker, я во всём разобрался!

[scottparker]

sneeskaa, сомневаюсь)

[sneeskaa]

scottparker, сто плюсов тебе в карму)