Discussion :

[vonitiana]

Je suis sous jsp msql!

Sur ma page jsp, je veux afficher la liste de mes mots clés. L problème est ça marche pour une autre fonction qui ne se sert pas de "id_bouquet" (variable que j'ai dû rajouter pour ma 2è méthode).

Les codes sont les mêmes à une variable près et voilà ce qu'il me renvoie!

java.sql.SQLException: Syntax error or access violation: You have an error in your SQL syntax near 'id_bouquets=1' at line 1
Problem requestion Keywords for user with id 11

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
/**
          * lit les mots clés dans la classe.
          * @return <var>true</var> si pas de pb, <var>false</var> sinon.
          */
	private boolean read()
	{
		try
		{
			String query="select id, mot_cle from PMUserBouquetsKeywords where "
				+"id_user="+id_user+" and id_bouquets="+id_bouquet;
 
                        ResultSet rs=DBRequestor.executeQuery(query);
			int i=0;
			while(rs.next())
			{
				id[i]=rs.getInt(1);
				mot_cle[i++]=rs.getString(2);
			}
			nb_keyword=i;
			return true;
		}
		catch(Exception e)
		{
			System.err.println("Problem Requesting Keywords for user with id "+this.id_user);
			return false;
		}
	}

[mavina]

Salut,

Le premiere chose qui me vienne à l'esprit est le fait qu'il n'y ai pas de quotes dans ta requete :

Code :

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

1
2
3
 
String query="select id, mot_cle from PMUserBouquetsKeywords where "
+"id_user='"+id_user+"' and id_bouquets='"+id_bouquet+"'";

Ensuite, fais un

Code :

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

e.printStackTrace();

pour voire où exactement, et l'intitulé exact de ton exception.

Fred

[in]

tu devrais afficher la requete, la solution sauterait surement aux yeux.

peut être en mettant des ' ' autour des valeur ->

Code :

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

1
2
String query="select id, mot_cle from PMUserBouquetsKeywords where "
				+"id_user='"+id_user+"' and id_bouquets='"+id_bouquet+"'";

normalement c'est mieux

[EDIT] grillé !

[vonitiana]

merci à tous les deux!

ça marche!

ce qui m'étonne c'est que mes variables sont des "int". Je croyais qu'on utilise les quotes dans le cas seulement des string!

[mavina]

tout ce qui n'est pas nom de colonne est entre quotes

Fred

[in]

ps : et si tu as des String n'oublie pas de les traiter (remplacer ' par '', virer les # -- ; et autres caractères "dangereux")

[vonitiana]

d'accord!

[adiGuba]

Salut,

Le plus simple serait d'utiliser des PreparedStatement :
Les instructions paramétrées - PreparedStatement

a++