condizionali più brevi in ​​js

Question

ho condizionali come questo:

if (foo == 'fgfg' || foo == 'asdf' || foo == 'adsfasdf') { 
// do stuff 

} 

Sicuramente c'è un modo più veloce per scrivere questo?

Grazie.

Answer 1

Vorrei mantenere i condizionali come sono. Qualsiasi modo intelligente di accorciarli renderebbe il codice meno idiomatico e meno leggibile.

Ora, se si fa cura di leggibilità, è possibile definire una funzione per fare il confronto:

if(foo_satisfies_condition(foo)) { 
    // ... 
} 

Oppure:

if(is_month_name(foo) { 
    // ... 
} 

Se si dà la funzione di un nome che descrive fedelmente ciò lo fa, sarà più facile capire l'intento del codice.

L'implementazione di tale funzione dipende dal numero di confronti necessari. Se hai un numero molto elevato di stringhe a cui stai confrontando, potresti usare un hash. Tuttavia, i dettagli di implementazione sono irrilevanti durante la lettura del codice chiamante.

Answer 2

if (/^(fgfg|asdf|adsfasdf)$/.test(foo)) { 

o:

if (["fgfg", "asdf", "adsfasdf"].indexOf(foo) != -1) { 

supporto cross-browser Array.indexOf è ancora limitata. Inoltre, questi sono più veloci da scrivere, probabilmente non più veloci da eseguire.

Answer 3

A seconda della situazione che si possa fare ..

 
//At some point in your code 
var vals = new Array('fgfg', 'asdf', 'adsfasdf'); 
//... 
if(vals.indexOf(foo) >= 0) 

Answer 4

si potrebbe considerare uno switch-case dichiarazione

switch(foo) { 
    case "fgfg": 
    case "asdf": 
    case "adsfasdf": 
    // ... 
} 

Non è proprio alcuna più breve, ma potrebbe essere più leggibile a seconda di quanti condizioni che usi.

Answer 5

Non c'è bisogno di utilizzare indexOf o una regex se si utilizza una tabella di hash:

var things = { 'fgfg' : 1, 'asdf' : 1, 'asdfasdf' : 1 }; 
if (things[foo]) { 
    ... 
} 

Answer 6

Ecco un modo semplice:

String.prototype.testList = function(lst) { 
lst = lst.split('|'); 
for(var i=0; i<lst.length; i++){ 
    if (this == lst[i]) return true; 
} 
return false; 
}; 

Per utilizzare questa funzione, è possibile solo fare questo:

if (foo.testList('fgfg|asdf|adsfasdf')) { 

si può anche rinominare testList per quello che vuoi, e cambiare il del imiter da | a tutto ciò che vuoi.

Answer 7

Ternary operator sembra buono se vi piace e ha altro