-1

每 3 秒我发出一个 AJAX POST 请求以获取进程的状态。这很好用。

当进程达到 100% 时，将执行回调函数（如下所示）以向页面添加新元素，然后取消用于每 3 秒连续获取进度的 setTimeout 方法。但是，我的用户告诉我它有时无法取消并且新元素没有添加到页面中，而且我一直卡在显示“100%”。

我已经一次又一次地测试了它，它永远不会卡住我。代码看起来也不错，但是我的 JavaScript 技能不是很好，所以我希望有人能指出是否有可能发生这个问题？

我已经评论了代码，很抱歉它很长。我试图减少它。

function convertNow(validURL){

    startTime = setTimeout('getStatus();', 6000); 
       //AJAX CALL TO RUN PROCESS
       $.ajax({
       type: "GET",
       url: "main.php",
       data: 'url=' + validURL + '&filename=' + fileNameTxt,
       success: function(msg){
       //ON SUCCESS CLEAR SETTIMEOUT AND SHOW ELEMENTS (text)
       clearTimeout(continueTime);
       clearTimeout(startTime);    
        $("#loading").hide("slow");
        $("#done").html("Done");   
       }//function

     });//ajax

}//function convertNow

function getStatus(){
        //AJAX CALL TO GET STATUS OF PROCESS
        $.ajax({
        type: "POST",
        url: "fileReader.php",
        data: 'textFile=' + fileNameTxt,
        success: function(respomse){
        textFileResponse = respomse.split(" ");
        $("#done").html("Processing...");
        }
        });//ajax
        clearTimeout(continueTime);

        if(textFileResponse[0]=='100.0%'){
            clearTimeout(continueTime);
        }
        else{
            clearTimeout(startTime);
            continueTime = setTimeout('getStatus();', 3000); 
        }
}
4

1 回答 1

4

在某些边缘情况下，textFileReponse[0]=='100.0%' 中可能存在解析错误，响应中的值不完全等于 100.0%（可能有额外的空格，或者在某些平台上可能存在一些细微差别， ETC...）。这将导致代码落入 else {} 块，并且您的 getStatus 函数将再次排队。

编辑：鉴于评论中的线程，两个 Ajax 代码块之间存在竞争条件的可能性也相同。（只是为了读者的利益而把它放在这里）。结束编辑

然而，除了解析解析之外，您可能想要的是使用 setInterval()，只有一个计时器，而不是 startTime 和 continueTime 计时器。setTimeout 只执行一次，而 setInterval 每 x 毫秒重复一次，所以你只需要一个。要取消 setInterval，请使用 clearInterval。

于 2009-03-04T19:53:24.613 回答