使用nodejs + express处理服务器端和客户端错误的最佳方法是什么

时间:2019-06-26 12:22:22

标签: javascript node.js reactjs express try-catch

我想知道处理响应中的错误的最佳方法-请求。 我有一条收到请求的路线:

app.get('/getInfo', function (req, res, next) {
    let obj = {}
    try {
        obj = { 
            ...                
            date: lastUpdatedDate('./utils/appVersion.js'),
            ...
        }
        res.status(200).send(obj)
    } catch (error) {
        console.log(error.message)
        res.send({error: "The data wasn't load"})
    }       
})

此函数用于发出请求

getInfo () {
    axios.get(process.env.REACT_APP_HOST + '/getInfo')
      .then(resp => {
        this.appInfoHandler(resp.data)
      })
      .catch(function (error) {    
        console.log(error)
      })
  }

如果错误发生在服务器端,最好的解决方法是什么? 让我们假设该代码块中的目录不存在:lastUpdatedDate('./directoreyDoesntExists/appVersion.js'),

因此,我的代码转到了catch块。

我应该发送这样的错误:

res.send({error: "The data wasn't load"})

我应该设置这样的状态吗?

  res.status(500).send({error: "The data wasn't load"})

还是应该使用其他状态代码设置状态?

基于此,在前端方法getInfo()中处理错误并在Web界面上显示错误消息的最佳方法是什么?

我应该像这样在if else块内做一个.then吗?

 getInfo () {
      axios.get(process.env.REACT_APP_HOST + '/getInfo')
      .then(resp => {
            if(resp.status === 200){
                 this.appInfoHandler(resp.data)
            }else if (resp.status === 400){
                  //print error message on web interface
            }else if (resp.status === 500){
                  //print error message on web interface
          })
          .catch(function (error) {    
            console.log(error)
          })

还是应该像这样直接在catch块中处理此错误

getInfo () {
    axios.get(process.env.REACT_APP_HOST + '/getInfo')
      .then(resp => {
        this.appInfoHandler(resp.data)
      })
      .catch(function (error) {    
        //print error message on web interface
      })
  }

4 个答案:

答案 0 :(得分:4)

在这种情况下

res.send({error: "The data wasn't load"}) 

vs

res.status(500).send({error: "The data wasn't load"})

发送状态只是更详细,但都可以。 检查Proper way to set response status and JSON content

在这种情况下,取决于您的需求

then(resp => {
            if(resp.status === 200){
                 this.appInfoHandler(resp.data)
            }else if (resp.status === 400){
                  //print error message on web interface
            }else if (resp.status === 500){
                  //print error message on web interface
          })
          .catch(function (error) {    
            console.log(error)
          })

vs

getInfo () {
    axios.get(process.env.REACT_APP_HOST + '/getInfo')
      .then(resp => {
        this.appInfoHandler(resp.data)
      })
      .catch(function (error) {    
        //print error message on web interface
      })
  }

您可以处理将其发送到catch块的所有错误

else if (resp.status === 400){
                  //print error message on web interface

不在此处打印错误,而是抛出一个新错误,将其发送到catch块

throw new ApiError("UserNotFount",400,"not found");
throw new Error('Error 400, not found');

答案 1 :(得分:3)

在这种情况下

res.send({error: "The data wasn't load"}) 

vs

res.status(500).send({error: "The data wasn't load"})

我建议发送错误以及状态代码,因为这将对客户端更具描述性。

第二种情况

getInfo () {
      axios.get(process.env.REACT_APP_HOST + '/getInfo')
      .then(resp => {
            if(resp.status === 200){
                 this.appInfoHandler(resp.data)
            }else if (resp.status === 400){
                  //print error message on web interface
            }else if (resp.status === 500){
                  //print error message on web interface
          })
          .catch(function (error) {    
            console.log(error)
          })

vs

getInfo () {
    axios.get(process.env.REACT_APP_HOST + '/getInfo')
      .then(resp => {
        this.appInfoHandler(resp.data)
      })
      .catch(function (error) {    
        //print error message on web interface
      })
  }

在这种情况下,我建议您在遇到错误时直接使用catch块,因为响应状态取决于错误,而不是相反的情况

答案 2 :(得分:2)

除200以外的任何状态码均表示未成功,因此您无需使用这些if-else语句。更好的选择是捕获错误并按原样发送并发送响应。这样做的好处是,您无需对状态码进行硬编码就可以收到发生的错误类型。 (例如,我们将此处的状态代码设为400不成功)

 .catch(function (error) {    
        //print error message on web interface
        res.status(400).send(JSON.stringify(error, undefined, 2));
      });

通过使用stringify方法,您还可以在控制台上打印确切的错误。

.catch(function (error) {    
            console.log(JSON.stringify(error, undefined, 2));
          });

这里的stringify方法中的参数是:

  1. 错误对象

  2. 未定义:包含用于过滤对象中键的键的数组(此处为error)。该数组中存在的所有那些键只是未被过滤掉的那些键。

  3. 2:用于在对象表示中引入空格

答案 3 :(得分:2)

作为从事REST Api的初学者,您应该看看准则-微软的合法性是:https://docs.microsoft.com/en-us/azure/architecture/best-practices/api-design

基本上,您需要为每个请求返回正确的HTTP代码,并查看https://http.cat/-例如,如果请求格式错误,则返回400,如果用户未经授权,则返回401:

if (!req.body.name) {
  res.status(400).send({ error: 'missing user name' }); // 400 bad request
}
const user = getUser(req.body.name, req.body.pass);

if(!user) {
  res.status(401).send({ error: 'user does not exist' }); // 401 unauthorized
}

try {
  const token = createToken(user);
  // better to set a cookie
  res.status(200).send({ token }); // 200 success
} catch(e) {
  res.status(500).send({ erroe: e.message }); // 500 internal error
}

if(isTeapot) {
  res.status(418).send({ error: 'I can only make tea' }); // 418 teapot, totally real
}

为了使事情变得更容易,有很多库可以帮助您生成更好的错误消息并更好地处理错误,我最喜欢的一个是celebrate