dedlfix: mehrspaltige Tabellen generieren

Beitrag lesen

echo $begrüßung;

Achtung: "quick'n dirty" und ungetestet ;)

Genauso sieht es auch aus.

$i=0;
$dir = '../../images/pics/'.$jahr.'/'.$category.'/';
$verz = opendir ($dir);

$fileFound = true;

echo "<table>";
while($fileFound) {

Selbst wenn es keine darzustellenden Inhalte gibt, wird eine Tabelle erzeugt, die eine Zeile mit vier leeren Spalten (wenn der Schleifenkörper nicht noch anderweitig grob falsch wäre). Das empfinde ich als überflüssig.

echo "<tr>";
    for($i = 0; i <= 3; i++)

Neben den vergessenen $-Zeichen läuft deine Schleife jedes Mal 4 Runden: 0,1,2,3
Das Anliegen, durch diese for-Schleife immer 3 (4) Spalten darzustellen, ist erstmal begrüßenswert ...

{
        echo "<td>";
        if($file = readdir($verz))
        {
            if ($file != "." && $file != "..")
                echo "<img src="" . $file . "">";

Das heißt also, für . und .. wird zwar kein Inhalt dargestellt, die Zelle existiert trotzdem.

else
                $fileFound = false;

und alles andere beendet dann die while-Schleife

}

wohingegen hier, wenn readdir() am Ende ist (oder '0' liefert) einfach weitergemacht wird bis zum Abbruch durch max_execution_time

echo "</td>";
    }
    echo "</tr>";
}
echo "</table>";

echo "$verabschiedung $name";