Warum sind Globals schlecht?

Question

Es macht total Sinn, es hier zu verwenden. Was wäre die Alternative? Wie kann ich im Allgemeinen vermeiden, sie zu verwenden und vor allem, warum es laut jsLint schlecht ist, Globals zu verwenden.

(function($){ 
    $(function(){ 
    $body = $('body'); //this is the BAD Global 

    $.each(somearray ,function(){ $body.dosomething() }); 

    if (something){ 
    $body.somethingelse(); 
    } 

    }); 
}(jQuery)); 

Können Sie mir helfen, dies zu verstehen? Und gib mir eine bessere Lösung?

Answer 1

Globals sind schlecht, weil sie sofort keine Probleme verursachen. Erst später, nachdem du sie überall benutzt hast, verursachen sie sehr hässliche Probleme - die du nicht mehr lösen kannst, ohne deinen Code von Grund auf neu zu schreiben.

Beispiel: Sie verwenden $body, um einige Funktionen zu definieren. Das funktioniert gut. Aber schließlich brauchst du auch einen Wert. Sie verwenden also $body.foo. Funktioniert gut. Dann fügen Sie $body.bar hinzu. Und dann, Wochen später, benötigen Sie einen anderen Wert, so dass Sie $body.bar hinzufügen.

Sie testen den Code und es scheint zu funktionieren. Aber Sie haben dieselbe Variable zweimal "hinzugefügt". Dies ist kein Problem, da JavaScript das Konzept "einmal eine neue Variable erstellen" nicht versteht. Es weiß nur "create, es sei denn es existiert bereits." So verwenden Sie Ihren Code und schließlich wird eine Funktion $body.bar ändern eine andere Funktion zu ändern. Selbst wenn Sie das Problem finden, brauchen Sie viel Zeit.

Deshalb ist es besser sicherzustellen, dass Variablen nur bei Bedarf angezeigt werden können. Auf diese Weise kann eine Funktion nicht eine andere unterbrechen. Dies wird umso wichtiger, je größer der Code wird.

Answer 2

sollten Sie es mit var $ Körper zu definieren, dann wäre es in den Anwendungsbereich dieser Funktion lokal sein, ohne var es von jedermann überschrieben werden könnte

(function($){ 
    $(function(){ 
    var $body = $('body'); //this is the local variable 

    $.each(somearray ,function(){ $body.dosomething() }); 

    if (something){ 
    $body.somethingelse(); 
    } 

    }); 
}(jQuery)); 

Answer 3

Globale Variablen mit anderen Scripts kollidieren könnte oder überschrieben werden . Wenn Sie kein globales benötigen, ist es ratsam, sie zu vermeiden. Verwenden Sie einfach var (oder let wenn Ihr JS-Version-Support ist größer als 1,7):

(function() { 
    var foo = 'bar'; 
    alert(foo); 
})(); 

Answer 4

Sie umschreiben könnte, dass als

var $body = $('body'); 

Das (die Verwendung des var Schlüsselwort) es machen würde eine lokale Variable, die für Ihre Zwecke ausreicht. Es ist immer noch in Reichweite in Ihrem each Rückruf.

Der Grund, warum es schlecht ist, Globals zu verwenden, ist, dass es von irgendetwas anderem überschrieben werden kann. Damit Ihr Code gut skaliert wird, hängt er davon ab, welche anderen Skripts Sie verwenden. Es ist vorzuziehen, das Skript so autark wie möglich zu halten, mit möglichst wenig Abhängigkeiten, die auf die Welt außerhalb davon zeigen.

Answer 5

jsLint ist sehr streng. Es ist wahrscheinlich nicht notwendig, sich damit aufzuhalten.

Aber wenn Sie sich schlecht fühlen, können Sie es tun, wie, wie Sie jQuery scoped:

(function($){ 
    $(function(){ 
    $.each(somearray ,(function($body){ $body.dosomething() })($('body')); 

    if (something){ 
    $('body').somethingelse(); 
    } 

    }); 
}(jQuery));