在Adapter中使用Holder的那些坑

在使用GridView、ListView时，通常会在Adapter中采用Holder缓存每一项以提高效率，但如果没有用好Holder，这个缓存机制会导致许多意想不到的问题，结合自己的经验特地总结一下，以免今后再犯。

内容错乱

  在Adapter的getView方法中通过position更新每一项的内容，对于根据判断条件给每一项设置属性的情况，每个判断条件下都需要给每一项的每个属性赋值，否则在滑动ListView或GridView时会导致内容错乱，比如下面这段代码是在getView中调用的一个方法，如果在使用Holder时，只在if(position == mSelectPosition)分支下给项背景设置属性，在滑动过程中会导致本来不应该是蓝色背景的项背景变成蓝色了：

[java]  view plain copy

1. private void setFileInfo( FileInfo fileInfo, int position ){  
2.     mFileHolder.mFilePathTxtView.setText( fileInfo.getFilePath( ) );  
3.     mFileHolder.mVoiceTypeTxtView.setText( fileInfo.getVoiceType( ) );  
4.     mFileHolder.mFileItemLayout.setTag(position);  
5.   
6.     if(position == mSelectPosition){  
7.         mFileHolder.mFileItemLayout.setBackgroundColor( Color.BLUE );  
8.     }else{  
9.         // 每一项的else分值也需要为项设置背景，否则会在滑动过程中导致内容错乱  
10.         mFileHolder.mFileItemLayout.setBackgroundColor( Color.RED );  
11.     }  
12. }  

  这个问题虽然很容易处理，但在开发的过程中经常遇到，比如根据判断条件为每一项设置不同的背景、头像或者文本时，稍有不注意或者偷懒都会导致意想不到的bug，特别是对于每一项需要同时修改多个属性的时候就要更加注意了，一定要保证每个条件分值下都为项的每个视图都设置属性了。

setTag

  在上面那段代码中增加监听项点击事件，由于用到了缓存，所以不能通过getId来获取具体点击了哪一项，通常我们都是通过给每一项的layout设置tag，在监听回调中getTag来获取具体点击了哪一项，比如下面这样：

[java]  view plain copy

1. private void setFileInfo( FileInfo fileInfo, int position ){  
2.     mFileHolder.mFilePathTxtView.setText( fileInfo.getFilePath( ) );  
3.     mFileHolder.mVoiceTypeTxtView.setText( fileInfo.getVoiceType( ) );  
4.   
5.     // 为每一项setTag，在onClick方法中通过v.getTag()方法获取点击了哪一项  
6.     mFileHolder.mFileItemLayout.setTag(position);  
7.     mFileHolder.mFileItemLayout.setOnClickListener(new OnClickListener( ) {  
8.         @Override  
9.         public void onClick(View v) {  
10.             int position = ( Integer )v.getTag( );  
11.             mSelectPosition = position;  
12.             if( null != mOnFileListOnClickListener ){  
13.                 mOnFileListOnClickListener.onItemClick(mFileInfoList.get(position));  
14.             }  
15.   
16.             notifyDataSetChanged( );  
17.         }  
18.     });  
19.   
20.     if(position == mSelectPosition){  
21.         mFileHolder.mFileItemLayout.setBackgroundColor( Color.BLUE );  
22.     }else{  
23.         mFileHolder.mFileItemLayout.setBackgroundColor( Color.RED );  
24.     }  
25. }  

  如果按上述这种方式编码，在程序运行后点击某一项你会发现程序会挂掉，会报下面的异常：

[java]  view plain copy

1. 02-01 15:57:49.340: E/AndroidRuntime(1949): java.lang.ClassCastException: java.lang.Integer cannot be cast to com.uperone.view.FileListAdapter$FileHolder  
2. 02-01 15:57:49.340: E/AndroidRuntime(1949): at com.uperone.view.FileListAdapter.getView(FileListAdapter.java:72)  

  具体原因也是因为Holder缓存导致的，因为在getView方法中，我是通过为convertView设置tag来做到Holder重用的：

[java]  view plain copy

1. @Override  
2. public View getView(int position, View convertView, ViewGroup parent) {  
3.     if( null == convertView ){  
4.         convertView = mLayoutInflater.inflate( R.layout.list_file_item_layout, null);  
5.   
6.         mFileHolder = new FileHolder();  
7.         mFileHolder.mFileItemLayout = ( LinearLayout )convertView.findViewById(R.id.fileItemLayoutId);  
8.         mFileHolder.mFilePathTxtView = ( TextView )convertView.findViewById(R.id.filePathTxtId);  
9.         mFileHolder.mVoiceTypeTxtView = ( TextView )convertView.findViewById(R.id.voiceTypeTxtId);  
10.         mFileHolder.mFileItemLayout.setClickable(true);  
11.         convertView.setTag( mFileHolder );  
12.     }else{  
13.         mFileHolder = ( FileHolder )convertView.getTag( );  
14.     }  
15.   
16.   
17.     if( !isDataEmpty() ){  
18.         setFileInfo( mFileInfoList.get( position ), position );  
19.     }  
20.   
21.     return convertView;  
22. }  

  在点击ListView中的某一项时，由于设置每一项内容的setFileInfo方法是在convertView.getTag()后调用的，所以在系统调用getView()方法更新ListView时，convertView取到的tag是mHolder.mFileItemLayout设置的tag，使用了Holder，就不能在每项中有多个视图设置tag和取tag。

  对于上述的异常，通常的解决方式是在ListView实例化的地方，通过setOnItemClickListener方法监听每一项的点击，避免tag冲突的问题。