Discussion :

[italia57]

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

[Ryu2000]

Est-ce que dans getView de ton Adapter tu ne ferais pas ce genre de chose :

Code :

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

view.setVisibility(View.GONE);

?

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

Code :

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

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 ?

[Hizin]

Je mettrai plutôt le compte sur le lazy loading :p

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

Code :

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

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

[italia57]

Voici mon adapter:

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
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

[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.

[italia57]

Code :

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

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...

[Hizin]

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 Java :

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
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 ?

[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.

[Hizin]

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).

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.

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.

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é).

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