Discussion :

[maysa]

bonjour,

j'ai un petit soucis, bon c'est la premiere fois que j'utilise les exceptions. voici une partie de ma classe .
Je voudrais que si le parametre numLivre n'est pas numérique, je veux que la requete ne soit pas exécutée.
je lui demande donc que si mon param n'est pas un entier, on lance une exception ensuite on essaye de faire la requete et on attrape l'exception..
mais ça marche pas...
Je bloque depuis hier, alors toute aide est la bienvenue

Merci

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
class Livre extends BaseApp
	{
		var $numLivre; 
		var $jaquette; 
		var $nom;
		var $editeur;
		var $numEditeur;
 
		var $historique;
 
		function LivreInfo($numLivre)
		{
			global $basePath;
 
			$this->BaseApp();
 
			if(is_integer($numLivre))
			{
				$this->numLivre= $numLivre;
			}
			else
			{
				throw new Exception();
			}
			try{
			$q .= "SELECT C.nomLivre,C.historique,E.nomEditeur,E.numEditeur,LPAD(C.numLivre,5,'0') AS padNumCollec";
			$q .= " FROM Livre AS C";
			$q .= " LEFT JOIN editeur AS E ON (C.numEditeur=E.numEditeur)";
			$q .= " WHERE numLivre=" . $this->numLivre;
			$collec = $this->db->selectLine($q);
                        } 
catch(Exception $e)
{
 $e->getMessage();
}
 
 
			$this->nom = $collec["nomCollection"];
			$this->editeur = $collec["nomEditeur"];
			$this->numEditeur = $collec["numEditeur"];
			$this->historique = $collec["historique"];
		}

[Bisûnûrs]

Ton throw new Exception doit être dans un try pour être capturé, que ce soit à même le try, ou dans une fonction appelée dans le try.

Dans ton cas, on pourrait l'utiliser comme ça :

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
try {
   if(is_integer($numLivre)) {
      $this->numLivre= $numLivre;
   }
   else {
      throw new Exception('Le numéro du livre est erroné.');
   }
 
   $q .= "SELECT C.nomLivre,C.historique,E.nomEditeur,E.numEditeur,LPAD(C.numLivre,5,'0') AS padNumCollec";
   $q .= " FROM Livre AS C";
   $q .= " LEFT JOIN editeur AS E ON (C.numEditeur=E.numEditeur)";
   $q .= " WHERE numLivre=" . $this->numLivre;
   $collec = $this->db->selectLine($q);
} 
catch(Exception $e) {
   echo $e->getMessage();
}

[maysa]

Merci ,

bon finalement j'ai fait comme tu m'as dit. Mais je ne comprends pas car si le mets une chaine de caractère au lieu d'un numéro, j'affiche ma requete pour tester et j'ai bien le message d'erreur mais la requete s'exécute tout de meme avec le string dans le where .
Pourtant je dis bien

Code :

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

1
2
3
 if(is_integer($numLivre)) {
      $this->numLivre= $numLivre;
   }

donc comme dans mon where j'ai $this->numLivre, elle ne devrait pas le connaitre ??

[Bisûnûrs]

Normalement, si ton script passe dans le else, il ne devrait pas exécuter le reste du try.
Donc soit ton numLivre est bien un entier, soit tu n'exécutes pas ta requête au bon endroit.

Peut-on voir le code que tu as maintenant ? Et nous dire à quel endroit s'exécute ta requête ?

[maysa]

et voilà, merci pour ta réponse

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
try {
   if(is_integer($numLivre)) {
      $this->numLivre= $numLivre;
$q .= "SELECT C.nomLivre,C.historique,E.nomEditeur,E.numEditeur,LPAD(C.numLivre,5,'0') AS padNumCollec";
   $q .= " FROM Livre AS C";
   $q .= " LEFT JOIN editeur AS E ON (C.numEditeur=E.numEditeur)";
   $q .= " WHERE numLivre=" . $this->numLivre;
   $collec = $this->db->selectLine($q);
   }
   else {
      throw new Exception('Le numéro du livre est erroné.');
   }
}
catch (Exception $e)
{
$this->erreur = $e->getMessage();
}
 
 
}

[Bisûnûrs]

Heu, pourquoi as-tu mis la requête dans le if et non pas comme je l'ai mise ?

Et je pense qu'on aimerait avoir un exemple complet du code qui ne fonctionne pas.

[maysa]

Hello, eh bien tu exécutes la requete que si c'est du int... c'est pour cela que je l'ai mise là.
Je ne peux pas mettre d'autre code. ça fonctionne, c'est juste quand j'affiche mes requetes que j'ai quand meme un param en string dedans. Mais c'est peut etre tout simplement parce que je n'ai pas vidé mon cache

[Bisûnûrs]

Je comprends pas, c'est bon ou c'est pas bon alors ?

tu exécutes la requete que si c'est du int... c'est pour cela que je l'ai mise là.

Heu, mouais ... T'as pas dû tout capter aux exceptions alors. Parce que comme tu l'utilises, un simple if/else suffit, et pas besoin d'exception.

[maysa]

oui ça fonctionne, j'ai remis le code comme tu l'avais mis..
En effet peut etre pas tout compris aux exceptions. mais bon quand on entre dans un catch l'execution s'arrete non ? c'est comme si y'avait un exit. ?
je crois que je viens de dire une grosse bétise...

[Bisûnûrs]

je crois que je viens de dire une grosse bétise...

En effet.

Pour te donner un exemple simple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
try {
   throw new Exception( 'Message' );
 
   // Le code se trouvant ici ne sera pas exécuté
   echo 'Du texte';
}
catch ( Exception $e ) {
   // Exception lancée
}
 
echo 'Fin';

Dans cet exemple, le premier echo ne sera pas affiché, mais le second si. Le catch ne joue pas le rôle du exit, après l'exception le reste du code continue d'être exécuté.

[maysa]

Merci,

je comprends mieux...ça ira mieux quand je les utiliserai régulièrement