Discussion :

[Madfrix]

Bonjour, je m'entraine à manipuler les méthodes findDependentRowset, findParentRow et findManyToManyRowset.

Dans ce but j'ai construis ce petit modèle :

Code sql :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
 
CREATE TABLE IF NOT EXISTS `personnes` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `nom` varchar(50) NOT NULL,
  `prenom` varchar(50) NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE=InnoDB  DEFAULT CHARSET=latin1 AUTO_INCREMENT=4 ;
 
 
INSERT INTO `personnes` (`id`, `nom`, `prenom`) VALUES
(2, 'nom1', 'prenom1'),
(3, 'nom2', 'prenom2');
 
 
CREATE TABLE IF NOT EXISTS `livres` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `id_personne` int(11) NOT NULL,
  `nom` varchar(50) NOT NULL,
  PRIMARY KEY (`id`),
  KEY `id_personne` (`id_personne`)
) ENGINE=InnoDB  DEFAULT CHARSET=latin1 AUTO_INCREMENT=5 ;
 
 
INSERT INTO `livres` (`id`, `id_personne`, `nom`) VALUES
(2, 2, 'livre php'),
(3, 3, 'livre java'),
(4, 2, 'livre C');
 
ALTER TABLE `livres`
  ADD CONSTRAINT `livres_ibfk_1` FOREIGN KEY (`id_personne`) REFERENCES `personnes` (`id`) ON DELETE CASCADE;

Mon bootstrap contient entre autres ces 2 méthodes :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
 protected function _initDb() {
        $db = $this->getPluginResource('db')->getDbAdapter();
        $db->setFetchMode(Zend_Db::FETCH_OBJ);
        Zend_Db_Table_Abstract::setDefaultAdapter($db);
        return $db;
    }
 
    protected function _initAutoLoader() {
        $autoLoader = new Zend_Loader_Autoloader_Resource(array(
                    'BasePath' => APPLICATION_PATH,
                    'namespace' => 'Sim'
                ));
 
        $autoLoader->addResourceType('db', 'models/tables', 'Table');
 
        return $autoLoader;
    }

J'ai également crée 2 modèles pour mes tables que j'ai défini ainsi :

Table Personnes :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
class Sim_Table_Personnes extends Zend_Db_Table_Abstract {
 
    protected $_name = 'personnes';
    protected $_primary = 'id';
    protected $_dependentTables = array('Sim_Table_Livres');
 
    public function getPersonnes() {
        $select = $this->select();
        return $this->fetchAll($select);
    }
 
    public function getDependantLivres() {
        return $this->find(2)->current()->findDependentRowset('Sim_Table_Livres');
    }
 
}

Table Livres:

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
class Sim_Table_Livres extends Zend_Db_Table_Abstract {
 
    protected $_name = 'livres';
    protected $_primary = 'id';
    protected $_referenceMap = array(
        'relation' => array(
            'columns' => array('id_personne'),
            'refTableClass' => 'Sim_Table_Personnes',
            'refColumns' => array('id')
        )
    );
 
    public function getParentPersonne() {
        return $this->find(2)->current()->findParentRow('Sim_Table_Personnes');
    }
 
}

Pour finir j'ai crée 2 controllers Personnes et Livres ainsi qu'une action list listant respectivement tous les livres de la personne ayant l'id 2 et la personne qui est le lecteur du livre dont l'id est 2 (exemple un peu triviaux mais bon...)

Pour mon cas 1, c'est bon :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
  public function listAction()
    {
        $personnes = new Sim_Table_Personnes();
        $this->view->livres = $personnes->getDependantLivres();
    }

Ma vue récupère bien mes livres correspondant à l'id 2. Par contre dans le cas 2, j n'arrive pas à remonter au lecteur connaissant l'id d'un livre, j'ai cette erreur :

Notice: Trying to get property of non-object in C:\wamp\www\zftable\application\views\scripts\livres\list.phtml on line 4

Ma vue est la suivante :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
foreach($this->personne as $pers)
      echo $pers->nom;

et mon action est la suivante :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
 public function listAction()
    {
        $livres = new Sim_Table_Livres();
        $this->view->personne = $livres->getParentPersonne();
    }

Je pense pas que ce soit grand chose, mais ca me turlupine...d'autant plus que le cas de récupérer les livres connaissant le lecteur (cas inverse donc) marche parfaitement...

Petite question subsidaire ( ) :

Comment faire pour récupérer que certains champs en utilisant un find() ?


Merci et désolé pour mon explication "usine à gaz"


EDIT: info complémentaire : un print_r($this->personne) m'informe que je récupére bien l'enrgistrement (2, nom1, prenom1) mais je n'arrive pas à fetcher en objet comme dans le cas 1 et je vois pas pourquoi...

[manuscle]

Salut!

Normalement quand tu fait un getParentQuelquechose tu n'est pas censé récupérer un RowSet mais un Row.

De plus, le fonction getParent[nomDeTaTable] est une methode magique de ta classe de table, tu n'a juste qu'à faire getParentSim_Table_Personnes() ou getParentRow('Sim_Table_Personnes')

Essaie ça:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
// Action
 
 public function listAction()
    {
        $livres = new Sim_Table_Livres();
        $this->view->livre = $livres->find(2)->current();
        $this->view->personne = $livres->getParentSim_Table_Personnes();
    }
 
// Vue
echo $this->personne->nom;

[Madfrix]

Salut et merci de t'intéresser à mon soucis

Normalement quand tu fait un getParentQuelquechose tu n'est pas censé récupérer un RowSet mais un Row.

Mais un objet de type row doit pouvoir se fetcher comme un objet non ? Mon bootstrap ne paramètre pas cela comme ca par défaut ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
protected function _initDb() {
        $db = $this->getPluginResource('db')->getDbAdapter();
        $db->setFetchMode(Zend_Db::FETCH_OBJ);
        Zend_Db_Table_Abstract::setDefaultAdapter($db);
        return $db;
    }

je sais que ton code doit marcher (vu qu'une fois je l'avais fait marcher via un tuto), mais j'arrive pas à le faire marcher maintenant :

Fatal error: Call to undefined method Sim_Table_Livres::getParentSim_Table_Personnes() in C:\wamp\www\zftable\application\models\tables\Livres.php on line 19

Mais bon dans la mesure du possible je souhaite passer par les fonctions "classiques" et pas "magiques"

[manuscle]

Je ne sais pas pourquoi tu fait une fonction _initDb dans ton bootstrap, normalement si tu utilise Zend_Application tu devrais avoir ce genre de config dans application.ini :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
 
;; DATABASE
resources.db.params.charset        = utf8
resources.db.adapter               = "pdo_mysql"
resources.db.params.host           = "127.0.0.1"
resources.db.params.username       = "*****"
resources.db.params.password       = "*****"
resources.db.params.dbname         = "*****"
resources.db.params.date_format    = "YYYY-MM-ddTHH:mm:ss"
resources.db.isDefaultTableAdapter = true

Peut-etre est-il possible d'ajouter le parametre setFetchMode (voir la doc) , d'ailleur je ne savais même pas que l'on pouvait le définir, tu m'apprend quelque chose! lol

C'est peut etre ce parametre aussi qui pose probleme, essaie de l'enlever pour voir.... Moi je ne l'utilise pas et tout va bien....

Enfin bref toujours est-il que normalement ça devrait marcher...... je ne comprend pas.
Car chez moi lorsque je fait

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$livre = $livres->find(2)->current();

J'obtiens bien un Objet Zend_Db_Table_Row, donc je suis en mesure de faire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$personne = $livre->findParentRow('Sim_Table_Personnes');

Que te renvoi le dump de ton row livre?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
        $livres = new Sim_Table_Livres();
        $this->view->livre = $livres->find(2)->current();
        Zend_Debug::dump($this->view->livre);

[Madfrix]

J'avoue me perdre un peu dans toutes les méthodes de récupération de données avec Zend à force de lire les tutos

Par exemple, une requête de ce type :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
$result = $db->fetchRow('SELECT id, name FROM membres WHERE country = :country', array('country'=>'France');

est-elle protégée finalement ?

Autre question : une requête comme ceci ne sert juste qu'à éviter de taper la requête entièrement finalement, pas d'autre intérêt ?

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
 
$select = $db->select(); // récupération de notre objet de Zend_Db_Select
// équivalent à $select = new Zend_Db_Select($db);
$select->from('membres',array('id','name'));
$select->where('country = ?','usa');
$select->order('name ASC');
$select->limit(2);
$result = $db->fetchAll($select);

Au final, je suis en train de me demander si les méthodes de type findDependantRowset, findParentRow ont un véritable intérêt : pourquoi ne pas écrire manuellement la requête je pense que ces méthodes rajoutent des couches supplémentaires et sont plus longues non ? Un prepare/exec serait il pas plus rapide ?

Merci

[manuscle]

Pfiou! La question que tu me pose nécessiterais un tuto pour te répondre!
Oh non, pas encore un tuto!

Par exemple, une requête de ce type est-elle protégée finalement ?

Zend_Db_Mysql utilise l'extension PHP PDO, donc utiliser des requêtes préparées élimine le risque d'injection SQL et accélère les performances

Au final, je suis en train de me demander si les méthodes de type findDependantRowset, findParentRow ont un véritable intérêt : pourquoi ne pas écrire manuellement la requête je pense que ces méthodes rajoutent des couches supplémentaires et sont plus longues non ? Un prepare/exec serait il pas plus rapide ?

ça tu le découvrira à la pratique.... Zend_Db_Table est vraiment pratique et mérite que tu te prenne la tête dessus.
Sinon je suis d'accord avec toi dans le sens où apprendre à maitriser Zend_Db_Table n'est pas chose simple au début... Moi quand j'ai appris ZF je l'ai appris en même temps que j'ai appris la POO, c'est dire les journées, soirées, nuits même à me faire des noeuds au cerveau parce que je pigeais que dalle à tout ça!

Ce qu'il faut comprendre dans les grandes lignes, c'est que Zend_Db_Table te permet deux choses principales:
1. D'avoir un jeux de résultats, RowSet. Le RowSet étant un Objet issue de Zend_Db_Table_Rowset_Abstract, lui-même contenant diverses méthodes utiles

2.Chaque ligne de résultat est un Row. Le Row étant un objet issu de Zend_Db_Table_Row_Abstract, lui-même contenant diverses methodes utiles

L'avantage d'utiliser les résultats sous forme d'objet est que les classes contiennent des méthodes permettant de réaliser des actions de manière simple (ah bon? lol) et rapide.
ça permet de coder plus rapidement.
ça permet aussi plein d'autres choses que des développeurs plus chevronnés que moi t'expliquerons.

Prenons un exemple simple:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
$TMembres = new Table_membres();
// Je veux tout les membres
$membres = $TMembres->fetchAll(); // J'obtiens un Rowset
// Je veux tout les Français
$membres = $TMembres->fetchAll(array('pays' => 'fr'); // J'obtiens un Rowset
$membre = $TMembres->fetchRow(array('pays' => 'fr'); // J'obtiens un Row 
// car je demande tout les Français mais je ne veux récupérer que le premier résultat, donc le premier membre dans le résultat de la requete.
 
// Je tout les membres dont l'id = 5
$membre = $TMembres->find(5); // Rowset
 
// Je veux LE membre n°5
$membre = $TMembres->find(5)->current(); // Row
// Je veux le suivant
$membre = $TMembres->find(5)->next(); // Row
 
// Je veux la liste des adresses du membre 5
$adresses = $membre->findDependantRowSet('Table_Adresses'); // RowSet
// ou 
$adresses = $membre->findTable_Adresses(); // RowSet

Maintenant là où ça craint, c'est ce genre de chose

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
 
// Je veux tout les membres ainsi que leurs adresses
$membres = $TMembres->fetchAll(); // J'obtiens un Rowset
 
foreach ($membres as $membre) {
    $adresses = $membre->findTable_Adresses();
}
// Car dans cette boucle tu fait une requete à chaque changement de membre
// La simple requete avec jointure est plus apropriée
 
//EDIT: A partir de là ça craint plus
// Le fait d'utiliser l'objet est que c'est facile d'ajouter des parametres dans ta requete:
 
$select = $db->select(); 
$select->from('membres',array('id','name'));
$select->where('country = ?','usa');
$select->order('name ASC');
// Je peux faire cela par exemple, je suis pas admin, j'ai pas le droit de voir les admins
if(Zend_Auth::getInstance()->getIdentity()->isAdmin == 0) {
$select->where('is_admin', 0);
}
 
$result = $db->fetchAll($select);

Bref, à toi de choisir ce qui conviens le mieux à ton application... il n'y a pas de regle.

[Madfrix]

Zend_Db_Mysql utilise l'extension PHP PDO, donc utiliser des requêtes préparées élimine le risque d'injection SQL et accélère les performances

Yep je suis d'accord
Mais il me semble (dis moi si je me trompe O_o^^) que seule l'utilisation de prepare/execute empeche les injections. Or un fetchXXX utilise t-il nativement un prepare ou un query ou un exec ?

Si non, est il possible d'associer un fetchAll par exemple avec un prepare ?

L'utilisation des méthodes de sélection de données par fetchXXX, findDependantRowset etc..est à peu près bien comprise, je me demande juste si au niveau performances, on perd ce qu'on gagne en facilité de codage...

J'ai surtout peur d'une perte de perfs...

Abuser des méthodes de récupération des dépendances va générer une charge insoutenable pour le SGBD. Ces méthodes sont là, mais en pratique on les utilise très très peu. Utiliser une méthode de récupération de dépendance dans une boucle foreach, par exemple, amène à une dégradation certaine des performances de l'application.
Créez vous même vos jointures, limitez les champs retournés (*, ne sert que très très rarement), et si possibles, utilisez des vues et des procédures stockées. N'agissez pas à l'aveugle en vous reposant sur le framework, ayez toujours en tête les requêtes envoyées à votre SGBD. ZF ne fait que proposer une interface avec les SGBDs, à vous de l'utiliser de manière correcte, réfléchie et sensée.

[manuscle]

Yep je suis d'accord
Mais il me semble (dis moi si je me trompe O_o^^) que seule l'utilisation de prepare/execute empeche les injections. Or un fetchXXX utilise t-il nativement un prepare ou un query ou un exec ?

Si non, est il possible d'associer un fetchAll par exemple avec un prepare ?

Va voir la methode Zend_Db_Table_Abstract::fetchAll(); Regarde ce que ça fait, suis le code..... tu auras ta réponse!

L'utilisation des méthodes de sélection de données par fetchXXX, findDependantRowset etc..est à peu près bien comprise, je me demande juste si au niveau performances, on perd ce qu'on gagne en facilité de codage...

J'ai surtout peur d'une perte de perfs...

C'est justement ce que je te montre dans le deuxième exemple où 'ça craint'. C'est donc pour cela qu'il faut utiliser Zend_Db_Table avec prudence.
Mais surtout, regarde le code, n'hésite pas à aller fouiller dans les classes.... fait des debug pour voir ce que contiennent réellement tes variables de classe, tu decouvrira beaucoup.
Utilise Zend_Db_Profiler pour afficher à la fin de ton script le total des requetes executées, le temps que ça a pris etc......
Si tu fait ça une fois au début, même si ça te prend du temps, tu sauras, et la prochaine fois tu n'aura plus à te poser la question entre quelle methode est la plus judicieuse pour conduire à bien ton projet!

Par exemple, voici une requete que j'ai fait hier et que je ne ferais JAMAIS avec Zend_Db_Table car bien trop long...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
 
            SELECT tp.prix_1 AS 'P_PUHT1', tp.prix_1 AS 'P_PUHT2', tp.prix_1 AS 'P_PUHT3', tp.prix_1 AS 'P_PUHT4', tp.prix_1 AS 'P_PUHT5', 
                   p.reference AS 'P_REF', p.poids_moyen AS 'P_POIDS', CONCAT('/fiche/',p.alias_url) AS 'P_URL_STOCK', 
                   pl.titre AS 'P_NOM', pl.descriptif AS 'P_DESC', 
                   m.nom_marque AS 'P_MARQUE',
 
            (SELECT GROUP_CONCAT(DISTINCT CONCAT(couleur_code,'#',nom_couleur) separator ',') 
                FROM produits_details 
                LEFT JOIN couleurs_lang ON produits_details.id_produits_details = couleurs_lang.id_produit_detail 
                LEFT JOIN tarifs_produits_details ON produits_details.id_produits_details = tarifs_produits_details.id_produit_detail 
                WHERE id_produit = p.id
                AND tarifs_produits_details.id_tarif_produit = tp.id_tarif_produit
                AND couleurs_lang.id_langue = 1) AS 'P_COLORIS',
 
            (SELECT GROUP_CONCAT(DISTINCT taille_code ORDER BY taille_ordre ASC separator '/') 
                FROM produits_details 
                LEFT JOIN tarifs_produits_details ON produits_details.id_produits_details = tarifs_produits_details.id_produit_detail 
                WHERE id_produit = p.id
                AND tarifs_produits_details.id_tarif_produit = tp.id_tarif_produit) AS 'P_TAILLE',
 
            CONCAT(p.reference,'_', IFNULL(p.top_couleur, (SELECT couleur_code FROM produits_details  ORDER BY RAND() LIMIT 1)),'_fiche.png' ) AS 'P_IMAGE'
 
            FROM tarifs_produits AS tp
 
            LEFT JOIN produits         AS p  ON tp.id_produit = p.id 
            LEFT JOIN produits_lang    AS pl ON pl.id_produit = p.id 
            LEFT JOIN marques          AS m  ON m.id = p.id_marque
 
            WHERE p.active = 1
            AND tp.tarif_code = 'F'
            AND pl.id_langue = 1
 
            /** Si il n'y pas d'article details de créé
                couleurs et tailles seront à null **/
            AND 'P_COLORIS' IS NOT NULL
            AND 'P_TAILLE'  IS NOT NULL
 
            /** Par sécurité on ne retourne pas les prix <= 0 **/
            AND prix_1 > 0
 
            ORDER BY 'P_COLORIS' ASC ";

C'est une requete qui prend beaucoup de ressource MySql mais je l'exécute une fois par mois donc je m'en fiche un peu...