JavaScript: не могу остановить установку

Question

Я работаю над проверкой прокси -сервера и имею следующий код, чтобы запустить запросы с интервалом примерно 5 секунд, используя функцию SetTimeout;

        function check() {

            var url = document.getElementById('url').value;
            var proxys = document.getElementById('proxys').value.replace(/\n/g,',');

            var proxys = proxys.split(",");

            for (proxy in proxys) {

                var proxytimeout = proxy*5000;

                t = setTimeout(doRequest, proxytimeout, url, proxys[proxy]);

            }
        }

Однако я не могу остановить их, как только они начали!

        function stopcheck() {

            clearTimeout(t);

        }

Исправление или лучший метод будет больше, чем ценится.

Спасибо сообщество переполнения стека!

Answer 1

Есть 2 основные проблемы с вашим кодом:

1. t перезаписан для каждого тайм -аута, теряя ссылку на предыдущий тайм -аут каждую итерацию.
2. t может не быть глобальной переменной, таким образом stopcheck() Может быть, не сможет "увидеть" t.

Обновленные функции:

function check() {
    var url         = document.getElementById('url').value;
    var proxys      = document.getElementById('proxys').value.replace(/\n/g,',');
    var timeouts    = [];
    var index;
    var proxytimeout;

    proxys = proxys.split(",");
    for (index = 0; index < proxys.length; ++index) {
        proxytimeout                = index * 5000;
        timeouts[timeouts.length]   = setTimeout(
            doRequest, proxytimeout, url, proxys[index];
        );
    }

    return timeouts;
}

function stopcheck(timeouts) {
    for (var i = 0; i < timeouts.length; i++) {        
        clearTimeout(timeouts[i]);
    }
}

Пример использования:

var timeouts = check();

// do some other stuff...

stopcheck(timeouts);

Answer 2

Где определяется «t»? Он постоянно переопределяется в петле, так что вы будете терять каждую ручку времени ...

Вы можете сохранить множество ручек:

var aTimeoutHandles = new Array();
var iCount = 0;
for (proxy in proxys) {

    var proxytimeout = proxy*5000;

    aTimeoutHandles[iCount++] = setTimeout(doRequest, proxytimeout, url, proxys[proxy]);

}

Answer 3

Определять t сначала вне обеих функций. Кроме того, вы перезаписываете t с каждой итерацией ваша for петля. Возможно, создание коллекции ссылок, а затем остановить их, вы пробегаете, и clearTimeout на каждого.

Answer 4

Вы перезаписываете t Каждый раз, когда вы устанавливаете интервал. Таким образом, вы только в конечном итоге очищаете последний набор.

Answer 5

Looks like you're setting multiple timeouts (one for each proxy), but trying to save them in the same variable. You probably need to use an array there, instead of a simple variable.

Answer 6

You have a few problems there:

1. The main one is that you're overwriting t on each iteration of your for loop; you need an array of ts for your structure to work.
2. You're using for..in to loop through the indexes of the array. That's not what for..in is for (although there are a lot of people confused about that; see this article). for..in loops through the property names of an object, not the indexes of an array, and therefore this usage breaks in non-trivial situations. Just use an old-fashioned for loop.
3. You're declaring proxys twice. This is actually harmless, but...
4. You're not declaring proxy at all (which isn't harmless; it becomes an implicit global).

I've updated the code in Jordan's excellent answer to address those.