Unübersichtlich und „aufgeblähter" Code?

[Gunnar Bittersmann]

@@Jörg Reinholz

Da wäre aber noch was. Notiere mindestens:

> $errors=array();
> ...
> if(empty($_POST['hersteller'])) { $errors[] = "Geben Sie bitte einen Hersteller an"; }
> if(empty($_POST['url']))        { $errors[] = "Geben Sie bitte eine Hersteller URL an"; }
> if (! empty($errors) {
>    ...
> }

Das Sammeln von Fehlermeldungen kommt mir zweifelhaft vor. Ich ahne Schlimmes.

Was soll innerhalb von if (! empty($errors) {} stehen? Die Ausgabe aller Fehlermeldungen in einem Block?

Nei-en!!

Es ist dem Nutzer nicht zuzumuten, sich bei einer Fehlermeldung das entsprechende Eingabefeld selbst im Formular suchen zu müssen. Und womöglich mehrmals. Und das womöglich in einem längeren Formular, wo der Nutzer den Fehlermeldungsblock (am Seitenanfang) und die Eingabefelder nicht in einem Blick hat und hoch- und runterscrollen muss.

Die Meldungen gehören jeweils direkt an die entsprechenden Eingabefelder, wo der „Fehler“ zu berichtigen ist.

Da erscheint es mir unsinnig, die Fehler_meldungen_ in einem Array zu sammeln.

LLAP