"Mejor" forma de escribir esto?

Question

¿Existe alguna forma mejor (es decir, más legible) para escribir esto?

if (isset($input_vars['directive']) && $input_vars['directive'] == 'edit') { 

Answer 1

Desafortunadamente no. Puede envolver este código en una función y simplemente llamarlo cada vez que lo necesite.

function compareArrayItem($array, $key, $value) { 
    return isset($array[$key]) && $array[$key] == $value; 
} 

if (compareArrayItem($input_vars, 'directive', 'edit')) { 
    // Do something 
} 

Pero eso parece una especie de sentido para mí (y menos legible que el código original). O bien, podría reducir el nivel de informe de errores para no incluir E_NOTICE, para que no necesite la primera expresión.

error_reporting(E_ALL^E_NOTICE); 

if ($input_vars['directive'] == 'edit') //... 

Pero no recomendaría hacer esto solo por el bien de acortar su código.

Si yo fuera usted, solo lo dejo en paz. Está bien tal como está.

Answer 2

Lo siguiente arrojaría el mismo resultado cada vez.

if($input_vars['directive'] == 'edit'){ 

Esto es porque si no está configurado, entonces no es 'editar', si es 'editar' entonces su conjunto.

Esto devuelve un aviso pero puede desactivarlo (no diciendo que debería) de su instalación de PHP.

Answer 3

yo diría:

if(@$input_vars['directive']=='edit') { 

sería un poco más fácil de leer y que no produce advertencias si la directiva no está allí.

Answer 4

Voy a adivinar que probará $ input_vars ['directive'] contra más de un valor (de lo contrario, ¿por qué no tendría simplemente un booleano simple almacenado en $ input_vars ['edit'] o ¿similar?). También me atrevería a adivinar que estás haciendo esas pruebas una tras otra (si 'edit' haces X, de lo contrario si 'display' lo haces).

En tal caso, simplemente coloque la prueba isset() en una instrucción if y anote las otras dentro de esa (switch/case flow no sería una mala elección).

Answer 5

Si se conoce el conjunto de valores permitidos en $input_vars y los controles que mencionas son por todo el lugar la siguiente función auxiliar hará:

function item ($array, $key) { 
    if (isset ($array [$key])) 
     return $array [$key]; 
    else 
     return NULL; // Or use whatever is more appropriate 
}

A continuación, el código original se puede cambiar para parecerse

if (item ($input_vars, 'directive') == 'edit') { ... 

Esto no solo es más legible, sino que también elimina la duplicación: tanto la variable de matriz como la clave aparecen solo una vez.