Verifier le type avec la méthode getParameter()

Version imprimable

Bonjour,

J'ai un soucis avec mon servlet. Je récupère les données d'un formulaire grâce à la méthode getParameter() et je souhaiterais verifier si le type de ce que je récupère est bien un entier pour la getion des erreurs.

Je me suis pas mal renseigné et je n'ai pas trouvé. Si quelqu'un pouvait m'aider.

Voici mon code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 String isbn = request.getParameter("isbn"); if (isbn.length() == 0) { out.println("<body>"); out.println("L'enregistrement du numéro ISBN de ce livre est vide<br>"); out.println("</body>"); } else if (isbn ???????????? Int) { out.println("<body>"); out.println("L'enregistrement du numéro ISBN de ce livre n'est pas valide (le numéro ISBN doit être un entier)<br>"); out.println("</body>"); } else { out.println("<body>"); out.println("<font size=\"3\">"); out.println("<br>ISBN :" + isbn); out.println("</body>"); }
Merci d'avance

Tu peux te définir une fonction pour tester si c'est un entier

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public boolean testSiEntier(String chaine)
{
  try
  {
    int i = Integer.parseInt(chaine);
    return true;
  }
  catch (NumberFormatException e)
  {
     return false;
   }
}

26/05/2008, 09h32
charlix

Bonjour,

Merci pour ta réponse. Mais est ce qu'il n'existe pas une méthode du genre instanceof pour faire cela directement dans une condition d'une boucle if ?

Sinon je vais essayer ton bout de code.

Les expressions régulières ça te conviendrait?

Code:

1
2
3
4
5
6
7
 
else if (!isbn.matches("\\d*")) //vérifie si la chaine est un entier
    	{
    		out.println("<body>");
    		out.println("L'enregistrement du numéro ISBN de ce livre n'est pas valide (le numéro ISBN doit être un entier)<br>");
    		out.println("</body>");	
    	}

26/05/2008, 09h55
adiGuba

Salut,

Citation:

Envoyé par charlix

Merci pour ta réponse. Mais est ce qu'il n'existe pas une méthode du genre instanceof pour faire cela directement dans une condition d'une boucle if ?

instanceof vérifie le type de l'objet. Or getParameter() te renvoi forcément une String donc ce sera toujours faux...

Citation:

Envoyé par Deadpool

Les expressions régulières ça te conviendrait?

A mon avis la solution via parseInt() est préférable, car cela permet de vérifier en même temps les valeurs minimum et maximum, alors que l'expression acceptera toutes les valeurs même énorme.

a++
26/05/2008, 10h01
Deadpool

Citation:

Envoyé par adiGuba

A mon avis la solution via parseInt() est préférable, car cela permet de vérifier en même temps les valeurs minimum et maximum, alors que l'expression acceptera toutes les valeurs même énorme.

a++

Je suis d'accord mais le monsieur il voulait quelque chose sur une ligne. ;)

Cela dit ta précision n'est pas inutile.
26/05/2008, 10h54
guigui5931

En même temps si tu as défini une fonction l'appel de cette fonction se fait sur une ligne ;)
26/05/2008, 11h33
charlix

Bonjour à tous,

Merci de vos réponses.

Citation:

Je suis d'accord mais le monsieur il voulait quelque chose sur une ligne.

Nan pas forcement, ce n'est pas cela que je souhaite, mais pour ma part je pense qu'il exite bien une méthode qui fait cela.

J'ai testé les deux solutions mais aucune ne fonctionne. Je ne comprend pas bien encore pourquoi. Alors que cela devrait fonctionner.
26/05/2008, 12h09
guigui5931

Montre nous ce que tu a fait mais normalement les 2 solutions marchent.

Je ne sais pas pourquoi mais je crois que mon tomcat me joue des tours.

Voici mon code :

J'ai le code de la fonction testSiEntier un peu plus haut dans la code

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
	public boolean testSiEntier(String parametre)
	{
	  try
	  {
	    int i = Integer.parseInt(parametre);
	    return true;
	  }
	  catch (NumberFormatException e)
	  {
	     return false;
	   }
	}

Code:

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
 
String annee = request.getParameter("annee");
    	if (annee.length() == 0)
    	{
 
    		out.println("L'enregistrement de l'année de sortie de ce livre est vide" + "<br>");
 
    	}
    	else if (!annee.matches("\\D*"))
    	{
 
    		out.println("L'enregistrement de l'année de sortie de ce livre est invalide (l'année doit être un entier)" + "<br>");
 
    	}
    	/*else if (testSiEntier(annee) == false)
    	{
    		
    		out.println("L'enregistrement de l'année de sortie de ce livre est invalide (l'année doit être un entier)" + "<br>");
    		
    	}*/
    	else
    	{
 
    		out.println("<font size=\"3\">");
    		out.println("Année :" + annee + "<br>");
 
    	}

Cela devrait fonctionner, je ne comprend pas

26/05/2008, 12h23
adiGuba

Salut,

C'est \d et non pas \D qui signifie exactement l'inverse...

a++

PS : il y a quand même des solutions existantes pour la validation des formulaires...
26/05/2008, 12h29
charlix

C'est bon tout fonctionne bien avec la méthode matches() ainsi qu'avec parseInt().

Merci à tous.

PS : j'avais un soucis avec mes librairies server.jar et servlet.jar.