Discussion :

[ulquiorra]

Bonjour

J ' essaie d ' appliquer le pattern ViewHolder pour ma listview , mais je n arrive pas a le passer a mes fonctions pour que celles ci puissent l ' utiliser .

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
public View getView(int position, View convertView, ViewGroup parent) {
 
		final ViewHolder holder ;
 
 
		   if (convertView == null) {
               convertView = inflater.inflate(R.layout.mediaplayer, null);
 
               holder = new ViewHolder();
               	tab=new TabVoiceMail();
 
              holder.playButton=(ImageButton)convertView.findViewById(R.id.button_play);
              holder.barre=(ProgressBar)convertView.findViewById(R.id.progress_bar);
              holder.fondEcran=(AbsoluteLayout)convertView.findViewById(R.id.affichage);
 
 
 
              	//updateTime();
 
              	holder.playButton.setOnClickListener(new OnClickListener() {
 
              			public void onClick(View view) {
 
 
 
 
              				play();
              				holder.fondEcran.setBackgroundColor(Color.BLUE);
 
              				calculProgressBar();
              				etatLecteur();
 
 
 
 
        		}});
              //tab.play();
 
 
 
 
               convertView.setTag(holder);
           } else {
 
               holder = (ViewHolder) convertView.getTag();
           }
 
 
 
		   holder.playButton.setTag(position);
           return convertView;
       }

la j ' ai un null pointer levé quant j ' appelle ma fonction etatlecteur() alors que j ' ai bien appelé le view holder dedans

ma fonction est composée ainsi

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
	public void etatLecteur()
	{
		final ViewHolder holder ;
		holder=new ViewHolder();
		  if (mMediaPlayer.isPlaying()) {
 
				mMediaPlayer.pause();
 
				holder.playButton.setImageResource(R.drawable.button_play);
			} else {
				mMediaPlayer.start();
				holder.playButton.setImageResource(R.drawable.button_pause);
			}
	}

null pointer sur le holder.playButton comme si il reconnaissait pas mon viewholder

je precise que ca marche si je fout directement le code dans mon onclick sans faire de fonctions mais ca ferait un gros foutoir apres vu les fonctions qui necessitent le viewholder

merci de votre aide

[MrDuChnok]

Salut,

Premièrement si tu veux qu'on puisse rapidement t'aider, il serait bon de bien indenter ton code avant de le publier ici. Ca facilite grandement la lecture / debogage.

sinon ton nullpointerexception provient bien de ta méthode etatLecteur() :
Tu instancie un nouvel objet ViewHolder, et juste derrière tu cherches à utiliser le membre "playButton". Je ne vois pas dans ton code à quel moment ce bouton est initialisé.
As-tu compris l'intérêt du mécanisme de ViewHolder ?
Sans tester, je te suggererais de faire quelques choses comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public void etatLecteur(View aView)
{
  ViewHolder holder = null;
  holder=(ViewHolder)aView.getTag();
  if (mMediaPlayer.isPlaying()) {
	mMediaPlayer.pause();
	holder.playButton.setImageResource(R.drawable.button_play);
  } else {
	mMediaPlayer.start();
	holder.playButton.setImageResource(R.drawable.button_pause);
  }
}

et lorsque tu appelles ta méthode, de modifier en conséquence :

Code :

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

1
2
3
4
5
6
7
8
9
holder.playButton.setOnClickListener(new OnClickListener() {
   public void onClick(View view) {
      play();
      holder.fondEcran.setBackgroundColor(Color.BLUE);
      calculProgressBar();
      etatLecteur(view);
   }
});

(tu peux même carrément passer ta ViewHolder en paramètre)

[ulquiorra]

Salut
Merci de ta réponse et de ton aide

Désolé pour l ' indentation , je change tellement mon code que j ' oublie d ' indenter systematiquement apres.
Ca ne se reproduira plus

apres test et corrections , toujours un nullpointer a cette ligne

Code :

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

holder.playButton.setImageResource(R.drawable.button_pause);

un viewholder consiste a garder une reference des elements declares .
J ' ai raison ?

D' autant que je ne comprends pas pourquoi eclipse me signale justement ce bouton comme jamais utilise(never read) alors que le viewholder l' utilse

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
public static class AdapterTest extends BaseAdapter implements FilenameFilter{
	private LayoutInflater inflater;
	public MediaPlayer mMediaPlayer;
	   private static final String TAG = "Test";
	   private ImageButton playButton;
	   private int currentPosition = 0;
           private ProgressBar barre;
	   private String path;
 
 
 
           public AdapterTest(Context context)
	 {
		 inflater=LayoutInflater.from(context);
	  }

Merci de ton aide

[MrDuChnok]

Salut,

Alors concernant le warning sur Eclipse, c'est normal, tu utilises la variable "playButton" de ta classe "ViewHolder", mais pas celle de ta classe "AdapterTest".

Concernant ton nullpointerexception, il faudrait que tu utilises le debuguer pour voir si tes holders sont bien stockés dans chaque vue.

[ulquiorra]

merci apres quelque modif, il me sort maintenant un
classCastException sur cette ligne

Code :

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

 holder=(ViewHolder)aView.getTag();

comme si il ne reconnaissait pas le cast

Merci de ton aide

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
public void etatLecteur(View aView)
	{
	  ViewHolder holder = null;
	  holder=(ViewHolder)aView.getTag();
	  if (mMediaPlayer.isPlaying()) {
		mMediaPlayer.pause();
		holder.playButton.setImageResource(R.drawable.button_play);
	  } else {
		mMediaPlayer.start();
 
		holder.playButton.setImageResource(R.drawable.button_pause);
	  }
	}

[MrDuChnok]

passe directement le holder en paramètre, t'auras peut être moins de soucis, et ça sera peut être plus clair

[ulquiorra]

merci ca marche impec