CLASS HELPER

Commentaire de malalam le 28/01/2006 10:24:36 administrateur CS

Ensuite, sans vouloir paraître trop critique... : là, tu crées 25 classes différentes, avec 4 méthodes différéntes...qui font la même chose pour chaque classe...c'est pas très POO ça. Manque une classe d'abstraction générique, dans ce cas. Et pourquoi pas une interface. Et tes méthodes devraient avoir le même nom.

Commentaire de galadriann le 28/01/2006 20:15:07

j'oubliais ... je fais des classes differentes car chaque table va interagir differemment avec d'autres ... certaines vont appeller un objet existant (genre asset -> asset_type) et d'autre vont etendre d'autres tables (pc -> asset ou  server -> pc -> asset) ...

Si tu as le temps, je te montre le design complet de la base et tu me dis ce que tu en penses ...

a+

Commentaire de FhX le 30/01/2006 16:22:40

Moi, ce qui me fait le plus peur, c'est quand je vois ca :
"global $db;"

La dernière fois que j'ai fait ca, j'ai eu le droit à une belle erreur dans une langue totalement inconnu :)
Sachant que $db est un objet, utilise plutôt le singleton pour pouvoir récupérer ton objet plus facilement et surtout plus rapidement ! (car tu gagnes des ressources au passage).


Pareil :
if (isset($id_assettype))
        {
            $where='WHERE id_assettype="'.$id_assettype.'"';
        }
$sql="SELECT id_assettype, partnumber, modeltype, id_class FROM assetype $where";
Que c'est bien de la ressource de perdu :)
Regarde plutot :
$sql = 'SELECT id_assettype, partnumber, modeltype, id_class FROM assetype ';
if (isset($id_assettype) ) {
    $sql .= 'WHERE id_assettype="'.$id_assettype.'"';
}
$db->get_results($sql);


Au passage, si tu dois faire ce morceau de code :
if ($debug) {$db->debug();}
        return($res);

}
A chaque fois que tu fais une requète, c'est que ta classe est incomplète. Tu dois faire un test d'erreur dans ta classe de DB et certainement pas à l'extérieur. Sinon tout le coté objet perd son interêt ici ! (en plus du reste).

Bref, je rentre pas dans le code car au peu que j'ai vu, je préfère m'arreter la :)
Si tu veux des idées de classes SQL, tu peux les trouver facilement. C'est certes un peu plus compliqué mais pleinement fonctionnel :)

Commentaire de malalam le 31/01/2006 09:24:49 administrateur CS

Il faudrait que tu passes ton objet $db à ta classe
public function __construct (& $db, ...)

Commentaire de FhX le 31/01/2006 23:10:14

Ah non !

Pour passer un objet par référence, il faut utiliser le singleton (soit en francais l'instance unique) soit le multiton (soit l'instance unique multiple).

Le singleton (ou le multiton) va te permettre d'utiliser ton objet dans un contexte global. En effet, au lieu de faire :
$db = new mysql;
tu va devoir faire :
$db = mysql::NomDeLaMethodeQuiCreerLinstance();

De ce fait, c'est un peu comme si tu faisais une copie d'objet. Un clone en quelque sorte en PHP4 :)
Donc après, tu peux appeler ton objet de n'importe où sans rien perdre au niveau vitesse contrairement à un passage par référence via une méthode de classe où d'une fonction.

De plus, toutes les méthodes font passer les arguments par références Malalam, je te l'ai déja dit :) Donc le & n'a plus lieu d'être =)

D'ailleurs, un constructeur de classe quel qu'il soit n'a PAS à récupérer un objet de connection SQL dans ses arguments, ni même dans ses propriétés. Ca reste de la duplication sinon, et ca n'apporte rien mise à part une perte de mémoire :)

Voila voila !