Sistema de login con POO

[Malenko]

Si vas a agregar más métodos a la clase que requieran conexión contra la BBDD y justifiquen mantener la conexión abierta, entonces tiene sentido dejarla en el constructor. Pero has pedido que te comenten el código que has puesto, no el que quizás en un futuro hagas. Y ahora mismo en ningún momento liberas la conexión.

Por otro lado, si estás realizando POO y quieres seguir los principios SOLID tendrías que desacoplar el acceso a BBDD de la clase de Login y permitir poder intercambiar de un SGBD a otro, etc.