Comment utiliser l'opérateur ternaire au lieu de if-else dans PHP

Question

J'essaye de raccourcir mon code en utilisant l'opérateur ternaire.

Ceci est mon code d'origine:

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

Comment puis-je utiliser l'opérateur ternaire dans mon code au lieu de if de/s else de?

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

Voici le code que j'ai essayé, mais ne sais pas comment ajouter une « elseif partie » à elle.

Answer 1

Ceci est appelé l'opérateur ternaire ;-)

Vous pouvez utiliser deux de ces:

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

Cela peut être lu comme:

• si $type est 'recent'
• puis utilisez 'sid DESC'
• autre
  • si $type est 'pop'
  • puis utilisez 'counter DESC'
  • utiliser autre 'RAND()'

Quelques notes:

• Vous devez utiliser == ou ===; et non =
  • Les deux premiers sont comparison operators
  • Le dernier est le assignment operator
• Il est préférable d'utiliser (), pour rendre les choses plus faciles à lire
  • Et vous shouldn N'utilisez pas trop d'opérateurs ternaires comme ça: ça rend le code un peu difficile à comprendre, je pense

Et, comme une référence au sujet de l'opérateur ternaire, citant le Operators section of the PHP manual:

Le troisième groupe est le ternaire opérateur: ?:.
Il doit être utilisé pour sélectionner entre deux expressions en fonction d'un troisième, plutôt que pour sélectionner deux phrases ou chemins d'exécution .
Entourant ternaire des expressions avec des parenthèses est une très bonne idée.

Answer 2

Je suggérerais plutôt d'utiliser une instruction case. il est un peu plus lisible mais plus maintenable lorsque vous voulez ajouter des options supplémentaires

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