Почему не работает скрипт?

Question

[sidorikv]

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

function doCheck() {
    const now = new Date();
    if (now.getHours() !== 17) { // часы
    timeout(60);
    } 
    else {
    if (now.getMinutes() !== 39) { // минуты
    timeout(30);
    } 
    else {
    if (now.getSeconds() !== 00) { // секунды
    timeout(15);
    } 
    else {
    if (now.getMilliseconds() !== 000) { // milекунды
    timeout(15);
    } 
    else {
    document.querySelector("какая то кнопка").click();
    }
    }
    }
    }
    }
    
function timeout(ms = 1) {
    setTimeout(() => {
    doCheck();
    }, ms);
    }
    
doCheck();

Answer 1

[Stalker_RED]

Он вообще-то работает, но только с некоторым шансом.

if (now.getSeconds() !== 00) { // секунды 
    timeout(15);
    }

если секунда неподходящая, он ждет еще 15 миллисекунд.
И если следующая проверка now.getMilliseconds() !== 000 не произойдет РОВНО через 15 msec, то промах, и надо ждать еще сутки.

Вообще это плохой подход, не надо так. Проще посчитать на старте разницу между текущим временем и нужным и вызвать setTimeout сразу на нужное время.

Comments

[sidorikv]

function timeout(ms = 1) {
    setTimeout(() => {
    doCheck();
    }, ms);
    }

А на что влияет эта функция?:D Просто я уже запутался если честно! Вернее параметр ms = 1, зачем он? Если мы таймауты выставляем под часами минутами и секундами

[Stalker_RED]

sidorikv, = 1 это значение по умолчанию.

Чтобы не запутываться, просто почитайте учебник какой-нибудь про event loop и таймеры, и этот код будет вызывать у вас улыбку.

Answer 2

[Тимур Покровский]

А зачем везде таймауты? Убери их и всё заработает

Comments

[sidorikv]

То есть если проверка не проходит, то просто ничего не делать?

[Тимур Покровский]

sidorikv, ну да

[Тимур Покровский]

sidorikv, можно вообще объединить это всё в один if

[sidorikv]

Тимур Покровский, хмм, а можно код, как именно это сделать? Заранее спасибо

[Andrey Vasilev]

sidorikv,

if (now.getHours() !== 17
   && now.getMinutes() !== 39
   && now.getSeconds() !== 00
   && now.getMilliseconds() !== 000) {
} else {
  document.querySelector("какая то кнопка").click();
}

[sidorikv]

Andrey Vasilev, а если часы будут верные, но минуты и секунды нет?

[Stalker_RED]

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

[Andrey Vasilev]

sidorikv, наверное, суть все-таки в этом?

if (now.getHours() !== 17
   || now.getMinutes() !== 39
   || now.getSeconds() !== 00
   || now.getMilliseconds() !== 000) {
} else {
  document.querySelector("какая то кнопка").click();
}

[Andrey Vasilev]

sidorikv, в первом варианте должны выполнится все условия, во втором - хотя бы одно