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.