Таймер (обратный вызов) внутри раскрывающегося модуля для обновления открытого свойства

Question

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

Вот простой модуль

var Module = (function () {


    // private
    var isLoaded=false;
    var timer = null;

    var _privateMethod = function () {
        console.log('hello from private')
    }

    //public
    var publicMethod = function(){
        console.log('in public method')
        _privateMethod();
    }

    var start = function(){
        console.log('starting timer')
        timer = setInterval( function() {
            console.log('timer callback')
            isLoaded = !isLoaded;
        },1000)
    }

    var stop = function(){
        console.log('stopping timer')
        clearInterval(timer)
    }

    return {
        publicMethod: publicMethod,
        isLoaded: isLoaded,
        start: start,
        stop: stop,
    }

})();

module.exports = Module;

и я проверяю это с этим

function sleep(milliseconds) {
    var start = new Date().getTime();
    for (var i = 0; i < 1e7; i++) {
      if ((new Date().getTime() - start) > milliseconds){
        break;
      }
    }
  }

var DM = require('./module.js')

DM.publicMethod()
DM.start()

console.log(DM.isLoaded)
 sleep(500)

console.log(DM.isLoaded)
 sleep(500)

console.log(DM.isLoaded)
 sleep(500)

console.log(DM.isLoaded)
 sleep(500)

 DM.stop();

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

Answer 1

Для этого (с такой общей структурой) у вас есть два варианта:

Держите объект

Сохраните ссылку на возвращаемый объект:

var obj;

// ...

obj = {
    publicMethod: publicMethod,
    isLoaded: false,
    start: start,
    stop: stop,
};
return obj;

... и удалите переменную isLoaded, а вместо этого установите obj.isLoaded = true в обратном вызове.

Пример из жизни:

var Module = (function() {
    var obj;
    var timer = null;

    var start = function(){
        console.log('starting timer')
        timer = setInterval( function() {
            console.log('timer callback')
            obj.isLoaded = !obj.isLoaded;
        },1000)
    };

    var stop = function(){
        console.log('stopping timer')
        clearInterval(timer)
    };
    
    obj = {
        //publicMethod: publicMethod,
        isLoaded: false,
        start: start,
        stop: stop,
    };
    return obj;
})();
Module.start();

var t = setInterval(function() {
  console.log("Module.isLoaded = " + Module.isLoaded);
}, 500);
setTimeout(function() {
  console.log("Stopping");
  clearInterval(t);
  Module.stop();
}, 10000);

.as-console-wrapper {
  max-height: 100% !important;
}

Использовать свойство аксессора:

Преимущество в том, что он доступен только для чтения:

return {
    publicMethod: publicMethod,
    get isLoaded() {
        return isLoaded;
    },
    start: start,
    stop: stop,
};

Живой пример:

var Module = (function() {
    var isLoaded = false;
    var timer = null;

    var start = function(){
        console.log('starting timer')
        timer = setInterval( function() {
            console.log('timer callback')
            isLoaded = !isLoaded;
        },1000)
    };

    var stop = function(){
        console.log('stopping timer')
        clearInterval(timer)
    };
    
    obj = {
        //publicMethod: publicMethod,
        get isLoaded() {
            return isLoaded;
        },
        start: start,
        stop: stop,
    };
    return obj;
})();
Module.start();

var t = setInterval(function() {
  console.log("Module.isLoaded = " + Module.isLoaded);
}, 500);
setTimeout(function() {
  console.log("Stopping");
  clearInterval(t);
  Module.stop();
}, 10000);

.as-console-wrapper {
  max-height: 100% !important;
}