Question

我的express.js应用程序中有一个基本控制器。然后，我尝试执行某些服务，该服务会在短暂延迟后获取模拟数据。该服务继承自EventEmitter，在接收到数据后发出SUCCESS事件。

这是我的控制人：

const express = require('express');
const router = express.Router();
const GetAllUsers = require('../GetAllUsers');
const getAllUsers = new GetAllUsers();

router.get('/', function(req, res, next) {
  getAllUsers
    .on('SUCCESS', (users) => {
      res
        .status(200)
        .json({ users });
    })
    .on('ERROR', next);

  getAllUsers.execute();
});

module.exports = router;

服务：

const EventEmitter = require('events');

class GetAllUsers extends EventEmitter {
  async execute() {
    const data = [{ id: 1, name: 'user 1' }, { id: 2, name: 'user 2' }];

    try {
      const users = await new Promise(resolve => {
        setTimeout(() => {
          resolve(data);
        }, 1000);
      })

      this.emit('SUCCESS', users);
    } catch (error) {
      this.emit('ERROR', error);
    }
  }
}

module.exports = GetAllUsers;

问题是，当我第一次进入/users路径时，实际上得到了一个用户列表。但是当我第二次及以后尝试时，出现以下错误：

Cannot set headers after they are sent to the client
Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
    at ServerResponse.setHeader (_http_outgoing.js:471:11)
    at ServerResponse.header (/home/debian/dev/sandbox/emitterTest/node_modules/express/lib/response.js:767:10)
    at ServerResponse.send (/home/debian/dev/sandbox/emitterTest/node_modules/express/lib/response.js:170:12)
    at ServerResponse.json (/home/debian/dev/sandbox/emitterTest/node_modules/express/lib/response.js:267:15)
    at GetAllUsers.getAllUsers.on (/home/debian/dev/sandbox/emitterTest/routes/users.js:12:10)
    at GetAllUsers.emit (events.js:187:15)
    at GetAllUsers.execute (/home/debian/dev/sandbox/emitterTest/GetAllUsers.js:14:12)

据我了解，标头是在发送回复之前设置的。

当我尝试在没有发射器的情况下实现控制器时，一切正常：

router.get('/', async function(req, res, next) {
  const data = [{ id: 1, name: 'user 1' }, { id: 2, name: 'user 2' }];

  const users = await new Promise(resolve => {
    setTimeout(() => {
      resolve(data);
    }, 1000);
  })

  res
    .status(200)
    .json({ users });
});

该问题如何解决？

Answer 1

之所以会发生这种情况，是因为每次发出请求时，您都会运行路由器功能：

function(req, res, next) {
  getAllUsers 
    .on('SUCCESS', (users) => { //subscribe to event every time
      res
        .status(200)
        .json({ users });
    })
    .on('ERROR', next);

  getAllUsers.execute();
}

您已订阅这些事件并运行一次。当时，您提出了第二个请求，您获得了2个订阅者：从上一个请求到当前的订阅者。然后，execute()函数触发该事件。因此，一次执行2个功能，其中一个尝试再次发送响应，但这是不可能的，因此会导致错误。

如果确实需要这些事件发射器，则可以使用getAllUsers.once函数仅订阅一次fire事件。

首次请求后，事件发射器无法正常工作

1 个答案: