ListView et sélection/clic

Version imprimable

29/01/2011, 18h36
j0o0

ListView et sélection/clic

Bonjour à tous,

Je continue mon apprentissage de l'API android, voici ce que j'aimerais faire : lorsqu'un item d'une ListView est cliqué/sélectionné, je voudrais changer sa couleur de fond (pour indiquer quel item de la liste est sélectionné). L'item garderait cette nouvelle couleur jusqu'à ce qu'un autre item soit cliqué/sélectionné...

J'ai essayé plusieurs façons de faire, notamment en changeant le fond dans un AdapterView.OnItemClickListener, le problème est que lorsque l'adapteur recycle les vues (cf. le paramètre convertView de Adapter.getView), la vue avec le fond modifié réapparaît autre part dans la ListView.
J'ai également essayé d'utiliser l'attribut android:listSelector, mais l'item ne change de couleur que pendant la durée du clic...

Quelle est la solution à ce problème ?
Merci !
31/01/2011, 10h22
Feanorin

Bonjour,

Citation:

J'ai essayé plusieurs façons de faire, notamment en changeant le fond dans un AdapterView

Comment as tu fait ?

Car je pense que en passant par l' inflater de ta view, tu dois avoir moyen de pouvoir changer les couleurs de fond comme tu veux , ou tu peux aussi passer par la procédure

Code:

public View getView(int position, View contentView, ViewGroup arg2)

Avec le OnItemClickListener j'avais teste comme ca :
Code:

1 2 3 4 5 6 7 mListView.setOnItemClickListener(new ListView.OnItemClickListener() { @Override public void onItemClick(AdapterView<?> parent, View view, int position, long id) { view.setBackgroundColor(Color.RED); } });
Le fond passe bien en rouge lors d'un clic, le probleme est que lorsque je fais defiler la liste, un autre item apparait en rouge a cause du recyclage de la View par l'adapteur...

Essaye plutôt en créant toi même ton adapter ,

http://developer.android.com/referen...w.ViewGroup%29

Il te suffira pour cela de modifier ta vue obtenue en fonction de la position sélectionnée .

Un exemple

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
public class TonAdapter extends ArrayAdapter {
 
     @Override
     public View getView(int position, View convertView, ViewGroup parent) {
         View view = convertView;
 
            if (view == null)
            {            
                LayoutInflater li = (LayoutInflater) this.getContext().getSystemService(Context.LAYOUT_INFLATER_SERVICE);
                //le layout représentant la ligne dans le listView
                view = li.inflate(R.layout.list_view, null); 
            }         
 
           // maintenant tu peux travailler ta view qui correspond à une ligne 
          // si la ligne correspond à l'élément sélectionnée préalablement tu change son fond .
 
 
         return view;
 
     }
}

31/01/2011, 15h03
j0o0

Effectivement, c'est mieux en changeant le fond a partir de l'adapteur...Encore une fois merci Feanorin pour ton aide !

j'ai un problèem un peu différent,

Je voudrais changer la couleur de fond de ma listView mais à partir d'un bouton du menu que j'ai créer ... et je n'y arrive pas.

Voici mon 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
 
public class ManagementAlarm extends Activity implements AlarmeAdapterListener{
    /** Called when the activity is first created. */
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);
 
        ArrayList<ListAlarme> listP = ListAlarme.getAListOfAlarms();
        AlarmeAdapter adapter = new AlarmeAdapter(this, listP);
        adapter.addListener(this); 
        ListView list = (ListView)findViewById(R.id.listalarme);
        list.setAdapter(adapter);   
    }
 
    public void onClick(ListAlarme item, int position) {
 
    	String idalr = ListAlarme.getAListOfAlarms().get(position).idAlarme;
    	String typealr = ListAlarme.getAListOfAlarms().get(position).typeAlarme;
    	int genrealr = ListAlarme.getAListOfAlarms().get(position).genre;
 
      	Intent intenalr1 = new Intent();
    	intenalr1.putExtra("id", idalr); 
    	intenalr1.putExtra("type", typealr); 
    	intenalr1.putExtra("genre", genrealr); 
    	intenalr1.putExtra("Position", position);
    	intenalr1.setClass(this, DetailAlarm.class);
	    startActivity(intenalr1); 
   } 
 
    /**-----------------------------Barre de Menu----------------------------*/
 
    public boolean onCreateOptionsMenu(Menu menu) 
    {
       super.onCreateOptionsMenu(menu);
       MenuInflater inflater = new MenuInflater(this);
       inflater = getMenuInflater();
       inflater.inflate(R.menu.menu, menu);
       return true;
    }
     public boolean onOptionsItemSelected(MenuItem item) {
         switch (item.getItemId()) {
             case R.id.ack:Toast.makeText(this, "Acknowledge this alarm", Toast.LENGTH_LONG).show();
             //Mettre l'alarme choisi en gris
                                 break;
             case R.id.ackall:Toast.makeText(this, "Acknowledge all alarms", Toast.LENGTH_LONG).show();
             ((TextView)findViewById(R.id.alarme_id)).setBackgroundColor(Color.GRAY);
             ((TextView)findViewById(R.id.alarme_type)).setBackgroundColor(Color.GRAY);            
                                 break;
             case R.id.reset:Toast.makeText(this, "Reset this alarm", Toast.LENGTH_LONG).show();
                                 break;
             case R.id.resetall:Toast.makeText(this, "Reset all alarms", Toast.LENGTH_LONG).show();             		
             					 break;    
             case R.id.update:Toast.makeText(this, " alarms update", Toast.LENGTH_LONG).show();          
             					 break;             
         }
         return true;
     }
}

Je voudrais mettre cette action sur le bouton ackall de mon menu, mais les lignes de codes que j'ai mises fonctionnent uniquement sur la première ligne de ma listView, soit la ligne 0 passe en grise mais pas les autres, une idée de solution?

j'ai essayé avec :

Code:

1
2
 
((ListView)findViewById(R.id.listalarme)).setBackgroundColor(Color.GRAY);

Mais ça ne change pas du tout la couleur.

Merci

31/05/2011, 11h47
Ryu2000

En fait peut être que la ListView change de background, met t'as tes items par dessus.
Pour vérifier tu peux par exemple mettre qu'un item dans une ListView qui prend tout l'écran.
Ou mieux, les layouts de tes items fait les transparentes :

Code:

android:background="@android:color/transparent"

Peut être que là ça va changer de couleur.
31/05/2011, 11h55
Invité

Non ça n'a rien changé, enfin c'est toujours la 1ére ligne qui passe en gris
(PS on s'est pas déja vu sur un autre forum ^^ de nouveau merci pour ton aide)
31/05/2011, 12h17
Feanorin

Bonjour,

Citation:

J'ai essayé avec :

Code:

((ListView)findViewById(R.id.listalarme)).setBackgroundColor(Color.GRAY);

Mais ça ne change pas du tout la couleur.

Essaye de coller un invalidate sur ta liste après le changement de couleur.

Citation:

Je voudrais mettre cette action sur le bouton ackall de mon menu, mais les lignes de codes que j'ai mises fonctionnent uniquement sur la première ligne de ma listView, soit la ligne 0 passe en grise mais pas les autres, une idée de solution?

plutôt que de changer la couleur de ton texte directement dans le menu , change juste une variable Color , puis met à jour dans l'adapter de ta listview qui affichera tes views avec la couleur contenu dans la variable Color .
31/05/2011, 12h19
Invité
En fait je le vois si je met :
Code:

1 2 android:background="#000000"
mais il se trouver que la couleur de ma listView n'est pas déclaré dans mon xml mais dans une classe comme ceci :
Code:

1 2 3 4 5 6 7 8 if (mListA.get(position).genre == ListAlarme.grave) { layoutItem.setBackgroundColor(Color.RED); } else { layoutItem.setBackgroundColor(Color.YELLOW); }
C'est dans toute la ligne qui a la même couleur, et pas les item de la listView, et c'est toute la ligne que je veux repasser en gris lorsque je clique sur mon bouton. Une idée?

En mettant les item transparents, j'ai quand même le tour des items (le reste de la listView) qui est soit rouge soit jaune selon le cas.
31/05/2011, 13h48
Invité

@Feanorin

Comment je met l'invalidate?

et où je déclare ma variable Color, dans mon adapter, ou dans ma classe principale? vu que je déclare le background de ma listview(de départ) dans l'adapter il faut que je déclare ma variable à cette endroit la non?
31/05/2011, 14h26
Feanorin
Bonjour,
Déjà première question a quel endroit est ce code (onCreate , onResume , ... )?
Code:

1 2 3 4 5 6 7 if (mListA.get(position).genre == ListAlarme.grave) { layoutItem.setBackgroundColor(Color.RED); } else { layoutItem.setBackgroundColor(Color.YELLOW); }
Après ,

Citation:

Comment je met l'invalidate?

La fonction invalidate te permet de demander à un composant de se redessiner (mettre à jour).
Pour utiliser cette fonction rien de plus facile , il faut juste l'appeler après une modification comme cela :
Code:

1 2 3 4 5 // Tu modifies ton composant soit ta liste ListView liste = findViewById(R.id.listalarme); liste.setBackgroundColor(Color.GRAY); // tu lui demandes de se mettre à jour liste.invalidate();
A tester je ne sais pas si dans ton cas cela marche :roll:.
NB : lorsque on modifie un élément graphiques essaye de le mettre en menbre de la classe cela t'éviteras de le rechercher à chaque coup vie les ressources.

Citation:

et où je déclare ma variable Color, dans mon adapter, ou dans ma classe principale? vu que je déclare le background de ma listview(de départ) dans l'adapter il faut que je déclare ma variable à cette endroit la non?

Tu déclare ta variable dans ta classe principale. Maintenant si ton adapter est déclarer dans ton activity. Tu peux utiliser cette même variable, sinon il faut que tu crées une deuxième variable dans ton adapter que tu mets à jour depuis ton activity (ce qui est plus propre mais plus lourd au niveau du code ).
Je parle toujours de ton exemple ou tu as essayer de modifier la couleur des éléments de ta liste et non la liste elle-même.

Le bout de code en question se trouve dans une classe AlarmeAdapter qui extends BaseAdapter, et dans ma classe principale j'appelle cette classe :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 public void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.main); ArrayList<ListAlarme> listP = ListAlarme.getAListOfAlarms(); AlarmeAdapter adapter = new AlarmeAdapter(this, listP); adapter.addListener(this); ListView list = (ListView)findViewById(R.id.listalarme); list.setAdapter(adapter); }
J'ai mis le code suivant sur le bouton de mon menu :
Code:

1 2 3 4 ListView liste = (ListView)findViewById(R.id.listalarme); liste.setBackgroundColor(Color.GRAY); liste.invalidate();
Mais ça ne change pas les couleurs (que ce soit les items ou la liste)

Et pour répondre à ta derniére question, je veux changer la couleur soit des items, soit de la liste, ce qui est le plus simple à coder.

31/05/2011, 14h46
Feanorin

Bonjour,

J'ai testé vite fait chez moi et cela marche ! avec ou sans invalidate() pour le coup !

J'avais pas vu mais si le bout de code se trouve dans le GetView de ton adapter alors pour le coup cela devient normal que le fond de la liste ne se modifie pas car tu l'écrases à chaque coup via ce bout de code !:roll:.

Cela serait donc plutôt à cet endroit ou il faudrait que tu définisses la couleur de fond souhaité pour ta liste ?

Mais j'ai du mal à comprendre pourquoi tu veux modifier entièrement le fond de ta liste si tu associes un fond par item en fonction d'une condition ?
31/05/2011, 14h55
Invité

En fait j'ai une liste avec deux types d'alarmes, soit grave (afficher en rouge) soit juste des défauult (afficher en jaune).
J'ai donc cette liste au départ, lorsque je lance mon appli

J'ai un bouton (qui s'appelle ici ackall) qui permetter "d'acquitter toutes les alarmes", une fois les alarmes acquitter, pour montrer qu'elles le sont bien je veux qu'elles s'affichent en grises

Peut-être qu'il faut que je déclare ailleur le fond de ma liste alors? mais si je le fait pas dans l'adapter, ni dans le xml, je le fais où?

Citation:

J'ai un bouton (qui s'appelle ici ackall) qui permetter "d'acquitter toutes les alarmes", une fois les alarmes acquitter, pour montrer qu'elles le sont bien je veux qu'elles s'affichent en grises

Peut-être qu'il faut que je déclare ailleur le fond de ma liste alors? mais si je le fait pas dans l'adapter, ni dans le xml, je le fais où?

Non dans ce cas tu es bon , il faut juste que dans ton getView tu positionnes le background correspondant à ton item.

Du style

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 // create a new  for each item referenced by the Adapter
	        public View getView(int position, View convertView, ViewGroup parent) {
	         	View v;
                        ListeItem item = mListA.get(position)
                        if (item.genre == ListAlarme.grave)
                             // rouge
                             v.setBackgroundColor(Color.RED);
                        else if (item.genre == ListAlarme.default)
                             // rouge
                             v.setBackgroundColor(Color.YELLOW);
                        else if (item.genre == ListAlarme.ackked)
                             // rouge
                             v.setBackgroundColor(Color.GRAY);

C'est juste un exemple ;). On doit pouvoir faire mieux je pense .

31/05/2011, 15h14
Invité
Ok je vois ce que tu veux dire, mais comment tester que je clique sur mon bouton ack qui est un bouton du menu dans cette ligne :
Code:

1 2 3 4 else if (item.genre == ListAlarme.ackked) // rouge v.setBackgroundColor(Color.GRAY);
Parce que je ne test pas le "genre" de mon item, je veux tester si "j'appuie sur le bouton ack du menu"

Et là je séche un peu pour savoir comment coder ça :s

Citation:

J'ai un bouton (qui s'appelle ici ackall) qui permetter "d'acquitter toutes les alarmes", une fois les alarmes acquitter, pour montrer qu'elles le sont bien je veux qu'elles s'affichent en grises

Citation:

Parce que je ne test pas le "genre" de mon item, je veux tester si "j'appuie sur le bouton ack du menu"

Je suis allez trop vite , dans ce cas créer une variable 'state' qui elle te précisera si l'utilisateur à appuyer l'acquittement du menu .
Dans le getView :

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 finale static  int ACKKED = 1;
public finale static  int NO_ACKKED = 0;
private int state;
 
// dans le constructeur tu l'initialise à zéro
state = NO_ACKKED;
 
// Dans le menu 
state = ACKKED;
 
// create a new  for each item referenced by the Adapter
public View getView(int position, View convertView, ViewGroup parent) {
	         	View v;
                        if (state = ACKKED) {
                                v.setBackgroundColor(Color.GRAY);
                        } else {
                                ListeItem item = mListA.get(position)
                                if (item.genre == ListAlarme.grave)
                                    // rouge
                                   v.setBackgroundColor(Color.RED);
                               else if (item.genre == ListAlarme.default)
                                   // jaune
                                  v.setBackgroundColor(Color.YELLOW);
                        }

31/05/2011, 16h05
Invité
bon alors mauvaise nouvelles, aucune erreur dans le code mais ça ne marche, pour essayer de comprendre j'ai changer cette ligne :
Code:

1 2 if (ManagementAlarm.etat == ManagementAlarm.ACKKED )
en cette ligne :
Code:

1 2 if (ManagementAlarm.etat == ManagementAlarm.NO_ACKKED )
lorsque je lance mon appli, tous mes item ont un background gris, mais il ne se colore pas quand je clique sur mon bouton (ce que ça devrait faire en changeant cette ligne), c'est donc normal que dans l'autre sens ça ne marche pas non plus.

je ne pouvais pas juste mettre
Code:

1 2 if (etat == ACKKED)
Eclipse me mettait une erreur.
31/05/2011, 16h08
Feanorin

Rends public ta variable etat ;). et de revenir vers une condition du type

Code:

public int state;

Code:

if (etat == ACKKED)

Sinon tu peux toujours passer cette variable à ton adapter .
31/05/2011, 16h18
Invité
Je pense que le code est bon, cependant j'ai l'impression que le teste est effectué uniquement lors du démarrage de l'apli, et donc lorsque je clique sur le bouton, comme je n'effectue pas le test ... je ne peux pas changer les couleurs.

J'ai déclaré ma variable dans ma classe AlarmAdapter
Code:

1 2 public int etat
mais cela ne change pas grand chose...
31/05/2011, 16h34
Feanorin

Peux tu me poster tout ton code sur la partie liste , cela sera plus simple je pense ?

Merci.

Je déclare ma liste dans cette classe :

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
 
public class ListAlarme {
	public final static int defaut = 1;
	public final static int grave = 2;
 
	public String idAlarme;
	public String typeAlarme;
	public int genre;
 
	public ListAlarme(String aid, String atype, int aGenre) {
		idAlarme = aid;
		typeAlarme = atype;
		genre = aGenre;
	}
 
	public static ArrayList<ListAlarme> getAListOfAlarms() {
		ArrayList<ListAlarme> listAl = new ArrayList<ListAlarme>();
 
		listAl.add(new ListAlarme("alarme1 ", "type1 ", defaut));
		listAl.add(new ListAlarme("alarme2 ", "type2 ", grave));
		listAl.add(new ListAlarme("alarme3 ", "type3 ", grave));
		listAl.add(new ListAlarme("alarme4 ", "type4 ", defaut));
		listAl.add(new ListAlarme("alarme5 ", "type5 ", defaut));
		listAl.add(new ListAlarme("alarme6 ", "type6 ", grave));
		listAl.add(new ListAlarme("alarme7 ", "type7 ", defaut));
		listAl.add(new ListAlarme("alarme8 ", "type8 ", grave));
		listAl.add(new ListAlarme("alarme9 ", "type9 ", grave));
		listAl.add(new ListAlarme("alarme10", "type10", defaut));
		listAl.add(new ListAlarme("alarme11", "type11", grave));
		listAl.add(new ListAlarme("alarme12", "type12", grave));
		listAl.add(new ListAlarme("alarme13", "type13", defaut));
		listAl.add(new ListAlarme("alarme14", "type14", grave));
 
		return listAl;
	}
 
	public String toString() {
	     return "id : "+idAlarme+", type : "+typeAlarme+", genre : "+genre+".";
	}
}

Mon adapter est déclaré ici :

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
 
public class AlarmeAdapter extends BaseAdapter{
 
	private static List<ListAlarme> mListA;
	public int etat;
 
	private Context mContext;
	private LayoutInflater mInflater;
	public AlarmeAdapter(Context context, List<ListAlarme> aListA) {
		  mContext = context;
		  mListA = aListA;
		  mInflater = LayoutInflater.from(mContext);
		}
	public int getCount() {
		  return mListA.size();
		}
 
	public Object getItem(int position) {
		  return mListA.get(position);
		}
 
	public long getItemId(int position) {
		  return position;
		}
	public View getView(final int position, View convertView, ViewGroup parent) 
	{
		  LinearLayout layoutItem;
		  if (convertView == null) {
		    layoutItem = (LinearLayout) mInflater.inflate(R.layout.alarme_layout, parent, false);
		  } 
		  else 
		  {
		  	layoutItem = (LinearLayout) convertView;
		  }      
		  TextView alarmeID = (TextView)layoutItem.findViewById(R.id.alarme_id);
		  TextView alarmeType = (TextView)layoutItem.findViewById(R.id.alarme_type);		            
		  alarmeID.setText(mListA.get(position).idAlarme);
		  alarmeType.setText(mListA.get(position).typeAlarme);
 
		if (etat == ManagementAlarm.NO_ACKKED ) 
          {
        	  layoutItem.setBackgroundColor(Color.GRAY);
          } 
          else 
          {
        	  ListAlarme item = mListA.get(position);
              if (item.genre == ListAlarme.grave)
            	  layoutItem.setBackgroundColor(Color.RED);
             else if (item.genre == ListAlarme.defaut)
             {
            	 layoutItem.setBackgroundColor(Color.YELLOW);
             }
 
          }
			layoutItem.setTag(position);
			layoutItem.setOnClickListener(new OnClickListener()
			{
					@Override
					public void onClick(View v) 
					{
						Integer position = (Integer)v.getTag();
						sendListener(mListA.get(position), position);						
					}		        	
		     }); 
		        return layoutItem;
	}		    
    private ArrayList<AlarmeAdapterListener> myListListener = new ArrayList<AlarmeAdapterListener>();
    public void addListener(AlarmeAdapterListener aListener) 
    {
    	myListListener.add(aListener);
    }
    private void sendListener(ListAlarme item, int position) 
    {
    	for(int i = myListListener.size()-1; i >= 0; i--) 
    	{
    		myListListener.get(i).onClick(item, position);
    	}
    }
 
 
	public interface AlarmeAdapterListener 
	{
	  	public void onClick(ListAlarme item, int position);
 
	}
}

Et voici ma classe principal, avec le menu :

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
 
public class ManagementAlarm extends Activity implements AlarmeAdapterListener{
    /** Called when the activity is first created. */
    public final static  int ACKKED = 1;
    public final static  int NO_ACKKED = 0;
    public int etat;
    @Override
    public void onCreate(Bundle savedInstanceState) 
    {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);
 
        ArrayList<ListAlarme> listP = ListAlarme.getAListOfAlarms();
        AlarmeAdapter adapter = new AlarmeAdapter(this, listP);
        adapter.addListener(this); 
        ListView list = (ListView)findViewById(R.id.listalarme);
        list.setAdapter(adapter);   
        etat = NO_ACKKED;
 
    }
 
    public void onClick(ListAlarme item, int position) {
 
    	String idalr = ListAlarme.getAListOfAlarms().get(position).idAlarme;
    	String typealr = ListAlarme.getAListOfAlarms().get(position).typeAlarme;
    	int genrealr = ListAlarme.getAListOfAlarms().get(position).genre;
 
      	Intent intenalr1 = new Intent();
    	intenalr1.putExtra("id", idalr); 
    	intenalr1.putExtra("type", typealr); 
    	intenalr1.putExtra("genre", genrealr); 
    	intenalr1.putExtra("Position", position);
    	intenalr1.setClass(this, DetailAlarm.class);
	    startActivity(intenalr1); 
   } 
 
 
 
 
    /**-----------------------------Barre de Menu----------------------------*/
 
    public boolean onCreateOptionsMenu(Menu menu) 
    {
       super.onCreateOptionsMenu(menu);
       MenuInflater inflater = new MenuInflater(this);
       inflater = getMenuInflater();
       inflater.inflate(R.menu.menu, menu);
       return true;
    }
     public boolean onOptionsItemSelected(MenuItem item) {
         switch (item.getItemId()) {
             case R.id.ack:Toast.makeText(this, "select an alarm", Toast.LENGTH_LONG).show();
                                 break;
             case R.id.ackall:Toast.makeText(this, "Acknowledge all alarms", Toast.LENGTH_LONG).show(); 
             etat = ACKKED;         
                                 break;
             case R.id.reset:Toast.makeText(this, "Select an alarm", Toast.LENGTH_LONG).show();
                                 break;
             case R.id.resetall:Toast.makeText(this, "Reset all alarms", Toast.LENGTH_LONG).show();      				
             					 break;    
             case R.id.update:Toast.makeText(this, " alarms update", Toast.LENGTH_LONG).show();   
             					 break;             
         }
         return true;
     }

Encore merci pour ton aide =)

01/06/2011, 10h34
Invité

J'ai beau tenté plusieurs façon de déclarer la variable etat le problème reste le même, est-ce que je l'ai déclaré au mauvais endroit? je ne pense pas qu'il faille la déclarer dans les 2 classes comme je l'ai fait ci-dessus ....

Bonjour,

Citation:

J'ai beau tenté plusieurs façon de déclarer la variable etat le problème reste le même, est-ce que je l'ai déclaré au mauvais endroit? je ne pense pas qu'il faille la déclarer dans les 2 classes comme je l'ai fait ci-dessus ....

Effectivement

Donc enlève la variable etat de ton activity elle sert à rien après dans le constructeur de ton adapter .

Code:

1
2
3
4
5
6
public AlarmeAdapter(Context context, List<ListAlarme> aListA, int state) {
		  mContext = context;
		  mListA = aListA;
		  mInflater = LayoutInflater.from(mContext);
                 etat = state;
		}

Pour construire ton adapter :

Code:

AlarmeAdapter adapter = new AlarmeAdapter(this, listP,NO_ACKKED); // ou ACKKED

Pour mettre à jour ta variable etat :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
public boolean onOptionsItemSelected(MenuItem item) {
         switch (item.getItemId()) {
             case R.id.ack:Toast.makeText(this, "select an alarm", Toast.LENGTH_LONG).show();
                                 break;
             case R.id.ackall:Toast.makeText(this, "Acknowledge all alarms", Toast.LENGTH_LONG).show(); 
            ListView list = (ListView)findViewById(R.id.listalarme);
            AlarmeAdapter adapter = (AlarmeAdapter)list.getAdapter();
            adapter.setEtat(ACKKED);
            list.invalidate(); // ou tu fais un noitify sur ton adapter 
                                 break;
             case R.id.reset:Toast.makeText(this, "Select an alarm", Toast.LENGTH_LONG).show();
                                 break;
             case R.id.resetall:Toast.makeText(this, "Reset all alarms", Toast.LENGTH_LONG).show();      				
             					 break;    
             case R.id.update:Toast.makeText(this, " alarms update", Toast.LENGTH_LONG).show();   
             					 break;             
         }
         return true;
     }

Dans ton adapter :

Code:

1
2
3
public void setEtat (int nvetat) {
   etat = nvetat;
}

Après je pense que d'avoir ta liste dans l'adapter en static n'est pas forcément nécessaire.

01/06/2011, 11h16
Aerinder

Dans le code ci-dessus il n'y aucune relation entre la variable etat de la classe "AlarmeAdapter" et "ManagementAlarm".

Pourquoi ne pas laisser etat dans "AlarmeAdapter" ensuite faire un mutateur (setEtat).

Dans "ManagementAlarm" au niveau de "onOptionsItemSelected" il faudra faire mAlarmeAdpter.setEtat(nouvelle_valeur).

Ensuite certainement qu'il faut notifier le changement ou forcer le refresh de la liste pour que l'affichage change.

P.S : j'ai pas tout lu donc peut-être que la solution a déjà été proposé.

EDIT : Feanorin a parfaitement illustré ce que je disais

@Aerinder

ta solution a été proposé par feanorin.

cependant que je mette

Code:

list.invalidate();

Code:

adapter.notifyDataSetChanged();

cela ne fonctionne pas.

Est-ce que le test est bien codé dans l'adapter :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
		if (etat == ManagementAlarm.ACKKED ) 
          {
        	  layoutItem.setBackgroundColor(Color.GRAY);
          } 
          else 
          {
        	  ListAlarme item = mListA.get(position);
              if (item.genre == ListAlarme.grave)
            	  layoutItem.setBackgroundColor(Color.RED);
             else if (item.genre == ListAlarme.defaut)
             {
            	 layoutItem.setBackgroundColor(Color.YELLOW);
             }
 
          }

01/06/2011, 11h35
Invité

Citation:

Envoyé par Feanorin

Après je pense que d'avoir ta liste dans l'adapter en static n'est pas forcément nécessaire.

Effectivement, j'ai enlevé le static ;)

Je suppose qu'il faut quand même laissé la déclaration des int ACKKED et NO_ACKKED dans mon activity non?
01/06/2011, 11h42
Feanorin

Citation:

Je suppose qu'il faut quand même laissé la déclaration des int ACKKED et NO_ACKKED dans mon activity non?

Hmm à ta place je les déplacerais dans l'adapter. Il faut que les deux classes y ont accès puisque les deux l'utilisent , comme ton activity doit déjà surement importé la class de ton adapter alors la place de ces déclarations seront plus intelligentes dans ton adapter.

Après tu peux les laisser dans ton activity ou même les positionner dans un fichier Commun qui te permettra de centraliser toutes tes déclarations static.

Edit :
J'avais pas fais gaffe au premier post , essaye de mettre des Log pour voir si la valeur de ta variable etat ainsi que pour voir si tu rentres bien dans le bon cas lors de la sélection de ton background.

Lorsque je lance mon appli, je rentre bien dans le bon cas du test (cad NO_ACCKED) et ma liste et rouge et jaune.

cependant dés que je clique sur le bouton je ne rentre pas dans le test, je pense que tout le problème vient de là, comment faire un sorte que mon programme refasse ce test à chaque fois que je clique sur le bouton.

si ça peut t'aider lorsque jai cliqué sur le bouton mon logcat m'a affiché ça :
Code:

1 2 06-01 09:51:11.568: WARN/KeyCharacterMap(414): Using default keymap: /system/usr/keychars/qwerty.kcm.bin 06-01 09:51:13.458: WARN/InputManagerService(58): Window already focused, ignoring focus gain of: com.android.internal.view.IInputMethodClient$Stub$Proxy@44f76f30
je ne sais pas si c'est utile.

En tout cas si le test ne s'effectue pas, le problème ne vient pas de la déclaration des variables ...

01/06/2011, 11h57
Invité

Est-il possible de faire le test en dehors du getView? car cette méthode ne se lance qu'au lancement de l'appli, on en revient pas dessus je me trompe?
01/06/2011, 12h04
Feanorin
Essaye avec cela
Code:

1 2 list.invalidateViews();
http://developer.android.com/referen...ateViews%28%29
01/06/2011, 12h06
Invité

Merci ça marche avec invalidateViews() Génial =D

Citation:

Envoyé par Feanorin

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
public class TonAdapter extends ArrayAdapter {
 
     @Override
     public View getView(int position, View convertView, ViewGroup parent) {
         View view = convertView;
 
            if (view == null)
            {            
                LayoutInflater li = (LayoutInflater) this.getContext().getSystemService(Context.LAYOUT_INFLATER_SERVICE);
                //le layout représentant la ligne dans le listView
                view = li.inflate(R.layout.list_view, null); 
            }         
 
           // maintenant tu peux travailler ta view qui correspond à une ligne 
          // si la ligne correspond à l'élément sélectionnée préalablement tu change son fond .
 
 
         return view;
 
     }
}

J'ai ajouter ceci à ton code car j'ai une custom listView

Code:

1
2
3
4
5
 
            String SingleNameItem = getItem(position);
            TextView tV= (TextView) customView.findViewById(R.id.tV);
            tV.setText(SingleNameItem);
            // Condition ??

Et dans le listner

Code:

1
2
3
4
 
 public void onItemClick(AdapterView<?> parent, View view, int position, long id) {
                view.setBackgroundColor(Color.RED);
            }

Le problème c'est que j'arrive pas à trouver la condition adéquate afin d'éviter de retrouver en bas de la liste des items sélectionnés à cause du recyclage de la liste. Merci d'avance.