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

Question

[fandorin_official]

Всем добра.

Есть юзерскрипт (некоторые переменные и шелуху, не очень информативную, я выбросил):

(function (window, undefined) {  // [2] нормализуем window
    var w;
    if (typeof unsafeWindow != undefined) {
        w = unsafeWindow;
    } else {
        w = window;
    }
    if (w.self != w.top) {
        return;
    }

    function start() {
        var read = getread();
        checkResult(read);
    }

    function checkResult(last_read) {
        setTimeout(function() {
            if (last_read < getread()) {
                
            } else if (last_read > getread()) {

            } else {
                checkResult(last_read);
            }
        }, 500);
    }

    function getread() {
        var read_text = $('.read_select .chosen-single span').text();
        var regex = /[+-]?\d+(\.\d+)?/g;
        var floats = read_text.match(regex).map(function(v) {
            return parseFloat(v).toFixed(8);
        });
        return floats[0];
    }
    
    function minimum_rate() {
    }

    function callnetwork() {
    }
})(window);

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

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

Comments

[fandorin_official]

Может быть стоит сделать задержку секунд в 15 в третьем условии функции checkResult?

Answer 1

[Антон Спирин]

Иногда на функции checkResult скрипт не может получить ответ от сайта и он выполняет третье условие и перезапускает функцию checkResult снова.

Ерунда. Нет никакого ответа. Код синхронный(не считая setTimeout) и без запросов. Проверьте свои условия. Просто выведите в теле колбека setTimeout в консоль:

console.log(last_read);
console.log(getRead());
console.log(last_read < getread());
console.log(last_read > getread());

Наверняка там совсем не то, что вы задумали.
А скрипт ваш не "устает", а завершает свое выполнение. Кратчайший путь для завершения: start() -> getRead() -> checkResult() -> getRead()
Не знаю что вы задумали, но чтобы скрипт не останавливался достаточно после условий выполнять checkResult:

function checkResult(last_read) {
        setTimeout(function() {
            if (last_read < getread()) {
                
            } else if (last_read > getread()) {

            } else {
                checkResult(last_read);
            }

            checkResult(getRead());
        }, 500);
    }

Comments

[fandorin_official]

Насчет ответа от сайта.
Скрипт "устает" после того, как в консоле появляется "Failed to load resource: the server responded with a status of 429 ()".
В таком случае:
console.log(last_read < getread());
console.log(last_read > getread());
равно false и в итоге мы получаем бесконечное выполнение checkResult по третьему условию.

Добавление строчки checkResult(getRead()); не очень выручает)

[Антон Спирин]

fandorin_official, вот видите он не "устает", а бесконечно выполняет одно и то же условие, а вы писали, что он перестает выполняться и получили рекомендацию, как сделать так, чтобы не переставал.
Вам надо хорошо поработать над логикой алгоритма. Я, например, совершенно не представляю, что он должен делать, но знаю точно, что его можно переписать так, чтобы он всегда выполнялся корректно.
Перепишите условия и обработайте исключение. Скрипт будет работать.