Discussion :

[laurentSc]

Bonjour,

j'ai une classe (C1) dont une méthode doit lancer une méthode d'une autre classe (C2). Pour ce faire la classe C1 instancie l'autre classe (C2) pour en créer un référent puis appelle sa méthode :

Code php :

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

1
2
$ref_model_CVSImport = new CSVImport();
$ref_model_CVSImport->uploadFile($this->filepath);

Par contre, j'ai lu, donc me demande si ça serait mieux que si une classe C1 communique avec une autre (C2), il vaudrait mieux que le référent vers C2 soit un attribut de C1. Quel est votre avis ?

[laurentSc]

Bonjour,

submergé par les réponses , j'ai finalement décidé de mettre tous les référents de classe en attributs car quand on relit le code, c'est beaucoup plus clair : on voit des attributs private $ref_xxx, donc on sait tout de suite avec quelles classes la classe présente communique.

[grunk]

C'est important d'éviter de lier tes classes entre elle trop fortement.

Quand C1 fait un new C2 dans une de ses méthode tu créer un couplage fort entre ces 2 classes. (sous entendu que C1 ne peut plus exister sans C2)
Au contraire quand tu injecte C2 dans C1 (via le constructeur par exemple) ta classe C1 n'as pas besoin de connaitre les problématiques d'instanciation de C2 et ne sera pas impactée en cas de changement.

Donc :
Pas bien :

Code :

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

1
2
3
4
5
6
class C1 {
	public function foo() {
		$c2 = new C2();
		$c2->bar();
	}
}

Mieux

Code :

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

1
2
3
4
5
6
7
8
9
10
11
class C1 {
    private C2 $c2;
 
	public __construct(C2 $c2) {
	    $this->c2 = $c2;
	}
 
	public function foo() {
		$this->c2->bar();
	}
}

[laurentSc]

Ouh là, là, tu me fais prendre conscience d'une grosse erreur de conception

[laurentSc]

Je suis embêté pour implémenter ta solution : en effet dans ta solution "mieux" tu passes en argument du constructeur de C1 un référent vers C2. Autrement dit, il faut avoir créé C2 avant C1 pour que ce référent existe. Donc même si le new C2(); n'est pas dans C1, il faut le mettre quelque part. Mais où ?

[cavo789]

Un truc du genre...

Code :

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

$c1 = new C1(new C2());$c1->run();

ou

Code :

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

1
2
3
4
$c2 = new C2();
 
$c1 = new C1($c2);
$c1->run();