MYSQL_SELECT + MYSQL_FETCH_ARRAY.. SIMPLIFIÉ

Commentaire de kankrelune le 30/07/2006 22:11:12

Salut...

Je vois pas trop l'interet de réinventer la roue... tu aurais fait une classe d'abstraction SGDB d'accord mais il n'y aucun interet de faire une classe pour encapsuler les fonctions sql... c'est faire de l'objet histoire de faire de l'objet... tu gagne peut être en charactères à taper pour coder mais tu perd en performances... .. .

function sql()    
{
    /* Connection a la base de donnée */
    $connect = @mysql_connect(host_sql, login_sql, passwd_sql);

D'où viennent host_sql, login_sql et passwd_sql... pareil pour db_sql... .. ?

Personnellement je trouve incorrect que ta classe gère l'affichage des erreurs... ce n'est pas son role... qui plus est si la connection échoue tu affiche un message d'erreur... c'est bien (quoique) mais tu ne shunte pas le mysql_select_db ce qui fera planter ton script... ou alors il faut remplacer le echo par un die mais pour un site pro en production c'est crade... .. .

incorrect => echo "<span style='color:red; font-size:20px;'>...
correct => echo '<span style="color:red; font-size:20px;">...

Quel interet de mettre le mysql_fetch_array dans la methode select()... aucun... il faut faire une methode pour éxécuter les requete et une pour les traiter... .. .

SELECT * est à éviter car gourmand en resources d'autant plus qu'il n'est pas utile de rapatrier le contenu de tous les champs d'une table si tu n'a besoin que du contenu d'un seul champ... et tu fais comment si tu veux rajouter une clause WHERE... .. ?

Voili voilou... je te conseil de jeter un coup d'oeil à la classe qu'a posté FhX sur ce site elle est beaucoups plus complete... .. .

^_^

@ tchaOo°

Commentaire de FhX le 30/07/2006 23:47:12

Jmy met moi aussi :

function select($query, $sql_result) { // }
Méthode à 2 arguments et pourtant :
while($row = $sql->select("SELECT * FROM table"))

^==^ la haut y'en a qu'un seul ! Déja y'a un problème :)

"#  if (!$sql_result)
#         {
#             $this->sql_result = mysql_query($query);
#         }"

Tu fais un test sur quoi ? J'ai pas tout saisi la ... !

JE vois ce que tu veux faire. Pour faire ce que tu veux, tu dois faire quelque chose de plus grossier :

class mysql {
public $query;
public $sql;
public $iterate = FALSE;

// J'épargne la connection.

// $sql est la chaine SQL.
public function query($sql) {
  
  if ( ! $this->iterate ) { // On vérifie si y'a pas déja eu un query()
     $this->sql = $sql;
     $this->query = mysql_query($this->sql) or die (mysql_error());
      if ( mysql_num_rows($this->query) > 1 ) $this->iterate = TRUE;
  } // Si y'a plus d'un enregistrement (donc faut boucler, on initialise Iterate pour éviter de refaire une requète.
  
  if ( FALSE !== ( $data = mysql_fetch_array($this->query) ) ) {
       return $data; // Si ca boucle tjs, on renvoit
  } else {
       $this->iterate = FALSE; // Sinon on stop.
  }

}

}

Bien entendu, ne jamais utiliser la méthode que j'ai mise :) C'est juste pour l'exemple lol !