Du verwendest Variablen in der Sub, welche jedoch gar nicht als Parameter übergeben wurden. Das ist nicht nur schlecht, weil du scheinbar globale Variablen verwendest, es ist sogar sehr schlecht, weil so niemand sagen kann, welche Variablen in welcher Sub benötigt und/oder nicht benötigt werden. Anstatt die Übersichtlichkeit und Wartbarkeit deines Codes zu erhöhen, bewirkst du so genau nur das Gegenteil davon.