Que tal Shock!
Estoy totalmente de acuerdo, es un desastre como está diseñada la clase y el uso que se le da al constructor y perfectamente habría que abstraer esa funcionalidad en un nuevo método.
Lo que hay que tener en cuenta que un constructor con sus parámetros estamos definiendo qué es lo básico y mínimo para poder crear la instancia. Si esto no se cumple, no se puede crear (y eso refuerza el diseño).
PD: amén de todo eso, está usando sintaxis de PHP4, variables globales, etc (todo lo que no deberías usar).
Saludos!