listView checkbox错乱

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

package com.haier.frozenmallselling.adapter;

import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;

import android.content.Context;
import android.graphics.drawable.Drawable;
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.TextView;

import com.haier.frozenmallselling.R;
import com.haier.frozenmallselling.vo.RegionInfo;

public class ListChooseSalesAreaAdapter extends BaseAdapter {
    public List<RegionInfo> list = new ArrayList<RegionInfo>();
    private LayoutInflater myInflater;
    private int width = 0;
    private Context mContext;
//    public Map<Integer, RegionInfo> isCheckMap = new HashMap<Integer, RegionInfo>();;
//    public List<RegionInfo> isCheck = new ArrayList<RegionInfo>();
    public ListChooseSalesAreaAdapter(List<RegionInfo> liste,
            LayoutInflater mInflater, int mwidth, Context context) {
        super();
        this.list = liste;
        this.myInflater = mInflater;
        this.width = mwidth;
        this.mContext = context;
            
    }

    @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(final int position, View convertView, ViewGroup parent) {
        View row = convertView;

        ViewHolder viewHolder = null;
        final RegionInfo groupInfo = list.get(position);
        final int index = position;
        if (row == null) {
            row = myInflater.inflate(R.layout.choose_salesarea_item, null);
            viewHolder = new ViewHolder();
            viewHolder.item_name = (TextView) row.findViewById(R.id.item_name);
            viewHolder.item_check = (CheckBox) row
                    .findViewById(R.id.item_check);
            viewHolder.item_line = (View) row.findViewById(R.id.item_line);
            viewHolder.item_line2= (View) row.findViewById(R.id.item_line2);
            if (position == (list.size() - 1)) {
                viewHolder.item_line.setVisibility(View.GONE);
                viewHolder.item_line2.setVisibility(View.VISIBLE);
            }else {
                viewHolder.item_line2.setVisibility(View.GONE);
            }
            
            row.setTag(viewHolder);
        } else {
            viewHolder = (ViewHolder) row.getTag();
        }
        
        viewHolder.item_name.setText(list.get(position).group_name);
        viewHolder.item_check.setTag(position);
      
        if(list.get(position).type .equals("1")){
             viewHolder.item_check.setChecked(true);
        }else{
             viewHolder.item_check.setChecked(false);
        }
        
        Drawable rechoose;
        rechoose = mContext.getResources().getDrawable(R.drawable.check_rechoose);
        rechoose.setBounds(0, 0, rechoose.getMinimumWidth(), rechoose.getMinimumHeight());
        if("1".equals(list.get(position).rechoose)){
            viewHolder.item_check.setEnabled(false);
            viewHolder.item_check.setCompoundDrawables (rechoose,null,null,null);
        }else{
            viewHolder.item_check.setEnabled(true);
        }
         viewHolder.item_check
        .setOnCheckedChangeListener(new OnCheckedChangeListener() {
            @Override
            public void onCheckedChanged(CompoundButton buttonView,
                    boolean isChecked) {
                if (isChecked) {
                    list.get(index).type = "1";
                } else {
                    list.get(index).type = "0";
                }
            }
        });


        return row;
    }
    

    class ViewHolder {
        TextView item_name;
        CheckBox item_check;
        View item_line;
        View item_line2;

    }

}

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

解决方案:

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

　　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真的是对的吗?测试一下便知:

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

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

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

package com.haier.frozenmallselling.adapter;

import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;

import android.content.Context;
import android.graphics.drawable.Drawable;
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.TextView;

import com.haier.frozenmallselling.R;
import com.haier.frozenmallselling.vo.RegionInfo;

public class ListChooseSalesAreaAdapter extends BaseAdapter {
    public List<RegionInfo> list = new ArrayList<RegionInfo>();
    private LayoutInflater myInflater;
    private int width = 0;
    private Context mContext;
//    public Map<Integer, RegionInfo> isCheckMap = new HashMap<Integer, RegionInfo>();;
//    public List<RegionInfo> isCheck = new ArrayList<RegionInfo>();
    public ListChooseSalesAreaAdapter(List<RegionInfo> liste,
            LayoutInflater mInflater, int mwidth, Context context) {
        super();
        this.list = liste;
        this.myInflater = mInflater;
        this.width = mwidth;
        this.mContext = context;
            
    }

    @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(final int position, View convertView, ViewGroup parent) {
        View row = convertView;

        ViewHolder viewHolder = null;
        final RegionInfo groupInfo = list.get(position);
        final int index = position;
        if (row == null) {
            row = myInflater.inflate(R.layout.choose_salesarea_item, null);
            viewHolder = new ViewHolder();
            viewHolder.item_name = (TextView) row.findViewById(R.id.item_name);
            viewHolder.item_check = (CheckBox) row
                    .findViewById(R.id.item_check);
            viewHolder.item_line = (View) row.findViewById(R.id.item_line);
            viewHolder.item_line2= (View) row.findViewById(R.id.item_line2);
            if (position == (list.size() - 1)) {
                viewHolder.item_line.setVisibility(View.GONE);
                viewHolder.item_line2.setVisibility(View.VISIBLE);
            }else {
                viewHolder.item_line2.setVisibility(View.GONE);
            }
            
            row.setTag(viewHolder);
        } else {
            viewHolder = (ViewHolder) row.getTag();
        }
        
        viewHolder.item_name.setText(list.get(position).group_name);
        viewHolder.item_check.setTag(position);
           viewHolder.item_check
        .setOnCheckedChangeListener(new OnCheckedChangeListener() {
            @Override
            public void onCheckedChanged(CompoundButton buttonView,
                    boolean isChecked) {
                if (isChecked) {
                    list.get(index).type = "1";
                } else {
                    list.get(index).type = "0";
                }
            }
        });
        if(list.get(position).type .equals("1")){
             viewHolder.item_check.setChecked(true);
        }else{
             viewHolder.item_check.setChecked(false);
        }
        
        Drawable rechoose;
        rechoose = mContext.getResources().getDrawable(R.drawable.check_rechoose);
        rechoose.setBounds(0, 0, rechoose.getMinimumWidth(), rechoose.getMinimumHeight());
        if("1".equals(list.get(position).rechoose)){
            viewHolder.item_check.setEnabled(false);
            viewHolder.item_check.setCompoundDrawables (rechoose,null,null,null);
        }else{
            viewHolder.item_check.setEnabled(true);
        }
    
        return row;
    }
    

    class ViewHolder {
        TextView item_name;
        CheckBox item_check;
        View item_line;
        View item_line2;

    }

}

总结:

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