Der generelle Aufbau des Skriptes sieht mir ein wenig wirr aus.

Erst weist Du einzelne Elemente eines Array an skalare Variablen, um nachher diese Skalare wieder einem Array zuzuweisen. Warum nicht direkt zuweisen?



Du deklarierst innerhalb einer for-Schleife eine benannte Subroutine... Scheint mir auch nicht korrekt zu sein.

Du schreibst ein print hinter ein return in der Subroutine. Das print kommt nie zur Ausführung, weil vorher die Subroutine durch das return verlassen wurde...

Erstmal soviel nach dem ersten Überfliegen... Für eine genauere Analyse fehlt gerade die Zeit...

Eine gesunde Einrückung würde den Code auch lesbarer und übersichtlicher machen...