ダメなコード（自分主観）は…
・ショートタグ
・文字コードがShift-JIS
・extract
・タグを閉じたり開いたり忙しい
・echo "{$var}";
・ファイルの先頭・要所にコメントがない、
　あってもコード見たら分かる事しか書いてない（仕様を書かないと意味がない）
・1行がやたらと長い
・$_GETとか$_POST配列扱う時にisset使わず分岐したりしてる（警告出るちゅーねん！）

…書き出したらきりがないのでこのあたりにしておく