ListView的CheckBox状态的保存及项目中会出现的奇怪情况

最近小组成员，在使用listview中出现了奇怪的现象，先描述一下这个奇怪的现象：首先他给checkbox设定了OnCheckedChangeListener监听器，当点击的时候，就修改该条item，对应的bean中保存的checkbox状态值，并且将该item添加到选中的集合中，反之就去除集合中的该元素，当convertView复用的时候，他又重新设定当前的状态。可是其奇怪的现象出现了，当复用convertView的使用，任然没有保存状态。那么好，我们看看该问题所在的getView（）方法。

我仿照他的写法写了一个：

    private Context mContext;    private List<ListBean> mAdapterDataList;    private LayoutInflater mInflater;    public MyAdapter(Context context, List<ListBean> dataList) {        this.mContext = context;        this.mAdapterDataList = dataList;        mInflater = (LayoutInflater) context.getSystemService(Context.LAYOUT_INFLATER_SERVICE);    }    @Override    public int getCount() {        if (mAdapterDataList == null) {            return 0;        }        return mAdapterDataList.size();    }    @Override    public ListBean getItem(int position) {        return mAdapterDataList.get(position);    }    @Override    public long getItemId(int position) {        return position;    }    @Override    public View getView(int position, View convertView, ViewGroup parent) {        if (convertView == null) {            convertView = mInflater.inflate(R.layout.item_activity_main, parent, false);            convertView.setTag(new ViewHolder(convertView));        }        ViewHolder holder = (ViewHolder) convertView.getTag();        //设定checkbox的状态        **holder.mCheckBox.setChecked(getItem(position).isCheck());**        holder.mTextView.setText(getItem(position).getText());        //给监听器中设定当前的需要操作的对象        holder.mCheckBox.setTag(getItem(position));        holder.mCheckBox.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {            @Override            **public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {                // 在这里进行保存自己选中的item                ListBean bean = (ListBean) buttonView.getTag();                //保存checkbox对应的状态                bean.setCheck(isChecked);                if (isChecked) {                    if (!mSelectItem.contains(bean)) {                            mSelectItem.add(bean);                        }                } else {                    mSelectItem.remove(bean);                }            }**        });        return convertView;    }}public static class ViewHolder {    private CheckBox mCheckBox;    private TextView mTextView;    public ViewHolder(View view) {        mCheckBox = (CheckBox) view.findViewById(R.id.check_box);        mTextView = (TextView) view.findViewById(R.id.text_view);    }}`

不知道你有没有发现这段代码中的bug，我来看看他需要改进的地方
第一：在convertView 不为null的时候，他在调用holder.mCheckBox.setChecked(getItem(position).isCheck());方法的时候，会触复用前一convertView的OnCheckedChangeListener的onCheckedChanged（）的方法，就会把之前的那个状态给改变掉，这也就为什么出现没有没有保存状态的原因。
第二：每次复用convertView 的时候，都会new OnCheckedChangeListener的对象，就会造成创建很多OnCheckedChangeListener对象，那么对内存管理来说也不是很好。

解决办法：
1、解决第一个bug，将设置OnCheckedChangeListener监听器和holder.mCheckBox.setChecked(getItem(position).isCheck())方法的顺序调换一下，那么就不会出现下一个复用，调用上一个的onCheckedChanged（）方法。
2、解决第二个bug，当convertView==null的时候设定监听器，那么就不会创建多余的监听器。

下载源码