Sauber Programmieren

[BloodReaver]

Original geschrieben von TheUser
OOP

da wo es sinn macht

toll sowas kann man sich aber schenken *fg*

ich mein man sollte ALLES egal was nur da einsetzen wos sinn macht

[Hopka]

Hinweise für sauberen Code gibt es auch bei PEAR:
http://pear.php.net/manual/en/standards.php

[Floriam]

Mal ne Frage zum Thema:

Was ist sauberer

if($foo = $fooz)
{
echo "richtig";
} 

oder

if($foo = $fooz) {
echo "richtig";
} 

[mrhappiness]

das untere ist verbreiteter

sauber ist beides ziemlich gleich

aber ich würd mit == vergleichen und nicht mit =

[Happy Nihilist]

Und TAB benutzen

[TobiaZ]

jepp

es ist dann auch egal, an welcher stelle du dein else einsetzt.

[Floriam]

aber ich würd mit == vergleichen und nicht mit =

Wir sprechen aber vom Aussehen, nicht vom Code

Und TAB benutzen

mach ich auch immer, nur hier hab ich es mal mit der Hand im Post geschrieben, und da fällt sowas nicht auf.

es ist dann auch egal, an welcher stelle du dein else einsetzt.

Mhn, versteh ich nicht ganz, es ist doch nie egal wo man sein else einsetzt


Floriam

[Titus]

Zum sauberen Programmieren gehört auch Fehlervermeidung. Und wenn ein Fehler auftritt, sollte man den Besucher nicht unbedingt mit einer PHP-Fehlermeldung belasten; dafür aber selbst erfahren, dass und wo (uind warum) ein Fehler aufgetreten ist:

• ab 4.1 IMMER die Superglobals ($_GET/$_POST/$_COOKIE/$_REQUEST/$_FILES) benutzen, um an HTTP-Parameter zu kommen. Man kann ja nie wissen, ob man nicht mal zu einem Provider umzieht, bei dem register_globals ausgeschaltet ist. (Dafür muss zwar track_vars=On sein, das ist seit 4.0.3 im Gegensatz zu register_globals seit 4.1 der Default; und damit steigen die Chancen, dass die Werte erreichbar sind.)
• Die Benutzung von mit global eingebundene Variablen wird bei längeren Funktionen schnell unübersichtlich; daher benutze ich gern den Superglobal $GLOBAL.
• Bei MySQL-Queries immer or die (mysql_error()) anhängen oder eventuelle Fehler auf andere Weise abfangen.
• Für Live-Websites ist die Verwendung eines eigenen Error Handlers angebracht.
  Der könnte z.B. auf ein Fehler-Skript weiterleiten und/oder eine Mail mit den Inhalten der Superglobals (damit hat man mehr Info, um rauszufinden warum es nicht funktioniert hat) verschicken.
• ganz gut zum Fehlertracking in dem Zusammenhang:

  $fnstack = array();
  function call($file, $line, $php)
  {
    // diese Funktion ist (noch) kurz genug für global statt $GLOBAL
    global $fnstack;
    $fnstack[] = array($file, $line, $php);
    eval($php);
    array_pop($fnstack);
  } 

  Damit kann im Error-Handler ganz einfach rausgefunden werden, welchen Weg der Interpreter bis zum Fehler genommen hat, solange man bei möglichen Fehlerquellen funktionsaufruf(parameter) durch call(__FILE__, __LINE__, "funktionsaufruf(parameter)"); ersetzt.
  Ein Problem ist natürlich, dass das die Ausführungszeit um einiges erhöht, wenn man´s zu oft einsetzt; und es wär schöner, wenn man einfach den Stack des PHP-Interpreters auslesen könnte ... und seit 4.3 geht das auch mit der Funktion debug_backtrace

[TobiaZ]

Mhn, versteh ich nicht ganz, es ist doch nie egal wo man sein else einsetzt

naja, ich meinte damit, ob du

} else {
  bla
} 

oder

}
else {
  bla
} 

oder halt

}
else
{
  bla
} 

nimmst.

ein else kann man natürlich nur da verwenden, wo ein if ist. das ist ja klar!

[seballa]

Original geschrieben von Happy Nihilist

.....

Aber mal ein paar Sachen, die ich wie die Pest hasse:

......

- Alles-in-eine-Zeile-Packer

$query = "SELECT * FROM table WHERE name = '". $var_a.
         "' AND time = '". $var_b."' OR wurst = '". $schnuppenpoo.
         "' OR kindergarten = 'true' ORDER BY iq LIMIT 0,3"; 

- Code-Plenker

$foo="bar";
$query="SELECT * FROM tabelle WHERE id='".$foo."'"; 

;

1. Wie würde ich denn die Query besser machen im mehren Zeilen?
2. Was ist ein Code Plenker? Hab das Beispiel nicht kapiert.

-Seballa

[rythms]

1. alle mysql befehle groß schreiben.

2. mysql querys einrücken:

SELECT
    foo,
    bar
FROM
    tab
WHERE
    ka = '5' AND
    (
        punkte > '0' OR
        fehler < '0'
    )
ORDER BY
    Datum DESC,
    punkte ASC
LIMIT
    10,10; 

3. sich bei funktionsbennenung entweder für studlycaps (function fooBar()) oder underscores (function foo_bar()) entscheiden.

4. PEAR Coding Standards beachten.
5. PHPDoc Kommentarstil beachten und ! benutzen.