tengo una duda, se debe cerrar la etiqueta <?php en algunos manuales dice que no

[dashtrash]

Repito, no es sólo una cuestión de gusto.Previene que caracteres en blanco extra al final de ficheros, inicien la salida HTML por lo que no es posible enviar posteriormente cabeceras.
Por eso, los ficheros donde nunca cierro con ?>, son en los ficheros de configuración.Se cargan antes que cualquier otra cosa.Un simple "enter" al final del fichero, o un espacio, puede darte un pequeño problema, que así te ahorras.
En un fichero cargado una vez que la salida HTML ya se ha iniciado, es cuando es simplemente una cuestión de gusto.