PHP améliorer une simple déclaration if/else

Question

<?php echo isset($areas['footer']) ? $areas['footer'] : null; ?> 

Une façon d'améliorer cela?

Answer 1

Notez que vous faites écho et que la valeur null est sans effet. Vous pourriez dire comme 'empty' ou ' ' ou 'not found' à la place. Autre alternative est d'obtenir la valeur de retour de isset:

$return = isset($areas['footer']) ? $areas['footer'] : null; 

if ($return) 
{ 
    // $return contains footer info 
} 
else 
{ 
    // footer was not set :(
} 

Answer 2

Selon l'endroit où $ les zones provient de cela pourrait être plus propre pour l'assigner à une variable:

$footer = isset($areas['footer']) ? $areas['footer'] : null; 

vous pouvez utiliser $ pied sans toute vérification supplémentaire d'isset.

Answer 3

echo $areas['footer']; 

Simple et a exactement le même effet que la ligne d'origine.

Modifier en réponse à Felix Cela donne un avis, mais à moins que vous êtes censé tourner cela en tant que devoirs ultra-parfait, il n'a pas vraiment d'importance. Vous pouvez soit remplir votre code avec des appels isset ou ignorer les petites choses comme ça. Je m'inquiéterais si je travaillais sur ... Java, mais pragmatiquement, personne ne s'en souciera si le code PHP qui fonctionne produit des notices.

Answer 4

Vous pouvez également épargner la branche else, en définissant un défaut:

$footer = null; 
if (isset($areas['footer'])) { 
    $footer = $areas['footer']; 
} 

echo $footer; 

Answer 5

Non, c'est la façon la plus concise de gérer ce genre de sortie.

Answer 6

«J'utilise ce genre de code très souvent »

Peut-être que vous devriez éviter la question tout à fait à l'aide d'un langage de modèle ou d'encapsulation ce comportement dans une fonction?

comme ceci:

function get_area($area) { 
    if... //your code 
    return $area 

Answer 7

Une version plus courte que je peux penser serait:

<?php !isset($areas['footer']) or print $areas['footer']; ?> 

Mais je ne sais pas s'il est plus rapide ou plus élégant. Qu'est-ce que vous en pensez?