bonjour,
Et oui tout mes messages ont le même titre :-)
Je suis surpris d'une technique de codage que je voie pour la 2eme fois, que je trouve affreusement sale, mais car je l'ai vue 2 fois je me demande qd meme si ce n'est pas mon jugement qui est altéré. Donc votre avis m'intéresse :
Le site web contient une dizaine de page, et pour ne pas avoir de requete SQL éparpillées, une classe technique contient beaucoup de méthodes sont créées pour les différentes requêtes à effectuer. Pourquoi pas ? ca va dans le sens du MVC. Mais que penser du code html mélanger au code SQL dans ces mêmes méthodes ?
Code :
- class technique
- {
- // Génération de la liste des projets
- public function get_all_projet($default_project = "" )
- {
- $requete = "SELECT *
- FROM ".$this->table_projet;
- $result = $this->executeSQL($requete);
- $form = "<table width='50%' border='0' cellspacing='1' cellpadding='1' align='center'>";
- $form .= "<tr><td width='30%'><span class='txtform'>Choix du projet :</span></td></tr><tr><td><select name='id_p'>";
- while ($ligne = mysql_fetch_array($result))
- {
- $form .= "<option value='".$ligne['id_projet']."' id='".$ligne['nom_projet']."'";
- if ($default_project != "" && $ligne['id_projet'] == $default_project)
- $form .= " selected";
- $form .= ">".$ligne['nom_projet']."</option>";
- }
- $form .= "</select>
- <input id='sub' type='submit' name='submit1' value='configurer'/>
- </td></tr>
- </table>";
- return ($form);
- }
- // Récupérer l'id d'un projet
- public function get_id_projet($projet)
- {
- $requete = "SELECT id_projet
- FROM ".$this->table_projet."
- WHERE nom_projet = '".$projet."'";
- $result = $this->connect->executeSQL($requete);
- $id = mysql_fetch_array($result);
- return ($id['id_projet']);
- }
- }
|
Merci
---------------
Du tofu en Alsace : www.tofuhong.com