Discussion :

[mike3012]

Bonjour,

Je souhaite afficher les données d'une base de données dans une ListView (à priori rien de méchant).

Voici les éléments de mon code :

J'ai une classe pour créer la base de données :

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
 
public class FavoriteOpenHelper extends SQLiteOpenHelper {
	public static final String TABLE_FAV = "favorites_table";
	public static final String COL_ID = "ID";
	public static final String COL_CITY = "City";
	public static final String COL_COUNTRY = "Country";
 
	public static final String CREATE_BDD = "CREATE TABLE " + TABLE_FAV + " (" + COL_ID + " INTEGER PRIMARY KEY AUTOINCREMENT, " + COL_CITY + " TEXT NOT NULL, " + COL_COUNTRY + " TEXT NOT NULL); ";
 
	public FavoriteOpenHelper(Context context, String name, CursorFactory factory, int version) {
		super(context, name, factory, version);
	}
 
	@Override
	public void onCreate(SQLiteDatabase db) {
		//on créé la table à partir de la requête écrite dans la variable CREATE_BDD
		db.execSQL(CREATE_BDD);
	}
 
	@Override
	public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion) {
		//On peut fait ce qu'on veut ici moi j'ai décidé de supprimer la table et de la recréer
		//comme ça lorsque je change la version les id repartent de 0
		db.execSQL("DROP TABLE " + TABLE_FAV + ";");
		onCreate(db);
	}

Ensuite j'ai une classe pour définir la base de données :

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
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 FavoritesBDD {
 
	public static final int VERSION_BDD = 1;
	private static final String NOM_BDD = "favorites.db";
 
	private static final String TABLE_FAV = "favorites_table";
	private static final String COL_ID = "ID";
	private static final int NUM_COL_ID = 0;
	private static final String COL_CITY = "City";
	private static final int NUM_COL_CITY = 1;
	private static final String COL_COUNTRY = "Country";
	private static final int NUM_COL_COUNTRY = 2;
 
	private SQLiteDatabase bdd;
	private FavoriteOpenHelper favoriteOpenHelper;
 
	public FavoritesBDD(Context context){
		favoriteOpenHelper = new FavoriteOpenHelper(context, NOM_BDD, null, VERSION_BDD);
	}
 
	public void open(){
		bdd = favoriteOpenHelper.getWritableDatabase();
	}
 
	public void read(){
		bdd = favoriteOpenHelper.getReadableDatabase();
	}
 
	public void close(){
		bdd.close();
	}
 
	public SQLiteDatabase getBDD(){
		return bdd;
	}
 
	public long insertFavorite(Favorite favorite){
		ContentValues values = new ContentValues();
		values.put(COL_CITY, favorite.getCity());
		values.put(COL_COUNTRY, favorite.getCountry());
		return bdd.insert(TABLE_FAV, null, values);
	}
 
	public int updateFavorite(int id, Favorite favorite){
		ContentValues values = new ContentValues();
		values.put(COL_CITY, favorite.getCity());
		values.put(COL_COUNTRY, favorite.getCountry());
		return bdd.update(TABLE_FAV, values, COL_ID + " = "+id, null);
	}
 
	public int removeFavorite(int id){
		return bdd.delete(TABLE_FAV, COL_ID + " = "+id, null);
	}
 
 
	public Cursor queueAll(){
		String[] column = new String[]{COL_CITY, COL_COUNTRY};
		Cursor c = bdd.query(TABLE_FAV, column, null, null, null, null, null);
		return c;
	}
 
	public int deleteAll(){
		return bdd.delete(TABLE_FAV, null, null);
	}
 
	public Favorite getFavoriteWithCity(String city){
		Cursor c = bdd.query(TABLE_FAV, new String [] {COL_ID, COL_CITY, COL_COUNTRY}, COL_CITY + " LIKE \"" + city + "\"", null, null, null, null);
		return cursorToFavorite(c);
	}
 
	private Favorite cursorToFavorite(Cursor c){
		if (c.getCount() == 0)
			return null;
		c.moveToFirst();
		Favorite favorite = new Favorite();
		favorite.setId(c.getInt(NUM_COL_ID));
		favorite.setCity(c.getString(NUM_COL_CITY));
		favorite.setCountry(c.getString(NUM_COL_COUNTRY));
		c.close();
		return favorite;
	}
 
 
}

Puis dans mon activité principale, j'affecte des données a la bdd. Puis je récupère ces données avec un Cursor que je lie à la listView via un SimpleCursorAdapter :

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
28
29
30
31
32
 
         favorite_cities = (ListView) findViewById(R.id.favorites);
         FavoritesBDD bdd = new FavoritesBDD(this);
 
        /* Fill the bdd */
        bdd.open();
		bdd.deleteAll();
		bdd.insertFavorite(new Favorite("Paris", "France"));
		bdd.insertFavorite(new Favorite("Nice", "France"));
		bdd.insertFavorite(new Favorite("Bordeaux", "France"));
		bdd.insertFavorite(new Favorite("Bergerac", "France"));
		bdd.insertFavorite(new Favorite("New York", "United States"));
		bdd.insertFavorite(new Favorite("Dublin", "Ireland"));
		bdd.insertFavorite(new Favorite("Milan", "Italy"));
		bdd.insertFavorite(new Favorite("London", "United Kingdoms"));
 
        bdd.close();
 
        /* Read the bdd */
        bdd.read();
        Cursor c = bdd.queueAll();
        startManagingCursor(c);
 
        String[] from = new String[]{FavoriteOpenHelper.COL_CITY, FavoriteOpenHelper.COL_COUNTRY};
        int[] to = new int[]{R.id.city, R.id.country};
 
 
        /* Display the bdd */
        SimpleCursorAdapter mAdapter = new SimpleCursorAdapter(this.getBaseContext(), R.layout.item, c, from, to);
        favorite_cities.setAdapter(mAdapter);
 
        bdd.close();

Concernant ma listView elle est définie dans le main.xml :

Code :

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

1
2
3
4
5
6
7
8
9
10
<ListView
	    android:id="@+id/favorites"
	    android:layout_width="fill_parent"
	    android:layout_height="fill_parent"
	    android:layout_marginLeft="20dp"
	    android:layout_weight="1"
	    android:footerDividersEnabled="false"
	    android:headerDividersEnabled="true" >
 
	</ListView>

Et j'ai un layout pour chaque item de la liste :

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
28
29
30
31
32
33
34
35
36
37
38
39
40
<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:orientation="horizontal"
    android:layout_width="fill_parent"
    android:layout_height="wrap_content"
    >
 
 
    <ImageView
        android:id="@+id/img"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_gravity="center_vertical"
        android:padding="10px"
        android:scaleType="matrix" />
 
	<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
	    android:orientation="vertical"
	    android:layout_width="fill_parent"
	    android:layout_height="wrap_content"
	   	android:layout_gravity="center_vertical"
	   	android:paddingLeft="10px"
	    android:layout_weight="1"
	    >
 
	    <TextView android:id="@+id/city"
	         android:layout_width="fill_parent"
	         android:layout_height="fill_parent"
	         android:textSize="16px"
	         android:textStyle="bold"
	         />
 
	    <TextView android:id="@+id/country"
	         android:layout_width="fill_parent"
	         android:layout_height="fill_parent"
	         />
 
    </LinearLayout>
 
</LinearLayout>

Normalement, ca devrait fonctionner ! J'ai verifié avec sqlite3 que ma bdd existait bien et qu'elle contenait les données insérées. Pas d'erreur de compilation, mais quand je lance l'application, elle crashe (The application has stopped unexpectedly. Please try again).

La je bloque. Est-ce que quelqu'un pourrait m'aider ?



Merci
PS: Je précise que je débute sur le sdk android donc merci de votre indulgence !

[Simon MARQUIS]

Si tu veux qu'on t'aide, il faut d'abord que tu nous montre le message d'erreur généré. Il se trouve dans le LogCat (Window->Views->Logcat) copie l'erreur (elle est en rouge)

[mike3012]

Ah oui pardon désolé
Voila l'erreur que j'obtiens (merci pour l'info sur le logcat effectivement c'est plus parlant qu'un crash d'app !!! )

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
28
29
30
31
32
08-25 14:10:14.776: D/dalvikvm(2625): GC_FOR_ALLOC freed 174K, 4% free 6424K/6663K, paused 15ms
08-25 14:10:14.776: I/dalvikvm-heap(2625): Grow heap (frag case) to 6.823MB for 513740-byte allocation
08-25 14:10:14.836: D/dalvikvm(2625): GC_FOR_ALLOC freed 2K, 4% free 6923K/7175K, paused 10ms
08-25 14:10:14.886: D/dalvikvm(2625): GC_CONCURRENT freed 2K, 4% free 6931K/7175K, paused 1ms+1ms
08-25 14:10:14.896: D/AndroidRuntime(2625): Shutting down VM
08-25 14:10:14.896: W/dalvikvm(2625): threadid=1: thread exiting with uncaught exception (group=0xa000c708)
08-25 14:10:14.906: E/AndroidRuntime(2625): FATAL EXCEPTION: main
08-25 14:10:14.906: E/AndroidRuntime(2625): java.lang.RuntimeException: Unable to start activity ComponentInfo{com.mikecorp.weather.tab/com.mikecorp.weather.tab.MainActivity}: java.lang.IllegalArgumentException: column '_id' does not exist
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:1815)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:1831)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at android.app.ActivityThread.access$500(ActivityThread.java:122)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1024)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at android.os.Handler.dispatchMessage(Handler.java:99)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at android.os.Looper.loop(Looper.java:132)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at android.app.ActivityThread.main(ActivityThread.java:4123)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at java.lang.reflect.Method.invokeNative(Native Method)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at java.lang.reflect.Method.invoke(Method.java:491)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:841)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:599)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at dalvik.system.NativeStart.main(Native Method)
08-25 14:10:14.906: E/AndroidRuntime(2625): Caused by: java.lang.IllegalArgumentException: column '_id' does not exist
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at android.database.AbstractCursor.getColumnIndexOrThrow(AbstractCursor.java:299)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at android.widget.CursorAdapter.init(CursorAdapter.java:169)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at android.widget.CursorAdapter.<init>(CursorAdapter.java:117)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at android.widget.ResourceCursorAdapter.<init>(ResourceCursorAdapter.java:52)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at android.widget.SimpleCursorAdapter.<init>(SimpleCursorAdapter.java:78)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at com.mikecorp.weather.tab.MainActivity.onCreate(MainActivity.java:139)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at android.app.Activity.performCreate(Activity.java:4397)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1048)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:1779)
08-25 14:10:14.906: E/AndroidRuntime(2625): 	... 11 more
08-25 14:10:18.326: I/Process(2625): Sending signal. PID: 2625 SIG: 9

[Simon MARQUIS]

Je pense que ton problème vient du fait que tu as modifié la structure de ta base de donnée plusieurs fois.

Je te conseille de désinstaller complètement ton application de ton smartphone, puis de la réinstaller.

Dis nous si le problème persiste

[mike3012]

Je viens d'essayer. J'ai désinstallé complètement l'application (J'ai vérifié que la data avait bien été supprimée dans data/data/mon_app/. Puis, je l'ai réinstallé et j'ai toujours le même problème.

Pour info j'utilise un emulateur (x86 + VirtualBox) - API 3.2.2

Je viens d'essayer sur ma tablette (Asus eeePad - API 4.0.3). Même problème.

En tout cas, merci de ton aide. De mon côté, j'essaye de regarder s'il n'y a pas un problème de construction au niveau du curseur ou autre.

[mike3012]

Pour info, voici la classe "Favorite" sur laquelle se base la construction de ma classe FavoritesBDD. Si cela peut aider ...

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
28
29
30
31
32
33
34
35
36
37
38
39
40
41
public class Favorite {
 
	private int id;
	private String city;
	private String country;
 
	public Favorite(){}
 
	public Favorite(String city, String country){
		this.city = city;
		this.country = country;
	}
 
	public int getId(){
		return this.id;
	}
 
	public void setId(int id){
		this.id = id;
	}
 
	public String getCity(){
		return this.city;
	}
 
	public void setCity(String city){
		this.city = city;
	}
 
	public String getCountry(){
		return this.country;
	}
 
	public void setCountry(String country){
		this.country = country;
	}
 
	public String toString(){
		return "ID : "+this.id+"\nCity : "+this.city+"\nCountry : "+this.country;
	}
}

Merci pour vos aides.

[Hizin]

Ton erreur dit "la colonne _id n'existe pas". Donc, tu fais une requête qui utilise cette colonne (explicitement ou implicitement). Celle-ci se trouve dans ta classe MainActivity, ligne 139.

[Simon MARQUIS]

Qu'y a-t-il à la ligne 139 de la classe MainActivity ?

Code :

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

at com.mikecorp.weather.tab.MainActivity.onCreate(MainActivity.java:139)

[mike3012]

La ligne 139 correspond a cette ligne :

Code :

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

SimpleCursorAdapter mAdapter = new SimpleCursorAdapter(this, R.layout.item, c, from, to);

[mike3012]

Ton erreur dit "la colonne _id n'existe pas". Donc, tu fais une requête qui utilise cette colonne (explicitement ou implicitement). Celle-ci se trouve dans ta classe MainActivity, ligne 139.

Oui c'est bien ce que je vois. Mais comme indiqué dans le post précédent, mon "CusrsorAdapter" utilise le Cursor defini plus haut (c) qui lui appelle la methode queueAll() qui est une query :

Code :

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

1
2
3
4
5
public Cursor queueAll(){
		String[] column = new String[]{COL_CITY, COL_COUNTRY};
		Cursor c = bdd.query(TABLE_FAV, column, null, null, null, null, null);
		return c;
	}

Or les valeurs de COL_CITY et COL_COUNTRY sont définies comme suit :

Code :

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

1
2
private static final String COL_CITY = "City";
private static final String COL_COUNTRY = "Country";

Du coup, effectivement il doit y avoir une appel implicite mais je ne vois pas ou

[nicroman]

CursorAdapter utilise implicitement la colonne "_id" comme identifiant de row...
Voila pourquoi il est recommandé d'utiliser "_id" (et non "id").... directement comme nom de colonne....
Sinon, il est possible de faire un alias dans la query (custom query obligatoire): select ID as _Id ....



J'ai vu une autre chose dans ton code
Tu est sur de la fonction getFavoriteWithCity ??
Il me semble que d'une part elle est fausse (en SQL on marque les strings avec ' et non avec "), d'autre part sujette au SQL injection (il n'y a pas d'encodage du parametre city)....

Il vaut mieux passer city en tant que parametre et juste faire "LIKE ?" dans le sql.
Cette derniere remarque est aussi vraie pour tout le reste:
plutot que faire:
db.update(TABLE,values,"id="+Integer.toString(id),null);
il vaut mieux faire:
db.update(TABLE,value,"id=?",Integer.toString(id));

[mike3012]

@nicroman :

Merci pour tous ces précieux conseils. Je vais essayer cela tout de suite :
- Remplacer ma colonne ID par _id
- Securiser mes requêtes SQL

Concernant les strings dans les requêtes, il me semblait que SQL acceptait aussi les " mais je vais suivre ton conseil et remplacer par des '.

Je vous tiens au courant pour vous dire si j'y arrive (je comprends vite mais il faut m'expliquer longtemps !! )

[nicroman]

Concernant les strings dans les requêtes, il me semblait que SQL acceptait aussi les " mais je vais suivre ton conseil et remplacer par des '.

Tous les SQL non, certainement pas (Oracle ne supporte pas " )
Peut-être SQLite (il est tellement over-simplifié)... Dans tous les cas, un remplacement par un parametre (?) permet de virer les quotes qu'ils soient simples ou non

D'ailleurs je reviens sur cette fonction

TOTO_COL like 'abc' va renvoyer toutes les rows qui ont 'AbC', 'abC', ou 'ABC', mais pas 'AbcD' par exemple... (ou alors il manque un '%')

[mike3012]

Trop coool ca fonctionne !!!!

Alors, pour info j'ai du dans ma méthode queueAll

remplacer

Code :

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

String[] column = new String[]{COL_CITY, COL_COUNTRY};

Code :

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

String[] column = new String[]{COL_CITY + " AS _id", COL_COUNTRY + " AS _country"};

Ma query ne récupère pas les id de la table. J'ai donc considéré que city serait mon _id.

Enfin pour que ca matche, j'ai remplacé :

Code :

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

String[] from = new String[]{FavoriteOpenHelper.COL_CITY, FavoriteOpenHelper.COL_COUNTRY}

par

Code :

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

String[] from = new String[]{"_id", "_country"}

Maintenant, je m'attaque aux ?

Merci pour tout (je vais passer le post en résolu)

[nicroman]

ouais... enfin... rien n'empeche de les récupérer les ID
Finalement c'est la seule chose qui identifie une ligne... quid du click sur l'item ? c'est quand même bien sympa de l'avoir cet ID unique..

En fait le CursorAdapter fait un match entre un NOM de colonne, et une entrée dans les list-item:

Code :

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

1
2
3
 
        String[] from = new String[]{FavoriteOpenHelper.COL_CITY, FavoriteOpenHelper.COL_COUNTRY};
        int[] to = new int[]{R.id.city, R.id.country};

Et il rajoute une colonne pour l'identifiant de la ligne... ("_ID")...

Donc si ta colonne s'appelle de base "_ID" dans la table, tu peux faire un select * from ... et le CursorAdapter se débrouille...

La il suffit que queueAll renvoit au moins:
_ID, COL_CITY et COL_COUNTRY
donc:
select ID as _ID, COL_CITY, COL_COUNTRY
et c'est tout !

[mike3012]

Oui pas faux !!! Je viens de m'en rendre compte. Allez c'est reparti pour un tour.

Tant que j'y suis j'ai créé cela pour pouvoir interagir avec les éléments de la listView :

Code :

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

1
2
3
4
5
6
favorite_cities.setOnItemClickListener(new OnItemClickListener(){
 
        	@SuppressWarnings("unchecked")
        	public void onItemClick(AdapterView<?> a, View v, int position, long id) {
...
}

Comment est-ce que je peux récupérer les données d'un item cliqué de la listView ?

Merci (promis après j'arrête !!! ),

[mike3012]

Cette derniere remarque est aussi vraie pour tout le reste:
plutot que faire:
db.update(TABLE,values,"id="+Integer.toString(id),null);
il vaut mieux faire:
db.update(TABLE,value,"id=?",Integer.toString(id));

@nicroman

Pour info, j'ai essayé de faire cela pour la méthode update et ca ne passe pas.

Avec :

Code :

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

return bdd.update(TABLE_FAV, values, COL_ID + " = ?", Integer.toString(id));

J'obtiens :

The method update(String, ContentValues, String, String[]) in the type SQLiteDatabase is not applicable for the arguments (String, ContentValues, String,
String)

et avec

Code :

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

return bdd.update(TABLE_FAV, values, COL_ID + " = ?", null, Integer.toString(id));

J'obtiens :

The method update(String, ContentValues, String, String[]) in the type SQLiteDatabase is not applicable for the arguments (String, ContentValues, String, null,
String)

Par contre pas de problème pour la méthode query.

Merci encore,

[nicroman]

The method update(String, ContentValues, String, String[]) in the type SQLiteDatabase is not applicable for the arguments (String, ContentValues, String,
String)

Donc ?

update(TABLE,values,"_id=?",new String[] { Integer.toString[id] });

Je sais... il est bizarre quand même qu'ils n'aient pas utilisé l'ellipsis pour les paramètres... enfin bon...