Hola
@syntex
El código no está mal, pero es una mala práctica de desarrollo.
No debes imprimir código html desde php, utiliza el método de escape.
Separa tu presentación de tu lógica de negocios. Esto lo puedes solucionar utilizando el
Patrón clásico de diseño web Modelo Vista Controlador (MVC).
Aquí te dejo 2 link:
http://arleytriana.blogspot.com/2009...lasico-de.html http://web2development.blogspot.com/...atron-mvc.html
Saludos.