Usar @$var ? en vez de isset( $var ) } Buena práctica?

#1 (**permalink**) 09/08/2011, 07:36

Estoy programando un conjunto de clases para facilitar el manejo de base de datos usando configuraciones y salidas de pantalla entre otras cosas, y en una parte de mi script en una condición if para saber si existían alguna de las variables especificadas, preferí usar:

Código PHP:

       if( @$var || @$var2 || @$var3 || ... || @$varN ) ;
     #En vez de:
     if( isset($var) || isset($var2) || isset($var3) || ... || isset($varN) ) ;

Ambas formas no alertan errores de nivel E_WARNING al no estar definida la variable.

Es una buena práctica hacer esto pero no siempre, sólo en casos específicos? Yo lo hice porque eran muchas las variables las que tenía que verificar y eran elementos de arrays en realidad, entonces así podía mantener la condición del if en una sola línea y no en más líneas perjudicando la legilibilidad del código debido a la extensión que resultaba al user la función isset().

Qué piensan?

andresdzphp · #2 (**permalink**) 09/08/2011, 09:29

Obviamente no es una buena práctica. Lo que hace la (@) es "ignorar" el error... No muestra el mensaje de error pero tampoco te soluciona el problema por arte de magia. Además de consumir recursos innecesarios.

Isset lo que hace es comprobar si la variable está definida y no es NULL, creo que se acerca más a tus necesidades.

skiper0125 · #3 (**permalink**) 09/08/2011, 09:33

Hola que tal

Como dice andresdzphp el @ te ignora el error por lo cual no es recomendable para la programación, recuerda que también puedes utilizar empty pasa saber si una variable contiene valor

Saludos

#4 (**permalink**) 09/08/2011, 10:11

Justamente como pensé, no es una buena práctica, estoy totalmente de acuerdo con ustedes. Dejaré que la condición se alargue para no hacer algo poco profesional. Muchas gracias!

pateketrueke · #5 (**permalink**) 09/08/2011, 20:22

Pésima practica.

Por cierto, isset() admite múltiples parámetros por si no quieres hacer muy largo y confuso el código:

Código PHP:

  if (isset($foo, $bar, $candy)) // ...

Y no olvides usar empty() también, pues te ahorra trabajo la mayoría del tiempo.

Sourcegeek · #6 (**permalink**) 09/08/2011, 20:42

Todo lo que tenga que ver con @ es malo

@mysql_query
@file_get_contents
@mail
Etc...

Saludos!