Android ListView CheckBox状态错乱

listView中包含checkBox的时候,经常会发生其中的checkBox错乱的问题,大多时候的代码如下：

先看一下效果图：奇数行为选中状态，偶数行为非选中状态

 

具体代码：

布局文件：

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:layout_width="fill_parent"
    android:layout_height="fill_parent"
    android:orientation="horizontal" >
    
    <LinearLayout 
        android:id="@+id/layout"
        android:layout_width="fill_parent"
        android:layout_height="fill_parent">
        
        <TextView 
            android:id="@+id/textView"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"/>
    
        <CheckBox 
            android:id="@+id/checkBox"
            android:layout_width="wrap_content"
                android:layout_height="wrap_content"/>
        
    </LinearLayout>
    
</LinearLayout>

 

JAVA CODE:

package com.tony.ui.listview;

import java.util.ArrayList;
import java.util.List;

import com.tony.R;

import android.app.Activity;
import android.os.Bundle;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.BaseAdapter;
import android.widget.CheckBox;
import android.widget.CompoundButton;
import android.widget.CompoundButton.OnCheckedChangeListener;
import android.widget.LinearLayout;
import android.widget.ListView;
import android.widget.TextView;

public class ListViewCheckBox extends Activity{
    
    private ListView listView;
    private List<A> list;
    private Adapter1 adapter1;
    
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.listview_checkbox);
        initDate();
        listView = (ListView)findViewById(R.id.listView);
        adapter1 = new Adapter1();
        listView.setAdapter(adapter1);
    }
    
    
    /**
     * 模拟40个数据，奇数数据为选中状态，偶数数据为非选中状态
     */
    private void initDate(){
        list = new ArrayList<A>();
        A a;
        for(int i=0;i<40;i++){
            if(i%2==0){
                a = new A(i+"号位",A.TYPE_NOCHECKED);
                list.add(a);
            }else{
                a = new A(i+"号位",A.TYPE_CHECKED);
                list.add(a);
            }
        }
    }
    
    class Adapter1 extends BaseAdapter{

        @Override
        public int getCount() {
            return list.size();
        }

        @Override
        public Object getItem(int position) {
            return null;
        }

        @Override
        public long getItemId(int position) {
            return 0;
        }

        @Override
        public View getView(int position, View convertView, ViewGroup parent) {
            final int index = position;
            ViewHolder viewHolder;
            if(convertView == null){
                viewHolder = new ViewHolder();
                convertView = LayoutInflater.from(ListViewCheckBox.this).inflate(R.layout.listview_checkbox_item, null);
                viewHolder.layout = (LinearLayout)convertView.findViewById(R.id.layout);
                viewHolder.textView = (TextView)convertView.findViewById(R.id.textView);
                viewHolder.checkBox = (CheckBox)convertView.findViewById(R.id.checkBox);
                convertView.setTag(viewHolder);
            }else{
                viewHolder = (ViewHolder)convertView.getTag();
            }
            
            
            viewHolder.textView.setText(list.get(position).name);
            if(list.get(position).type == A.TYPE_CHECKED){
                viewHolder.checkBox.setChecked(true);
            }else{
                viewHolder.checkBox.setChecked(false);
            }
            
            /*点击checkBox所在行改变checkBox状态*/
            /*final ViewHolder vv = viewHolder;
            viewHolder.layout.setOnClickListener(new OnClickListener() {
                @Override
                public void onClick(View v) {
                    if(vv.checkBox.isChecked()){
                        vv.checkBox.setChecked(false);
                        list.get(index).type = TYPE_CHECKED;
                    }else{
                        vv.checkBox.setChecked(true);
                        list.get(index).type = TYPE_NOCHECKED;
                    }
                    
                }
            });*/
            
            viewHolder.checkBox.setOnCheckedChangeListener(new OnCheckedChangeListener() {
                @Override
                public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
                    if(isChecked){
                        list.get(index).type = A.TYPE_CHECKED;
                    }else{
                        list.get(index).type = A.TYPE_NOCHECKED;
                    }
                }
            });
            
            return convertView;
        }
    }
    
    class ViewHolder{
        LinearLayout layout;
        TextView textView;
        CheckBox checkBox;
    }
    
    class A {
        
        public static final int TYPE_CHECKED = 1;
        public static final int TYPE_NOCHECKED = 0;
        
        String name;
        int type;
        
        public A(String name,int type){
            this.name = name;
            this.type = type;
        }
    }
}

 

以上代码就是根据List集合中的对象的类型来设置checkBox是否为选中状态，当用户点击checkBox的时候，程序根据checkBox是否选中来将其状态保存至list集合对象中，相信很多人第一次做的时候会信心满满地认为这样的逻辑简直天衣无缝。但结果是,测试的时候还没有点击checkBox来改变其状态,只是简单地上下拉动listView的时候就会发现,好像事情没有想象的那么简单。

效果图如下：

只要视力不算太差的人便一眼可以看出，按照程序的逻辑怎么连续两个checkBox的状态会一样呢，恩，肯定是模拟器神经错乱了。

 

解决方案:

　　很多人给出的两种解决办法

　　1:上来就说是因为convertview对象共用的原因,不能用convetView,而是每次getView()的时候都new一个对象的view出来.这种办法大概是用屁股想出来的.

　　2:即然错乱,那我就自己再弄一个集合保存checkBox的状态,再错乱,弄死你.即然adapter里有一个list集合里保存checkBox的状态了,为什么还要自己再保存一次checkBox的状态呢,不是多此一举吗?

 

　　PS:提供这两种办法的人都没有解释到底是为什么错乱.下面来尝试分析一下:

　　1:首先分析下viewHolder.checkBox.setOnCheckedChangeListener(new OnCheckedChangeListener()...);

　　   这句话,就是给checkBox添加一个监听器,如果checkBox的状态改变了,那么系统就会自动回调里面的onCheckedChange()方法.

         而文中onCheckedChange()方法里写的是记录这次改变后checkBox状态的代码.

　　2:再接着分析下if(list.get(position).type == A.TYPE_CHECKED){...},这部分代码是根据list集合里的对象属性初始化view里checkBox是否应该是选择状态.

　　3:我上下滑动listView的时候,checkBox的状态就错乱了,根据第二点的分析,无论如何checkBox的状态都不会错乱,除非list集合里对象的属性已经被改变了,到底是什么地方改变了它?

　　4:文中只有一个地方写了改变list集合里checkBox对象属性的地方,那就是第一点里提到的OnCheckedChangeListener()方法。它被执行了?

　　　怎么回事，不可能吧,打个断点,跑一下便知上下滑动listView的时候确实停了下来.

　　5:这便是convertView的功能,因为不管listView里显示多少条数据,都只是共用那么几个对象,然后我们的代码每一次把得到的对象重新赋值而已。

　　　对了,正是在这赋值的时候出了问题,假设android系统给我们生成了10个共用view对象,第一个view对象在第一屏的时候需要显示成"未选择"状态,而到了第二屏的时候,却要显示成"选择"状态,但由于是共用的同一个对象,根据第一点得知当checkBox的状态改变的时候,会调用onCheckedChange()方法。

　　6:也许有人会怀疑就算它调用了onCheckedChanged()方法,那又如何?onCheckedChange()方法里的代码还是将当前是否为选中状态保存到了list集合里,当我再次显示时还是会根据第二点里提到的代码来正确地显示,是的,代码会根据当前index来改变list集体的属性.关键就在这里,这个index真的是对的吗?测试一下便知:

　　　　

　　　　在onCheckedChange()方法里打印一下index,　当快速向下滑动的时候,index的值如下:　

     

　　7:共用的对象有10(这里举例,并不是一定)个,当onCheckedChange()方法调用的时候,至少也是共用对象用光的时候,再从第一个共用对象用的时候才会打印,那时的index也应该是从10开始，为什么打印的结果里会出现0?

　　8:这是由于代码的顺序决定的，根据上面的代码可以看出,添加监听器的代码在初始化checkBox属性的代码之后,也就是说当初始化checkBox属性时,由于可能改变其状态,导致调用了onCheckedChange()方法,而这个监听器是在上一次初始化的时候添加的,那么当然其index就是上一次的positon值,而不是本次的,所以每次保存checkBox属性状态的时候，都把值赋到的list集合里其它对象上去了,而不是与本次index相关的对象上,这才是发生莫名其妙错乱的真正原因.

　　9:解决办法:由于是因为index错误造成的,那么只要保证index值与当前positon保持一至即可,只要把添加监听器的方法加到初始化view中checkBox状态的代码之前即可.这样即始由于初始化造成调用了onCheckedChange()方法，也因为其中index值是最新的,而依然不会错乱.

 

代码示例:

class Adapter1 extends BaseAdapter{

        @Override
        public int getCount() {
            return list.size();
        }

        @Override
        public Object getItem(int position) {
            return null;
        }

        @Override
        public long getItemId(int position) {
            return 0;
        }

        @Override
        public View getView(int position, View convertView, ViewGroup parent) {
            final int index = position;
            ViewHolder viewHolder;
            if(convertView == null){
                viewHolder = new ViewHolder();
                convertView = LayoutInflater.from(ListViewCheckBox.this).inflate(R.layout.listview_checkbox_item, null);
                viewHolder.layout = (LinearLayout)convertView.findViewById(R.id.layout);
                viewHolder.textView = (TextView)convertView.findViewById(R.id.textView);
                viewHolder.checkBox = (CheckBox)convertView.findViewById(R.id.checkBox);
                convertView.setTag(viewHolder);
            }else{
                viewHolder = (ViewHolder)convertView.getTag();
            }
            
            viewHolder.checkBox.setOnCheckedChangeListener(new OnCheckedChangeListener() {
                @Override
                public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
                    if(isChecked){
                        list.get(index).type = A.TYPE_CHECKED;
                    }else{
                        list.get(index).type = A.TYPE_NOCHECKED;
                    }
                }
            });
            
            viewHolder.textView.setText(list.get(position).name);
            if(list.get(position).type == A.TYPE_CHECKED){
                viewHolder.checkBox.setChecked(true);
            }else{
                viewHolder.checkBox.setChecked(false);
            }
            return convertView;
        }
    }

 

 

总结:

　　其实解决办法就一句话,"只要把添加监听器的方法加到初始化view中checkBox状态的代码之前即可. "