Unübersichtlich und „aufgeblähter" Code?

[Jörg Reinholz]

Moin!

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

danke,

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) {
   ...
}

Der Programmierer sucht sonst vergeblich den Array errors, an den Du was anhängst und PHP schmeisst sogar eine Notiz wenn es keinen der von Dir vorgesehenen Eingabefehler gab. Im Übrigen sind Deine Prüfungen etwas zu oberflächlich: Ein Leerzeichen reicht jeweils als Eingabe im HTML-Formular. Ich bin mir nicht sicher ob das Absicht ist, mag das aber nicht recht glauben.

Jörg Reinholz