Perte de données avec une ListView

Version imprimable

Voir 40 message(s) de cette discussion en une page

08/08/2012, 08h51
italia57

Perte de données avec une ListView

Bonjour à tous,

je viens de rencontrer un phénomène étrange qui à mon avis ne le sera pas pour vous, mais je ne comprend pas.

J'ai une listeView qui comporte un certain nombre d'items (les infos des items je les récupères sur une BDD MySQL). Si tous les items entre sur l'écran sans que j'ai à scrollé pour voir les derniers aucun problème. Cependant si j'ai plus d'items et qu'il faut que je scroll pour aller voir les derniers, quand je remonte au haut de la listView j'ai perdu des informations concernant les items du haut (que je n'avais plus à l'écran une fois que j'étais en bas) alors qu'elles étaient bien présentes avant que je scroll.

Je vous remercie.

Italia57
08/08/2012, 09h07
Ryu2000
Est-ce que dans getView de ton Adapter tu ne ferais pas ce genre de chose :

Code:

view.setVisibility(View.GONE);

?

Dans le getView tu recycle une View.
Du coup il faut faire :
Code:

1 2 3 4 5 6 if(...){ view.setVisibility(View.GONE); } else{ view.setVisibility(View.VISIBLE); }
Si dans ton getView tu ne changes pas de visibilité, il faudra être plus précis sur ton problème.

Quel Adapter utilise tu ?
Avec quoi le créer tu ?
08/08/2012, 09h17
Hizin
Je mettrai plutôt le compte sur le lazy loading :p

Dans ton getView, tu n'aurais pas un truc comme :
Code:

1 2 3 4 5 6 SI (ma ligne n existe pas) création ligne allocation des textes et images FINSI RETOUR la cellule
?

Le code de ton getView pourrait être pratique dans tout les cas :)

Voici mon adapter:

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
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
 
package com.restomaniak.alpha;
 
import java.text.NumberFormat;
 
 
import org.json.JSONArray;
import org.json.JSONException;
import org.json.JSONObject;
 
import android.content.Context;
import android.graphics.Typeface;
import android.location.Location;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.BaseAdapter;
import android.widget.LinearLayout;
import android.widget.RatingBar;
import android.widget.TextView;
 
public class AdapterPersoSurPlace extends BaseAdapter {
 
	private JSONArray maListe = new JSONArray();
	private JSONArray maListe2 = new JSONArray();
	private Context mContext = null;
	private LayoutInflater mInflater = null;
 
	private Typeface robotoCond = null;
	private Typeface robotoBold = null;
 
	private float longFloat = 0;
	private float latFloat = 0;
 
	private float resultat =0;
	private String resultatString = new String();
 
	private Location locationRestaurant = new Location("Restaurant");
	private Location locationDetecter = new Location("Detecter");
 
 
	private int km = 0;
 
	private float moyenneCuisine = 0;
	private float moyenneService = 0;
	private float moyenneQP = 0;
	private float moyenneTotal = 0;
 
	private JSONObject json_data = new JSONObject();
	private JSONObject json_data2 = new JSONObject();
 
	private boolean used = true;
	private int indice = 0;
 
	private int id_category_resto = 0;
	private int id_category_note = 0;
 
	public AdapterPersoSurPlace (Context context, JSONArray jArray, JSONArray jArray2, Location detecte, Float lg, Float lt)
	{
 
		mContext = context;
		maListe = jArray;
		maListe2 = jArray2;
		mInflater = LayoutInflater.from(mContext);
		robotoCond = Typeface.createFromAsset(mContext.getAssets(), "Roboto_Condensed.ttf");
		robotoBold = Typeface.createFromAsset(mContext.getAssets(), "Roboto_Bold.ttf");
		locationDetecter = detecte;
		longFloat = lg;
		latFloat = lt;
 
	}
 
	public int getCount() {
 
			return maListe.length();
 
		// TODO Auto-generated method stub
 
	}
 
	public Object getItem(int position) {
		// TODO Auto-generated method stub
 
 
			try {
				return maListe.get(position);
			} catch (JSONException e) {
				// TODO Auto-generated catch block
				e.printStackTrace();
			}
			return position;
 
 
	}
 
	public long getItemId(int position) {
		// TODO Auto-generated method stub
		return (position);
	}
 
	public View getView(int position, View convertView, ViewGroup parent) {
		// TODO Auto-generated method stub
		LinearLayout layoutItem;
		  //(1) : Réutilisation des layouts
		  if (convertView == null) {
		  	//Initialisation de notre item à partir du  layout XML "personne_layout.xml"
		    layoutItem = (LinearLayout) mInflater.inflate(R.layout.affichageitembis, parent, false);
		  } else {
		  	layoutItem = (LinearLayout) convertView;
		  }
 
 
		  //(2) : Récupération des TextView de notre layout      
		  TextView tv_Nom = (TextView)layoutItem.findViewById(R.id.nom);
		  //TextView tv_Adresse = (TextView)layoutItem.findViewById(R.id.adresse);
		  TextView tv_PrixMoyen = (TextView)layoutItem.findViewById(R.id.prixMoyen);
		  TextView tv_NombreAvis = (TextView)layoutItem.findViewById(R.id.avis);
		  TextView tv_prixMoyenBDD = (TextView)layoutItem.findViewById(R.id.infoprixMoyen);
		  TextView tv_signeEuro = (TextView)layoutItem.findViewById(R.id.prixMoyeneuro);
		  TextView tv_Distance = (TextView)layoutItem.findViewById(R.id.distance);
		  TextView tv_Type = (TextView)layoutItem.findViewById(R.id.type);
		  RatingBar moyenneEtoile = (RatingBar)layoutItem.findViewById(R.id.etoileListeRestoMoyenne);
 
 
		  //(3) : Renseignement des valeurs  
 
 
 
		  try {
 
			  if(position == 0) {
				  used = true;
				  indice = 0;
			  }
 
			  json_data = maListe.getJSONObject(position);
 
 
			  tv_Nom.setText(json_data.getString("name"));
			  tv_Type.setText(json_data.getString("type"));
			  tv_PrixMoyen.setText(json_data.getString("average_price"));
 
			  id_category_resto = Integer.parseInt(json_data.getString("id_category"));
 
 
			  if (indice < maListe2.length() && used)
			  {
				  json_data2 = maListe2.getJSONObject(indice);
				  indice++;
				  id_category_note = Integer.parseInt(json_data2.getString("prout"));
				  used = false;
			  }
 
 
			 if (id_category_resto == id_category_note)
			  {
 
				  moyenneCuisine = Float.parseFloat(json_data2.getString("moyennecuisine"));
				  moyenneService = Float.parseFloat(json_data2.getString("moyenneservice"));
				  moyenneQP = Float.parseFloat(json_data2.getString("moyenneqp"));
 
				  moyenneTotal = (moyenneCuisine + moyenneService + moyenneQP)/3;
				  moyenneEtoile.setRating(moyenneTotal);
 
				  tv_NombreAvis.setText("("+json_data2.getString("nombreCommentaire")+" avis)");
 
 
				  used = true;
			  } else {
 
				  moyenneEtoile.setRating(0);
				  tv_NombreAvis.setText("(0 avis)");
			  }
 
 
 
			//longFloat = Float.parseFloat(json_data.getString("villedelivered"));
			//latFloat = Float.parseFloat(json_data.getString("cpdelivered"));
 
 
		} catch (JSONException e) {
			// TODO Auto-generated catch block
			e.printStackTrace();
		}
 
 
 
 
	  //longFloat = (float) (longFloat/Math.pow(10,6));
      //latFloat = (float) (latFloat/Math.pow(10,6));
 
      locationRestaurant.setLongitude(longFloat);
      locationRestaurant.setLatitude(latFloat);
 
      resultat = locationDetecter.distanceTo(locationRestaurant);
 
	  resultat = Math.round(resultat);
 
 
 
  		if (resultat >= 1000)
  		{
  			resultat = (float) (resultat/Math.pow(10,3));
  			km = 1;
  		}
 
  		NumberFormat Myformat = NumberFormat.getInstance();
		  	Myformat.setMinimumFractionDigits(1);       //Nb de Digit mini
		  	Myformat.setMaximumFractionDigits(1);       //Nb de Digit Maxi
		  	resultatString = Myformat.format(resultat);
 
  		if (km == 1)
  		{
 
  			resultatString = resultatString + " km";
 
  		}
  		else {
 
  			resultatString = resultatString + " m";
 
  		}
 
  		 tv_Distance.setText(resultatString); 
 
 
 
  		 tv_Nom.setTypeface(robotoBold);
  		 tv_NombreAvis.setTypeface(robotoBold);
  		 tv_PrixMoyen.setTypeface(robotoCond);
 
 
		tv_prixMoyenBDD.setTypeface(robotoCond);
		tv_signeEuro.setTypeface(robotoBold);
		tv_Type.setTypeface(robotoCond);
		tv_Distance.setTypeface(robotoBold);
 
 
 
 
  		//tv_Adresse.setTypeface(arial);
 
 
 
 
		  /*//(4) Changement de la couleur du fond de notre item
		  if (mListP.get(position).genre == Personne.MASCULIN) {
		    layoutItem.setBackgroundColor(Color.BLUE);
		  } else {
		  	layoutItem.setBackgroundColor(Color.MAGENTA);
		  }*/
 
		  //On retourne l'item créé.
 
		  return layoutItem;
	}
 
}

Je ne vois pas ou es-ce que ce problème peut se produire.

Les infos que je père sont moyenneTotal et nombreCommentaire

Je vous remercie

08/08/2012, 09h25
Ryu2000

Ton logcat n'indique aucun problème ?

Quels TextView ne sont pas remplit exactement ?

Edit :
J'avais mal lu, donc c'est une RatingBar moyenneEtoile et un TextView tv_NombreAvis.

Et qu'est-ce que ça affiche exactement ?
Faire voir un screenshot quand ça fonctionne et un autre quand ça ne fonctionne pas.
08/08/2012, 09h32
italia57
Code:

1 2 3 TextView tv_NombreAvis = (TextView)layoutItem.findViewById(R.id.avis); RatingBar moyenneEtoile = (RatingBar)layoutItem.findViewById(R.id.etoileListeRestoMoyenne);
Tout est correst mais quand je redescend en bas de la listView et puis remonte sur les items qui était plus à l'écran, je perd le nombre Avis et la note, les deux textview que j'ai mis au dessous.

Non rien dans le logcat. De plus c'est bizarre car parfois que j'enchaine les scroll haut et bas parfois les infos reviennent, parfois pas etc...

Outre la qualité du code, qui est perfectible, je te déconseilles fortement de faire un parsing JSON AU MOMENT du getView().

Pourquoi ne pas le faire à un autre moment, pour remplir une liste d'objet et ne les parcourir qu'une unique fois ?
À chaque fois qu'un getView est appelé, il y a un parcours JSON.

Sinon, c'est marrant, mais les deux informations que tu perds sont côte-a-côte, dans le même bloc conditionnel, à la fin dans un bloc try/catch.
Je pencherai donc pour un problème sur ta condition :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 if (id_category_resto == id_category_note) { float moyenneCuisine = Float.parseFloat(json_data2.getString("moyennecuisine")); float moyenneService = Float.parseFloat(json_data2.getString("moyenneservice")); float moyenneQP = Float.parseFloat(json_data2.getString("moyenneqp")); float moyenneTotal = (moyenneCuisine + moyenneService + moyenneQP) / 3; moyenneEtoile.setRating(moyenneTotal); tv_NombreAvis.setText("(" + json_data2.getString("nombreCommentaire") + " avis)"); used = true; } else { moyenneEtoile.setRating(0); tv_NombreAvis.setText("(0 avis)"); }
Remarque :
Indente bien ton code
soit cohérent dans le placement des accolades
mContext, resultat, resultatString, moyenneCuisine, moyenneService, moyenneOP, moyenneTotal et id_category_resto sont plutôt des variables locales, pas des attributs
nomme bien tes variables, et respecte la convention Java (pas de snake_case)
pense à l'internationalisation
km est utilisé comme un booléen, donc autant qu'il le devienne, non ?

08/08/2012, 09h45
italia57

Je ne comprend pas ta remarque à propos de JSON, qu'elle serait ta suggestion ?

Oui il y a certainement un problème sur le bloc mais c'est bizarre tous de même. Je vais regarder ça.

Pour ce qui est attribut ou variable locale, à part vouloir être puriste ou pas, y a t-il vraiment un changement ?

Pour l'indentation j'ai pour habitude d'indenter à chaque fois que j'ai fini un bout de code.

Je ne connais pas le terme snake_case je ne vois donc pas ce que tu veux dire.
08/08/2012, 09h54
Hizin

Citation:

Envoyé par italia57

Je ne comprend pas ta remarque à propos de JSON, qu'elle serait ta suggestion ?

Faire le parsing de TOUT ces JSON à la création, ou au lancement, ou une fois que tu les récupère ... pour les mapper sous forme d'objet. Tu as chargé tes données en mémoire, tu n'as pas besoin de les parcourir X fois (je pense qu'Android fait une mise en cache pour éviter les multi-parsing, mais ce n'est pas une raison).

Citation:

Envoyé par italia57

Oui il y a certainement un problème sur le bloc mais c'est bizarre tous de même. Je vais regarder ça.

Je t'aurai bien conseiller le debugger, mais vu la place (dans le getView), ça va être très chiant de le faire avec je pense. Quelques traces donc, à mon avis.

Citation:

Envoyé par italia57

Pour ce qui est attribut ou variable locale, à part vouloir être puriste ou pas, y a t-il vraiment un changement ?

Pour moi, c'est du même ordre "Je mets en global, ça fonctionne bien comme ça". Accessoirement, les variables en attributs gardent les valeurs jusqu'à ce qu'elles soit de nouveau affecté, à la différence des variables locales qui n'ont pas la "mémoire" de ce qui s'est passé avant.
Peut-être que tu es en train d'expérimenter un effet de bord de ce côté-ci justement.

Citation:

Envoyé par italia57

Pour l'indentation j'ai pour habitude d'indenter à chaque fois que j'ai fini un bout de code.

Comme tu veux, mais pense à ceux qui lisent ton code. Si tu peux faciliter la vie des personnes à qui tu demandes de l'aide, tu auras plus de chance d'avoir de l'aide (accessoirement, sous chaque IDE tu as un raccourci te permettant de faire l'indentation automatique selon le profil d'indentation que tu as réglé).

Citation:

Envoyé par italia57

Je ne connais pas le terme snake_case je ne vois donc pas ce que tu veux dire.

UpperCamelCase : MonNomDeVariable
lowerCamelCase : monNomDeVariable
snake_case : mon_nom_de_variable
08/08/2012, 10h03
italia57

Pour ce qui est des variables locales et attribut bien entendu, mais je pense pas que le problème viennent d'ici.

Après personnellement je trouvé pas que mon code était mal indenté, c'est une manière d'indenté le code comme une autre, il n'y a pas vraiment de convention là dedans.

Pour mon problème je ne sais pas d'où cela peut provenir du coup. Je vais regarder ça encore de plus près.

EDIT: une fois que j'arrive en bas de la liste et que je retourne vers le haut de la listView pourquoi il refait appel à getView alors que les items ont déjà été crée. Je ne comprend pas en quoi cela est utile.
08/08/2012, 10h09
Hizin

Ma remarque était surtout pour les accolades. Une fois à la fin de la ligne, une fois sur une ligne séparée. Je t'accorde que je suis chiant, là, par contre :)
08/08/2012, 10h11
italia57

:D

J'ai mis un Edit au dessus.
08/08/2012, 10h16
Hizin

getView est appelée à chaque fois qu'une cellule doit être créée.

Par exemple : ton écran peut afficher 5 items en même temps (l'utilisateur en voit 5), donc, getView chargera les 5 items visible, ainsi que les premiers non-visible (en dehors du champ).
Tu scrolls d'un écran entier : les 5 premiers items sont gardé en cache (si c'est bien fait), et tu as 5 nouveaux items d'affiché, et ainsi de suite.

Le getView charge les cellules qui sont directement visible (ou le seront bientôt). Il sera appelé à chaque fois qu'un nouvel item devra s'afficher.
08/08/2012, 10h20
italia57

Mais si il garde les infos en cache il devrait tout de même mes les redonnées comme je les aient quitté...

Ça m'échappe franchement. Car je ne vois d'ou peut bien provenir le bug. Il rentre dans le "else" alors qu'il faut pas, mais avec tous les appels de getView pour pouvoir débugger ça pas simple. Bizarre certains comportement dans le développement Android.
08/08/2012, 10h41
Hizin

Euh ... non.

Il va réinvoquer getView à chaque fois, donc dérouler le code de cette méthode à chaque fois. Il n'a pas de cache au niveau du contenu, juste au niveau du conteneur.

Exemple : Tu as 10 cellules, chacune contenant un TextView. Tu scrolls en bas, 10 TextView ont été créés. Tu scrolls tout en haut à présent, les 10 TextView n'ont pas changé. Tu scrolls en bas, toujours pareil.
PAR CONTRE, leur contenu à été mis à jour dans le getView à chaque appel de cette méthode.

Citation:

Envoyé par italia57

Il rentre dans le "else" alors qu'il faut pas

Donc, tu sais où il va. Il ne te reste qu'a savoir pourquoi maintenant ;)

La méthode getView est invoquée à chaque fois qu'une ligne/item/cellule de la ListView doit être affichée.

Ps : ce système de "cache" n'est pas fait par Android, mais par le développeur.
08/08/2012, 10h53
italia57

ah bon ? je m'en suis jamais préoccupé... Comment faire ça ? Je ne suis plus là, j'ai bien peur que se soit une erreur que je mette beaucoup de temps à trouver...
08/08/2012, 11h04
Hizin

Tu as tout ton code, tu devrais savoir ce qu'il se passe, tu peux le débugguer et tu sais ce qui ne va pas (passer dans le "else").
Tu as toutes les cartes en mains pour, au minimum, résoudre ton problème :)
08/08/2012, 11h05
italia57

Ok, je vais me concentrer la dessus alors... merci beaucoup

EDIT: Sujet clos. J'ai géré ce bug différement en traitant un petit peu les données dans le PHP avant de les encodé en JSON et de les récupérer en JAVA.
09/08/2012, 14h06
Substitut
Pour avoir eut la même erreur, la partie qui te fait planter c'est ton
Code:

1 2 3 4 if(convertView == nul){ ... }else{ ... }
En faite faire une vérification ici permet de ne pas recharger entièrement les élements de la liste donc tu gagnes en performances. Malheureusement des fois ca ne fonctionne pas très bien (je n'ai pas encore trouvé pourquoi ). Si tu remplace ta condition juste par le code present dans ton if, cela fonctionne normalement.
09/08/2012, 14h11
Hizin

Je n'ai jamais eu ce type de bug avec cette vérification personnellement (pas faute d'utiliser les ListView, de les mettre à jour dynamiquement ... pourtant).

Je pense que tu as trouvé un endroit de ton problème, mais pas la véritable cause (avis personnel, je ne sais pas du tout ce que tu as fait, ni comment, ni pourquoi, ni ... bref, impression ^^ ).

Voir 40 message(s) de cette discussion en une page