Come dovrebbero essere scritte le espressioni booleane in PHP?

Question

Come dovrebbe la seguente espressione booleana essere scritto in PHP:

$foo = ""; 
if($var==TRUE){ 
    $foo = "bar"; 
} 

o

if($var==TRUE){ 
    $foo = "bar"; 
}else{ 
    $foo = ""; 
} 

o

$foo = ($var==TRUE) ? "bar": ""; 

Answer 1

Prima di tutto, true non è una costante, si tratta di un segno, quindi per favore non maiuscolo esso (so che alcune norme farlo, ma penso che confonde il significato) ...

In secondo luogo, non è necessario il confronto ridondante $var == true all'interno dello if. È esattamente lo stesso di if ($var) { (Per un confronto doppio ==. Un confronto identico === dovrebbe essere esplicito).

In terzo luogo, preferisco la preinizializzazione. Quindi:

$foo = ''; 
if ($var) { 
    $foo = 'one status'; 
} else { 
    $foo = 'another status'; 
} 

Se non è necessario il ramo else, è sufficiente rimuoverlo. Preferisco la preinizializzazione poiché obbliga a inizializzare la variabile e previene i casi in cui si dimentica di inizializzarla in uno dei rami. Inoltre, ti dà un suggerimento quando torni a leggere la funzione più tardi ...

E per un ramo semplice come quello, l'uso della sintassi ternaria va bene. Se c'è più logica complessa, mi piacerebbe stare lontano però:

$foo = $var ? 'bar' : ''; 

Answer 2

Tutti coloro lavoro. È una preferenza. Prenderò in considerazione l'inizializzazione della variabile come prima nel primo esempio. Ma per qualcosa di così semplice, la terza opzione va bene nel mio libro.

Inoltre, il 3 ° non deve essere così verbose se $ var è solo un valore booleano:

$foo = $var ? "bar" : ""; 

Answer 3

mi piace la prima:

$foo = ""; 
if($var==TRUE){ 
    $foo = "bar"; 
} 

Dal momento che è chiaro, conciso e facile da leggere.

Answer 4

Non importa molto. Mi piace il primo quando c'è un sacco di elseif in modo che tu sappia che la variabile è sempre inizializzata. Ma è davvero solo una questione di preferenza.

Come le virgolette, mi piace usare quelle singole in php. Nessuna buona ragione :)

Answer 5

La risposta giusta, come è spesso il caso è "dipende". In questo caso,

if ($var==TRUE) $foo = "bar"; 
else $foo = ""; 

è molto chiaro. Ma qual è il tuo contesto?

In generale, l'operatore terziario, la terza opzione, deve essere utilizzata con estrema cautela, poiché diventa molto difficile da leggere.

Ma pensa in termini di ciò che vuoi che il tuo codice MEAN, più di quello che fa. Vuoi impostare il tuo $ foo su un valore "normale" e poi sostituirlo? O vuoi impostare qualcosa che dipende da cosa è $ var?

Qualcosa che trovo utile per cambiare, che non è direttamente quello che chiedi, ma che è simile, è questo, da

function func() { 
    ... 
    if ($condition) { 
     do plenty 
     of things 
    } 
    else { 
     do plenty 
     of things 
    } 
} 

Che, io in genere piace cambiare a:

function func() { 
    ... 
    if ($condition) { 
     do plenty 
     of things 
     return; 
    } 
    do plenty 
    of things 
} 

Generalmente ha senso.

Basta chiedersi: "Se qualcuno che non sapeva nulla del mio codice lo leggesse, avrebbe senso per lui? O per lei?"

Answer 6

Preferisco il primo (eccetto per il test ridondante per il booleano) perché funziona in modo coerente tra le lingue, in particolare quelle che richiedono di dichiarare la variabile (e magari la tipizzano) prima di impostarla.
Java:

String foo = ""; 
if (var) { 
    foo = "Something"; 
} 

JavaScript o JavaFX:

var foo = ""; 
if (var) { 
    foo = "Something"; 
} 

Ecc
si può usare il terzo modulo di troppo, ma se la condizione (o assegnazione) è complesso, è un po 'meno leggibile .