Wie man veränderbare Variable von der Schließung vermeidet

Ich habe einige code wie dieser:

for(var id=0; id < message.receiver.length; id++){
   var tmp_id = id;
   zlib.gzip(JSON.stringify(message.json), function(err, buffer){
                        ...
   pushStatusPool[message.receiver[tmp_id]] = null; //fix memory leak
   delete pushStatusPool[message.receiver[tmp_id]];
   ...
   });
}

Und ich bekam eine Warnung, dass die Verwendung tmp_id im Verschluss kann zu Problemen führen, denn es ist eine veränderbare variable.

Wie könnte ich das vermeiden? Ich meine, wie könnte ich das senden einer unveränderlichen variable um Rückruf, da diese eine for-Schleife und ich kann nicht den code ändern von zlib.gzip? Oder in anderen Worten, wie konnte ich passiere ein argument für eine Schließung?

Kommentar zu dem Problem - Öffnen
Wie können Sie vermeiden, was? Ihre Frage ist nicht klar. Bitte viel mehr spezifisch, was Sie wollen, helfen Sie mit. Kommentarautor: jfriend00
Ich begann zu schreiben, eine Antwort ist... aber es wird wirklich klar, dass dies könnte ein umgestalten 🙁 Du bist Gzip die gleiche Sache, die - Meldung.- Empfänger.Länge - mal. Post die ganze Sache? Kommentarautor: brianreavis

InformationsquelleAutor der Frage bxshi | 2012-12-11

  1. 41

    Müssen Sie einen Bereich erstellen, richtig zu erfassen tmp_id mit einem self-executing-Funktion. Das ist, weil die gesamte for-Schleife ist ein Bereich, was bedeutet, das jeder Zeit durch, Sie erfassen die gleiche variable. So wird der callback am Ende mit den falschen ids, weil temp_id's Wert geändert werden, bevor die callback aufgerufen wird.

    Würd ich ignorieren (oder Abschaltung) der Warnung, obwohl, das scheint zu sein, beschwerte sich, weil temp_id ist veränderbar, Sie könnte zuweisen. Das ist sowas von albern. Wenn Sie wirklich wollen, um es zu beheben, versuchen Sie es mit der const - Schlüsselwort anstelle von var.

    for(var id=0; id < message.receiver.length; id++){
   (function(){
       const tmp_id = id;
       zlib.gzip(JSON.stringify(message.json), function(err, buffer){
                        ...
           pushStatusPool[message.receiver[tmp_id]] = null; //fix memory leak
           delete pushStatusPool[message.receiver[tmp_id]];
           ...
       });
   })();
}

    InformationsquelleAutor der Antwort

  2. 10

    Ich habe vor dem gleichen problem und löste es leicht zu modifizieren: die Antwort von user24359, durch die übergabe der id, um die Schließung:

    for(var id=0; id < message.receiver.length; id++){
   (function(tmp_id){
       zlib.gzip(JSON.stringify(message.json), function(err, buffer){
                        ...
           pushStatusPool[message.receiver[tmp_id]] = null; //fix memory leak
           delete pushStatusPool[message.receiver[tmp_id]];
           ...
       });
   })(id);
}

    InformationsquelleAutor der Antwort gennadi.w

  3. 2

    hier eine Vereinfachung der user24359 die tolle Antwort.
    Das ist die Lösung:

    var object = {a:1,b:2};

for (var y in object){
    (function(){const yyy = y;
        setTimeout(function(){console.log(yyy)},3000);})();
}

    Den oben genannten code, meldet sich ein b und ist die Lösung.
    Die folgenden code-Protokolle b b :

    var object = {a:1,b:2};
for (var y in object){

    setTimeout(function(){console.log(y)},3000);
}

    InformationsquelleAutor der Antwort Michael Moeller

  4. 0

    Ich habe vor dem gleichen problem Winkelmesser. Gelöst mit folgenden code -

    (function(no_of_agents){
              ptor.element.all(by.repeater('agent in agents').column('displayName')).then(function(firstColumn){
                    console.log(i, '>>>>>Verifying the agent Name');
                    var agentsSorted = sortAgentsByName();
                    //verify the agent name
                    expect(firstColumn[no_of_agents].getText()).toEqual(agentsSorted[no_of_agents].name);
                    //now click on the agent name link
                    firstColumn[no_of_agents].click();
                    ptor.sleep(5000);
              });
            })(no_of_agents);

    InformationsquelleAutor der Antwort Piyush Jajoo

  5. 0

    @user24359 Antwort ist eine gute Lösung, aber Sie können ersetzen Sie einfach die var Schlagwort von der let Stichwort.

    for(var id=0;

    wird

    for(let id=0;

    Details siehe hier.

    Bearbeiten : Als Heriberto Juarez vorgeschlagen, es funktioniert nur für Browser, die unterstützt EcmaScript6.

    InformationsquelleAutor der Antwort Bludwarf

  6. 0

    Erstellen Verschlüsse in einer Schleife mit var (tmp_id) wird im oberen Bereich der callback-Funktion ist eine allgemeiner Fehler, die vermieden werden sollten aufgrund der var nicht-block-Bereich. Deshalb, und weil jede Schließung, erzeugt in der Schleife, teilt die gleichen lexikalische Umgebung, die variable wird immer der letzten iterierten Wert (d.h. message.receiver.length - 1 als tmp_id) wenn die callback-Funktion wird aufgerufen. Die IDE erkennt dieses Verhalten und beschwert sich zu Recht.

    Zur Vermeidung der Warnung, es gibt mehrere Lösungen:

    • Ersetzen var mit lassen dafür sorgen, dass jede erstellte Schließung, seinen eigenen Gültigkeitsbereich tmp_id definiert, in jeder iteration:

      for (var id = 0; id < message.receiver.length; id++) {
  let tmp_id = id;
  zlib.gzip(JSON.stringify(message.json), function(err, buffer) {
    //Do something with tmp_id ...
  });
}

    • Erstellen Sie eine lexikalische Umgebung in jeder iteration durch die Nutzung IIEF wie gennadi.w hat.

    • Erstellen Sie eine callback-Funktion in jeder iteration durch die Verwendung einer factory-Funktion (createCallback):

      const createCallback = tmp_id => function(err, buffer) {
  //Do something with tmp_id ...
};
for (var id = 0; id < message.receiver.length; id++) {
  zlib.gzip(JSON.stringify(message.json), createCallback(id));
}

    • bind die variable(N) auf die callback-Funktion, in der Sie erhalten vorangestellt Ihrer Parameter:

      for (var id = 0; id < message.receiver.length; id++) {
  zlib.gzip(JSON.stringify(message.json), function(tmp_id, err, buffer) {
    //Do something with tmp_id (passed as id) ...
  }.bind(this, id));
}

    Wenn möglich var sollten vermieden werden, da der ECMAScript 2015 wegen solcher fehleranfälligen Verhalten.

    InformationsquelleAutor der Antwort Boghyon Hoffmann

Schreibe einen Kommentar