Как остановить таймер?

Question

[XpeH Петрович]

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

const pool = [];

function Some() {
  this.id = uuid();
  this.timer = setInterval(() => {
    console.log('чем-то гажу в консоль');
  }, 1000 * 1);
  this.stop = () => {
    clearInterval(this.timer);
  };
}

var http = require('http');
var bodyParser = require('body-parser');

var express = require('express');
var app = express();

app.use(bodyParser.json());
var server = require('http').Server(app);

app.route("/")
  .post((req, res) => {
    let some = new Some();
    pool[some.id] = some;
    res.json({
      message: "created"
    });
  })
  .delete((req, res) => {
    let id = req.body.id;
    if (id in pool) {
      var some = pool[id];
      some.stop();
      delete pool[id];
      res.json({message: "deleted"});
    } else {
      res.json({message: "cannot delete"});
    }
  });
});

app.listen(8000, () => {
  console.log('listening on *:8000');
});

где бы я и как не пытался убить this.timer - он не убивается((( я так понял что что-то не так с контекстом вызова, но попроверял и вроде все ок. что я не так делаю и куда копать???;(

UPD.: наткнулся на тостере что каким-то макаром js умеет делать "скрытые" замыкания. Может у меня как раз тот случай?

Comments

[XpeH Петрович]

премного-премного извиняюсь. бессонная ночь дала о себе знать. поправил код, функция была не закрыта, сервер не запущен(

Answer 1

[XpeH Петрович]

UPD.: прикрепляю ссылку на pastebin
pastebin.com/Z4WbQQP1

Решение: надо было подробнее почитать про this. В общем, таймер прикреплялся не к тому объекту к которому я думал) всем спасибо

Answer 2

[Константин Китманов]

"Скрытые замыкания", ЕМНИП, это страшилка времен IE6-8.

Я упростил ваш код до такого и он работает:

function Some() {
    this.id = Math.floor(Math.random() * 1000);
    this.timer = setInterval(() => {
        console.log('чем-то гажу в консоль');
    }, 1000 * 1);
    this.stop = () => {
        console.log(this); // тут видно, что контекст действительно правильный
        clearInterval(this.timer);
    };
}

const some0 = new Some();

setTimeout(() => {
    some0.stop();
}, 3000);

Замечу, однако, что в вашем коде функция Some не закрыта и включает в себя app.route и прочее, то есть вы не рассказываете всей правды и понять, что же именно не так, довольно трудно.
Еще у вас pool это массив, а используется как объект. Плюс вы используете стрелочные функции, но эмулируете класс через стремные костыли.

Comments

[Stalker_RED]

Плюс const pool, а потом попытки его изменить.

[Константин Китманов]

Stalker_RED: это нормально, const в ES2015 означает иммутабельность ссылки, а не объекта по ссылке. Хотя code-style может это запрещать, криминала в этом нет и большинство так и пишет.

[XpeH Петрович]

Stalker_RED: почитайте про то как работают константы в js. immutable сам тип значения, но никто не мешает туда пихать mutable массив

[XpeH Петрович]

Константин Китманов app.route это express. остальное - нода. ничего не скрываю)

[Константин Китманов]

XpeH Петрович: Ну в текущем виде код просто нерабочий.

[Stalker_RED]

Константин Китманов: XpeH Петрович: вот оно что, а я уж думал константы завезли! :)

[Константин Китманов]

Stalker_RED: завезли. Нельзя сделать const pool = 12; pool = 42;. Просто они не такие, как, скажем в Haskell.

[XpeH Петрович]

Константин Китманов на драсьти нерабочий. у меня крутится на ноде 7.5.0 с флагом --harmony, экспресс последний stable, и все работает. не работает только эта фигня с таймаутом)
UPD.: последняя строчка лишняя, сорри

[XpeH Петрович]

Константин Китманов в смысле закрывающие две скобки последние, не заметил их

[Константин Китманов]

XpeH Петрович: вы уверены? Потому что так как он есть, этот код на каждый new Some импортит экспресс, создает приложение и вешает роуты. Но если вы уверены, что это именно то, чего вы добивались, то мне нечем вам помочь:)
Ну и кстати --harmony не нужен (а вот 'use strict'; не помешал был). Держите причесанный код:

'use strict';

const express = require('express');
const bodyParser = require('body-parser');
const uuid = require('uuid');

class Some {
    constructor() {
        this.id = uuid();
        
        this.timer = setInterval(() => {
            console.log('чем-то гажу в консоль');
        }, 1000 * 1)
    }
    
    stop() {
        clearInterval(this.timer);
    }
}

const pool = {};

const app = express();

app.use(bodyParser.json());

app.route('/')
    .post((req, res) => {
        const some = new Some();
        pool[some.id] = some;

        res.json({
            message: 'created'
        });
    })
    .delete((req, res) => {
        const id = req.body.id;

        if (id in pool) {
            pool[id].stop();
            delete pool[id];

            res.json({message: 'deleted'});
        } else {
            res.json({message: 'cannot delete'});
        }
    });

[XpeH Петрович]

Константин Китманов написал в комментарии к самому топику. извиняюсь( симптомы красноглазия налицо

[Константин Китманов]

XpeH Петрович: бывает) Отдыхайте чаще, перегорите — совсем работать не сможете.

[XpeH Петрович]

товарищи, а на тостере можно размещать код с pastebin'а например? чтобы прям здесь не писать. я бы тогда туда лучше скинул

[Константин Китманов]

XpeH Петрович: вроде только ссылкой.