Discussion :

[marry]

Salut tout le monde
j'ai une jsp dans laquelle je récupére 2 champs date d'une autre jsp, pour les utiliser dans une requete, voilà le 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
 
String date1=request.getParameter("textfield");
String date2=request.getParameter("textfield");
 
SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd" );
Date d1 = null;
Date d2 = null;
try {
d1 = sdf.parse(date1);		
d2 = sdf.parse(date2);
} 
catch (ParseException e) 
{			
e.printStackTrace();
}
//connexion à la base et chargement de driver
PreparedStatement stm = con.prepareStatement ("SELECT * FROM maTable WHERE (date1='"+d1+"') AND (date2='"+d2+"')");

le probleme c'est que lorsque la date1 et date2 sont egals à null, il m'affiche l'erreur suivante: syntax error at or near "null"

j'espere que vous pouvez m'aider à resoudre ce probleme
merci d'avance

[tchize_]

si date1 ou date2 est null, tu va avoir du mal à les transformer en objet Date. Rajouet simplement un test avec un if pour ces cas là.

[Lorantus]

Pour avoir un code stable, il est impératif de faire un test des valeurs retournées par getParameter(...) :
- si null, pour eviter une NullException
- si au bon format (try/catch, il faut en abuser)
- transformer les "<" en &lg; pour eviter les modification d'affichage HTML des texts soumis par les utilisateurs
- encodeURL(...) pour garder les sessions dans les flux de navigation des utilisateurs.

Ne jamais supposer que les valeurs transmisses en paramètre de requête sont bonnes !

On le répétera jamais assez.

[luminien]

Bonjour,
Je suis d accord avec Lorantus sur le test de tes valeurs
avec un ( request.getParameter(date1)!=null) et toute les fonctions qui teste la validité des valeurs entée par l'utilisateur( faut supposer que l'utilisateur est bete )
et dans ton cas ta requete sql
tu declare une date d1 null et tu fais :

Code :

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

(date1='"+d1+"')

si date1 est null c'est que d1 est null donc a mon avis tu entre pas dans ton try
essaye un System.out.println() dans ton try et regarde dans ton fichier log tomcat est ce qu'il entre dedans ou non !
ou plutot regarde juste la trace de ton exception vu que tu fais un

Code :

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

e.printStackTrace();

luminien