no , no esta mal separarlo.
De hecho php ofrece un lenguaje de "plantilla", entonces en lugar de escribir
Código PHP:
Ver original<?php while($condicion) { ?>
tu HTML
<?php } ?>
escribes:
Código PHP:
Ver original<?php while($condicion): ?>
<?php endwhile ?>
Por cierto, no utilices las funciones mysql_ utiliza PDO o mysqli. Tu codigo es vulnerable a inyeccion de SQL.