nicht angemeldet
Es handelt sich um eine Methode für das Abholen von Daten aus der DB, [...]
Das kann sie ja machen. Das ist ihre Aufgabe. Ihre Aufgabe ist es aber nicht (auch noch), die Verbindung zum DBMS zu verwalten. Faustregel: Wenn du ein und in der Aufgabenbeschreibung brauchst, machst du zu viel an der Stelle.
Er sprach von zwei Methoden und davon abgesehen ist nichts dagegen einzuwenden, wenn eine Methode ihren eigenen Kram auch selbst aufräumt, ganz im Gegenteil.
Nur, wenn dieser Kram auch woanders benötigt wird, wie in diesem Fall nach der Erweiterung wohl vorliegend, sollte die Verwaltung aus der Methode ausgelagert werden - und das hat er anscheinend gemacht, wie in seiner Eingangsfrage beschrieben.