COUCHE D'ABSTRACTION DE LA BASE DE DONNÉES ET GESTION ORIENTÉ OBJET DES ACCÈS AUX TABLES

Commentaire de malalam le 15/12/2006 16:50:37 administrateur CS

Hello,

j'ai pas tout maté, mais c'est un joli package, de ce que j'en avais vu.
C'eut été plus joli en PHP5, mais bon...faut bien du PHP4 :-)
Pas grand chose à en dire là, je n'ai pas assez approfondi.

Commentaire de malalam le 15/12/2006 22:43:44 administrateur CS

Re,

c'est gentil ça ;-)
Bon, j'ai maté un peu plus...mais pas encore testé, ni téléchargé à dire vrai donc je vais te faire quelques commentaires de plus, mais loin d'être exhaustif.
Je maintiens que ce serait vraiment chouette en PHP5 ;-) Avec des classes abstraites, de vraies constantes de classes, des ùéthodes abstraites (paskeu les méthodes qui retournent false parce qu'elles sont destinées à être surchargées...ça fait pauvre ;-) ).  Et des exceptions! Et un bel itérateur pour ta classe recordset...!
- 1er conseil : ajoute une doc! Et un ou plusieurs exemples d'utilisation. Parce que bon...déjà pour un bon codeur, faut un temps d'adaptation à mon avis pour utiliser ta classe...mais pour un moins bon codeur, même pas la peine d'y penser!
- ensuite : tu devrais laisser le choix, pour le log des requêtes. C'est bien d'avoir un logger. C'est mieux d'avoir le choix quant à son utilisation. Parce que moi je veux bien logger des update ou des insert, des truncate, des create table etc...mais des select, bof...je m'en fous un peu.
- mydatabase::quoteString () : attention... : c'est valable pour pas mal de base de données, dont mysql, et pas mal d'autres...mais pas pour certaines : au hasard, mssql. Ce qui fait que cette méthode ne devrait pas être définie ici. D'ailleurs, t'as un mysql::escape(), donc si ça se trouve un truc m'échappe là...mais bref, si tu fous un addslash sur une chaîne destinée à être insérée dans une base mssql, ben...ça va pas beaucoup t'aider pour les apostrophes.
- dans mysql, ou du moins la classe fille directement liée à la bdd utilisée, j'aurais vien vu des vérifications plus poussées sur les ressources. Via gettype (), et get_resource_type (), tu devrais bien t'en sortir en faisant bien gaffe...mssql_* encore, en php 5.0x, renvoie des entiers et non des ressources. Alors qu'avec php5.1+, t'as tes ressources).

Bon enfin...j'arrête là pour ce soir. C'est quand même très joli.

Commentaire de wizad le 16/12/2006 11:57:58

Zut Malalam à été plus rapide que moi... :)

Donc en php5 ce serait (déja dit mais je le redit quand même... gniark).

Sinon ça à l'air d'être du costaud et d'être vachement intérréssant.  Cependant une chose ma surpise : le nombre de fichier. mais bon ce n'est pas une critique dans le sens ou il est vrai que le système d'accés à un SGBD est généralement le plus lourd dans un site donc autant faire quelque chose de bien.

Sinon je regarderais le code et testerai plus tard (suis pas sur mon pc).

Commentaire de vanadium1 le 16/12/2006 14:35:59

La seule documentation qu'il y ait pour le moment, c'est les commentaires apportés au code (toutes les fonctions ont une entête aux standards Java) ainsi qu'un fichier exemple reprenant de façon très résumée les possibilités offertes.
Pour ce qui est du passage des informations de la base ça se déroule ainsi :
// instanciation d'un manager d'objets par inversion de controle
$link_m =& get_manager('link');
la fonction get_manager va inclure la classe définissant l'objet link et le manager d'objet link toute seule (suivant des conventions de nommage du fichier classe et des classes dans ce fichier).
dans class.object.php, tu remarqueras que dans le constructeur du manager d'objet nous avons :
$this->con =& DatabaseFactory::getConnection();
Ce code appelle la Factory qui s'occupe de l'instanciation d'une connexion à la base de données. On crée ainsi un découplage entre le type de base de données que l'on veut (SQL, PostGre...) et on laisse l'usine à connexion choisir pour nous le type de connexion approprié en fonction de $CONFIG['database'].
Database::getInstance() fait la meme chose que DatabaseFactory::getConnection();. Ce code est juste Deprecated, je l'utilisais dans un script et je l'ais conservé pour la compatibilité.
La Factory inclut le bon driver de connexion à la base et retourne une instance de celui-ci par inversion de controle suivant $CONFIG['database'] et en lui passant les paramètres de connexion à la base de données.
Le driver de base de données dérive de la classe MyDatabase qui est une interface indiquant les méthodes devant être implémentées dans les sous-classes (dans les drivers).

Commentaire de kankrelune le 18/12/2006 11:05:21

Ah la class d'abstraction SGDB de Xoops... bien quelle présente quelques lourdeur c'est ma préférée elle est assez simple à prendre en main mais là ou ça se complique c'est lorsque l'on veux l'utiliser avec la class critéria qui est plus complexe à apréhender... par contre je pense que plutot que te passer par un tableau (déclaré en global je suppose) tu aurais du garder les constantes qui sont bien plus efficace à mon sens (déclarés dans le fichier conf et basta)... .. .

Sinon l'idée est bonne je regrette juste le "manque d'innovation" si je puis dire ça ainsi vu qu'une très grosse partie du package vient de xoops et n'a, à première vue, quasiement pas été modifié... ce qui est domage car certaines parties de ces classes (je pense notament au modèle object de xoops) demande à être retravaillé pour alléger le bousin... cependant ça reste un bon package... .. .

@ tchaOo°