javascript - 我的 NodeJS promise 代码竞争条件安全吗？

Question

我目前正在用 nodejs 编写一个小型多人游戏。

当玩家加入时，我想将他们加载到游戏世界中他们要求的位置。但是，这个位置可能已被占用，如果是这样，我会四处寻找最近的可用空间并将它们加载到该位置。

我的做法如下:

    function playerJoin(){    
        let playerTank = new Tank();
        this.findEmptyArea(playerTank, 100, 100).then((result) => {
            if (!result.success) {
                return;
            }
            addObjectToWorld(playerTank, result.x, result.y);    
        } 
    }


    function findEmptyArea(object, x, y){   
        return new Promise((resolve, reject) => {
          // Psuedo code: code iterates over game objects testing for 
          // collisions . if the requested position is free uses that else 
          // finds an empty nearby location and returns it
          return resolve({success: true, x: freeX, y: freeY});

          // fails to find empty location for object
          return resolve({ success: false });
        } 
  }

这不是实际代码，而是一个精简版本，以使其更清晰。我的问题是:

当用户通过网络套接字连接时，函数 playerJoin 运行。然后它创建一个新的 player tank ，找到空闲区域并返回一个 promise，如果成功则 player tank 被添加到世界的位置。

仔细查看后，我想知道这段代码是否存在缺陷。 addObjectToWorld 是否有可能在实际上不是空闲的位置上被调用？

例如:

1. player1 连接为 player1 调用的 playerJoin
2. 在 playerJoin 中调用了 player1 的 findEmptyArea
3. player 2 连接 playerJoin 调用 player2
4. 在 playerJoin 中为 player2 调用的 findEmptyArea
5. findEmptyArea promise for player1 在 10,10 找到一个空闲空间并且返回 promise 。

6. findEmptyArea promise 为 player2 找到一个免费的10,10 处的空格并返回 promise。

7. .then() Promise 代码块在 playerJoin 中(在 findEmptyArea 之后)让玩家 1 运行，将玩家置于 10,10 通过添加对象到世界

8. playerJoin 中的 .then() Promise 代码块玩家 2 运行，通过 addObjectToWorld 将玩家置于 10,10..并且游戏崩溃了

所以我想我的问题是，当一个 promise 解决时，.then 代码块会立即运行吗？运行 addObjectToWorld 的代码块会立即运行，还是其他代码可能会首先运行(例如另一个玩家也在寻找空闲区域)

谢谢你的帮助

Answer 1

因为 findEmptyArea 是异步的，所以它很可能访问外部的东西，对吧？

Node.js 是用于执行 JS 代码的单线程(尽管在最新版本中有实验性的多线程功能)。但是 Node.JS 使用子线程来访问外部资源。因此，只有当您访问文件等外部资源或执行 API 请求时，您才会出现竞争条件。

如果您执行 API 请求，您访问的资源将负责确保它仅返回一次 true 以防止竞争条件。

但是，如果您只检查 Promises 中的本地对象，您就不应该有任何竞争条件(只要您不使用 setTimeout 和类似的东西)。 Node.JS 事件循环将一次执行每个已解决/拒绝的 Promise 的代码。这意味着一旦 Promise 被解决，then 代码块就会被执行。

我找到了 this article结合 Promises 对事件队列有帮助。

您问题的答案取决于 findEmptyArea 检查空白区域的方式。

一些编码建议:

调用resolve 和reject 时不需要使用return。你可能想使用 reject 因为你提到了变量:

this.findEmptyArea(playerTank, 100, 100).then((result) => {
  addObjectToWorld(playerTank, result.x, result.y);    
}).catch((err) => {
  // findEmptyArea failed because the Promise was "rejected"
});

return new Promise((resolve, reject) => {
  // On success call this
  resolve({x: freeX, y: freeY});

  // On failure call this
  reject();
});