javascript - Javascript 中的线程安全？

Question

我有一个名为 save() 的函数，该函数收集页面上的所有输入，并对服务器执行 AJAX 调用以保存用户工作的状态。

save() 当前在用户点击保存按钮时被调用，或者执行一些其他需要我们在服务器上拥有最新状态的操作(例如从页面生成文档)。

我添加了每隔一段时间自动保存用户工作的功能。首先，我想防止自动保存和用户生成的保存同时运行。所以我们有以下代码(我削减了大部分代码，这不是 1:1，但应该足以让这个想法得到理解):

var isSaving=false;
var timeoutId;
var timeoutInterval=300000;
function save(showMsg)
{
  //Don't save if we are already saving.
  if (isSaving)
  { 
     return;
  }
  isSaving=true;
  //disables the autoSave timer so if we are saving via some other method
  //we won't kick off the timer.
  disableAutoSave();

  if (showMsg) { //show a saving popup}
  params=CollectParams();
  PerformCallBack(params,endSave,endSaveError);

}
function endSave()
{  
    isSaving=false;
    //hides popup if it's visible

    //Turns auto saving back on so we save x milliseconds after the last save.
    enableAutoSave();

} 
function endSaveError()
{
   alert("Ooops");
   endSave();
}
function enableAutoSave()
{
    timeoutId=setTimeOut(function(){save(false);},timeoutInterval);
}
function disableAutoSave()
{
    cancelTimeOut(timeoutId);
}

我的问题是这段代码是否安全？主流浏览器一次只允许一个线程执行吗？

我的一个想法是，如果我们是自动保存的，那么用户点击保存却得不到任何响应会更糟(而且我知道如何修改代码来处理这个问题)。有人在这里看到任何其他问题吗？

Answer 1

浏览器中的 JavaScript 是单线程的。您在任何时间点都只会参与一个功能。功能将在输入下一个功能之前完成。您可以依靠这种行为，因此如果您在 save() 函数中，在当前函数完成之前您将永远不会再次进入它。

当您有异步服务器请求(或 setTimeouts 或 setIntervals)时，这有时会让人感到困惑(但仍然如此)，因为那时感觉您的函数是 interleaved .他们不是。

在您的情况下，虽然两个 save() 调用不会相互重叠，但您的自动保存和用户保存可能背靠背发生。

如果您只想至少每 x 秒保存一次，您可以在保存函数上执行 setInterval 并忘记它。我认为不需要 isSaving 标志。

我认为您的代码可以大大简化:

var intervalTime = 300000;
var intervalId = setInterval("save('my message')", intervalTime);
function save(showMsg)
{
  if (showMsg) { //show a saving popup}
  params=CollectParams();
  PerformCallBack(params, endSave, endSaveError);

  // You could even reset your interval now that you know we just saved.
  // Of course, you'll need to know it was a successful save.
  // Doing this will prevent the user clicking save only to have another
  // save bump them in the face right away because an interval comes up.
  clearInterval(intervalId);
  intervalId = setInterval("save('my message')", intervalTime);
}

function endSave()
{
    // no need for this method
    alert("I'm done saving!");
}

function endSaveError()
{
   alert("Ooops");
   endSave();
}