Как сделать условие JavaScript с помощью setinterval?

Question

[Flipflap]

Мне нужно, чтобы скрипт срабатывал раз в опредлённое время, только если условие вернуло TRUE.

Для этого, я написал такой код, но подозреваю, что он не верный

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

function show () {
	if (document.URL.indexOf("/testurl/") != -1);
        modal.style.display = "block";
}

         btn.onclick = function() {
         modal.style.display = "none";
     }

setInterval(show, 10000);
</script>

Дело в том, что скрипт работает с интервалом игнорируя условие, будто оно вечно TRUE. ( познания в JS очень не глубокие, поэтмоу прошу не закидывать тухлыми помидорами )

Comments

[Absolute138]

у вас синтаксис нарушен. Хотя бы так:

function show () {
  if (document.URL.indexOf("/testurl/") != -1){
        modal.style.display = "block";
  }
  btn.onclick = function() {
         modal.style.display = "none";
     }
}

[Ярослав Иванов]

Absolute138, зачем каждый раз в таймере объявлять функцию?

[Absolute138]

Ярослав Иванов, понял Вас. Так наверное:

function show () {
  if (document.URL.indexOf("/testurl/") != -1){
        modal.style.display = "block";
  }
}
  btn.onclick = function() {
         modal.style.display = "none";
  }

[Ярослав Иванов]

Absolute138, неси зачётку

[Absolute138]

Ярослав Иванов, я не для рейтинга здесь, а больше для повышения квалификации, но ответом - записал, чтобы по здешним канонам вопрос был решен и закрыт)

[Александр]

А зачем здесь таймер нужен во обще? Я так понял что при загрузке страницы с определенным url-ом нужно показать блок?
Тогда может вот так:

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

btn.onclick = function() {
  modal.style.display = "none";
}

if (document.URL.indexOf("/testurl/") != -1) {
  modal.style.display = "block";
}

[Flipflap]

Absolute138, спасибо, работает)

[Flipflap]

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

Answer 1

[Absolute138]

function show () {
  if (document.URL.indexOf("/testurl/") != -1){
        modal.style.display = "block";
  }
}
  btn.onclick = function() {
         modal.style.display = "none";
  }

Comments

[Flipflap]

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

[Absolute138]

Flipflap, я здесь больше указываю на ошибки в синтаксисе.

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

Тогда надо куда-то записывать значение, что показ при загрузке страницы уже был, а дальше по таймеру. Здесь я пишу в sessionStorage:

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

function show () {
  if (document.URL.indexOf("/testurl/") != -1){
     modal.style.display = "block";
  }
        
}

btn.onclick = function() {
  modal.style.display = "none";
}
if(sessionStorage.getItem('show') === null){
    modal.style.display = "block";
    sessionStorage.setItem('show', '1');
} else {
    setInterval(show, 10000);
}

Answer 2

[Дмитрий Беляев]

Ошибка в точке запятой после ifif (document.URL.indexOf("/testurl/") != -1);if обрабатывает пустую операцию, все что дальше уже вне if

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

Нормальный вариант уже предложен Absolute138

Но остается вопрос, зачем здесь вообще интервал?

Comments

[Flipflap]

Потому что нужный урл - каталог. При открытии всех последующих каталогов блок снова всплывает

[Дмитрий Беляев]

Flipflap, так проверяйте URL при входе на страницу, ну или перехватывайте роутинг если у Вас SPA