Discussion :

[mely_84]

Bonjour,

J'ai un petit soucis et je viens à vous car je ne vois vraiment pas comment faire :/

j'ai une classe Dispatcher qui s'occupe en fonction du fichier passé en paramètre de créer un nouvel objet correspondant à une nouvelle classe kml par exemple le problème est que mes return ne se font pas ou du moins l'objet à la base Dispatcher ne se change pas en objet Kml.. j'espère avoir été claire

Voici mon code :

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
class Dispatcher {
 
	public function __construct($sFile) {
		$this->controlFile($sFile);
	}
 
	public function controlFile($sFile) {
		//Si le fichier existe
		if(file_exists($sFile)) {
			//Vérification de l'extension
			$extension = pathinfo($sFile, PATHINFO_EXTENSION);
 
			//Si l'extension est en "kml" ou e "KML"
			if(($extension == "kml") || ($extension == "KML")) {
				return $object = new Kml($sFile);
			}
			else {
				return new autre objet;
			}
		}
	}
}

index.php

Code :

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

1
2
3
$object = new Dispatcher("FILES/cardio.kml");
$object->getChFile();
print_r($object);

J'ai donc une erreur pour getChFile() car dispatcher ne l'a connais pas et que $object ne s'est pas changé en fait en objet KML.

L'autre classe fonctionne...

Avez vous une idée :s

Merci d'avance à vous

Cordialement

[TERRA126]

Bonjour,

Tout d'abord un return dans la méthode __construct de fonctionnera pas. Lorsque tu instancie une classe avec le mot clé new alors il te sera renvoyé l'instance de cette classe et rien d'autre.

Ensuite dans tout les cas cela n'aurais pas marché car tu as justement oublié le return dans $this->controlFile($sFile); .

A priori dans ta classe Dispatcher tout est dans une seule méthode, pourquoi ne pas externaliser cette méthode comme suite :

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
 
function controlFile($sFile) {
		//Si le fichier existe
		if(file_exists($sFile)) {
			//Vérification de l'extension
			$extension = pathinfo($sFile, PATHINFO_EXTENSION);
 
			//Si l'extension est en "kml" ou e "KML"
			if(($extension == "kml") || ($extension == "KML")) {
				return $object = new Kml($sFile);
			}
			else {
				return new autre objet;
			}
		}
	}

Avec pour l'appeler juste :

Code php :

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

1
2
 
$object = getChFile("FILES/cardio.kml");

[mely_84]

C'est justement ce que j'ai fais du coup j'ai tout externalisé parce que je me suis bien rendu compte que ce que j'essayais de faire était un peu fastidieux et inutile donc tampis ma fonction marche très bien sans classe et ça le restera comme ca !

Merci beaucoup d'avoir pris du temps pour me répondre en tout cas