l'ultimo echo non è chiuso da ; per il fatto che l'ultima riga di codice può non averlo, ma è buona usanza farlo a priori, visto che poi se aggiungi altri righe possono succedere casini (vedi anche con include vari)

il codice dove c'è la funzione è un abominio comunque, non ha senso chiudere e riaprire ad ogni riga il tag di php