优化Node.js

时间:2017-05-24 09:25:26

标签: javascript json node.js api

我在Node.js中有一个API。我的路线看起来像

exports.getUserList = (req, res, next) => {
  User.find().sort({ name: 1 }).then(users => {
    res.json({
      status: 'success',
      users
    });
  }).catch(next);
};

如示例中所示,我使用.catch(next)。但这是正确的方法吗?路线不应该总是打印json吗?

所以我想做像

这样的事情
exports.getUserList = (req, res, next) => {
  User.find().sort({ name: 1 }).then(users => {
    res.json({
      status: 'success',
      users
    });
  }).catch(err => {
    res.json({
      status: 'error',
      msg: err
    });
  });
};

但不应该像res.status(some_status_code).json({})那样?

如何在错误处理方面执行简单的API?

如果我在代码中使用未定义的变量(即导致语法错误),该怎么办?我应该用JSON错误处理它还是应该确保我不做马虎编码? :-D

另外,这是打印json的最快方式吗?我的意思是,我应该使用User.find().lean()吗?我应该做一些缓存吗?是否更聪明地将我的API存储在普通网站上,或者是否存在针对此类情况的优化API服务器?

1 个答案:

答案 0 :(得分:0)

您是否尝试过异步/等待功能和自定义响应以获取成功和错误? 这里的例子:

responseHandler.js

// use this for success
function successResponse(message, data) {
  const success = {
    success: true,
    message: message,
    data: data
  }
  return success
}

// use this as middleware error handler
function errorResponse(err,req,res,next) => {
  return res.status(err.status || 500).json({
    success: false,
    message: err.message
  })
}

module.exports = {
  successResponse,
  errorResponse
}

myRouter.js

const { successResponse } = require('./responseHandler')

exports.getUserList = async (req, res, next) => {
  await User.find().sort({ name: 1 }).then(users => {
    res.status(200).json(
      successResponse(`data user`, users)
    )
  }).catch(err => {
    return err
  });
};