如何正确处理异步并发请求?

时间:2014-08-01 11:38:56

标签: node.js express concurrency sails.js

假设我有某种游戏。我有一个像这样的buyItem函数:

buyItem: function (req, res) {
    // query the users balance
    // deduct user balance
    // buy the item
}

如果我在扣除用户余额之前发送该路由(第二次查询),则用户的余额仍为正数。

我尝试过:

buyItem: function (req, res) {
    if(req.session.user.busy) return false;
    req.session.user.busy = true;
    // query the users balance
    // deduct user balance
    // buy the item
}

对于前5个请求,问题req.session.user.busy将是undefined。所以这也不起作用。

我们如何处理这种情况?我正在使用Sails.JS框架,如果这很重要的话。

2 个答案:

答案 0 :(得分:14)

  

更新2

     

通过transaction support方法,Sails 1.0现已完整.getDatastore()。例如:

// Get a reference to the default datastore, and start a transaction.
await sails.getDatastore().transaction(async (db, proceed)=> {
  // Now that we have a connection instance in `db`, pass it to Waterline
  // methods using `.usingConnection()` to make them part of the transaction:
  await BankAccount.update({ balance: 5000 }).usingConnection(db);
  // If an error is thrown, the transaction will be rolled back.
  // Or, you can catch errors yourself and call `proceed(err)`.
  // To commit the transaction, call `proceed()`
  return proceed();
  // You can also return a result with `proceed(null, result)`.
});
  

更新

     

正如一些评论者所指出的那样,启用connection pooling时,以下代码无效。在最初发布时,并非默认情况下所有适配器都已合并,但此时应假设它们确实存在,以便每个单独的方法调用(.query().findOne()等。)可以在不同的连接上,并在交易之外操作。 Waterline的下一个主要版本将提供事务支持,但在此之前,确保查询是事务性的唯一方法是使用原始数据库驱动程序包(例如pgmysql)。

听起来你需要的是transaction。 Sails不支持框架级别的事务(它在路线图上),但如果您使用的是支持它们的数据库(如Postgres或MySQL),则可以使用模型的.query()方法访问底层适配器并运行native commands。这是一个例子:

buyItem: function(req, res) {
  try {
    // Start the transaction
    User.query("BEGIN", function(err) {
      if (err) {throw new Error(err);}
      // Find the user
      User.findOne(req.param("userId").exec(function(err, user) {
        if (err) {throw new Error(err);}
        // Update the user balance
        user.balance = user.balance - req.param("itemCost");
        // Save the user
        user.save(function(err) {
          if (err) {throw new Error(err);}
          // Commit the transaction
          User.query("COMMIT", function(err) {
            if (err) {throw new Error(err);}
            // Display the updated user
            res.json(user);
          });
        });
      });
    });
  } 
  // If there are any problems, roll back the transaction
  catch(e) {
    User.query("ROLLBACK", function(err) {
      // The rollback failed--Catastrophic error!
      if (err) {return res.serverError(err);}
      // Return the error that resulted in the rollback
      return res.serverError(e);
    });
  }
}

答案 1 :(得分:0)

我没有测试过这个。但只要您不使用多个实例或群集,只能将状态存储在内存中。因为节点是单线程的,所以原子性不应该有任何问题。

var inProgress = {};

function buyItem(req, res) {
    if (inProgress[req.session.user.id]) {
        // send error response
        return;
    }

    inProgress[req.session.user.id] = true;

    // or whatever the function is..
    req.session.user.subtractBalance(10.00, function(err, success) {
        delete inProgress[req.session.user.id];

        // send success response
    });
}