Erreur SQLite requête select

Version imprimable

Bonjour,

Je fais une requête SELECT mais l'erreur suivante s'affiche:

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
06-11 11:27:11.051: E/AndroidRuntime(873): FATAL EXCEPTION: main
06-11 11:27:11.051: E/AndroidRuntime(873): android.database.sqlite.SQLiteException: near ""Petot"": syntax error: , while compiling: SELECT nbpizzaencours FROM client WHERE nomLIKE "Petot"
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.database.sqlite.SQLiteCompiledSql.native_compile(Native Method)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.database.sqlite.SQLiteCompiledSql.<init>(SQLiteCompiledSql.java:64)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.database.sqlite.SQLiteProgram.compileSql(SQLiteProgram.java:143)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.database.sqlite.SQLiteProgram.compileAndbindAllArgs(SQLiteProgram.java:361)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.database.sqlite.SQLiteProgram.<init>(SQLiteProgram.java:127)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.database.sqlite.SQLiteProgram.<init>(SQLiteProgram.java:94)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.database.sqlite.SQLiteQuery.<init>(SQLiteQuery.java:53)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.database.sqlite.SQLiteDirectCursorDriver.query(SQLiteDirectCursorDriver.java:47)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.database.sqlite.SQLiteDatabase.rawQueryWithFactory(SQLiteDatabase.java:1564)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.database.sqlite.SQLiteDatabase.queryWithFactory(SQLiteDatabase.java:1449)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.database.sqlite.SQLiteDatabase.query(SQLiteDatabase.java:1405)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.database.sqlite.SQLiteDatabase.query(SQLiteDatabase.java:1485)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at com.example.cartedefidelite.clientBDD.getNbpizzawithNom(clientBDD.java:99)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at com.example.cartedefidelite.FideliteActivity$1.onClick(FideliteActivity.java:39)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.view.View.performClick(View.java:3480)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.view.View$PerformClick.run(View.java:13983)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.os.Handler.handleCallback(Handler.java:605)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.os.Handler.dispatchMessage(Handler.java:92)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.os.Looper.loop(Looper.java:137)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at android.app.ActivityThread.main(ActivityThread.java:4340)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at java.lang.reflect.Method.invokeNative(Native Method)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at java.lang.reflect.Method.invoke(Method.java:511)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:784)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:551)
06-11 11:27:11.051: E/AndroidRuntime(873): 	at dalvik.system.NativeStart.main(Native Method)

Et voici le code:

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
public class FideliteActivity extends AffichageClientActivity /*implements View.OnClickListener*/ {
	clientBDD clibdd = new clientBDD(this);
	EditText nombre;
	Button detail;
	Button ok;
	String nomduclient;
 
	protected void onCreate(Bundle savedInstanceState) {
		super.onCreate(savedInstanceState);
		setContentView(R.layout.fidelite);
		//récupération de la valeur de l'edittext
		Bundle objetbundle = this.getIntent().getExtras();
	    nomduclient = objetbundle.getString("nomclient");
		detail = (Button) findViewById(R.id.Detail);
		ok = (Button) findViewById(R.id.valider);
		nombre = (EditText) findViewById(R.id.nb);
		ok.setOnClickListener(validerListener);
		detail.setOnClickListener(detailclientListener);
	}
 
	private OnClickListener validerListener = new OnClickListener(){
		public void onClick(View v){
			if(v==ok){
			//récupération de la valeur dans l'EditText
			String nb = nombre.getText().toString();
			int nbpizz = Integer.parseInt(nb);
			//connexion à la BDD et récupération du nombre de pizza existant
			cliBDD.open();
			cli = cliBDD.getNbpizzawithNom(nomduclient);
			int nbexist = cli.getNbpizzaencours();
 
			//calcul plus affichage des avantages fidelité
			nbexist = nbexist + nbpizz;
			if(nbexist >=30)
			{
				Toast.makeText(
						FideliteActivity.this,
						"3 pizzas gratuites!",
						Toast.LENGTH_LONG).show();
				nbexist = nbexist-30;
			}
			else if(nbexist >=20){
				Toast.makeText(
						FideliteActivity.this,
						"2 pizzas gratuites!",
						Toast.LENGTH_LONG).show();
				nbexist = nbexist-20;
 
			}else if(nbexist >=10){
				Toast.makeText(
						FideliteActivity.this,
						"1 pizza gratuite!",
						Toast.LENGTH_LONG).show();
				nbexist = nbexist-10;
			}
			else{
				Toast.makeText(
						FideliteActivity.this,
						"Pas de pizza gratuite pour cette fois!",
						Toast.LENGTH_LONG).show();
			}
 
			//mis à jour de la BDD
			//cliBBD.updateClient(int id, client cli);
 
 
			//fermeture de la BDD
			cliBDD.close();
 
			}
		}
	};
 
	private OnClickListener detailclientListener = new OnClickListener(){
		public void onClick(View v){
		Intent versDetail = new Intent(getApplicationContext(),
		    DetailClientActivity.class);
			versDetail.putExtra("nomclient",nomduclient);
			startActivity(versDetail);
 
		}
	};
 
}

Ainsi que ma fonction

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
public client getNbpizzawithNom(String nom) {
		Cursor c = bdd.query(TABLE_CLIENT, new String[] { COL_PIZZAENCOURS },
				COL_NOM + "LIKE \"" + nom + "\"", null, null, null, null);
		return cursorToClient(c);
	}
public client cursorToClient(Cursor c) {
		if (c.getCount() == 0) {
			return null;
		}
		c.moveToFirst();
		client cli = new client();
		cli.setId(c.getInt(NUM_COL_ID));
		cli.setNom(c.getString(NUM_COL_NOM));
		cli.setPrenom(c.getString(NUM_COL_PRENOM));
		cli.setAdresse(c.getString(NUM_COL_ADRESSE));
		cli.setCp(c.getString(NUM_COL_CP));
		cli.setVille(c.getString(NUM_COL_VILLE));
		cli.setMail(c.getString(NUM_COL_MAIL));
		cli.setTel(c.getString(NUM_COL_TEL));
		cli.setNbpizzaencours(c.getInt(NUM_COL_PIZZAENCOURS));
		cli.setNbpizzatotal(c.getInt(NUM_COL_PIZZATOTAL));
		c.close();
		return cli;
 
	}

Donc, je me suis dit que c'était comme la dernière fois, puisque j'ai rajouté cette fonction après, et que je devais donc désinstaller/réinstaller l'appli de l'émulateur, seulement l'erreur persiste :(

Comment remédier à ce problème?

Merci d'avance :)

11/06/2013, 11h49
Twixou

Bonjour,

Rajoute un espace avant "LIKE"
11/06/2013, 11h52
maxusn
Citation:

while compiling: SELECT nbpizzaencours FROM client WHERE nomLIKE "Petot"

remplace
Code:

1 2 ursor c = bdd.query(TABLE_CLIENT, new String[] { COL_PIZZAENCOURS }, COL_NOM + "LIKE \"" + nom + "\"", null, null, null, null);
tu fais COL_NOM + "LIKE \"" il te manque un espace.

Code:

COL_NOM + " LIKE \""

devrait resoudre le soucis

En effet, c'est mieux! :P Merci!
Seulement, j'ai encore une erreur:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
06-11 11:55:15.897: E/CursorWindow(1015): Failed to read row 0, column 1 from a CursorWindow which has 1 rows, 1 columns.
06-11 11:55:15.947: E/AndroidRuntime(1015): FATAL EXCEPTION: main
06-11 11:55:15.947: E/AndroidRuntime(1015): java.lang.IllegalStateException: Couldn't read row 0, col 1 from CursorWindow.  Make sure the Cursor is initialized correctly before accessing data from it.
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at android.database.CursorWindow.nativeGetString(Native Method)
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at android.database.CursorWindow.getString(CursorWindow.java:451)
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at android.database.AbstractWindowedCursor.getString(AbstractWindowedCursor.java:51)
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at com.example.cartedefidelite.clientBDD.cursorToClient(clientBDD.java:111)
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at com.example.cartedefidelite.clientBDD.getNbpizzawithNom(clientBDD.java:101)
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at com.example.cartedefidelite.FideliteActivity$1.onClick(FideliteActivity.java:39)
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at android.view.View.performClick(View.java:3480)
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at android.view.View$PerformClick.run(View.java:13983)
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at android.os.Handler.handleCallback(Handler.java:605)
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at android.os.Handler.dispatchMessage(Handler.java:92)
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at android.os.Looper.loop(Looper.java:137)
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at android.app.ActivityThread.main(ActivityThread.java:4340)
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at java.lang.reflect.Method.invokeNative(Native Method)
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at java.lang.reflect.Method.invoke(Method.java:511)
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:784)
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:551)
06-11 11:55:15.947: E/AndroidRuntime(1015): 	at dalvik.system.NativeStart.main(Native Method)

Je ne sais pas si c'est lié mais:
Au lieu de :
Code:

1 2 3 4 5 if (c.getCount() == 0) { return null; } c.moveToFirst();
Un simple:
Code:

1 2 3 if (!c.moveToFirst()) return null;
suffit....

Et aussi on ne construit jamais (ou presque) les requêtes SQL programmatiquement (pour éviter les problème au dessus):
Au lieu de faire:
Code:

1 2 3 Cursor c = bdd.query(TABLE_CLIENT, new String[] { COL_PIZZAENCOURS }, COL_NOM + " LIKE \"" + nom + "\"", null, null, null, null);
on fait:
Code:

1 2 3 4 private static final String WHERE_PIZZA_LIKE = COL_NOM+" LIKE ?"; Cursor c = bdd.query(TABLE_CLIENT, new String[] { COL_PIZZAENCOURS }, WHERE_PIZZA_LIKE, new String[] { nom }, null, null, null);
Celà évite de s'embeter à "escaper" soit meme les paramètre (une string est délmitée par ' et non " en SQL).

Pour finir, tu demandes une seule colonne: COL_PIZZAENCOURS, mais dans la fonction cursorToClient il y a plein d'autres colonnes d'utilisées => ca va planter.

Citation:

Envoyé par nicroman

Pour finir, tu demandes une seule colonne: COL_PIZZAENCOURS, mais dans la fonction cursorToClient il y a plein d'autres colonnes d'utilisées => ca va planter.

L'erreur semble venir de là en effet, mais en fait j'ai une autre fonction qu'utilise toute les colonnes, alors je me suis dit que je pouvais réutiliser celle là
Voilà les fonctions:
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 28 29 30 31 32 33 34 35 36 public client getClientwithNom(String nom) { Cursor c = bdd.query(TABLE_CLIENT, new String[] { _ID, COL_NOM, COL_PRENOM, COL_ADRESSE, COL_CP, COL_VILLE, COL_MAIL, COL_TEL, COL_PIZZAENCOURS, COL_PIZZATOTAL }, COL_NOM + " LIKE \"" + nom + "\"", null, null, null, null); return cursorToClient(c); } public client getNbpizzawithNom(String nom) { Cursor c = bdd.query(TABLE_CLIENT, new String[] { COL_PIZZAENCOURS }, COL_NOM + " LIKE \"" + nom + "\"", null, null, null, null); return cursorToClient(c); } public client cursorToClient(Cursor c) { if (c.getCount() == 0) { return null; } c.moveToFirst(); client cli = new client(); cli.setId(c.getInt(NUM_COL_ID)); cli.setNom(c.getString(NUM_COL_NOM)); cli.setPrenom(c.getString(NUM_COL_PRENOM)); cli.setAdresse(c.getString(NUM_COL_ADRESSE)); cli.setCp(c.getString(NUM_COL_CP)); cli.setVille(c.getString(NUM_COL_VILLE)); cli.setMail(c.getString(NUM_COL_MAIL)); cli.setTel(c.getString(NUM_COL_TEL)); cli.setNbpizzaencours(c.getInt(NUM_COL_PIZZAENCOURS)); cli.setNbpizzatotal(c.getInt(NUM_COL_PIZZATOTAL)); c.close(); return cli; } }
Donc voilà, est-ce que je peux réutiliser la fonction public client cursorToClient(Cursor c) en utilisant seulement la colonne COL_PIZZAENCOURS où dois je procéder autrement?
Je vois pas trop comment faire..

Et je vais suivre tes conseils pour les requêtes, si c'est mieux donc, merci! :)

Solution 1. Utiliser des fonctions différentes.

Solution 2. Utiliser une seule fonction, mais vérifier à chaque fois que la colonne est présente.
Code:

1 2 3 4 5 6 7 8 int idx; Client ret = new Client(); // <= En JAVA, un nom de type commence TOUJOURS par une majuscule ! idx = cursor.getColumnIndex(COL_NOM); if (idx >= 0 && !cursor.isNull(idx)) ret.setNom(cursor.getString(idx)); idx = cursor.getColumnIndex(COL_ID); if (idx >= 0 && !cursor.isNull(idx)) ret.setId(cursor.getInt(idx)); ...
Solution 3. Toujours utiliser le même tableau de colonnes (ce qui au passage permet d'être en phase avec les NUM_xxxx)
Code:

1 2 3 4 private static String[] ALL_COLS = { _ID, COL_NOM, COL_PRENOM, COL_ADRESSE, COL_CP, COL_VILLE, COL_MAIL, COL_TEL, COL_PIZZAENCOURS, COL_PIZZATOTAL };
Et utiliser ALL_COLS dans chaque query...

Je n'aime pas la solution 2 car elle retourne des objets non complets....
La solution 1 permet de retourner d'autres types d'objets (complets) mais multiplie la quantité de types.
La solution 3 quant à elle récupère plus de données de la DB (donc est légèrement, mais très légèrement plus lente).

11/06/2013, 17h38
Pepito39

Je pense donc opter pour la solution 1, mais comment convertir un cursor en entier?