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

Question

[dorohdds]

При первом вызове или по истечению таймера все хорошо, но если вызвать несколько раз.. при попытке досрочно завершить таймер выполнится n-раз от кол-ва вызова

exports.run = (m1) => {
     var timer1 = setTimeout(function () {
        calculate()        
    }, 1000*5)
    m1.on('finish', function () {
        console.log(`таймер сброшен`);        
        clearTimeout(timer1)
        calculate()
    })
function calculate() { 
    // тут некоторые действия
}

Comments

[Lynn «Кофеман»]

Дубль Почему clearTimeout не завершает setTimeout?

[dorohdds]

Алексей Тен, вопрос другой.

Answer 1

[xmoonlight]

Поставьте счётчик на количество активных.
При вызове - прибавляете 1.
При останове - отнимаете 1.
И условие, контролирующее очередной запуск в зависимости от нужного количества.
Нужно только один - сравнение с единицей и т.д.

Comments

[dorohdds]

хороший вариант! Но получается, что экспортированная функция не завершается и в ней остается живым обработчик m1.on('finish).. И чисто теоретически при достаточно долгом работе программы и частых вызовах их будет огромное множество.. Является ли проблемой это?
Можно ли как-то после выполнения расчета завершить выполнение экпортированной функции, чтобы от нее ничего не оставалось?

[xmoonlight]

dorohdds, return; - это выход из любой внутренней функции. типа break для циклов.

[dorohdds]

xmoonlight, xmoonlight, ага, только при добавление ретурна в функцию calculate, событие все равно отлавливается и получается так:
ПЕРВЫЙ РАЗ
1. выполняем экспортированную функцию.
2. событием финиш досрочно останавливаем таймер
3. выполняет функция calculate

ВТОРОЙ РАЗ
1. выполняем экспортированную функцию.
2. событием финиш досрочно останавливаем таймер
3. выполняет функция calculate
3. выполняет функция calculate

И так с каждый разом больше на 1 из-за того что обраточик почему-то остается и продолжает слушать

[xmoonlight]

dorohdds, А обернуть код с внешней переменной-счётчиком?

[dorohdds]

xmoonlight, Это решает проблему, что исполняется много раз от кол-ва запусков.. Но
m1.on('finish) продолжает ловиться в каждом... экземпляере(так называется?).. и выполняться
можно придумать id, который будет указывать для какой именно функции надо завершить досрочное выполнение.. Но проблема, которая меня волнует это не исправит.

exports.run = (m1) => {
     var kostil = 1
     var timer1 = setTimeout(function () {
        kostil -=1
     if(kostil == 0) calculate()   
    }, 1000*5)
    m1.on('finish', function () {
        console.log(`таймер сброшен`);        
        clearTimeout(timer1)
        kostil -=1
        if(kostil == 0) calculate()
    })
function calculate() { 
    // тут некоторые действия
    
}

[xmoonlight]

dorohdds, опишите нужный алгоритм работы кода по-порядку без привязки к существующему коду.

[dorohdds]

Bavashi, пока что xmoonlight предложил хороший вариант о котором я и сам думал и позволяет решать задачу, хоть и с частичным решением проблемы. И он единственный, кто откликнулся и пытается помочь

[dorohdds]

xmoonlight, есть функция, которая собирает данные определенное время из эвентов. После окончания таймера или после принудительного завершения через эвент производится расчет данных.

1. вызов функции
2. создание таймер срабатывания
2.1. досрочно завершение и выполнение функции расчетов
2.2. завершение таймера и выполнение функции расчетов

[xmoonlight]

dorohdds, Вариант1: вначале собрать данные, а потом - посчитать, не подходит?
Вариант2: проверять повторные вызовы и игнорировать, если таймер существует.

[xmoonlight]

dorohdds, (Bavashi - наш местный кислотник-новичок, админы - никак не реагируют)

[dorohdds]

xmoonlight, а разве сейчас, не так происходит, как вы описали в варианте 1?
Да и вариант, который вы предложили позволяет решать задачу, но повторюсь. Меня волнует, что при событие "finish" оно отрабатывается во всех экземплярах экспортированной функции и пытается отработать.
Можно полностью обернуть m1.on('finish'), чтобы все что в нем срабатывало 1 раз, а потом игнорировалось. Но при каждом таком событие будет это проверяться. Возможно, это не критично.. Но как-то не правило.

Хотелось бы после выполнения функции ничего не оставалось. Грубо говоря, если до инициализации и вызова(ниже будет код) будет эммитировано событие finish абсолютно ничего не произойдет. Вот к такому и хотелось бы придти

let commandFile = require(`./commands/${cmd}.js`)
commandFile.run()

[xmoonlight]

dorohdds, Вы ждёте завершения вызова, а не таймера.
Также, можно в таймер передавать свои параметры: }, 1000*5, myvar1, myvar2, ...)
Возможно стоит пересмотреть логику и передавать в таймер каунтер, а мониторить - завершение таймера, а не блока?

[dorohdds]

xmoonlight, Да! Скорее всего хочу закончить вызов функции. Но ретурн нигде не возвращает и не прекращает выполнение...
За время работы таймера идёт сбор данных и передать таймеру особо и нечего
Не понял про вторую часть.

[xmoonlight]

dorohdds, вот мысль пришла:
В calculate() добавляем параметр сброса таймера и если нужно сбросить - сбрасываем где угодно.
Пример:

exports.run = (m1) => {
     var timer1 = setTimeout(function () {
        calculate(timer1)        
    }, 1000*5)
    m1.on('finish', function () {
        console.log(`таймер сброшен`);        
        clearTimeout(timer1)
        calculate() // зачем тут ещё раз вызов?
    })
function calculate(t) { 
    // тут некоторые действия
    clearTimeout(t);
}

[dorohdds]

xmoonlight, Для начала спасибо огромное, что тратите время и пытаетесь помочь!
Не нашел способа сбросить таймер и выполнить код. При сбрасывании код внутри игнорится. Для этого и есть второй вызов внутри на случай окончания полного таймера. А чем код откчается от моего?) Похоже увел вас разговорами о таймере в другое русло..
таймер у меня сбрасывается. все ок.

Проблема вот в чем.. У меня не получается закончить выполнение этой экспортированной функции. она продолжает работать даже после расчетов. Не помогает ретурн. Пробовал различными способами. Она продолжает слушать события "finish"

exports.run = (m1) => {
console.log(`Запуск таймера сбора данных`)
     var timer1 = setTimeout(function () {
        console.log(`Таймер закончился`)
        calculate()        
    }, 1000*5)
    m1.on('finish', function () {
        console.log(`таймер сброшен`);        
        clearTimeout(timer1)
        calculate()
    })
function calculate() { 
    console.log(`Расчет`)
    // тут некоторые действия
}

Вывод в консоль:

Первый вызов:
Вызов команды
Запуск таймера сбора данных
таймер сброшен
Расчет

Второй вызов:


Вызов команды
Запуск таймера сбора данных
таймер сброшен
Расчет
таймер сброшен
Расчет

Даже и если после первого запуска попытаться сбросить таймер все из m1.on('finish')отработает.

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

[xmoonlight]

dorohdds, а если засунуть

m1.on('finish', function () {
        console.log(`таймер сброшен`);        
        clearTimeout(timer1)
        calculate()
    })

внутрь функции таймера, то при уничтожении, теоретически, должен убиваться и его листенер...

var timer1 = setTimeout(function () {
    m1.on('finish', function () {
        console.log(`таймер сброшен`);        
        clearTimeout(timer1)
        calculate()
    })
        console.log(`Таймер закончился`)
        calculate()
    }, 1000*5)