Discussion :

[Sarra90]

j'ai crée une base de données, elle contient un tableau nommé TABLE_CATEGORY qui est formé par 3 colonnes :[*][id_category,category_name,image]
Dans mon application Android je veux afficher les noms de ces catégories dans des TextView
J'ai crée une méthode, mais elle ne fonctionne pas, je ne sais pas ou est le probléme

Code :

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

1
2
3
4
5
6
7
8
9
10
 public String getCategoryName(int i)
   {
	   String selectQuery ="SELECT " + COL_CATEGORY_NAME + "FROM " +
			    TABLE_CATEGORY + " WHERE " + COL_ID_CATEGORY +" = '" +i +"')";
 
	   database=DBHelper.getWritableDatabase();
	   Cursor cursor = database.rawQuery(selectQuery, null);
 
	   return cursor.toString();
    }

[nicroman]

cursor.toString() probablement va renvoyer le 'nom' du cursor, et pas le contenu d'une des colonnes du cursor:

D'abord, se rendre sur la première ligne, et si elle existe, demander la valeur de la première colonne.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
public String getCategoryName(int i)
{
String selectQuery ="SELECT " + COL_CATEGORY_NAME + "FROM " +
TABLE_CATEGORY + " WHERE " + COL_ID_CATEGORY +" = '" +i +"')";
 
database=DBHelper.getWritableDatabase();
Cursor cursor = database.rawQuery(selectQuery, null);
 
return (cursor.moveToFirst()) ? cursor.getString(0) : null;
}

Maintenant il faut aussi:
* Fermer le cursor.
* Utiliser les "prepared" statement de SQL:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
private static final String SQL_CATEGORY_NAME =
   "SELECT " + COL_CATEGORY_NAME + "FROM " + TABLE_CATEGORY + " WHERE " + COL_ID_CATEGORY +" = ?";
 
public String getCategoryName(int i)
{
    Cursor cursor = null;
    try {
         database=DBHelper.getWritableDatabase();
         cursor = database.rawQuery(SQL_CATEGORY_NAME , new String[] { Integer.toString(i) } );
         return (cursor.moveToFirst()) ? cursor.getString(0) : null;
    } finally {
         if (cursor != null) cursor.close();
    }
}

On remarquera:
1. La construction de la requete SQL en dehors de la fonction (évite de la refaire à coups de StringBuilder à chaque appel de la fonction).
2. Utilisation de query "paramétrée" (le ?) et les paramètres associés dans l'appel => utilisation de prepared statement, et souvent "cache" de ceux-ci.
3. Utilisation de try / finally pour fermer en toute circonstance le cursor.

Par contre, il me semble qu'il faille faire de même (close()) sur la database obtenue par getWritableDatabase() (en tout cas, à un moment il faudra la fermer).