Come utilizzare l'operatore ternario invece di if-else in PHP

Question

Sto cercando di abbreviare il mio codice utilizzando l'operatore ternario.

Questo è il mio codice originale:

if ($type = "recent") { 
    $OrderType = "sid DESC"; 
} elseif ($type = "pop") { 
    $OrderType = "counter DESC"; 
} else { 
    $OrderType = "RAND()"; 
} 

Come posso usare l'operatore ternario nel mio codice, invece di if s/s else?

$OrderType = ($type = "recent") ? "sid DESC" : "counter DESC" ; 

Questo è il codice che ho provato, ma non hanno alcuna idea di come aggiungere un “elseif parte” ad esso.

Answer 1

questo è chiamato l'operatore ternario ;-)

Si potrebbe utilizzare due di questi:

$OrderType = ($type == 'recent' ? 'sid DESC' : ($type == 'pop' ? 'counter DESC' : 'RAND()')) 

Questo può essere letto come:

• se $type è 'recent'
• quindi utilizzare 'sid DESC'
• altro
  • se $type è 'pop'
  • quindi utilizzare 'counter DESC'
  • altro utilizzare 'RAND()'

Un paio di note:

• È necessario utilizzare == o ===; e non =
  • Le prime due sono comparison operators
  • L'ultima è la assignment operator
• E 'meglio usare (), per rendere le cose più facili da leggere
  • E si shouldn Usiamo troppi operatori ternari del genere: rende il codice un po 'difficile da capire, penso che

E, come riferimento circa l'operatore ternario, citando il Operators section of the PHP manual:

Il terzo gruppo è il ternario esercente: ?:.
Dovrebbe essere utilizzato per selezionare tra due espressioni a seconda di una terza, anziché per selezionare due frasi o percorsi di esecuzione .
ternario circostante espressioni con parentesi è una buona idea .

Answer 2

Suggerirei invece di utilizzare una dichiarazione caso. lo rende un po 'più leggibile ma più manutenibile per quando si desidera aggiungere ulteriori opzioni

switch ($type) 
{ 
case "recent": 
    $OrderType = "sid DESC"; 
    break; 
case "pop": 
    $OrderType = "counter DESC"; 
    break; 
default: 
    $OrderType = "RAND()"; 
}