Unübersichtlich und „aufgeblähter" Code?

[Auge]

Hallo

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

Würd ich nicht: "Die öffnende Klammer wird in der selben Zeile geschrieben wie die Bedingungsanweisung. Die schließende Klammer wird immer in einer eigenen Zeile geschrieben. Jeder Inhalt innerhalb der Klammer muß durch Verwendung von vier Leerzeichen eingerückt werden."

Wenn, wie Jörg vorschlägt, zwischen einzeiligen und mehrzeiligen Anweisungsblöcken unterschieden wird, ist diese Vorgabe Blödsinn.

if(empty($_POST['hersteller'])) { $errors[] = "Geben Sie bitte einen Hersteller an";
}
// Das sieht nicht aus!

// hier jedoch:
// einzeilig …
if(empty($_POST['hersteller'])) { $errors[] = "Geben Sie bitte einen Hersteller an"; }
// … vs. mehrzeilig
if(empty($_POST['hersteller'])) {
	$errors[] = "Geben Sie bitte einen Hersteller an";
	$badpoints++;
}

So geht's. Zudem ist es mMn gut unterscheidbar.

Tschö, Auge