### Brett vor'm Kopf ###

[Wolke]

Das ist leider sehr häufig das Problem: Es wird nur entwickelt, bis es funktioniert, und auch nur geprüft, dass es funktioniert, wenn alles korrekt läuft. Fehlerkonditionen geraten gerne aus dem Blickfeld.

Können aber auch zur endlosschleife werden,
weil es immer wieder etwas zu finden gibt dass
man noch besser machen könnte.

zb. Ich persönlich würde aus diesem ganzen Script
hier eine eigene Class machen, aber für Jemand
wie Phil wäre der Lerneffekt dann = 0.

Extrem nervig, wie ich finde. Verseucht den globalen Variablenraum mit unnötigen Variablen, deren sinnvolle Neubezeichnung man sich auch noch ausdenken muß - und an die man sich an anderer Stelle, wo es um den gleichen Wert geht, wieder erinnern muß.

Sobald es daran geht array Variabeln wieder in html einzubinden
ist Übersichtlichkleitsgewinn hierdurch enorm.
Zumal das integrieren auch einfacher ist anstatt die
Strings zu trennen oder mit geschweiften { zu arbeiten.
Die angesprochne Resourcenverschwendung befindet sich
im kaum messbaren Bereich.

Was ist, wenn in $_GET['pic'] etwas drinsteht, was PHP zu "false" evaluiert? Der String "0" beispielsweise fällt darunter.

Dann kann es getrost ignoriert werden, es sei denn man
rechnet damit, was aber speziell in diesem Fall zu
ignorieren wäre.

$pic = stripslashes($pic);
// Das darf nur passieren, wenn magic_quotes_gpc auf ON steht.

Ok, könnte man auch noch abfragen.

Ersetze "könnte man" durch "muß man".

Das verstehe ich nicht, warum ein Muß ?

// Der Aufruf von clearstatcache() an dieser Stelle ist mehr als sinnlos - er kostet bei Dateimetadatenoperationen (die du nicht benutzt) auch richtig Performance, wenn man ihn unbedacht anwendet.

Quelle? Die englische Version sagt nichts von "muß man". Und das fragliche Szenario hielte ich auch nicht für relevant.

Stimmt, weiss nicht ob es so jemals in einem alten
Manual drinstand, hatte es aber so aufgenommen und
immer benenutzt.

Von der Performanceeinbusse steht im Manul nichts, eben genau
umgekehrt, deshalb wird ja clearstatcache empfohlen,´aber erst beim mehrmaligen Aufruf für eine Datei.

readfile($bildpfad.$pic);
exit(); // keine weitere Ausgabe

// Das Skript wird nur dann abgebrochen, wenn tatsächlich erfolgreich ein Bild gefunden und ausgegeben wurde. In allen anderen Fällen läuft das Skript bei einem Request nach einem Bild weiter in den HTML-generierenden Teil.

Worauf bezieht sich das "Nein, soll es nicht"? Das Skript läuft in den HTML-Teil, wenn der gesetzte Parameter kein existierendes Bild beschreibt. Ist eine Unschönheit.

»

So ist dieses Script bereits aufgebaut.
Eine if else regelt das hier.
Einzige Ausnahme der Aufruf als Bild, hier soll
unter keinen Umständen das Script weiterlaufen.
Soll ja nur das Bild angezeigt werden. Und wenn das
nicht klappt kommt die Ausgabe gar nicht zum tragen.
Also bereits 100% gelöst.

Probier es aus.