Porchoman® is Back!

20 octobre 2008

7 ans d’éxperiences PHP pour un ingenieur ça donne ça…

Classé dans : Non classé — Squirrel @ 19:34 
Pour commencer, le fichier débute avec ce genre de choses...
Un joli doctype ^^"
  1. <!DOCTYPE HTML PUBLIC "-//SoftQuad Software//DTD HoTMetaL PRO 6.0::19990601::extensions to HTML 4.0 + WebTV//EN" "webtv.dtd">
Des assignations de variables aléatoires...
  1. if (isset($_REQUEST["aide"])) $aide=$_REQUEST["aide"];
  2. if (isset($_REQUEST["conf"])) $conf=$_REQUEST["conf"];
Aucune séparation affichage/traitement.
Des balises/attributs en upper case systématiquement.
Des balises <script></script> pour "protéger" le code et permettre de pré visualiser le rendu
HTML via l'IDE... mais bien sur, ça reste tel quel même une fois le développement terminé.
les fonctions scriptdebut() et scriptfin() sont aussi là pour ça. Le programme en question
est HotMetal Pro de SoftQuart, plus maintenu à jour depuis quelques années.
  1. <P><FONT FACE="Arial" SIZE="-1"><A
  2. HREF="<? echo $PHP_SELF; ?>?aide=1">Aide</A></FONT></P><SCRIPT>
  3. <?scriptdebut();
  4. /* Aide */
  5. if ($aide==1)
  6. {
  7. scriptfin();
  8. ?>
  9. </SCRIPT>
Pourquoi faire simple quand on peut faire compliqué....

  1. $fp_line = ereg_replace(chr(13),'',$fp_line);
  2. $fp_line = ereg_replace(chr(10),'',$fp_line);
  3. $fp_line = ereg_replace(' ','',$fp_line);
A ceci s'ajoute une indentation totalement farfelue, une conception boiteuse.
Le plus amusant dans tout ceci, est que la personne ayant réalisé ce code, est un ingénieur ayant passé 7 ans à programmer en PHP (entre autre).

3 commentaires »

  1. Mais c’est abbérant quoi (en même temps ca date de .. ::19990601:: [c'est bien ca ? ^^])

    Commentaire par Alexis Voisin (hackFr0sT) — 24 octobre 2008 @ 4:22

  2. la date n’est jamais mise a jour xD. le fichier est du 20080725

    Commentaire par grepsd — 24 octobre 2008 @ 7:40

  3. Heu…tu as pensé à te relire avant de poster ?

    Commentaire par Kristofik — 24 octobre 2008 @ 8:04

Flux RSS des commentaires de cet article. URL de rétrolien

Laisser un commentaire