Discussion :

[sqlnoob]

Bonjour,

j'essaie de me mettre à la POO avec php, et je rencontre un petit soucis de conception...

J'ai une classe "cellule" qui contient une une méthode abstraite "display".
Cette méthode est abstraite car elle doit être redéfinie selon ce qu'on met dans cellule.

J'ai d'autre part une classe "liste".

Je souhaiterais que cette classe "liste" contienne un membre de la classe "cellule".

Dans l'esprit, j'aurais donc quelque chose comme:

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
 
abstract class cellule {
    protected $value;
    /* @var dlist_cell */
    protected $next;
    /* @var dlist_cell */
    protected $previous;
 
    abstract public function display();
}
 
class liste {
    /* @var cellule */
    protected $sentinel;
    protected $len;
 
    public function __construct() {
        $this->sentinel = new cellule();
        $this-len = 0;
    }
}

Or je ne peux pas instancier une classe abstraite.

Comment concevoir / modéliser ce genre de chose?

[bacargoudiaby]

Je dirais que le mieux c'est que tu n'utilises pas directement la classe cellule().
Il faudra par exemple créer une classe qui héritera de cellule() ensuite l'utiliser dans liste().

[sqlnoob]

Oui, mais justement...
Je souhaite que "liste" fonctionne avec n'importe quelle classe dérivée de "cellule"
Si je précise la classe dans "liste", alors elle n'est a priori plus aussi générique...

[bacargoudiaby]

OK mais l’intérêt d'une méthode ou classe abstraite c'est puisse être utilisé et modifier par d'autres classes selon les besoins.
Est ce que ça ne serait pas mieux que tu déclare la classe "cellule" comme abstraite vu qu'elle contient une méthode abstraite.

[sqlnoob]

mmmm...
Je ne suis pas sûr de comprendre...
Dans mon exemple de code, la classe "cellule" est en effet déclarée abstraite car elle contient une méthode abstraite.

[bacargoudiaby]

Désolé j'avais pas vu l'abstract dans:

Code :

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

abstract class cellule

Une classe abstraite ne peut pas être instancié, les classes filles le sont.

[Tsilefy]

Utilise l'injection de dépendance (qui est d'ailleurs préférable à l'utilisation de "new" dans le constructeur) en créant l'objet sentinelle en-dehors de la classe liste

Code :

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

1
2
3
4
public function __construct(cellule $sentinel) {
        $this->sentinel = $sentinel;
        $this-len = 0;
    }

tu crées ensuite une (ou plusieurs) classe concrète qui hérite de cellule, et tu passes une instance de cette classe dans le constructeur de liste, ex:

Code :

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

1
2
3
4
5
6
7
8
class CelluleA extends cellule {
    public function display() {
    }
}
 
$cellule_a = new CelluleA;
 
$liste = new Liste($cellule_a);

De cette manière, tu peux utiliser n'importe quelle classe concrète héritant de cellule dans liste.

Petite remarque, l'usage en PHP est de nommer les classes avec une majuscule au début, par exemple Cellule ou Liste pour que l'on distingue immédiatement que c'est une classe et non d'autres identifiants.

[sqlnoob]

Instinctivement, j'aurais pensé qu'il fallait que ce soit défini dans la classe.
Passer par une "injection de dépendance" me semble moins "carré", mais je n'y connais pas grand chose en poo, et m'en remets à toi!

Du coup ça veux dire que je ne suis pas sûr que le classe que je passe par injection dispose effectivement d'une méthode display().
N'est-ce pas problématique?

Merci

[Tsilefy]

- L'injection de dépendance est plus flexible, plus robuste, plus testable, bref meilleure que créer l'objet dans la classe, pour des tas de raisons qui ont été présentées depuis des décennies, et qu'une recherche Google te permettra de retrouver. Je ne m'étendrai donc pas sur ce sujet :-)

- La signature du constructeur est

Code :

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

public function __construct(cellule $sentinel)

Ça signifie que le constructeur acceptera seulement des objets héritant de cellule (puisqu'on ne peut pas utiliser cellule directement car elle est abstraite). Comme tous les héritiers de cellule doivent implémenter la méthode display(), tu es sûr à 100% que tous les objets passés disposent de la méthode display();

Si tu essaies de passer autre chose, tu auras un message de ce genre, par exemple si on passe un objet stdClass

Fatal error: Uncaught TypeError: Argument 1 passed to Liste::__construct() must be an instance of Cellule, instance of stdClass given

[sqlnoob]

Merci beaucoup pour ton aide!

[Tsilefy]

N'oublie pas de marquer le fil comme résolu si ton problème a été résolu :-)