web-dev-qa-db-ja.com

ノードjs eventemitterでイベントリスナーを適切に削除する方法

GETとPOSTルートを設定しました。ルートへのPOSTはイ​​ベントをトリガーし、GETルートはPOSTが実行されるたびに起動するサーバー送信イベントストリームであるという考えですイベントがトリガーされます...しかし、イベントストリームサブスクライバが1つしかなくても、イベントリスナーが定期的に追加されるように見えるため、何か間違ったことをしていると思います...何が間違っていますか？

var events = require('events'),
EventEmitter = events.EventEmitter,
rr = new EventEmitter();

app.post('/api/:boardname/remoterefresh', function(req, res){
    var boardname = req.param('boardname'),
    data = new Date().getTime();
    rr.emit("refresh-"+boardname, data)
    res.json({data: data})
});

app.get('/api/:boardname/remoterefresh', function(req, res){
    var boardname = req.param('boardname')

    rr.on("refresh-"+boardname, function(data){
        setTimeout(function(){
            res.write('data: '+data+'\n\n');
        }, 1000)
    });

    req.socket.setTimeout(Infinity);

    res.writeHead(200, {
        'Content-Type': 'text/event-stream',
        'Cache-Control': 'no-cache',
        'Connection': 'keep-alive'
    });

    res.write('\n');

    req.on('close', function(){
        console.log('closed')
        rr.removeListener("refresh-"+boardname, function(){
            //
        })
    })

})
javascriptnode.jsexpresseventemitterevent-stream
28
coffeedoughnuts

イベントハンドラとしてアタッチする関数に名前を付ける必要があります。それを削除したら、関数を名前で渡すだけです：

app.get('/api/:boardname/remoterefresh', function(req, res){
    var boardname = req.param('boardname')
    function refreshHandler(data){
        setTimeout(function(){
            res.write('data: '+data+'\n\n');
        }, 1000)
    }
    rr.on("refresh-"+boardname, refreshHandler);

    req.socket.setTimeout(Infinity);

    res.writeHead(200, {
        'Content-Type': 'text/event-stream',
        'Cache-Control': 'no-cache',
        'Connection': 'keep-alive'
    });

    res.write('\n');

    req.on('close', function(){
        console.log('closed')
        rr.removeListener("refresh-"+boardname, refreshHandler);
    });
});

基本的に、removeListenerは、指定された関数を参照で検索します。その関数が見つかった場合、イベントハンドラーから削除します。

40
Farid Nouri Neshat

または、次のように危険な方法ですべてを破棄することもできます。

rr.removeAllListener(["refresh-"+boardname])

匿名リスナーコールバック関数を参照せずに。

0
goodhyun