Простая блокировка, которая возвращает ложь для одновременных вызовов - javascript - PullRequest
Новая
       14

Простая блокировка, которая возвращает ложь для одновременных вызовов - javascript

5 голосов
/ 09 апреля 2020

Я хочу заблокировать функцию asyn c от одновременного выполнения. Функция может быть выполнена только одним вызывающим абонентом, и она должна получить значение 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
    }
}

Но похоже, что это не работает, как ожидалось. Может кто-нибудь помочь мне улучшить это?

1 Ответ

4 голосов
/ 13 апреля 2020

Выполнение обещания не всегда гарантированно прекращается сразу после разрешения или отклонения; вам придется вернуть логическое значение после того, как вы проверите блокировку и выйдите или продолжите обещание соответствующим образом.

Запустите приведенный ниже фрагмент кода; Это пример с 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>
...