Wenn er RaiseError gesetzt hat, macht DBI im Fehlerfall ein die(). Von daher ist der eval-Block nicht ganz verkehrt.

Dann muss er jeden DB Aufruf in einen eval Block verpacken, ist das sinnvoll?

Hängt davon ab, was und wie man die Datenbankaufrufe macht.
Wenns nur in einem Skript ist wohl eher nicht, da hast du recht.

Hallo, es geht ja noch weiter :-)
Ich habe jetzt meine Datenbankaufrufe mit Platzhaltern realisiert, danke für den Hinweis und den Link auf's Wiki.
Zu dem eval-Block:
Die Datenbankzugriffe sind Funktionen einer DB-Klasse. mit new() erzeuge ich einen DB-Handle, der während der Laufzeit im Cace bleibt. Meine Überlegung war, dass ja nach initialisierung des Handles ein Verbindungsfehler entstehen könnte, daher eval().
Ist das richtig gedacht?

Gruß, tsy