使用Ajax调用从函数返回值

时间:2009-02-18 19:08:06

标签: javascript ajax asynchronous

有人可以告诉我如何将status的值作为函数的返回值返回。

function checkUser() {
    var request;
    var status = false;

    //create xmlhttprequest object here [called request]

    var stu_id = document.getElementById("stu_id").value;
    var dName = document.getElementById("dName").value;
    var fileName = "check_user.php?dName=" + dName + "&stu_id=" + stu_id;
    request.open("GET", fileName, true);
    request.send(null);

    request.onreadystatechange = function() {
        if (request.readyState == 4) {
            var resp = parseInt(request.responseText, 10);
            if(resp === 1) {
                alert("The display name has already been taken.");
                status = false;
            }
            else if(resp === 2) {
                alert("This student ID has already been registered");
                status = false;
            }
            else if(resp === 0) {
                status = true;
            }
        }
    }
    return status;
}

当您在注册表单中点击提交时,上述功能会触发(必须明显)。问题是,无论响应是什么,此表单都会提交,并且警报框有时会显示[空白],有时根本不显示。但是,如果我手动将结束status更改为false,即将status替换为false],警告框中会显示正确的值。

P.S。我比javascript中的菜鸟更糟糕,所以也欢迎有关一般改进代码的建议。

2 个答案:

答案 0 :(得分:11)

问题是onreadystatechange在......等待之前不会触发......状态发生变化。因此,当您return status;大部分时间状态都没有时间设置时。您需要做的是return false;始终和onreadystatechange内部确定您是否要继续。如果您这样做,那么您提交表格。简而言之,获取处理返回值的代码,而不是从readystatechange处理程序中运行它。您可以直接执行此操作:

request.onreadystatechange = function() {
    if (request.readyState == 4) {
        var resp = parseInt(request.responseText, 10);
        switch (resp) {
        case 0:
            document.getElementById('myform').submit();
            break;
        case 1:
            alert("The display name has already been taken.");
            break;
        case 2:
            alert("This student ID has already been registered");
            break;
        }
    }
}
return false; // always return false initially

或将continuation传递给进行异步调用的函数:

function checkUser(success, fail) {
    ...
    request.onreadystatechange = function() {
        if (request.readyState == 4) {
            var resp = parseInt(request.responseText, 10);
            switch (resp) {
            case 0:
                success(request.responseText);
            case 1:
                fail("The display name has already been taken.", request.reponseText);
                break;
            case 2:
                fail("This student ID has already been registered", request.reponseText);
                break;
            default:
                fail("Unrecognized resonse: "+resp, request.reponseText);
                break;
            }
        }
    }

除此之外,在默认return false;之前,你可能想要某种指示某些东西正在进行,也许禁用提交字段,显示加载圈等。否则用户可能会感到困惑,如果它需要很长时间才能获取数据。

进一步说明

好吧,所以你的方式不起作用的原因主要在于这一行:

request.onreadystatechange = function() {

正在做的是说当AJAX请求的onreadystatechange发生变化时,您正在定义的匿名函数将会运行。此代码不会立即执行。它正在等待事件发生。这是一个异步过程,因此在函数定义结束时,javascript将继续运行,如果状态在到达return status;时没有改变,变量status显然没有时间设置,您的脚本将无法按预期工作。在这种情况下,解决方案始终是return false;,然后当事件触发时(即服务器响应PHP的脚本输出),然后您可以确定状态并提交表单,如果一切正常。

答案 1 :(得分:2)

可能同步执行send方法,因此request.responseText将立即可用,但通常不是一个好主意。该请求将挂起浏览器,直到完成。

要将请求设置为同步,请将request.open的第三个参数设置为“false”。

function checkUser() {
    var request;
    var status = false;

    //create xmlhttprequest object here [called request]

    var stu_id = document.getElementById("stu_id").value;
    var dName = document.getElementById("dName").value;
    var fileName = "check_user.php?dName=" + dName + "&stu_id=" + stu_id;
    request.open("GET", fileName, false);
    request.send(null);

    if (request.status === 200) {
        var resp = parseInt(request.responseText, 10);
        if(resp === 1) {
            alert("The display name has already been taken.");
            status = false;
        }
        else if(resp === 2) {
            alert("This student ID has already been registered");
            status = false;
        }
        else if(resp === 0) {
            status = true;
        }
    }
    else {
        alert("Request failed");
        status = false;
    }

    return status;
}

正如我之前提到的,这不是一个好主意。以异步方式执行请求几乎总是更好的主意,以便在您等待响应时可以运行其他代码。有一个原因 A jax而不是 S jax。