Leider nicht besonders gut gemacht. Die Abfrage fürs ResultSet ist synchron geschrieben und es empfiehlt sich nicht die Connection dauerhaft aufrecht zu erhalten sondern sie nach jeder Abfrage zu schließen. Und in der OOP solltest du dir auch noch mal Nachhelfen.