Question

I have this if statement

 if (!(@$donnees['mode']['delete'] === true || @$donnees['mode']['display'] === true)){
     //doSomething only if mode is not delete nor display 
 }

i used @ to not get the notice when mode is not set.
because if i use isset instead it would be even uglier :

$cond_delete = isset($donnees['mode']['delete']) && $donnees['mode']['delete'] === true;
$cond_display = isset($donnees['mode']['display ']) && $donnees['mode']['display '] === true;

if (!($cond_delete || $cond_display)){
   //doSomething only if mode is not delete nor display 
}

Is there a more concise way to do this ?

Thanks

Answer 1

Something like this might be a good choice although it creates a couple of single use variables.

/*Either set values if setting available, or set false*/
$cond_delete = isset($donnees['mode']['delete'])? $donnees['mode']['delete']: false;
$cond_display = isset($donnees['mode']['display '])? $donnees['mode']['display ']: false;


/*Where either mode is set do something*/
if (!($cond_delete || $cond_display)){
 //doSomething only if mode is not delete nor display 
}