Hallo, ich habe ein kleines Modul geschrieben, welches es mir erleichtern sollte, meine thumbnails zu verwalten. Könnt ihr euch das bitte mal anschauen und sagen, was man wo verbessern kann? Schießlich will man ja immer dazulernen ;)

Hab es mal hier zwischengespeichert: http://www.rafb.net/paste/results/paAdpN82.html

zwei dinge sind mir bei new aufgefallen

1. mkdir setzt kein Rechte
2. das tmpdir wird nie gelöscht... nicht gut

In die SYNOPSIS könntest du reinschreiben, wie man das Modul typischerweise benutzt --- im Idealfall muss der Benutzer dann nur Copy'n'Paste machen.

Es würde sich auch anbieten, aus dem Modul gleichzeitig ein ausführbares Skript zu machen. Einfach aus der letzten Zeile "return 1;" ein machen. Danach könnte der Skriptcode folgen (wahrscheinlich wieder das gleiche wie in der SYNOPSIS, mit ein bisschen Option-Handling).

Temporäre Dateien und Verzeichnisse sollten nur mit File::Temp erzeugt werden. Das hat einige Vorteile (funktioniert auch unter Nicht-Unix-Systemen, keine Möglichkeiten von Exploits, unterschiedliche Benutzer auf der gleichen Maschine kommen sich in die Quere).

Du erlaubst anscheinend nur .gif und .jpg, aber es gibt auch andere populäre Bildformate (png!) oder andere Schreibweisen für die Extension (.jpeg). Du könntest, falls der User das Modul installiert hat, Image::Info benutzen, um festzustellen, ob die Datei ein Bild ist. Oder vielleicht hat Image::Magick eine ähnliche Funktion.

PS: der Syntaxhighlighter auf der Webseite kommt mit PODs nicht klar.

würde ich noch verwenden .
Und dafür:
kannst du IMHO auch den /i switch verwenden