leicht lesbarer Code, Spaghettitips

[Badboy46]

Hallo,

Die von mir gewählte Schreibweise

so sah es vorhin aber nicht aus ;-)

foreach ($von as $value)           ## Für jeden value aus von
 {
   $timestamp=strtotime($value);    ## timestamp bestimmen

if($timestamp != (-1))           ## Rückgabewert prüfen
   {
     echo "$timestamp<br>";         ## und ausgeben, wenn kein Fehlöer vorliegt
   }
   else                             ## sonst Fehlermeldungen ausgeben
   {
     echo "Datum <strong>$value</strong> ist fehlerhaft<br />";
   }
 }

was spricht jetzt dagegen, das z.B. so zu schreiben?

foreach ($von as $value)                                           ## Für jeden value aus von
 {
   $timestamp=strtotime($value);                                    ## timestamp bestimmen

if($timestamp != (-1)) echo "$timestamp<br>";                    ## Rückgabewert prüfen und ausgeben, wenn kein Fehlöer vorliegt
   else echo "Datum <strong>$value</strong> ist fehlerhaft<br />";  ## sonst Fehlermeldungen ausgeben
 }

bei den Rückgabewert muss ich dir Recht geben, dass das bei meiner Methode nicht so einfach wäre

naja.. ich glaub ich mach mich hier unbeliebt :-(