Mit PHP Bild-Daten aus einem Ordner auslesen

[TS]

Hello,

<?php
$i = 0;
$verzeichnis = "img/";
$handle = openDir($verzeichnis);

##Fehlerhandlung fehlt

while ($datei = readDir($handle)) {

##Du besorgst hier Dateinamen. Bei einer Datei mit Namen "0" wäre hier Ende!

if ($datei != "." && $datei != ".." && !is_dir($datei)) {

hier wird im Prinzip doppelt geprüft, ob es sich um ein Verzeichnis handelt.

##Du könntest mit "is_file()" prüfen, das schließt aber auch symbolische Links mit ein, was hier aber unschädlich sein sollte, da Du keine rekursive Abfrage durchführst.

if (strstr($datei, ".gif") || strstr($datei, ".png") || strstr($datei, ".jpg")) { $verzeichnis_datei [$i] = $verzeichnis . $datei; $verzeichnis_datei [$i] [0] = getImageSize($verzeichnis_datei [$i]);

ich würde dann lieber mittels stristr() prüfen, um auch andere Schreibweisen für die Endungen zu ermöglichen ".Jpg, .JPG, ...). Das ist hier aber eigentlich ohnehin die falsche Funktion, denn Du willst doch wissen, wie die ENDUNG lautet, und nicht, ob ".jpg" irgendwo in der Mitte vorkommt.

##Nimm deshalb lieber if (strtolower([substr(](http://sg2.php.net/manual/en/function.substr.php)$datei,-4) == '.jpg')) oder eine ähnliche Konstruktion.

echo "<img src="$verzeichnis_datei[$i]" width="$verzeichnis_datei [$i] [0] [0]" height="$verzeichnis_datei [$i] [0] [1]"> <br>\n"; $i++; } } } closeDir($handle); ?>

Wie dedlfix schon schrieb, kannst auch ganz bequem glob() benutzen. Aber den Filter GLOB_BRACE würde ich trotzdem nicht unbedingt nutzen, solange das nicht caseinsensitive nutzbar ist. Ich würde lieber das erhaltene Array aller Dateien mittels eines Foreach() und dem oben schon erwähnten Filter durchgehen (alle nicht passenden Dateien aus dem Array herauslöschen).

So könnte man dann das ganze Konstrukt auch wirklich modular (in Funktionen oder Methoden) aufbauen, die jeweils die Daten in einem Array weiterreichen. Erst zum Schluss müsste dann in einer separaten Funktion die Ausgabe aus dem Array produziert werden. Die könnte man dann zu gegebener Zeit auch gegen eine mit DOM-Parser/Composer ersetzen, so wie Felix das fürs "HighEnd" gefordert hat.

Jedenfalls würde ich immer vermeiden, Datenbeschaffung und Ausgabegenerierung zu mischen. Das hat man nur solange getan, wie man insgesamt nur 8KByte Speicher zur Verfügung hatte. Die Zeiten sind vorbei!

Liebe Grüße
Tom S.