Erreur JavaScript "n'est pas une fonction" lors de l'appel de la méthode définie

C'est mon code:

request_xml: function()
        {
        http_request = false;
                    http_request = new XMLHttpRequest();
                     if (http_request.overrideMimeType) 
                            {
                            http_request.overrideMimeType('text/xml');
                            }
                          if (!http_request)
                          {
                                return false;
                          }
                        http_request.onreadystatechange = this.response_xml;
                        http_request.open('GET', realXmlUrl, true);
                        http_request.send(null);
                        xmlDoc = http_request.responseXML;

},



response_xml:function ()
    {
        if (http_request.readyState == 4)
        {
            if(http_request.status == 404 && countXmlUrl<=3)
            {
                countXmlUrl++;

                realXmlUrl = xmlUrl[countXmlUrl];
                this.request_xml();
            }
            if (http_request.status == 200)
            {
                xmlDoc = http_request.responseXML;
                alert("need to update3");
                this.peter_save_data();
            }

        }
    },

peter_save_data:function()
    {
//removed function code
},

Étrangement, l'alerte se déclenche sans problème, mais l'appel de la fonction en dessous me donne cette erreur:

Error: this.peter_save_data is not a function

L'appel de la même sacrément fonction à partir d'une autre fonction ailleurs fonctionne très bien.

source d'informationauteur Ryan

  1. 26

    Vous pouvez faire cela, juste avant l'appel de la génération XML.

    var that = this;

    et à plus tard...

    that.peter_save_data();

    Parce que this change fréquemment lors de l'évolution de la portée en utilisant une nouvelle fonction, vous ne pouvez pas accéder à la valeur d'origine en l'utilisant. De l'Aliasing à qui permet toujours d'accéder à la valeur d'origine de cette.

  2. 6

    Une pièce importante du puzzle qui manque, c'est comment response_xml est appelé. Ceci est important, parce que cela va changer quoi this (voir Jared commentaire).

    Rappelez-vous que this peut être considéré comme (à peu près) "le récepteur de l'appel de la méthode". Si response_xml est passé directement à l'utiliser comme un rappel puis bien sûr, il ne fonctionne pas -- this sera probablement window.

    Tenir compte de ces:

    var x = {f: function () { return this }}
var g = x.f
x.f() === x    //true
g() === x      //false
g() === window //true

    Heureux de codage.

    Le "fix" est probablement juste de changer la façon dont response_xml est appelé. Il existe de nombreuses façons de le faire (en général avec une fermeture).

    Exemples:

    //Use a closure to keep he object upon which to explicitly invoke the method
//inside response_xml "this" will be "that",
//which was "this" of the current scope
http_request.onreadystatechange = (function (that) {
   return function () { return that.response_xml() }
}(this)

//Or, alternatively,
//capture the current "this" as a closed-over variable...
//(assumes this is in a function: var in global context does not create a lexical)
var self = this
http_request.onreadystatechange = function () {
   //...and invoke the method upon it
   return self.response_xml()
}

    Personnellement, je voudrais juste utiliser jQuery ou similaire 😉

  3. -1

    Si vous souhaitez une classe comme le comportement, utilisez le bouton droit de la syntaxe, Les bibliothèques qui utilisent, à l'aide de JSON pour passer un paramètre à une fonction qui fait une classe.

    function MyClass(CTOR paarams){
    var response_xml=function ()
    {
        if (http_request.readyState == 4)
        {
            if(http_request.status == 404 && countXmlUrl<=3)
            {
                countXmlUrl++;

                realXmlUrl = xmlUrl[countXmlUrl];
                this.request_xml();
            }
            if (http_request.status == 200)
            {
                xmlDoc = http_request.responseXML;
                alert("need to update3");
                this.peter_save_data();
            }

        }
    }

    var peter_save_data=function()
    {
       //removed function code
    }
}

var Test = new MyClass(somthing,another_something);
Test.response_xml();
//etc etc.

    Ou, utiliser les bibliothèques comme Mootools où vous pouvez le faire sous forme de JSON:

    var T = new Class({
    response_xml:function ()
    {
        if (http_request.readyState == 4)
        {
            if(http_request.status == 404 && countXmlUrl<=3)
            {
                countXmlUrl++;

                realXmlUrl = xmlUrl[countXmlUrl];
                this.request_xml();
            }
            if (http_request.status == 200)
            {
                xmlDoc = http_request.responseXML;
                alert("need to update3");
                this.peter_save_data();
            }

        }
    },

    peter_save_data:function()
    {
      //removed function code
    }

});
var X = new T();//etc etc