Sequelize findOrCreate() -- .catch() 块在没有错误期间在 spread() 之后执行

Question

我的错误类似于这个post:

使用 sequelize.js 作为 ORM 处理程序。 queryCondition 变量值：{where: {$or: [{username:user.username},{email:user.email}]},defaults:user}

问题：成功将用户实例保存到数据库后，调用findOrCreate() 后按推荐调用spread()。 spread() 中的功能块被调用，然后在.catch() 块被调用之后立即被调用。不知道为什么会这样。有人可以帮忙吗？

 model.User.findOrCreate(queryCondition).spread(function(user, created){
    callback && callback(user, created);
 }).catch(function(error){
     callback && callback(null, null, error);
 });

Answer 1

找到答案： Sequelize 使用 Bluebird.js 作为它的 promise 库（查看 package.json 文件）。 documentation 解释了 spread() 方法，推荐在 sequelize 3.0 中使用（参见 post）。本文档中的一个部分指出，spread() 在幕后调用 all().then(..)。因此，以下更改使用 Node.js 解决了我的问题：

model.User.findOrCreate(queryCondition) .all().then(function(result, isCreated){ callback && callback(null, result, isCreated); }, function(error) { callback && callback(error); });

正确解析的 Promise 不会落入错误块中，并且实际的数据库错误会被捕获在错误块中。最后，对于那些认为错误是由于回调参数引起的人，请注意，我只是根据下面 num8er 的建议对回调参数进行了重新排序。 :) 希望这对其他人有所帮助。

Answer 2

如果它正在调用.catch 块，那么它似乎有一些问题。

尝试调试一下，看看原因：

 model.User.findOrCreate(queryCondition).spread(function(user, created){
   callback && callback(user, created);
 }).catch(function(error){
   console.error(queryCondition, error); // add this and check the output
   callback && callback(null, null, error);
 });

我还看到您的 queryCondition 变量也可以。

{
  where:{$or: [{username:user.username},{email:user.email}]},
  defaults:user
}

让我们试试这个注解：

{
  where:{$or: [{username: {$eq: user.username}},{email: {$eq: user.email}}]},
  defaults:user
} 

阅读：http://docs.sequelizejs.com/en/latest/docs/querying/#where

如果使用另一个注释，它将无法正常工作，因此我建议您检查表列中的 null、not null 字段并将其与 user 对象进行比较，以查看 @987654329 中是否缺少字段@ 目的。


在阅读了 Bergi 的评论后，我也建议您尝试这样做：

 model.User
   .findOrCreate(queryCondition)
   .then(
     function(result, isCreated){
       callback && callback(result, isCreated);
     },
     function(error) {
       callback && callback(null, null, error);
     });

如果是spread 方法，我认为它不会抛出可捕获的异常，所以在这种情况下：

model.User
  .findOrCreate(queryCondition)
  .spread(function(result, isCreated){
    if(!result) {
      return callback && callback('Cannot create user');
    }
    callback && callback(null, result, isCreated);
  });

附：通常在大多数包中，将错误作为回调的第一个参数返回是惯例。所以我只能猜测这个问题不会发生在你的代码示例中，它发生在等待回调的某个地方。

所以尝试像这样修改回调：

callback(user, created);  =>  callback(null, user, created);

callback(null, null, error);  =>  callback(error);