Cleaner Weg diesen Code zu schreiben

Question

Ich habe den folgenden Code, der perfekt funktioniert, aber ich rufe die getItem Funktion zweimal, was unnötig scheint. Wie kann ich es aufräumen?

if(localStorage.getItem("preferences") == null { 
    //show set preferences page 
} else { 
    var preferences = localStorage.getItem("preferences"); 
    //do stuff 
} 

Answer 1

Just do die getItem einmal:

var preferences = localStorage.getItem("preferences"); 
if (preferences == null) { 
    //show set preferences page 
} else { 
    //do stuff 
} 

Und es sei denn, es gültig ist, dass preferences andere ein falsy Wert sein könnte als null, würde ich wahrscheinlich die Blöcke umkehren:

var preferences = localStorage.getItem("preferences"); 
if (preferences) { 
    //do stuff 
} else { 
    //show set preferences page 
} 

Answer 2

Sie müssen localStorage.getItem nicht zweimal lesen. Sie können es in eine Variable einlesen und dasselbe

verwenden

var preferences = localStorage.getItem("preferences"); 
 
if (preferences == null) { 
 
    //show set preferences page 
 
    } else { 
 
    //you can use this preferences here 
 
    //do stuff 
 
    }

Hoffe, dass es

Answer 3

Hoffnung hilft diese

let preferences = localStorage.getItem("preferences"); 

preferences ? do stuff : show set preferences page 

Answer 4

var preferences = localStorage.getItem("preferences"); 

if(preferences) { 
    //Show preference page 
} else { 
    //Do something 
} 

Wenn der Wert aus dem lokalen Speicher ist null hilft, haben Sie immer es zu einem var zuweisen und dann überprüfe es.

Sie können auch einfach die Variable innerhalb der if-Anweisung überprüfen, ohne mit Null zu vergleichen. Nehmen Sie folgendes zum Beispiel:

if(!a) { 
    //Will enter here if a is null, undefined, false, empty string, 0 or NaN 
} 

Weiterführende Literatur: http://james.padolsey.com/javascript/truthy-falsey/

Answer 5

es einmal aufrufen und speichern sie in einer Variablen.

var preferences = localStorage.getItem("preferences"); 
if(preferences==null){ 
    //.... 
} 
else{ 
    //.... 
} 

Answer 6

Einfach!

var preferences = localStorage.getItem("preferences"); 

if(preferences == null) { 
    //show set preferences page 
} else { 

    //do stuff with preferences 
} 

Answer 7

var preferences = localStorage.getItem("preferences"); 
if(!!preferences){ //check if preferences is undefined or false or null 
    //show set preferences page 
} else { 
    //do stuff 
} 

Answer 8

nur als Alternative, können Sie Präferenzen innerhalb des if-Anweisung selbst zuweisen und wie folgt vorgehen:

if ((preferences = localStorage.getItem("preferences"))) { 
    // use preferences here 
} 
else { 
    // set preferences here 
} 

Ich nehme im Allgemeinen diesen Ansatz, wenn seine etwas nur werde ich arbeiten werden on kann das Zuweisen innerhalb einer Bedingung einige Leute beim Debuggen verwirren.