Na ja, ich würde bei der Bewertung ob „Fehler“ oder nicht mehr darauf schauen, worauf eine entsprechende PHP-Ausgabe im Rahmen der Anwendung hindeutet. Hier zeigt die Nachricht vermutlich an, dass die auszulesende Datei nicht existiert. Wenn die Anwendung aber von der Existenz ausgeht, handelt es sich schon in gewisser Weise um einen Fehler. Der führt zwar hier nicht unmittelbar zum Scriptabbruch. Aber vermutlich ergeben nachfolgende Operationen ab der Stelle keinen Sinn mehr. Das ist dann kein „harter“ Fehler wie etwa ein Syntaxfehler, aber dennoch eine Begebenheit, die die Anwendung in einen „undefinierten“ oder unerwarteten Zustand versetzt. Vielleicht einigen wir uns auf: Es ist wahrscheinlich ein Bug.
Ich bekenne mich schuldig, die Tastenkombination auch schon gedrückt zu haben, aber ein @ zur Unterdrückung einer Fehlerausgabe ist trotzdem ein No-go, wenn wir über „sauberen“ Stil reden. Es sollten immer die Ursachen behoben werden. In diesem Fall wäre das wohl das Ausformulieren einer Prüfung auf
file_exists.
filesize dafür zu „missbrauchen“, erscheint mir verkehrt.
Ob das innerhalb der Funktion getan wird oder vor dem Aufruf, ist dann noch mal eine ganz andere – aber sehr interessante – Frage. Da kommt man dann aber schnell in Bereiche von Fragen wie: „Was passiert, wenn ich die Datei zu dem Zeitpunkt lösche, der zwischen dem Aufruf von
file_exists und
filesize liegt?“ Das ist eine Variation von: „Was passiert, wenn genau dann und dann der Strom ausfällt?“ – Bei sowas wird man paranoid und bekloppt. (Siehe dazu auch meinen Post zum Thema „Fehlerbehandlung“ neulich:
http://www.html.de/php/41468-fehlerbehandlung.html#post299454.)
Gerade vor dem Hintergrund ist es aber nicht ganz illegitim, die Fehlermeldungen zu unterdrücken, denn es ist im Code sonst wohl nicht möglich, zu verhindern, dass unter gewissen – meinetwegen sehr unrealistischen – Umständen eine Notice bei etwa
filesize geworfen wird – nämlich dann, wenn die Datei in dem „CPU-Takt“ zwischen
file_exists und
filesize gelöscht wird. (Wobei ich nicht genug Ahnung habe, um sagen zu können, ob und wie realistisch das Problem selbst in dem unrealistischen Fall ist, dass die Datei überhaupt gelöscht wird.)
Ein Stichwort dazu wäre jedenfalls:
-
PHP: set_error_handler - Manual
E_NOTICE wird im Handbuch übrigens auch zu den „error level constants“ gezählt. Es ist letztlich Haarspalt… Definitionssache, ob man die Dinger als Fehler bezeichnet oder nicht.
Pauschalaussagen sind schwierig, aber: In der PHP-Welt ist es heutzutage Standard, „
E_STRICT-kompatiblen“ Code zu schreiben. Das bedeutet: Notices sind erst mal als Bugs anzusehen. Es ist nicht beabsichtigt, dass der Code welche auslöst.
Ob ein solcher „Bug“ dabei zwangsläufig seinen Ursprung im PHP-Code haben muss oder in irgendeiner sonstigen Serverkonfiguration, ist dann auch mehr oder weniger Definitionssache… Ist es die Verantwortung eines Scripts, zu prüfen, ob etwa ImageMagick auf einem Server installiert ist? Für mich ist das ganz schwer zu beantworten.
