javascript - 可能的 javascript ajax 调用竞争条件

Question

所以我在 javascript 中实现了这个功能:当用户打开文档时，它的数据每 10 秒保存一次，当用户关闭时，它会再保存一次。它看起来很好用。这是实现:

var data = "blabla";
var saveagain = false;
var t1;

function onDocumentOpen() {
    saveagain = true;
    savedata();
}

function onDocumentClose() {
    saveagain = false;
    savedata();
}

function savedata() {
    if (!saveagain) clearTimeout(t1);
    SaveDataAjaxCall(data, function() {
        //data was saved
        if (saveagain) t1 = setTimeout("savedata()", 10000);
    });
}

我想知道我的方法是否正确，是否会在极端情况下导致某些可能的竞争条件，例如:

当从 onDocumentClose() 调用的 savedata() 实例在 if(!saveagain) 步骤之后时，从 setTimeout() 的前一个计时器调用的 savedata() 实例在该步骤之前，因此它会被调用再一次。这或任何更奇怪的事情会发生吗？

提前致谢

编辑:

考虑到 T.J. Crowder 和 Bengi 的评论我最终确定了代码:

var data = "";
var saveagain = false;
var t1;
var beingsaved = false;

function onDocumentOpen() {
    saveagain = true;
    savedata();
}

function onDocumentClose() {
    saveagain = false;
    savedata();
}

function saveData() {
    if (beingsaved) {
        if (!saveagain) setTimeout(saveData, 100);
        return false;
    }
    beingsaved=true;

    if (!saveagain) clearTimeout(t1);

    data=getData();

    SaveDataAjaxCall(data, function() {
        //data was saved
        beingsaved=false;
        if (saveagain) t1 = setTimeout(saveData, 10000);
    });

}

我想我现在已经处理好每一个场合了。我认为 beingsaved 解决方案等于 T.J Crowder 建议的原子计数器。

EDIT2:嗯，我不确定我是否解决了它，因为可能存在这样一种情况，即在 onDocumentClose 调用将 beingsaved 设置为 true 之前，if(beingsaved) 被 setTimeout 调用评估。这会发生吗？

Answer 1

我假设您的保存操作是异步的，因为所有好的 ajax 操作都应该是异步的。如果是这样，你会想在它周围设置某种保护条件，这样两个保存触发器就不会重叠。或者，当然，允许它们重叠但在服务器端处理(可能使用序列号或时间戳)，如果已经提交了较晚的保存，它会忽略较早的保存。