[POO] Objet se renvoyant lui-même en php5 [Résolu]

[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 :

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 :

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 :

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 :

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]

Envoyé par stailer

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 :

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 :

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

[is_null]

En parlant de performances, je propose une factorisation :

Code :

1
2
3
4
5
6
7
8
public function throwExceptions($flag = null)
    {
        if ( ! is_bool( $flag ) ) {
             return $this->_throwExceptions;
        }
        $this->_throwExceptions = $flag;
        return $this;
    }

J'éspère que tout le framework n'est pas codé comme cet example JK
Surtout que la variable préfixée avec underscore fait très PHP4 ...

[stailer]

Taum : Oui finalement j'ai fait un retour de $this et j'arrive au même résultat que le ZF... Mais j'aime pas trop faire transiter des objets complets, il me semblait que ça bouffait de la ressource.

Et non effectivement ce n'est pas de la récursivité, si ce n'est que dans l'esprit on envoie une fonction qui renvoie l'objet , qui envoie la fonction , qui renvoie l'objet .... etc..

Merci pour vos réponses,

[is_null]

J'y voie un autre avantage majeur.
Un exemple parlera de lui-meme :

Code php :

1
2
 
$o_objet = $panier->ajouter( new $s_objet() );

On economise une ligne dans le code, c'est toujours plus confortable puisque cela permet une meilleure lisibilité.

[dewsz]

Envoyé par stailer

Oui finalement j'ai fait un retour de $this et j'arrive au même résultat que le ZF... Mais j'aime pas trop faire transiter des objets complets, il me semblait que ça bouffait de la ressource.

Je pense que c'est la solution la plus simple à ton problème. Et comme le passage d'objet se fait par référence il n'y a pas de problème de ressource.

[is_null]

http://www.martinfowler.com/bliki/FluentInterface.html