Mala programación?
Negrada? 
No hay nada de malo en la programación usar if($variable) si lo que quiere hacer uno es verificar que la variable existe y no sea igual a cero ( que contenga algún valor, string, int, bool ->1, etc).
Dentro de los condicionales, cuando la variable tenga un valor diferente de cero es equiovalente al valor booleano true. Esto es teoría básica en php.
Cómo podemos calificar entonces este uso en la programación totalmente lícito y lógico como " programación con negradas" ? No nos apresuremos a dar calificativos porque las cosas que decimos a veces se nos vuelven a nosotros mismos.
No tiene por qué dar algún error lógico, al contrario es mas prolijo.
La función empty e isset y el operador de igualdad === (comprara valor y tipo) se usarán de acuerdo a las estrategias y necesidades de cada programador.
Pero repito,
no es mala programación y no tiene por qué dar errores lógicos.
Hasta siempre
PD: Te invito a hacer lo siguiente:
Código PHP:
$variable = true;
echo $variable;
Imprime
1