[POO] - CLASSE D'ACCÈS BDD MULTI SGBD

Commentaire de garfield90 le 10/02/2009 17:00:28

Aime pas :
- la fonction exit dans une classe, leve une exception et laisse le developpeur le soin de gérer celle ci comme il le souhaite.
- le @ qui n'est la que pour masquer l'incompétence d'un développeur.
- l'existence de plusieurs classes dans le même fichier (ajoute de la charge pour rien)
- l'absence d'interface pour définir les méthodes des différentes connexions DB


Aime:
- l'utilisation de la portée

Commentaire de NewSky le 10/02/2009 17:18:10

Salut,

peu étonnant que toutes les requêtes ne fonctionnent pas sur tous les SGBD : aucun ne respecte intégralement la norme SQL (essentiellement avec la clause LIMIT) et chacun y va de sa petite implémentation.

Déjà, depuis PHP5, il y a PDO qui est déjà une librairie d'abstraction pour les différents SGBD du marché.

Il existe une librairie très simple, écrite sous le modèle objet de PHP4 : ezsql. Je l'ai utilisé pendant bon nombre de temps.

Aujourd'hui, je me tourne plutôt vers des librairies ORM complètes telle que Zend_DB ou encore Doctrine. il en existe bien d'autres : propel, phpMyObject, ... Une recherche "ORM PHP" dans ton ami google te donnera quelques résultats utiles.
En outres, ces librairies répondent à tes questions (cache, accès a la structure des DB, correction du SQL pour les différents SGBD, ...)

En ce qui concerne ta source, j'ai juste jeté un oeil furtif et je ne pourrait pas dire grand chose...

Commentaire de webdeb le 10/02/2009 17:25:11

Grilled by NewSky ^^

Commentaire de garfield90 le 10/02/2009 17:31:21

pour la levée d'exception, je vais te donner un exemple :
dans ta classe
function connexion(){
     if ( $this->Connexion =  mysql_connect(   $this->Serveur, $this->Utilisateur, $this->MotDePasse ) ){
     if ( false === mysql_select_db($this->Base, $this->Connexion ){
       throw new Exception('Base inconnue');
     }
  }else{
     throw new Exception('Connexion au serveur impossible');
  }
}


et dans ton code
try{
   $oDBConnector = new BaseDeDonnee('mysql', 'root', 'password', 'localhost', 'maDB');
   $oDBConnector->connexion();
}catch( Exception $e){
    // je traite mon exception i.e. la non connexion a ma base de donnée ou a mon serveur de base de donnée
}


Tu utilises rarement plusieurs moteurs de bases de données en meme temps, donc tu ne charges que les fichiers nécessaires (c.a.d. que si tu n'utilises que MySQL, aucun interet de savoir gérer Oracle ou SQLite). => Mémoire libérée.

Classe abstraite != Interface

Une interface te permet de faire un prototypage obligatoire dont dérive tes classes l'implémentant (une classe en php peut avoir plusieurs interfaces)
http://fr.wikipedia.org/wiki/Classe_(informatique)#Interface

une classe abstraite te permet de définir des méthodes et aussi dans implémenter quelques unes. Cependant cette classe n'est pas appelable directement.

Faire des interfaces et des classes abstraites te permet de découpler encore plus tes fonctionnalités.

Commentaire de webdeb le 10/02/2009 23:47:38

@Delphiprog : c'est bien pour cette raison que l'on mentionne que les derniers projets d'ORM du moment s'appuient sur PDO. D'autre part, il ne faut pas non plus dire que PDO est une couche d'abstraction de base de données. Ce n'en est pas véritablement une puisque les requêtes SQL écrites dépendent toujours du SGBDR avec lequel on s'interface via PDO. PDO est donc à mon sens principalement un connecteur commun pour un bon nombre de SGBDR du marché plus qu'une véritable couche d'abstraction de bases de données.

Commentaire de codefalse le 12/02/2009 09:35:15 administrateur CS

Bon, si l'on reste sur le code tel qu'il est proposé, il y a plein de points à revoir, par l'auteur, et par les "commentateurs" :

faire un
try {
   $object->function_that_throw_sth ();
}
catch (Exception $oE) {
   throw new Exception ($oE->getMessage ());
}

Ca ne sert à rien, à part éventuellement changer le type d'exception (si à l'origine, l'exception lancée n'est pas de type Exception mais une fille étendant la classe Exception). Si c'est une exception, tu ne la catch pas : tant qu'elle sera pas catchée, elle remontera dans les fonctions. En fait, ce que tu fait là avec ton code, php l'aurait fait sans ton try/catch.

@Garfield: "le @ qui n'est la que pour masquer l'incompétence d'un développeur.". Je suis pas d'accord avec toi. Je suis tout à fait d'accord sur le fait que mettre le @ cache la misère, et est la méthode de facilité pour le développeur, mais masquer l'incompétence d'un développeur, non.
Un exemple de pourquoi, la fonction "parse_url".
Elle te retourne un tableau de tous les éléments d'une url (le schéma (ex: http), l'utilisateur, le mot de passe, le serveur, le port, le path, le query, etc (exemple, dans l'ordre : http://user:pass@serveur:80/path?query). Si celui qui donne l'url, retourne une url mal formée (dans notre cas, si tu te connecte à ta base de donnée en fournissant un dsn de la forme url (mysql://root:pass@localhost:3306/dbname), et que cette url est mauvaise, la fonction va génerer une erreur de type e_warning. Tu ne l'as pas forcément voulue, et tu ne peux rien faire pour l'éviter. Le @ (ou modifier le error_reporting) semble être la seule façon de ne pas afficher l'erreur. Bien sur il faudra tester le retour par la suite, savoir si la fonction à bien fait son travail ou pas.

Dans tes classes, il manque un gros point (entre autres), c'est l'usage de la SPL. Elle va te permettre de faire des itérations (dans ton cas) pour retourner une liste de résultats de tes requêtes.

Je ne peux que te conseiller d'aller voir les couches d'abstractions faites par certains membres de CS, tel que FhX, Malalam, moi et probablement d'autres. Regarde comment c'est fait, regarde la structure, les différents usages, et tu verra la complexité du truc :)

Bien sûr, comme je le dis tout le temps, utilisez PDO !!
C'est à peine plus lent que mysql_query (de quasiment rien, donc négligeable), et ca vous permet d'avoir une architecture modulaire, qui vous permettra d'utiliser du mysql comme du oracle, et avec un ensemble de classes bien faites, votre application sera complètement adaptée, rien qu'en changeant le dsn de connection !

Bon courage pour la suite :)