Discussion :

[Pillow]

Bonjours,

Je voudrais savoir comment améliorer le code suivant :

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
19
20
21
22
23
24
25
26
 
 
class recherche
{
    private $chaine;
 
         function __construct($chaine)
        {
        $this->_chaine = $chaine;
        }
	function fonction1($value = "")
	{
        /*****/
        return $this->_chaine;
	}
 
	function fonction2($value = "")
	{
        /*****/
        return $this->_chaine;
	}		
 
	function __destruct() 
	{
        unset($this->chaine);
        }

Pour l'instant je pense pas qu'il y est d'erreur mais c'est plutôt dans l'utilisation de la classe :

Code :

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

1
2
3
4
5
 
$var = new recherche("blablabla");
$temp= $var->fonction1();
$var2 = new recherche($toto);
echo $var2-> fonction2();

Je trouve ce code très lourd et je suis sur qu'il y a plus simple...
l'idée serait de faire qq chose ressemblant à :

Code :

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

1
2
3
4
 
$var = new recherche("blablabla");
$temp= ($var->fonction1())->fonction2();
//appliquer la fonction1 puis la fonction2 avec comme parametre blablabla

Ici il y a que 2 fonctions mais j'en est beaucoup plus et je me vois mal passer par un intermédiaire $temp à chaque fois....

Merci pour la lecture

[Celira]

Je n'ai absolument pas compris ce que tu veux faire. Le fait que toutes tes fonctions fassent la même chose n'est évidemment pas pour aider...

Mais bon, je vais essayer de te donner quelques conseils d'ordre général :
Par convention, un attribut privé (private) ou protégé (protected) est accédé depuis l'extérieur par des accesseurs (getters/setters en anglais) getNomAttribut qui récupère la valeur et setNomAttribut qui l'affecte.

On peut parfaitement utiliser des méthodes directement sur d'autres méthodes, sous réserve que la première méthode renvoie un objet de la même classe...

Ce qui donnerait quelque chose comme ça :
Classe Recherche :

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
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
class Recherche
{
    private $_chaine = '';
 
    function __construct($chaine) 
    {
        $this->_chaine = $chaine;
    }
	/**
	 * Set $_chaine
     * @param $chaine value to set
     */
    public function setChaine($chaine)
    {
        $this->_chaine = $chaine;
    }
 
	/**
	 * Return $_chaine
     * @return the $_chaine
     */
    public function getChaine()
    {
        return $this->_chaine;
    }
 
    /**
     * Applique un traitement à la chaine
     * @return $this
     */
    public function traiterChaine()
    {
        $this->_chaine = trim($this->_chaine);
        return $this;
    }
 
}

Utilisation :

Code :

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

1
2
3
$var = new Recherche('blabla   ');
$chaineNonTraitee = $var->getChaine(); // => "blabla   "
$chaineTraitee = $var->traiterChaine()->getChaine(); // => "blabla"