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

[is_null]

En parlant de performances, je propose une factorisation :

Code :

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

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 :

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

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]

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