Naja, den Programmierstil kann man immer nur subjektiv bewerten, also hier meine Meinung: Ich finde den Code ehrlich gesagt nicht wirklich übersichtlich. Vor allem bei den if-Schleifen habe ich Probleme nachzuvollziehen, was denn jetzt wann gemacht wird.
Ich halte es mit if-Schleifen immer so, dass ich grundsätzlich geschweifte Klammern verwende und diese auch immer in eine neue Zeile schreibe:
Code:
if ($variable == 'test')
{
echo 'Variable enthält "test"';
}
else
{
// sonst was...
}
Im Prinzip kann ich sagen, dass ich mich nahezu komplett an die
Coding Guidelines vom neuen phpBB halte, vielleicht findest du ja dort auch noch den einen oder anderen Tipp.
