ajax的onreadystatechange的问题

Question

自己用原生js造了一个轮子

var Ajax = function (config) {
    var xhr = window.XMLHttpRequest ? new XMLHttpRequest() : new ActiveXObject("Microsoft.XMLHTTP");
    xhr.dataType = config.dataType;
    xhr.responseType = config.responseType;
    xhr.open(config.method,config.url,config.async);
    // xhr.onreadystatechange = config.ondone(xhr);
    // xhr.onreadystatechange = function () {
    //     console.log( xhr )
    //     config.ondone(xhr);
    // };
    xhr.onerror = function () {
        console.info('readyState = ' + xhr.readyState)
        console.info('status = ' + xhr.status)
        config.onerror;
    }
    xhr.send(config.data);
}

以下为调用

    Ajax({
        url:'./',
        data:formData,
        async:'true',
        method:'POST',
        dataType:'type',
        responseType:'text',
        ondone:function (xhr) {
            console.log( xhr )
            console.log( xhr.readyState )
            },
        onerror:'',
    })

现在的问题是onreadystatechange方法，如果按照第一个写的话congole只会出现一次readyState为1。
如果我用第二个方法的话，是可以正常显示的readyState都能打印出来。
是哪里写错了还是？

Answer 1

xhr.onreadystatechange = config.ondone(xhr);

因为你第一种写法其实是相当于执行了一次ondone函数，执行完xhr.onreadystatechange == null，null其实就是ondone函数的返回值。
如果想要将一个函数赋值给xhr.onreadystatechange，需要这样写：

xhr.onreadystatechange = config.ondone;

虽然这样就无法直接传递参数，所以可以用第二种写法。

Answer 2

第一个你仔细看xhr.onreadystatechange = config.ondone(xhr)，你自己执行了一次ondone方法，然后将执行结果(这边没有返回结果，也就是undefined)赋值给xhr.onreadystatechange。然后每一次readystatechange的时候触发的是你放回的结果(也就是undefined)，不知执行你的ondone方法。

正确的应该xhr.onreadystatechange = config.ondone