1. Nur eine DB-Connection machen. Die Verbindung herzustellen ist teuer. Du kannst auch eine getConnection-Funktion haben, die bei Bedarf eine Verbindung erzeugt und sie als State speichert und beim nächsten Aufruf dann den State zurückgibt. Du kannst z.B. Class::Singleton verwenden.
2. Finish muss man nicht aufrufen. Normalerweise lässt man entweder das Handle out of scope gehen oder aber man holt alle Zeilen ab. (Zitat: You almost certainly do not need to call this method. Adding calls to "finish" after loop that fetches all rows is a common mistake, don't do it, it can mask genuine problems like uncaught fetch errors.)
3. Subroutinen ohne & aufrufen!