Hast du vorher auch geschaut ob diese Subroutine ausschlaggebend ist für eine Optimierung? Also hast du mit Profilern das überprüft?

Unabhängig von deinem Code, wenn du eine Subroutine hast die bei gleichen Übergabeparamtern immer das selbe Ergebniss liefern kannst du das Modul "memoize" nutzen. Dies fügt einen Cache zu deiner Subroutine hinzu. Ein schlüssel zur Performanceverbesserung ist es, einmal berechnete Sachen nur einmal berechnen zu lassen und das Ergebnis dann abzuspeichern. Das Modul memoize erledigt soetwas für dich.

Wenn eine Antwort von einer Subroutine nur für eine bestimmte Zeit gültig sein soll gibt es auch Das Modul Memoize::Expire.


Ansonsten unabhängig von der Performance würde ich immer die 3-argumenten form von open() nutzen, und weiterhin lexikalische Variablen nutzen. also soetwas hier:

in soetwas umwandeln:



Ansonsten wie Renee bereits anmerkte würde ich ein bereits vorhandees Template Module nehmen:
HTML:Template
Template::Toolkit
HTML:Template::Compiled