Pues yo no lo veo tan mal, eh? Todo depende del concepto que tienes de lo que es "programar bien". La única recomendación que te haría es usar echo para imprimir lo que tienes entre los ifs, en vez de abrir y cerrar tantas veces los bloques de PHP. Eso agilizará en unos cuantos microsegundos el proceso y lo hará ver más elegante.
