clearInterval () не останавливает setInterval () - Разработка расширения Firefox - PullRequest
2 голосов
/ 14 июня 2011

Я работаю над модификацией данных несанкционированного доступа, которая позволит мне отправлять HTTP-запрос / ответы, которые он наблюдает, на сервер.Пока что эта функциональность была реализована правильно.Следующим шагом является автоматизация этого процесса, и я хочу использовать кнопку меню панели инструментов типа 'checkbox' для включения и выключения этой функции.

Пока у меня есть этот фрагмент кода в .XUL:

<toolbarbutton id="tamper.autosend" label="&tamper.toolbar.autosend;" type="checkbox" oncommand="oTamper.toggleTimer();"/>

И эта функция в главном драйвере моего расширения:

toggleTimer : function() {
 var checked = document.getElementById('tamper.autosend').checked;

 var consoleService = Components.classes["@mozilla.org/consoleservice;1"].getService(Components.interfaces.nsIConsoleService);

 consoleService.logStringMessage(checked);

 if (checked) {
        var interval = window.setInterval(function(thisObj) { thisObj.sendResults(true); }, 1000, this);
 }

 else {
        window.clearInterval(interval);
 }
}

Используя consoleService, я вижу, что значение 'флажок' действительно корректен.Я считаю, что проблема заключается в том, как я вызываю clearInterval, но я не совсем уверен, как это исправить.

Любая помощь очень ценится!

Ответы [ 4 ]

5 голосов
/ 14 июня 2011

Вы определили интервал внутри, если попытаетесь объявить переменную в начале

var interval = 0;
toggleTimer : function() {
 var checked = document.getElementById('tamper.autosend').checked;

 var consoleService = Components.classes["@mozilla.org/consoleservice;1"].getService(Components.interfaces.nsIConsoleService);

 consoleService.logStringMessage(checked);

 if (checked) {
        interval = window.setInterval(function(thisObj) { thisObj.sendResults(true); }, 1000, this);
 }

 else {
        window.clearInterval(interval);
 }
}
3 голосов
/ 04 февраля 2014

Вы делаете это неправильно, каждый раз, когда вы хотите установить новый интервал, вы должны сначала очистить его

clearInterval(intervalID);

console.log('reset timer');

intervalID = setInterval(function () {
    console.log('tick');
}, refreshInterval);
2 голосов
/ 14 июня 2011

Конечно, потому что interval определяется как частная переменная. Он определяется в функции toggleTimer и уничтожается при завершении функции.

Используйте interval = window.setInterval() вместо var interval = window.setInterval(), чтобы определить глобальную переменную, которая будет доступна позже для clearInterval.

Ниже приведены некоторые примеры области видимости переменной JavaScript. var используется для определения переменной в области действия current . Выход var всегда создает или изменяет локальную переменную.

function func1() {
    i = 1; // global
}
func1();
alert(i); // 1

var j = 2;
function func2() {
    var j = 3; // private
}
func2();
alert(j); // 2

k = 4;
function func3() {
    k = 5; // global
}
func3();
alert(k); // 5

var l = 6;
function func4() {
    l = 7; // global
}
func4();
alert(l); // 7

function func5() {
    var m = 6; // private
}
func5();
alert(m); // undefined
2 голосов
/ 14 июня 2011

Вы храните interval в локальной переменной;значение теряется после возврата из функции, когда вы в следующий раз попытаетесь clearInterval неопределенную переменную.Вместо этого сохраните интервал, то есть глобальную переменную:

 if (checked) {
        window.interval = window.setInterval(function(thisObj) { thisObj.sendResults(true); }, 1000, this);
 }

 else {
        window.clearInterval(interval);
 }
...