Discussion :

[stailer]

Hello,

je suis vraiment désolé pour le titre mais je ne vois pas comment nommer mon problème.. ou alors "Objet récursif", mais c'est pas vraiment ça.

Donc je vous explique :
J'ai une classe : "Peche", qui va par exemple me permettre, via les iterator de rajouter dynamiquement des "poissons" (je répète bien que c'est un exemple lol) :

Code :

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

1
2
 
$this->getPeche()->addPoisson('truite', 25);

et à chaque fois que je veux ajouter un poisson, je dois utiliser cette ligne de code... Alors quand c'est dans une boucle pas de souci.. Mais si c'est en dur, c'est vraiment dommage de répéter autant de texte ! alors que je pourrais avoir quelque chose de plus élégant :

Code :

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

1
2
3
4
5
 
$this->getPeche()
   ->addPoisson('truite', 25)
   ->addPoisson('Saumon', 10)
   ->addPoisson('baleine', 30);

Quelqu'un a t il déjà implémenter cette technique de façon optimisée ? et pourrait-il me montrer un exemple ?

Merci d'avance pour vos réponses,

[mathieu]

pour l'ajout en chaine, j'ai souvent vu ça

Code :

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

1
2
3
4
5
$this->getPeche()->ajoutListePoisson(array(
    new Truite(25),
    new Saumon(10),
    new Baleine(30),
));

et là tu as une "Fatal error: Argument must be an list of Poisson" parce que Baleine n'hérite pas de Poisson (oui je sais que c'est un exemple )

[stailer]

Non j'aimerais vraiment pas passer par un tableau intermédiaire...

La solution que j'ai donc mis en place à l'heure actuelle est évidemment de retourner l'objet lui même :

Code :

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

1
2
3
4
5
 
public function addPoisson($type='', $nombre = 0) {
  ... code ajout ...
  return $this;
}

Bien sur ça marche... Mais n'y a t il pas mieux et plus optimisé que de retourner l'objet lui-même ?
Un design pattern n'existerait-il pas par exemple ?

(je sais qu'on ne peut pas ajouter baleine à poisson.. Mais c'était pour voir si tout le monde suivait héhé )

[Taum]

Mais n'y a t il pas mieux et plus optimisé que de retourner l'objet lui-même ?

Tu as un problème de performances ?
(à cause de l'ajout de la baleine qui prend beaucoup en mémoire ? )



Sérieusement, avant de chercher à optimiser un code, cherche d'abord à savoir si il constitue effectivement un goulot d'étranglement ... et dans ton cas à mon avis la réponse est non ...

De mémoire, il me semble d'ailleurs que le ZendFramework utilise la même technique pour les View ($view->assign('var1', $var1)->assign('var2', $var2)->render() par exemple).
Au passage je ne vois pas de récursivité là dedans, aucune fonction ne s'appelle elle-même

[Yoshio]

L'exemple avec le Zend Framwork aurait plutot été :

Code :

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

1
2
3
4
5
Zend_Loader::loadClass('Zend_Controller_Front');
$front = Zend_Controller_Front::getInstance()
	->addModuleDirectory('../application/modules')
	->throwExceptions(true)
	->dispatch();

J'aimerais bien savoir aussi comment faire ca.

[Taum]

Autant pour moi pour l'exemple mal choisi

J'aimerais bien savoir aussi comment faire ca.

Zend Framework est open-source, tu n'as qu'a regarder la source de Zend_Controller_Front :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public function throwExceptions($flag = null)
    {
        if (true === $flag) {
            $this->_throwExceptions = true;
            return $this;
        } elseif (false === $flag) {
            $this->_throwExceptions = false;
            return $this;
        }
 
        return $this->_throwExceptions;
    }

Il suffit de retourner $this