Discussion :

[Tchupacabra]

Bonjour à tous,

Comment faire en sorte que MaClasse return FALSE (au lieu de l'objet) en cas d'erreur à l'initialisation par exemple ?

J'essaye de faire un truc du genre :

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
<?php
 
class MaClasse {
	function MaClasse($param) {
		// initialisation [...]
 
		// en cas de problème :
		return FALSE;
	}
}
 
$o = new MaClasse('truc');
if ($o === FALSE)
	die('erreur');
else
	die('ok');
 
?>

Mais ce script affiche toujours 'ok'...
Une idée ?

Merci d'avance

[s.n.a.f.u]

En cas de problème, tu lèves une exception que tu peux ensuite intercepter par un bloc try/catch

[Momodedf]

Un constructeur renvoie toujours l'objet qu'il a créé, tu ne peux pas modifier ça.
Quand tu termines ton constructeur, tu n'as pas besoin de faire un

Code :

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

 return this;

ça se fait tout seul.

Comme le dit snafu le meilleur moyen est d'encadrer la création de ton objet par un try/catch et de lever une exception (throw new Exception) dans le constructeur en cas problème.

[Tchupacabra]

ok ! merci à vous 2...

par contre, d'après mes recherches,

Code :

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

throw new Exception('Une erreur est survenue.');

permet de gérer les erreurs bloquantes.

Est-il possible de gérer également les debug, warning, info & Co ?

J'ai essayé ainsi :

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
38
39
40
<?php
class MaClasse {
 
	function __construct($path) {
 
		if ( !is_string($path) )
			// erreur bloquante
			throw new Exception('Le paramètre est incorrecte.');
 
		if ( file_exists($path) ) {
			// erreur bloquante
			throw new Exception('Un fichier existe déjà à l\'emplacement indiqué.');
		} else {
			// info
			new Exception('Le chemin est libre.');
		}
 
	}
	function azerty() {
		// info
		new Exception('azerty()');
	}
}
 
 
 
try {
 
	$o = new MaClasse('/truc.jpg');
	$o->azerty();
	echo '<p>Traitement terminé avec succès.</p>';
 
	//echo '<p>INFOS :'.$e->getMessage().'</p>';
 
 
} catch (Exception $e) {
	echo '<p>Des erreurs sont survenues lors du traitement de l\'image.</p>';
	echo '<p>ERREUR : '.$e->getMessage().'</p>';
}
?>

Mais je ne sais pas comment récupérer les infos : "Le chemin est libre." et "azerty()"...

Encore une p'tite aide SVP.
Merci d'avance

@+

[Momodedf]

} else {
// info
new Exception('Le chemin est libre.');
}

}
function azerty() {
// info
new Exception('azerty()');
}
}
[/CODE]

Mais je ne sais pas comment récupérer les infos : "Le chemin est libre." et "azerty()"...

Encore une p'tite aide SVP.
Merci d'avance

@+

Tu as oublié le throw.

[s.n.a.f.u]

Une exception est levée normalement en cas d'erreur.
Il ne devrait pas s'agir d'information.

Peut-être pourrais-tu faire un truc comme cela :

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
38
39
40
41
42
43
<?php
class MaClasse {
 
    private $_messages;
 
	function __construct($path) {
	    $this->_messages = array();
		if ( !is_string($path) )
			// erreur bloquante
			throw new Exception('Le paramètre est incorrecte.');
 
		if ( file_exists($path) ) {
			// erreur bloquante
			throw new Exception('Un fichier existe déjà à l\'emplacement indiqué.');
		} else {
			// info
			$this->_messages['construct'] = 'Le chemin est libre.';
		}
 
	}
	function azerty() {
		$this->_messages['azerty'] = 'message azerty';;
	}
 
    public function getMessage($type) {
        return $this->_messages[$type];
}
 
 
 
try {
 
	$o = new MaClasse('/truc.jpg');
	$o->azerty();
	echo '<p>Traitement terminé avec succès.</p>';
 
	echo '<p>INFOS :'.$o->getMessage('azerty').'</p>';
 
 
} catch (Exception $e) {
	echo '<p>Des erreurs sont survenues lors du traitement de l\'image.</p>';
	echo '<p>ERREUR : '.$e->getMessage().'</p>';
}

[Tchupacabra]

Bonjour,

merci pour votre aide.

Tu as oublié le throw.

non, c'était volontaire, afin de mettre un log sans être bloquant...

Une exception est levée normalement en cas d'erreur.
Il ne devrait pas s'agir d'information.

c'est bien ce que je craignais...
Il faut donc passer par 2 systèmes (un pour gérer les exceptions et l'autre pour les logs)
Je trouve ça dommage...

comme toi, j'avais pensé à stocker mes logs dans un array définit dans la classe.
Je vais voir ce que je peux en tirer.

Je vous tient au courant si j'ai trouvé une autre méthode.

@+