GESTIONNAIRE DE MODÈLES SQL

Commentaire de destinyfr le 26/11/2008 19:15:39

Salut

Déjà merci pour l'exemple :)

J'ai regardé ta classe, et d'après moi, le gros avantage de ta classe c'est que tu déclare qu'une seul fois la connexion à la base dans une instance que tu utilise ensuite dans les autres fichiers. C'est bien pensé, le problème c'est que cette classe est plutôt faite pour les bases de type mysql, car dans le constructeur, tu vérifie que tous les champs nécessaires à la connexion existent, or si c'est du sqlite, un seul paramètre est utilisé.
Ce qui est bien, c'est le chargement de classes, tu charge la ou tu veux la classe nécessaire à ce que tu veux faire. Peut-tu par contre m'expliquer un petis poins dessus ?

$this->_aLoadedClasses[$sFile] = new $sFile ($this->_oPdo); -> A quoi cela sert t'il ?

Pour ce qui est de la différence avec ma classe, c'est que tu utilise 2 classes principales qui vont ensuite gérer les modules ce trouvant dans le dossier spécifié. C'est très pratique si tu as beaucoup de classe :)

Je vais essayer d'améliorer ma classe en me basant sur ton exemple :) (notamment pour la gestion des classes avec la fonction load).

Merci pour cette exemple, et merci de répondre à ma question ^^

Commentaire de destinyfr le 26/11/2008 21:00:00

Eu en faite oublie cette question débile XD je devais bader devant mon PC à ce moment (j'ai mal à la joue et des traces carré :o). Il est vrai qu'il n'y a pas de difficulté ici :)

En revanche j'étais sur que tu utilisé un variable pour rien, mais j'ai encore rêvé XD, elle est utilisé sur la ligne du dessous looooool.

Enfin merci quand même pour la réponse ^^.

Pour ce qui est de sqlite, étant donné que tu utilise isset pour vérifier si c'est existant, sqlite devrait passer. Sinon il suffisait juste de faire un petit teste : si user, password etc... faire cette action, sinon faire celle la.

Commentaire de destinyfr le 27/11/2008 07:18:20

Eu oO à quoi sert PDO dans la fonction construct dans PdoModel.php ?

Commentaire de codefalse le 27/11/2008 14:23:03 administrateur CS

Merci pour ton commentaire et ta notes :)

Pour ce qui est du manque de notion d'ORM, c'est volontaire. Le code présenté ici n'à que pour but de séparer le modèle du reste du code (si l'on prends une structure MVC). Un ORM serait une sorte de complément, extension au code présenté ici. La plupart des frameworks actuels proposent une séparation des différentes couches sans imposer un ORM, mais le propose en plus, si l'utilisateur le désir.

De toute facon la partie ORM doit plus te concerner d'après tes sources n'est-ce pas ;)

Cependant, ton histoire avec les pointeurs me perturbe, car à t'écouter, il faut mettre des & partout dans le code, genre une cinquantaine par lignes ;)

Pour le typage, j'avais entendu dire que php ne supportait que le typage de classes (comme PDO), pas des autres éléments (array, string, etc). D'ou une abscence de typage pour le Array et la présence pour PDO !
Va falloir que je regarde cela de plus prêt.

Tu aurais un article pour l'histoire des pointeurs, histoire que je comprenne mieux le pourquoi du comment s'il te plait ?

Merci :)

Commentaire de codefalse le 27/11/2008 15:27:11 administrateur CS

J'utilise la méthode magique __clone dans PdoManager mais pas dans PdoModel en effet, je vais regarder cela de plus prêt.

Pour ce qui est du typage, c'est une bonne nouvelle !
Merci de l'info :)

Commentaire de aKheNathOn le 27/11/2008 18:09:30

Non destiny, t'inquiettes c pas compliqué.

Déjà les variables des fonctions, la plus part du temps tu peux avoir :

function maFonction($var1, $var2) {
...
}

Mais à aucun moment tu ne dis à quoi correspond var1 ou var2 en tant que type de variable - du coup php accepterais que t'appelles ta fonction ainsi :
$resultat = maFonction(true, 10);
ou $resultat = maFonction("Salut !", new PDO(...));

Tu vois, on peut à peu près tout faire, et dans la pluspart des cas elle va planter à moins que tu tartines des tonnes de vérifications pour expliquer la vraie cause de l'erreur.

Ben sinon depuis php 5 tu peux typer tes arguments :

function maFonction(array $var1, string $var2) {
}

Si c'est un objet, tu indiques le nom de la classe, comme ...(PDO $varPDO)...

Commentaire de destinyfr le 27/11/2008 18:32:38

Salut merci pour l'explication :) Cependant, pourquoi ne pas avoir marquer cette fonction :

function maFonction(int &$argument) {
  $argument = -99;
}

Comme ceci :

function maFonction(int &$argument) {
  $argument =& -99;
}

Comme tu la fait dans cette exemple :

public function __construct (PDO &$oPdo) {
  $this->_oPdo =& $oPdo;
}

Commentaire de aKheNathOn le 28/11/2008 09:49:47

Lut malalam, tj un plaisir de lire tes commentaires très précis :)

Toujours en allant dans le sens de malalam, j'aurais trouvé ça fun de pouvoir directement faire :

$oPdoInstance->Users()->Update('...');

Une vérif aurait checké si users est chargé en tant que module, dans le cas contraire il l'aurais instancié et mis en cache. Il en retournerais son instance :)

Destiny, en réalité je me suis pas mal penché sur l'optimisation de certains codes dans des frameworks et je me suis rendu compte que même si PHP gère automatiquement les pointeurs, ce n'est pas toujours fait, du coup je prends des précautions :

Exemple :
public function __construct (PDO &$oPdo) {
  $this->_oPdo =& $oPdo;
}

Dans l'argument je lui dit que j'ai besoin d'une référence à l'objet pdo et non d'une valeur clonée. Ensuite je le sauvegarde dans une variable privée de ma classe, mais toujours en tant que référence. Sinon, dans certains cas il me duplique l'objet qui est pourtant une référence.

Malalam, je ne sais pas si ça relève du bug ou bien s'il faut bien tout déclarer, la plus part du temps je ne le fais pas - du coup des fois j'ai des perfs flinguées et des résultats incohérents.

Tu peux tester, la version SVN est à cet emplacement :
https://dotview.svn.sourceforge.net/svnroot/dotview/trunk

T'as rien à installer, juste déclarer le répertoire démo en tant qu'allias demo puis aller à sa racine.

Ca va te générer une erreur normale vu que t'as pas configuré les bases de données - et à droite de l'erreur t'as le debug stack. Il est vide...

dans dotview/core/debug.class.php modifies ça :
... &getInstance()... ligne 12

dans dotview/core/core.class.php
... &debug() ... ligne 8

Ce qui est étonnant c'est que le comportement change si on utilise une version antérieur de PHP.

Commentaire de codefalse le 28/11/2008 14:15:05 administrateur CS

Content de voir que mon code apporte de nouveaux code :p

Vos idées sont intéressantes et méritent réflexion. Je ne sais pas trop encore comment je pourrais m'y prendre, mais c'est vrai que jouer directement avec l'instance sans la retourner, ce n'est pas mal.

@Malalam : Ca faisait un moment non ? :)

Personnellement je préfère un mix de vous deux, c'est à dire un :

$o->attach ('modA');
$o->modA->getUsers ();

ou

$o->attach ('modA', 'users');
$o->users->getUsers ();

Sinon, déçu que CS ne fasse pas les balises bb ou autre pour le code :( du coup le post de malalam est dure à digérer (MALALAM ! On t'a dit quoi sur l'indentation !! RALALA !! :p)

Je vais voir pour retravailler ca ! :)

Commentaire de codefalse le 28/11/2008 15:52:38 administrateur CS

T'inquiète, je me doutais bien que c'était pas toi ! :)

Bon, j'espère que l'on te reverra rapidement ! :)
Bon taff à toi.

Commentaire de malalam le 28/11/2008 21:24:58 administrateur CS

Un exemple très simple et totalement incomplet...(et franchement inutile en l'état) :
<?php
class string {
protected $sValue;
public function __construct($sString) {
if(!is_string($sString)) {
throw new Exception('argument must be a string');
}
$this->sValue = $sString;
}


public function __toString() {
return $this->sValue;
}
}

$toto = new string('toto');
echo str_replace('o', 'i', $toto);
function say(string $bar) {
echo $bar;
}
echo say($toto);
?>

Commentaire de malalam le 29/11/2008 10:36:36 administrateur CS

Hello,

en fait, je vais nuancer tes propos Dighan : il y a des subtilités, j'avais lu cette remarque dans les commentaires de la doc :
<?php
class a {
public $bar = 1;
}

class b {
public $bar = 2;
}
$a = new a;
$b = new b;

$tmp = $a;
$a = $b;
$b = $tmp;

var_dump($a);
var_dump($b);

$a = new a;
$b = new b;

$tmp = $a;
$a = & $b;
$b = $tmp;

var_dump($a);
var_dump($b);
?>

Pour rester simple, de ce que j'en comprends, PHP5 utilise en fait des "copies"... : quand vous passez un objet en paramètre à une méthode/fonction, vous avez bien 2 variables distinctes. Mais ces 2 variables référencent un même identifiant (repéré par le #X dans un var_dump()), et cet identifiant POINTE vers une valeur. Etant donné qu'elles REFERENCENT le même identifiant, on parle de "références".
Avec le & (ampersand), vous créez un alias : c'est exactement la même variable, qui va posséder 2 noms différents, si vous voulez. Dans le 2ème cas de mon exemple donc, nous avons bien de vraies références. Contrairement au 1er cas.

C'est une petite subtilité qui peut amener à pas mal d'incompréhensions.

un clone effectue une copie conforme de l'objet. Une vraie copie : nous avons alors 2 identifiants.
Donc :
<?php
class a {
public $bar = 1;
}

class b {
public $bar = 2;
}

$a = new a;
$b = new b;
$a = $b;
$c = clone $a;
var_dump($a, $b, $c);

$a = new a;
$b = new b;
$a = & $b;
$c = clone $a;
var_dump($a, $b, $c);
?>

MAIS :
<?php
class a {
public $bar = 1;
}

class b {
public $bar = 2;
}

$a = new a;
$b = new b;
$c = clone $a;
$a = $b;
$b = $c;
var_dump($a, $b, $c);

$a = new a;
$b = new b;
$c = clone $a;
$a = & $b;
$b = $c;
var_dump($a, $b, $c);
?>

Bref, ce n'est pas si simple que ça...

Commentaire de vincent7766 le 01/12/2008 16:00:08

juste une ptite question facile

L'utilisation de la nomentlature
$o signifie que cette variable n'est instancié qu'une seule fois(once)?

$m signifie que cette variable est une variable membre d'une classe?

Commentaire de codefalse le 01/12/2008 16:25:22 administrateur CS

Non pas vraiment ;)

En règle générale, je nomme mes variables ainsi : typeNom, avec type la première lettre du type de variable (i pour integer, s pour string, o pour object, a pour array). Ensuite je lui donne un nom précis (genre pour pdo : oPdo).

$o, en fait c'est pour faire vite, donc c'est juste pour dire que c'est un objet, mais la variable n'a pas de nom au sens ou je lui en donne.

Ce n'est pas vraiment une nomenclature, juste une façon de procéder. Chaque personne à sa méthode, certain font pareils, ca dépend des gens :)

Commentaire de destinyfr le 01/12/2008 18:11:47

As-tu essayé la classe sur un serveur web ? parce que quand je remplis les champs dans le fichier index, avec ou sans la BDD name (informations pour faire la connexion), je reçois une erreur de type uncatchable.

peut-être que cela vien de moi ?

Commentaire de destinyfr le 01/12/2008 20:53:50

Salut

C'est en ce qui concerne la source ^^

Désolé je pensais avoir posté l'erreur :

Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[42000]: Syntax error or access violation: 1142 SELECT command denied to user '******'@'*******' for table 'Users'' in /testclass/classes/Models/Mysql/Users.php:24 Stack trace: #0 /testclass/classes/Models/Mysql/Users.php(24): PDOStatement->execute(Array) #1 /testclass/index.php(18): Users->selectFromId(1) #2 {main} thrown in /testclass/classes/Models/Mysql/Users.php on line 24

Y a du bordel ^^ (un message comme celui la jamais vu encore XD).

J'ai bien essayé de rajouter dans les dns (page index.php) le nom de la base de donnée mais rien à faire :(