对并发调用返回 false 的简单锁 - javascript答案

【问题标题】：Simple Lock That returns false for concurrent calls - javascript对并发调用返回 false 的简单锁 - javascript
【发布时间】：2020-04-26 21:58:27
【问题描述】：

我想锁定异步函数以防止同时执行。该函数只能由一个调用者执行，并且它应该在执行后得到 true 作为输出，而其他人得到 false 作为返回而不执行函数的逻辑内容。其他人不应等待当前调用者完成。

这是我迄今为止尝试过的：

class Lock {
    constructor() {
        this.locked = false;
    }
    lock(resolve) {
        if (this.locked) {
            resolve(false)
        }
        this.locked = true
        return
    }
    release(resolve) {
        this.locked = false;
        resolve(true)
    }


}

let lock = new Lock();

function myFunction() {
    return new Promise(resolve => {
        lock.lock(resolve);
        //do something - this takes time includes some api calls and db operations
        lock.release(resolve);
    })
}

async function callSite() {
    const executed = await myFunction();
    if(executed){
        //do something
    }
    else{
        //do another thing
    }
}

但似乎它没有按预期工作。谁能帮我改进一下？

【问题讨论】：

您期待什么，实际发生了什么？
当我快速调用这个调用站点函数时，它总是返回 true。为了测试，我在 lock.lock 和 then(lock.release) 之后使用了 fetch 函数。
它应该在未完成提取时挂断电话。
尝试const executed = myFunction(); - 删除await
另外，在lock类方法中，将最后一个命令return;替换为resolve (true);

标签： javascript async-await locking es6-promise

【解决方案1】：

承诺执行并不总是保证在解决或拒绝后立即停止；在检查锁并退出或相应地继续承诺后，您必须返回一个布尔值。

运行下面的sn-p；这是一个具有 3 个具有相同承诺和 3 秒延迟的按钮的示例。当一个按钮在运行时，其他的都不能执行。

class Lock {
  constructor() {
    this.locked = false;
  }

  lock(resolve) {
    if (this.locked) {
      resolve(false);
      return false;
    }
    this.locked = true
    return true;
  }

  release(resolve) {
    this.locked = false;
      resolve(true)
  }
}

let lock = new Lock();

function myFunction({ resultEl }) {
  resultEl.textContent = "Is running";
    
  return new Promise(resolve => {
    // Check if it's locked
    if (!lock.lock(resolve)) {
      // If it is, return and exit the function even if the promise is already resolved
      return;
    }

    // do something - this takes time includes some api calls and db operations
    // just wait 3 seconds for demontration
    setTimeout(() => {
      lock.release(resolve);
    }, 3000);
  })
}

async function callSite() {
  this.disabled = true;

  const executed = await myFunction({ resultEl : this.nextSibling });
    
  this.disabled = false;

  if (executed) {
    this.nextSibling.textContent = `Finished at ${(new Date).getTime()}`;
  }
  else {
    this.nextSibling.textContent = `Was NOT executed at ${(new Date).getTime()}`;
  }
}

document.getElementById('a-button').addEventListener('click', callSite);
document.getElementById('b-button').addEventListener('click', callSite);
document.getElementById('c-button').addEventListener('click', callSite);

div {
  margin-bottom: 10px;
}

button {
  margin-right: 5px;
}

<div>
  <button id="a-button">Run A</button><span></span>
</div>

<div>
  <button id="b-button">Run B</button><span></span>
</div>

<div>
  <button id="c-button">Run C</button><span></span>
</div>

【讨论】：