PHP Worst Practice Beispiele
Kommentare

Im PHP Subreddit haben wir wieder eine Perle entdeckt, die wir mit Euch teilen müssen. User Ozymandias-X hat mit seinem Kollegen auf Arbeit ein Script erstellt, das gegen alle Regeln verstößt, die er in seiner Erfahrung mit PHP gelernt hat.

Nun hat er mit anderen Redditors eine Liste zusammengetragen, die die Worst Practices in PHP beinhaltet. Ozymandias-X macht den Anfang mit folgenden Bonbons:

  • Multiple class definitions in a single file
  • Recreating register_globals in usercode (by parsing $_REQUEST in a foreach loop)
  • Inserting these created globals right into an SQL string, without any kind of validation or cleanup (little bobby tables would be proud of us)
  • directly requiring another class-file inside a method of a class
  • A mailing class that gets everything in its constructor method and sends immediately (no methods beyond __construct)
  • Saving passwords unhashed and unencrypted in a database
  • Using a global variable inside a class to get a database connection
  • No comments and where there are comments unhelpful comments like „this signs up a user“ on a method called „signup“
  • One letter variables
  • Reusage of variable names for different things
  • Pointlessly setting the signup method to being static
  • Automatically setting a $_SESSION[‚logged_in‘] = true right in the signup method, no double opt in or anything

Die Redditors schlossen sich begeistert an und übertrumpften sich in der Abgründigkeit ihres Codes. Weitere Goldstücke sind die Folgenden:

function add($i, $iAmount) {     eval("$i = $i + $iAmount;");     return $i; }  $i = add($i, 1);
$r = mysql_query(); while($r = mysql_fetch_row($r)) {     do_something(); }
function do_something() {      global $r; }

User RyanDywer verlängert die Liste um etliche weitere Einträge und sorgt für Schaudern in der Leserschaft. Zu allem Überfluss hat er seine Punkte als Tipps verpackt. Also könnt Ihr sie hier herunterladen, ausdrucken und Euren Kollegen als Best Practice Hinweise in die Kaffeeküche hängen; wobei das wahrscheinlich nur am ersten April lustig rüberkommt.

Aufmacherbild: Waste dumped in the countryside von Shutterstock / Urheberrecht: Jon Le-Bon

Unsere Redaktion empfiehlt:

Relevante Beiträge

Meinungen zu diesem Beitrag

X
- Gib Deinen Standort ein -
- or -