Discussion :

[Mehdi Fal]

Bonsoir

Donc voilà je voudrai exécuter une requête en suivant le MVC de Zend, donc voilà quelques bouts de code :

Le Modèle Tables.php :

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
 
<?php
    class Post extends Zend_Db_Table
    {
        protected $_name = "Post" ;
 
        public function select_acceuil_post()
        {
            $db = Zend_Db_Table::getDefaultAdapter() ;
            $select = new Zend_Db_Select($db) ;
            $select = $db -> select() ;
                          -> from('Post') ;
                          -> order('id_post DESC') ;
                          -> limit('3') ;
        }
    }
?>

Le Controller IndexController.php :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
<?php
    class IndexController extends Zend_Controller_Action
    {    
        public function indexAction()
        {
            require_once 'Tables.php' ;
            $this->view->title = "Contradictor but Homogen" ;
            $last_post = new Post() ;
            $req = $last_post -> select_acceuil_post() ;
            $this -> view -> last_post = $req -> fetchAll() ;
        }
    }
?>

La Vue index.phtml :

Code :

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

1
2
3
4
5
6
7
 
<?php
    foreach($this->last_post as $post)
    {
        echo $this-> escape($post -> id_post) ;
    }
?>

Et je reçois le Fameux Joli Message d'erreur :
Fatal error: Call to a member function fetchAll() on a non-object in /opt/lampp/htdocs/contradictors/application/controllers/IndexController.php on line 11

Est-ce quelqu'un aurait une idée d'où provient cette erreur ?

Merci d'avance

[Mehdi Fal]

C'est Bon maintenant ca marche:

Le Modèle Tables.php :

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
 
<?php
    class Post extends Zend_Db_Table
    {
        protected $_name = "Post" ;
 
        public function select_acceuil_post()
        {
            $db = Zend_Db_Table::getDefaultAdapter() ;
            $select = new Zend_Db_Select($db) ;
            $select = $db -> select() ;
                          -> from('Post') ;
                          -> order('id_post DESC') ;
                          -> limit('3') ;
            return $select->query() ;   // 1er Probleme
        }
    }
?>

et pour la view :

Code :

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

1
2
3
4
5
6
7
 
<?php
    foreach($this->last_post as $post)
    {
        echo $this-> escape($post['id_post']) ; // car fetchAll() return un tableau
    }
?>

Merci en tout cas

[5h4rk]

Un conseil, pour créer un modèle Post représentant la table post de ta BDD, donne lui le même nom et non pas Tables.php

Pour ma part, j'aurais fait ainsi:
- Le modèle serait dans le dossier : /application/models/DbTable/Post.php
- L'entête du fichier serait : Model_DbTable_Post extends Zend_DbTable

[m4riachi]

Tu dois travailler avec autoloader du zend pour ne plus faire des require_once pour appeler tes class

[Alcide_]

Autres petites remarques :

- utilise la notationCamel pour tes variables ou méthodes

- essaie de préfixer tes accesseurs par "get" ou "set"
ton code sera beaucoup plus lisible.

-Si tu as configurer ton adaptateur au niveau de ton bootstrap avec :

Code :

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

Zend_Db_Table::setDefaultAdapter($myAdapter);

Tu n'as plus besoin de récupérer l'adaptateur dans tes modèles.

- Tu devrais aussi mettre ton "fetchAll()" au niveau de ton modèle et non pas dans ton contrôleur (regarde bien la doc de fetchAll, car c'est un Zend_Db_Table_Rowset qu'il retourne et non un simple Array).

- (Edit:Je reformule) Ton modèle devrait plutôt retourner le résultat de "fetchAll()" et non celui d'un "$select->query()".
Sinon, regarde bien la doc de fetchAll() car il retourne un Rowset et non un simple Array.
Quand tu boucles dessus via un foreach, tu récupères un Row à chaque itération donc, faire "$row->monChamp" est possible.

Un peu de lecture : ZF coding standard

[5h4rk]

Ce que tu signale est exacte mise à part un bémol, il vaut mieux que le fetchAll soit côté controller, le modèle n'a pas pour but de retourner un array, c'est à toi côté applicatif à faire de ton rowset ce que tu veux et non pas au modèle.

Chaque méthode d'un modèle devraient retournée un row ou un rowset quand cela est possible mais ne jamais effectué des opérations sur ces résultats avant de les retransmettre.

Cela permet de pouvoir utiliser la méthode à plusieurs endroit, car si par exemple tu as besoin de travailler avec des objets dans une autre action, en transformant en array tu devra faire une nouvelle méthode

[Alcide_]

On est d'accord, pour que la méthode retourne un Row ou Rowset il faut que les méthodes proxyant _fetch() soient coté modèle...
Sous-entendu :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
Table_My extends Zend_Db_Table
{
   public function getFoo()
   {
       $select = $this->select();
       /* code */
       return $this->fetchAll($select);
   }
}

(regarde bien la doc de fetchAll, car c'est un Zend_Db_Table_Rowset qu'il retourne et non un simple Array).

Cette phrase faisait référence au commentaire de Mehdi Fal sur la valeur de retour d'un fetchAll(), non à une quelconque transformation des valeurs au sein du modèle... veuillez m'excuser si cela a porté à confusion...