ja, die Zeilen 12 und 13 werden nie ausgeführt. connect/disconnect würde ich sowieso globaler machen, nicht in einer Funktion, das sind sehr Zeitaufwändige Funktionen, die mn nur einmal pro Skript aufrufen sollte..

und die Zeilen 3 und 4 kannst du rausschmeissen

[EDIT]Ach und noch was. Niemals "$var" schreiben, das ist eine völlig unnötige Interpolation. In keiner Programmiersprache musst du die Variabeln in Anführungszeichen schreiben.