Il primo dei problemi e' che chiami la funzione prima di averla dichiarata, il secondo e' che chiudi la connessione all'interno del ciclo di estrazione while, per cui al massimo ti estrae solo il primo record.

Poi e' inutile fare una funzione per lanciare un'altra funzione. Lancia la funzione stessa. Semplifichi lo script.