【问题标题】:Simulate error handling with promises on router.js在 router.js 上使用 Promise 模拟错误处理
【发布时间】:2023-03-29 07:00:01
【问题描述】:

我正在尝试将传感器值保存到我的 mongodb 数据库中。我正在使用 promise 来进行正确的错误处理。我正在使用then()catch(),但我想我做得不对。我发送一个组 ID,以便它检查组 ID 是否存在,然后只存储传感器信息。 这是我的代码。

router.post('/data/:group_id', function(req,res,next){
    var group_id = req.params.group_id;
    User.find({group_id : group_id}).then(function(user){
        var object_id = user[0]._id;
        var datas = req.body;
        var data = new Data({
           user : object_id,
            value: datas.value,
            valueString : datas.valueString,
            sensorStatus : datas.sensorStatus,
            timeStamp : new Date().toJSON()
        });
        data.save().then(function(data){
            res.send('Data Saved');
        });
    }).catch(function(e){
        if(e.code === undefined){
            res.send('Group id does not exist');
        }else{
            res.send(e);
        }
    });
});

如果我发送一个不存在的组 ID,它确实说组 ID 不存在,但如果我发送错误的 json 数据来保存我的传感器信息,则没有错误,但我正在使用的邮递员应用程序发送信息卡住了。这是我故意使用有效组 ID 生成的错误。

我是新手,所以需要了解promise如何处理错误,首先如果找不到group_id,其次如果传感器json信息错误。

发送至http://localhost:3000/api/data/3

{
    "value1" : "79" ,
    "valueString" : "Small data",
     "sensorStatus" : "false"
}

额外信息用户和数据模型。注意(添加require和export部分)

var schema = new Schema({
    group_id : {type : Number, required: true},
    password : {type : String, default: null},
    project : {type : String , default : null}
});
var schema = new Schema({
    user: {type: Schema.Types.ObjectId, ref: 'User'},
    value :{type : Number , required : true} ,
    valueString : {type : String, required : true},
    sensorStatus : {type : Boolean , default : 0},
    timeStamp  : {type :String , required : true}
});

【问题讨论】:

  • @chridam 是的,当然!
  • 离题:我建议将 timeStamp 数据类型更改为 Date 并在应用程序层中转换为字符串,因为 Mongo 最适合使用日期类型存储的日期。问题可能与timeStamp : new Date().toJSON()这一行有关,我没有测试过,不太确定。
  • @chridam 我对此不太有信心,但我可以看到当我提供错误的 json 文件时,我认为我的错误处理是错误的。即使时间戳确实导致错误,它也应该捕获并显示它对吗?希望我说得通
  • data.save() 返回的promise 没有错误处理程序,我认为在这种情况下您需要返回promise,然后在第一个promise 上添加另一个then() 将发送响应,希望你能明白我的意思。
  • @chridam 我应该这样做吗? if(user == undefined || null){ throw new Error('Unable to find group id ');} 所以 catch 只会处理存储数据方

标签: javascript node.js mongodb promise


【解决方案1】:

是的,你陷入了一个常见的陷阱:不要嵌套 .then() 子句,而是返回 Promises 和链:

别这样

foo().then(() => {
  bar().then(() => {
    baz();
  }
}

.then() 和链内部返回 Promise,如下所示:

foo()
  .then(() => { return bar(); })
  .then(() => { return baz(); });

或更短的版本

foo()
  .then(() => bar())
  .then(() => baz());

在你的情况下:

router.post('/data/:group_id', function(req,res,next){
  var group_id = req.params.group_id;
  User.find({group_id : group_id})
    .then(function(user){
      // bla bla bla
      return data.save();
    })
    .then(function(data){
      return res.send('Data Saved');
    });
    .catch(function(e){
      if(e.code === undefined){
          res.send('Group id does not exist');
      }else{
          res.send(e);
      }
    });
});

一般规则:始终从.then() 处理程序returnthrow

【讨论】:

  • 非常感谢!它可以工作,但是有一个小问题,当组 id 错误时 (e) 返回 {} 我什至不知道如何告诉用户组 id 错误。无论如何要正确捕获该错误吗?
  • @MasnadNihit - 这不是if(e.code === undefined){ 在做什么吗?
  • @JaromandaX 由于某种原因,一个空数组不等于未定义。
  • 你说 "当组 id 错误时 (e) 返回 {}" ...如果 e{} 那么 e.code 不是一个空数组[]我可以向你保证
猜你喜欢
  • 2014-06-21
  • 2015-06-19
  • 2014-12-15
  • 2016-09-29
  • 1970-01-01
  • 2019-02-17
  • 1970-01-01
  • 2021-06-05
  • 2017-10-26
相关资源
最近更新 更多