php design pattern chaine de responsabilités

Version imprimable

Bonjour à tous,

J'aurais besoin de votre aide s'il vous plaît?

J’essaie d'implémenter le design pattern chaine de responsabilité pour créer un filtre de simple si la valeur est une chaine on la mets en majuscule sinon on renvoie le double.
Cela fonctionne ma question est que je n'utilise pas le typeint de php7 justement pour pouvoir filtrer sur la bonne classe, comment feriez vous? Mon code peut-être améliorer, avez vous des conseils?

Class abstraite

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
<?php
 
namespace Exercises\ChainResponsabilityPattern\Complete;
 
abstract class MainFilterAbstract 
{
  /**
   * @var MainFilterAbstract|null
   */
  protected $_successor = null;
 
  public function __construct(MainFilterAbstract $successor = null)
  {
      $this->_successor = $successor;
  }
 
  abstract public function filter();
}

le filtre pour les majuscules

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
<?php
 
namespace Exercises\ChainResponsabilityPattern\Complete;
 
class UpperCaseFilter extends MainFilterAbstract
{
    public function __construct(MainFilterAbstract $filter = null)
    {
        parent::__construct($filter);
    }
 
   public function filter($content = null)
   {
       if(is_string($content))
       {
           return strtoupper($content);
       }
 
       return $this->_successor->filter($content);
   }
}

Pour le double

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
<?php
 
namespace Exercises\ChainResponsabilityPattern\Complete;
 
class DoubleFilter extends MainFilterAbstract 
{
    public function __construct(MainFilterAbstract $filter = null)
    {
        parent::__construct($filter);      
    }
 
    public function filter($content = null)
    {
       if(is_int($content))
       {
           return $content * 2;
       }
 
       return $this->_successor->filter($content);
    }
 
}

le fichier d'execution

Code:

1
2
3
4
5
6
7
8
$upperCaseFilter = new UpperCaseFilter();
$myFilter = new DoubleFilter($upperCaseFilter);
 
$space = ' ';
 
echo $myFilter->filter('hello') . $space;
echo $myFilter->filter('24') . $space;
echo $myFilter->filter(360) . $space;

La sortie console

Code:

HELLO 24 720

Merci

Si je comprends bien, ton problème est le cas à la limite où tu reçois un nombre sous forme de chaine. Dans ce cas, tu veux le traiter comme un nombre et non pas comme une chaine. C'est bien ça ?

Tu peux essayer d'utiliser is_numeric (mais ça ne distingue pas entre entier et décimal) et/ou filter_var avec le filtre FILTER_VALIDATE_INT.

Code:

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 UpperCaseFilter extends MainFilterAbstract
{
   public function filter($content = null)
   {
       if(is_string($content) && !is_numeric($content))
       {
           return strtoupper($content);
       }
 
       return $this->_successor->filter($content);
   }
}
 
class DoubleFilter extends MainFilterAbstract 
{
    public function filter($content = null)
    {
       $contentFiltre = filter_var($content, FILTER_VALIDATE_INT);
       if($contentFiltre !== false) // attention au !==, le nombre peut être égal à 0
       {
           return $contentFiltre * 2;
       }
       return $this->_successor->filter($content);
    }
 
}

Remarque annexe : attention, si un de tes éléments n'a pas de successeur, ça va t'exploser à la figure. Je pense que tu devrais avoir une méthode filtrer() générique au niveau de l'abstract et une méthode abstraite à implémenter qui fait le filtrage proprement dit. Quelque chose comme ça :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
abstract class MainFilterAbstract 
{
    /**
    * @var MainFilterAbstract|null
    */
    protected $_successor = null;
 
    public function __construct(MainFilterAbstract $successor = null)
    {
        $this->_successor = $successor;
    }
 
    public final function filter($content) {
        // On fait le filtre
        $result = $this->doFilter($content);
        // Si on a un successeur, on l'appelle
        if ($this->_successor) {
            $result = $this->_successor->filter($result);
        }
        return $result;
    }
 
    abstract protected function doFilter();
}

EDIT : la méthode filter doit être final histoire d'éviter qu'on la surcharge par erreur

Merci Celira,

J'ai un problème de successeur null pour les valeurs entières

Code:

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
38
39
40
41
<?php
 
namespace Exercises\ChainResponsabilityPattern\Complete;
 
abstract class MainFilterAbstract 
{
  /**
   * @var MainFilterAbstract|null
   */
  protected $_successor = null;
 
  /**
   * constructor
   *
   * @param MainFilterAbstract $successor
   */
  public function __construct(MainFilterAbstract $successor = null)
  {
      $this->_successor = $successor;
  }
 
  /**
   * filter $content 
   *
   * @param [string|int] $content
   * @return string|int
   */
  public function filter($content)
  {
 
    $result = $this->doFilter($content);
    var_dump($this->_successor);
    if ($this->_successor) { 
        $result = $this->_successor->filter($result);
    }
 
    return $result;
  }
 
  abstract public function doFilter();
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
NULL
object(Exercises\ChainResponsabilityPattern\Complete\UpperCaseFilter)#3 (1) {
  ["_successor":protected]=>
  NULL
}
NULL
HELLO object(Exercises\ChainResponsabilityPattern\Complete\UpperCaseFilter)#3 (1) {
  ["_successor":protected]=>
  NULL
}
PHP Fatal error:  Uncaught Error: Call to a member function filter() on null in /home/kevin/php-interview-exercises/exercises/ChainResponsabilityPattern/Complete/UpperCaseFilter.php:19
Stack trace:
#0 /home/kevin/php-interview-exercises/exercises/ChainResponsabilityPattern/Complete/MainFilterAbstract.php(31): Exercises\ChainResponsabilityPattern\Complete\UpperCaseFilter->doFilter(48)
#1 /home/kevin/php-interview-exercises/exercises/ChainResponsabilityPattern/Complete/MainFilterAbstract.php(34): Exercises\ChainResponsabilityPattern\Complete\MainFilterAbstract->filter(48)
#2 /home/kevin/php-interview-exercises/exercises/ChainResponsabilityPattern/exercice.php(30): Exercises\ChainResponsabilityPattern\Complete\MainFilterAbstract->filter('24')
#3 {main}
  thrown in /home/kevin/php-interview-exercises/exercises/ChainResponsabilityPattern/Complete/UpperCaseFilter.php on line 19

Code:

1
2
3
4
5
6
7
8
$upperCaseFilter = new UpperCaseFilter();
$myFilter = new DoubleFilter($upperCaseFilter);
 
$space = ' ';
 
echo $myFilter->filter('hello') . $space;
echo $myFilter->filter('24') . $space; 
echo $myFilter->filter(360) . $space;

Le problème vient de '24' car le successeur est null, je recupère bien 48 mais si je fais ===null ou ===NULL dans le if même problème
Merci

Autant (*) pour moi : il faut adapter les enfants en fonction du parent, sinon ça sert à rien : on implémente la méthode abstraite et on ne touche surtout pas à la méthode public. Par exemple :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 namespace Exercises\ChainResponsabilityPattern\Complete; class UpperCaseFilter extends MainFilterAbstract { protected function doFilter($content = null) { if(is_string($content)) { return strtoupper($content); } return $content; } }
(*) ou au temps, je sais jamais et c'est pas important

01/10/2019, 16h20
grunk

D'un point vue purement conceptuel je définirais le successeur via un setter dédié et non via le constructeur.
Ça permet entre autre de pouvoir utiliser un des éléments de ta chaîne en toute indépendance sans définir aucun successeur.

Super merci Celira ça fonctionne,

Citation:

D'un point vue purement conceptuel je définirais le successeur via un setter dédié et non via le constructeur.
Ça permet entre autre de pouvoir utiliser un des éléments de ta chaîne en toute indépendance sans définir aucun successeur.

Merci
J'ai fais

Code:

1
2
3
4
5
6
7
8
9
$upperCaseFilter = new UpperCaseFilter();
$myFilter = new DoubleFilter();
$myFilter->setSuccessor($upperCaseFilter);
 
$space = ' ';
 
echo $myFilter->filter('hello') . $space;
echo $myFilter->filter('24') . $space;
echo $myFilter->filter(360) . $space;

Code:

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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
<?php
 
namespace Exercises\ChainResponsabilityPattern\Complete;
 
abstract class MainFilterAbstract 
{
  /**
   * @var MainFilterAbstract|null
   */
  protected $_successor = null;
 
  /**
   * getter $_successor
   *
   * @return MainFilterAbstract
   */
  public function getSuccessor() : MainFilterAbstract
  {
     return $this->_successor;
  }
 
  /**
   * setter $_successor
   *
   * @param MainFilterAbstract $succesor
   * @return void
   */
  public function setSuccessor(MainFilterAbstract $succesor = null)
  {
     $this->_successor = $succesor;
  }
 
  /**
   * filter $content 
   *
   * @param [string|int] $content
   * @return string|int
   */
  public function filter($content)
  {
 
    $result = $this->doFilter($content);
 
    if ($this->_successor) { 
        $result = $this->_successor->filter($result);
    }
 
    return $result;
  }
 
  abstract public function doFilter();
}

Par contre je n'ai pas compris l'utilité par rapport au constructeur :(

02/10/2019, 08h37
grunk

J'avais pas vu que dans ton constructeur tu définissais le successeur à null par défaut , donc effectivement pas de différence avec un setter.
Ceci étant dit je préfère passer par un setter. Quelqu'un qui reprend ton code qui veux juste filtrer un double par exemple va devoir se poser la question de ce paramètre dans le constructeur. est ce que je dois le définir , que se passe t'il si je le laisse à null , etc ... Avec un setter c'est plus transparent.