Utiliser fetch et ses amis dans un Controller

Version imprimable

Voir 40 message(s) de cette discussion en une page

10/06/2010, 14h31
abenlahsen

Bonjour,

Citation:

c'est ce code là qui fonctionne, pas l'autre super compliqué de [B]abenlahsen/B] (désolé abenlahsen )

Excuse moi mais cette solution casse le MVC!

Moi je ferais pas ce code dans mon contrôleur! Je créer une fonction dans mon modèle et je l'appel dans mon action

@tomtom94500 Le code que je t'ai donné illustre comment ajouté la clause where pas plus! Plus, bien évidemment, l'ajouter dans une fonction dans un modèle et pas dans le contrôleur.

Cordialement,
Ahmed.
10/06/2010, 14h37
tomtom94500

Je vais pas trop vous en demander non plus, mais bon...

Au niveau des instanciations je suis pas très bon ça serait possible d'avoir un petit code d'exemple svp ? :calim2:
10/06/2010, 14h39
Feng-Huang

@abenlahsen :
Tu ne précise pas dans quel contexte tu as, mais j'ai l'impression qu'on est dans ton modèle de table ( tu n'aurais pas fais de méthode comme celle-la dans un contrôleur, t'as pas fais ça hein ?!!! ).
Partant de ce postulat, tu n'utilise pas l'objet $this pour définir le from. Tu appel l'adapteur de Zend_Db_Table puis fais un micmac monumental pour instancier un objet Zend_Db_Select , alors que tu pourrais faire simplement :

Code:

$select = $this->select() // retourne Zend_Db_Table_Select

Pour reprendre ce que j'ai dis : compliqué et moche, ça c'est visuel :p
Ça casse le système MVC dans la mesure où tu définis des paramètres qui affectent ta méthode dans le modèle lui-même ( cf. ton prototype : public function listCommunautes($field = 'email', $value= '', $banni = 'all') )
Et enfin pas sécurisé dans la mesure où tu concatène directement la variable $field dans ton appel de la méthode where (au lieu de la binder comme $value.
10/06/2010, 14h41
abenlahsen

Merci @Feng-Huang
10/06/2010, 14h51
Feng-Huang
@tomtom94500 : la question est-elle " qu'est-ce qu'une instanciation ? "
Si oui alors ceci :

Code:

$DbTable = new Model_DbTable_Foo;

Est une instanciation de la classe Model_DbTable_Foo. Cette instanciation, ce traduit par la création d'un objet de type Model_DbTable_Foo qui est désormais contenu dans la variable $DbTable.
Code:

1 2 $instance1 = new Une_Classe_Bidon('foo'); $instance2 = new Une_Classe_Bidon('bar');
Alors les deux objets sont des instances Une_Classe_Bidon mais ces deux objets sont différents, car ils représentent chacun une instance différente.
Si tu construit 2 maisons avec le même plan, alors tu as 2 instances du plan.

Concernant ta requête tomtom94500 , ouvre un nouveau sujet dans la secion appropriée du forum SGBD -> SGBD & Sql -> <ton_sgbd>. Une fois que tu as ta requête, reviens ici et ouvre un sujet dans la section Zend_Db pour transformer ta requête SQL brute en objet Zend_Db_Table_Select ;)

Oui d'accord je vois à peu près... Mais moi je suis toujours sur le meme probleme : aller rechercher le pseudo de mon gagnant en fonction de l'id de l'enchere car je n'y arrive pas !

Bon je balance mes pages pour que ça soit plus simple :
models/DbTable/Encheres.php

Code:

1
2
3
4
5
6
7
 
<?php
 
class Model_DbTable_Encheres extends Zend_Db_Table_Abstract
{
    protected $_name = 'enchere';
}

models/DbTable/Gagnants.php

Code:

1
2
3
4
5
6
7
 
<?php
 
class Model_DbTable_Gagnants extends Zend_Db_Table_Abstract
{
    protected $_name = gagnant;
}

EnchereCocoController.php

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
<?php

class EnchereCocoController extends Zend_Controller_Action
{
    function indexAction()
    {
        $this->view->title = "Encheres Coco";
        $enchere = new Model_DbTable_Encheres();
        $sql = $enchere->select()
                       ->where('validation=?',2)
                       ->order('id DESC');
        $this->view->encheres = $enchere->fetchAll($sql);

// Comment faire pour aller chercher le pseudo de mon gagnant
        $gagnant = new Model_DbTable_Gagnants();
        $sql = $gagnant->select()
                       ->where('id=?',2);
        $this->view->gagnants = $gagnant->fetchRow($sql);
    }
}

views/scripts/enchere-coco/index.phtml

Code:

1
2
3
4
5
6
<?php foreach($this->encheres as $enchere) : ?>
id :<? echo $this->escape($enchere->id); ?>
titre :<? echo $this->escape($enchere->titre); ?>
Comment ressortir ici le pseudo de mon gagnant en fonction de l'id de l'enchere
<?php endforeach; ?>

10/06/2010, 15h34
tomtom94500

débloquez moi svp^:(
10/06/2010, 15h37
godzinho

Ne connaissant pas ta BD, on ne va pas pouvoir t'aider.
A un moment donné il te faut récupérer l'id du gagnant pour aller récupérer son pseudo. Si tu n'as pas cette info dans ta table enchere, tu dois la récupérer par un autre moyen.

Mais dans tous les cas tu dois récupérer l'id du gagnant et appeler la méthode adéquat pour avoir le pseudo.
10/06/2010, 15h48
tomtom94500

Ma BD est très simple !!!!

dans ma table gagnant j'ai enregistré l'id de l'enchere (id_enchere) et l'id du gagnant (id_gagnant)

J'aimerais retrouver id_gagnant en fonction de l'id_enchere dans ma boucle foreach...
voila c'est tou

Il te suffit donc de récupérer l'id du gagnant dans un premier temps.
Puis tu récupères le pseudo dans une autre table je suppose.
Code:

1 2 3 4 5 6 7 8 public function getIdGagnant($idEnchere){ $select = $this->_db->select(); $select->from($this->_name,'id_gagnant') ->where('idEnchere=?',$idEnchere); return $this->_db->fetchRow($select); }
Puis tu fais la même chose une fois que tu as l'id du gagnant.

Code:

SELECT * FROM encheres LEFT JOIN gagnants ON encheres.id = gagnants.enchere_id

Soit en version Zend :
Code:

1 2 3 4 $select = $this->select()->setIntegrityCheck(false) ->from($this,array('id','nom_produit')) ->joinLeft('gagnants','encheres.id = gagnants.enchere_id',array('id')) ->order('enchere.date DESC')
Youhou :D

10/06/2010, 16h06
Feng-Huang
@godzinho : tu fais des étapes inutiles dans ton modèle (outre le fait qu'il faut faire une jointure et pas une requête pour chaque enchère) :
Code:

1 2 3 4 5 6 7 public function getIdGagnant($idEnchere){ $select = $this->select(); $select->from($this,'id_gagnant') ->where('idEnchere=?',$idEnchere); return $this->fetchRow($select); }
C'est plus court et donne le même résultat ;)
10/06/2010, 16h11
tomtom94500

Bon d'accord je vois la démarche a effectuer... cependant j'insere la fonction getIdGagnant dans mon controller ? ou dans mon models ?
Et comment je fais pour m'en servir de cette fonction par la suite dans mon index.phtml ? (jsuis vraiment un foutu débutant très nul)

On a tous été un foutu débutant très nul :p

Comme je l'ai dis plutôt, pas de sql dans un contrôleur, il va de soit que ce n'est pas dans la vue évidemment, donc c'est dans ton modèle :)

Dans ton modèle :
Code:

1 2 3 4 5 6 7 8 public function getFullList(){ $select = $this->select()->setIntegrityCheck(false) ->from($this,array('id','nom_produit')) ->joinLeft('gagnants','encheres.id = gagnants.enchere_id',array('id')) ->order('enchere.date DESC') return $this->fetchAll($select); }
Dans ton contrôleur :
Code:

1 2 $DbTable = new Model_DbTable_Encheres; $this->view->encheres = $DbTable->getFullList()
Enfin dans ta vue :
Code:

1 2 3 foreach($this->encheres AS $enchere){ echo '<tr><td>',$enchere->nom_produit,'</td><td>',$enchere->gagnant_id,'</td></tr>'; }
Et la tu vois que tu dois encore faire une jointure pour récupérer le pseudo du membre (oui l'ID n'est pas très parlant pour l'utilisateur :p ).
Tu essaye de modifier le modèle ?

10/06/2010, 16h26
godzinho
Citation:
Envoyé par Feng-Huang

@godzinho : tu fais des étapes inutiles dans ton modèle (outre le fait qu'il faut faire une jointure et pas une requête pour chaque enchère) :

Code:

1 2 3 4 5 6 7 public function getIdGagnant($idEnchere){ $select = $this->select(); $select->from($this,'id_gagnant') ->where('idEnchere=?',$idEnchere); return $this->fetchRow($select); }

C'est plus court et donne le même résultat ;)
Tu m'en apprends encore.
Merci pour tes conseils.

Sinon tomtom94500 devrait s'en être tiré avec ce que tu viens de lui proposer.

Bon moi je ne veux pas utiliser la méthode de jointure c'est trop compliqué, tandis que créer une fonction pour récupérer l'id dans ma tête ça passe mieux

Bon je suis loin d'avoir réussi, je vais bientot craqué, merci de m'avoir suivi en tout cas, mais surfer entre les bases de données c'est ce qui fait qu'un site est un site internet !
Donc si j'arrive pas a surfer entre les base de données, je laisse tomber et je reste en php4.

Dernièr essai je balance mes pages parceque là j'en peux plus :

EnchereCocoController.php
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 <?php class EnchereCocoController extends Zend_Controller_Action { function indexAction() { $this->view->title = "Encheres Coco"; $enchere = new Model_DbTable_Encheres(); $sql = $enchere->select() ->where('validation=?',2) ->order('id DESC'); $this->view->encheres = $enchere->fetchAll($sql); De quelle manière je dois faire appelle à la fonction getIdGagnant() ?????? } }
index.phtml
Code:

1 2 3 4 5 6 <?php foreach($this->encheres as $enchere) : ?> id :<? echo $enchere->id; ?> titre :<? echo $enchere->titre; ?> De quelle manière j'affiche l'ID du gagnant (enfonction de l'id de l'enchere blabla.....) ! <?php endforeach; ?>
models/DbTable/Encheres.php
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 <?php class Model_DbTable_Encheres extends Zend_Db_Table_Abstract { protected $_name = 'enchere'; public function getIdGagnant($idEnchere) { $sql = $gagnant->select() ->from($gagnant,'id_gagnant') ->where('id_enchere=?',$idEnchere); return $gagnant->fetchRow($sql); } }
models/DbTable/Gagnants.php
Code:

1 2 3 4 5 6 7 <?php class Model_DbTable_Gagnants extends Zend_Db_Table_Abstract { protected $_name = 'gagnant'; }
Bref si des gens sur Paris sont dispo pour me donner un cours particulier je suis preneur...

Et si vous me dépannez beinh ça serait bien merci

10/06/2010, 16h52
Feng-Huang

Boucler pour récupérer les gagnants un à un est tellement scandaleux que je ne t'aiderai pas pour ça.
10/06/2010, 17h02
tomtom94500

@Feng-Huang j'ai l'impression que les messages privés ne marchent pas, tu as reçu mon message privé ou pas ??????

Il me semble que tu t'offusques vite Feng-Huang. Tout le monde a le droit à ses débuts...

@tomtom94500 : pour appeler la fonction getIdGagnant dans ton contrôleur. Tu instancies un objet sur la classe des encheres et tu fais appels à la méthode depuis cette instance.
Code:

1 2 3 4 5 6 7 8 9 10 class EnchereCocoController extends Zend_Controller_Action { function indexAction() { ... $enchere = new Model_DbTable_Encheres(); $idGagnant = $enchere->getIdGagnant($idEnchere); $this->view->idGagnant = $idGagnant; // on envoi à la vue } }
Puis dans la vue tu fais comme d'habitude :

Code:

idGagnant :<? echo $enchere->idGagnant; ?>

11/06/2010, 13h55
Feng-Huang

Est-ce que tu te rends compte de ce que ça représente une requête dans une boucle ? Quand il affichera la page des gagnants, mettons qu'il mette 20 enchères par page, ça veut dire 21 requêtes, rien que pour le corps de la page. Même avec des requêtes préparées, ça craind. Et plus si il refuse la jointures ça veut dire une requête de plus pour récupérer le pseudo du gagnant. Soit 41 requêtes, la où on peut en faire 1 seule.
Ce n'est pas s'offusquer, c'est juste que je veux bien apporter mon aide, mais pas encourager à faire des bétises.

Voir 40 message(s) de cette discussion en une page