PHP Programmierstil

[faux]

Zitat von Matze:

also deine if-Abfragen (auch genannt Erdbeerkuchen @Elvis ) sind graußlich, sorry, das ist ja nur unübersichtlich.

Ich formatiere auch generell meinen php-Code und zeige dir, wie ich das mache. Dein Code formatiert. Falls ich das vom Server nehmen soll, dann sag es bitte und ich schmeiß es runter.

Also nachdem mich S2B darauf aufmerksam machte, machte ich mich sofort an die Arbeit und setzte auch zu einzeiligen if-Erdbeerkuchen die Klammern.
Im Prinzip sieht mein Code jetzt wie dein Vorschlag aus, nur dass ich die Zeilenumbrüche in einer zusammengehörigen Zeile weggelassen habe (außer bei den SQL-Statements).

Danke für die Vorschläge!

Grüße
Faux

[alcaeus]

Zitat von Matze:

Ich formatiere auch generell meinen php-Code und zeige dir, wie ich das mache.

Sorry Matratze, aber bei deinem gibts auch noch was:
es gibt einen Unterschied zwischen Funktionen und Schluesselwoertern. Und genau deshalb schreibt man sie auch anders:

Code:

$foobar = doSomething($foobar);

aber

Code:

if ($foobar === 5) {}

Nach Schluesselwoertern ein Leerzeichen zu machen erhoeht die Lesbarkeit, vor allem wenn jemand ohne Syntaxhighlighter deinen Code an sieht (ja, solche Leute soll es geben ).

Weiters gibt es was Funktionsaufrufe betreffen noch ein paar Unsitten, hier mal alle in einen Block geworfen:

Code:

$foobar=foobar( $foobar );
$foobar = foobar ($foobar,$bar,$foo);
$foobar = foobar( $foobar , $bar , $foo);

Am leichtesten zu lesen finde ich immer noch das:

Code:

$foobar = foobar($foobar, $bar, $foo);

Also Leerzeichen um Operatoren, kein Leerzeichen nach Funktionsnamen, keines nach/vor den Klammern, aber eines nach dem Komma zur Parametertrennung.

Dasselbe gilt bei Ausdruecken:

Code:

if ($a == 5 || $b == 3 && $c == 5)

Das ist IMO nur haesslich und verwirrend. Benutze Klammern:

Code:

if (($a == 5 || $b == 3) && $c == 5)

Manche Leute (ich gehoere auch dazu) setzen dann nochmal Klammern um die einzelnen Ausdruecke; dies ist aber nicht unbedingt noetig um die Lesbarkeit zu erhoehen (bzw. um zu garantieren dass der Code funktioniert)

Dann: wenn du einen String hast, der keine Variablen enthaelt, verwende ' anstatt ". Wenn der Parser einen String in " sieht, dann laeuft er drueber um evtl. vorkommende Variablen zu ersetzen. Mit ' gehts also ein kleines bisschen schneller

Greetz
alcaeus

[faux]

Hallo alcaeus!

Also ich kann keinen deiner Vorwürfe in Matze's Code finden.

Grüße
Faux

[Matze]

Hi,

ich auch nicht so direkt. Nach if steht bei mir generell ein Leerzeichen, bei Funktionaufrufen folgt die Klammer direkt nach dem Funktionsnamen und die Parameter fangen direkt hinter der Klammer an ohne Leerzeichen. Also so:

Code:

functionname($foo, $blubb)

Ich finde das ganz gut leserlich, aber ok, ich nutze generell einen Syntaxhighlighter. Wer keinen nutzt, ist selber schuld.

[faux]

Was sagt ihr zur Kommentierung?
http://phpfi.com/111833

[Matze]

Zitat von faux:

Was sagt ihr zur Kommentierung?

Ich finde sie übersichtlich und komme damit gut zurecht. Auch wenn ich meinen Code nie so ausführlich kommentiere. Ok, aber meist spricht der Code auch für sich selbst.

[Airblader]

Also ich finde das so gut kommentiert Mein Bruder kommentiert wesentlich mehr, für mich aber unverständlich.
So ist es imo lesbar für jeden.

air

[alcaeus]

Zitat von faux:

Also ich kann keinen deiner Vorwürfe in Matze's Code finden.

Code:

foreach($answers as $answer)

Greetz
alcaeus

[Matze]

Zitat von alcaeus:

Code:

foreach($answers as $answer)

Ok, da habe ich persönich generell ein Leerzeichen dazwischen. ^^ Hatte den Code auch nur schnell überflogen.

[faux]

Zitat von Matze:

Zitat von alcaeus:

Code:

foreach($answers as $answer)

Ok, da habe ich persönich generell ein Leerzeichen dazwischen. ^^ Hatte den Code auch nur schnell überflogen.

Okay, das ist aber mein Fehler. Habe ich übersehen. Mache ich sont nämlich auch immer. Ist aber ein Einzelfall: Ausnahmen bestätigen die Regel.

Grüße
Faux