避免javascript竞争条件

时间:2008-12-03 17:59:38

标签: javascript javascript-events race-condition

以下是该方案:

我的用户会看到一个网格,基本上是一个电子表格的精简版本。网格中的每一行都有文本框。当他们更改文本框中的值时,我正在对其输入执行验证,更新驱动网格的集合,并重新绘制页面上的小计。这全部由每个文本框的OnChange事件处理。

当他们点击“保存”按钮时,我正在使用按钮的OnClick事件对金额执行一些最终验证,然后将他们的整个输入发送到Web服务,保存它。

至少,如果他们将表单标签到“提交”按钮,就会发生这种情况。

问题是,如果他们输入一个值,那么立即单击保存按钮,SaveForm()在UserInputChanged()完成之前开始执行 - 一个竞争条件。我的代码不使用setTimeout,但是我用它来模拟缓慢的UserInputChanged验证码:

 <!-- snip -->
 <script>
    var amount = null;
    var currentControl = null;

    function UserInputChanged(control) {
        currentControl = control;
        // use setTimeout to simulate slow validation code (production code does not use setTimeout)
        setTimeout("ValidateAmount()", 100); 
    }

    function SaveForm() {
        // call web service to save value
        document.getElementById("SavedAmount").innerHTML = amount;
    }

    function ValidateAmount() {
        // various validationey functions here
        amount = currentControl.value; // save value to collection
        document.getElementById("Subtotal").innerHTML = amount; // update subtotals

    }
</script>
<!-- snip -->
Amount: <input type="text" id="UserInputValue" onchange="UserInputChanged(this);" /> <br />
Subtotal: <span id="Subtotal"></span> <br />
<input type="button" onclick="SaveForm();" value="Save" /> <br /><br />
Saved amount: <span id="SavedAmount"></span>
<!-- snip -->

我认为我不能加速验证代码 - 它非常轻量级,但显然,代码试图在验证完成之前调用Web服务的速度很慢。

在我的机器上,〜95ms是验证代码在保存代码开始之前执行之间的幻数。根据用户的计算机速度,这可能更高或更低。

有没有人有任何想法如何处理这种情况?同事建议在验证代码运行时使用信号量,并在保存代码中使用繁忙循环等待信号量解锁 - 但我想避免在代码中使用任何类型的繁忙循环。

7 个答案:

答案 0 :(得分:21)

使用信号量(让我们称之为StillNeedsValidating)。如果SaveForm函数看到StillNeedsValidating信号量已启动,请让它激活自己的第二个信号量(我在这里称之为FormNeedsSaving)并返回。验证函数完成后,如果FormNeedsSaving信号量已启动,它将自行调用SaveForm函数。

在jankcode;

function UserInputChanged(control) {
    StillNeedsValidating = true;
    // do validation
    StillNeedsValidating = false;
    if (FormNeedsSaving) saveForm(); 
}

function SaveForm() {
    if (StillNeedsValidating) { FormNeedsSaving=true; return; }
    // call web service to save value
    FormNeedsSaving = false;
}

答案 1 :(得分:9)

验证期间禁用保存按钮。 将验证设置为禁用,并在验证完成后重新启用它。

e.g。

function UserInputChanged(control) {
    // --> disable button here --< 
    currentControl = control;
    // use setTimeout to simulate slow validation code (production code does not use setTimeout)
    setTimeout("ValidateAmount()", 100); 
}

function ValidateAmount() {
    // various validationey functions here
    amount = currentControl.value; // save value to collection
    document.getElementById("Subtotal").innerHTML = amount; // update subtotals
    // --> enable button here if validation passes --<
}

你必须在删除setTimeout时调整并使验证成为一个函数,但除非你的用户有超人的反应,否则你应该好好去。

答案 2 :(得分:4)

我认为超时导致你的问题...如果那是普通代码(没有异步AJAX调用,超时等),那么我认为在UserInputChanged完成之前不会执行SaveForm。

答案 3 :(得分:2)

信号量或互斥量可能是最好的方法,但不是繁忙的循环,只需使用setTimeout()来模拟线程睡眠。像这样:

busy = false;

function UserInputChanged(control) {
    busy = true;
    currentControl = control;
    // use setTimeout to simulate slow validation code (production code does not use setTimeout)
    setTimeout("ValidateAmount()", 100); 
}

function SaveForm() {
    if(busy) 
    {
       setTimeout("SaveForm()", 10);
       return;
    }

    // call web service to save value
    document.getElementById("SavedAmount").innerHTML = amount;
}

function ValidateAmount() {
    // various validationey functions here
    amount = currentControl.value; // save value to collection
    document.getElementById("Subtotal").innerHTML = amount; // update subtotals
    busy = false;
}

答案 4 :(得分:0)

您可以设置一个循环函数来监视整个网格的状态,并引发一个事件,指示整个网格是否有效。

然后,您的“提交表单”按钮将根据该状态启用或禁用自身。

哦,我现在看到类似的反应 - 当然也有效。

答案 5 :(得分:0)

使用异步数据源时,您肯定会遇到竞争条件,因为JavaScript进程线程继续执行可能依赖于尚未从远程数据源返回的数据的指令。这就是为什么我们有回调函数。

在您的示例中,对验证代码的调用需要有一个回调函数,可以在验证返回时执行某些操作。

然而,当制作具有复杂逻辑的东西或试图排除或增强现有的一系列回调时,你可以疯了。

这就是我创建proto-q库的原因:http://code.google.com/p/proto-q/

如果你做了很多这类工作,请查看它。

答案 6 :(得分:-1)

你没有竞争条件,因为javascript是单线程的,所以javascript中不会发生竞争条件,因此2个线程不能相互干扰。

您提供的示例不是一个很好的例子。 setTimeout调用将把被调用的函数放入javascript引擎的队列中,稍后运行。如果此时单击保存按钮,则在保存完成之后才会调用setTimeout函数。

你的javascript中可能发生的事情是,在调用onChange事件之前,javascript引擎会调用onClick事件。

作为提示,请记住javascript是单线程的,除非你使用javascript调试器(firebug,microsoft screipt debugger)。这些程序拦截线程并暂停它。从那时起,可以运行其他线程(通过事件,setTimeout调用或XMLHttp处理程序),使javascript可以同时运行多个线程。