在Adapter中使用Holder的那些坑

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

内容错乱

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

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

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

setTag

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

private void setFileInfo( FileInfo fileInfo, int position ){
    mFileHolder.mFilePathTxtView.setText( fileInfo.getFilePath( ) );
    mFileHolder.mVoiceTypeTxtView.setText( fileInfo.getVoiceType( ) );

    // 为每一项setTag,在onClick方法中通过v.getTag()方法获取点击了哪一项
    mFileHolder.mFileItemLayout.setTag(position);
    mFileHolder.mFileItemLayout.setOnClickListener(new OnClickListener( ) {
        @Override
        public void onClick(View v) {
            int position = ( Integer )v.getTag( );
            mSelectPosition = position;
            if( null != mOnFileListOnClickListener ){
                mOnFileListOnClickListener.onItemClick(mFileInfoList.get(position));
            }
            
            notifyDataSetChanged( );
        }
    });
    
    if(position == mSelectPosition){
        mFileHolder.mFileItemLayout.setBackgroundColor( Color.BLUE );
    }else{
        mFileHolder.mFileItemLayout.setBackgroundColor( Color.RED );
    }
}

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

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

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

@Override
public View getView(int position, View convertView, ViewGroup parent) {
    if( null == convertView ){
        convertView = mLayoutInflater.inflate( R.layout.list_file_item_layout, null);
        
        mFileHolder = new FileHolder();
        mFileHolder.mFileItemLayout = ( LinearLayout )convertView.findViewById(R.id.fileItemLayoutId);
        mFileHolder.mFilePathTxtView = ( TextView )convertView.findViewById(R.id.filePathTxtId);
        mFileHolder.mVoiceTypeTxtView = ( TextView )convertView.findViewById(R.id.voiceTypeTxtId);
        mFileHolder.mFileItemLayout.setClickable(true);
        convertView.setTag( mFileHolder );
    }else{
        mFileHolder = ( FileHolder )convertView.getTag( );
    }
    
    
    if( !isDataEmpty() ){
        setFileInfo( mFileInfoList.get( position ), position );
    }
    
    return convertView;
}

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

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

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 204,732评论 6 478
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 87,496评论 2 381
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 151,264评论 0 338
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 54,807评论 1 277
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 63,806评论 5 368
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 48,675评论 1 281
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,029评论 3 399
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,683评论 0 258
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 41,704评论 1 299
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,666评论 2 321
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 37,773评论 1 332
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,413评论 4 321
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 39,016评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 29,978评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,204评论 1 260
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 45,083评论 2 350
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,503评论 2 343

推荐阅读更多精彩内容