CheckBox Issue in List View With Different Layout For Rows

问题

I tried make list view has TextView and CheckBox. It is worked fine but when select CheckBox in row another CheckBox in another row selected too. Example: if I checked first row and scroll down I found another row selected too.

This my Adapter Code

public class ReadersrListAdapter extends BaseAdapter{
private static final int TYPE_SEPARATOR = 0;
private static final int TYPE_ITEM = 1;
private static final int TYPE_MAX_COUNT = 2;
private Activity activity;
private ArrayList<HashMap<String, String>> data;
private LayoutInflater mInflater=null; 



public ReadersrListAdapter(Activity a, ArrayList<HashMap<String, String>> d) {
    activity = a;
    data=d;
    mInflater = (LayoutInflater)activity.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
}

public int getCount() {
    return data.size();
}

public Object getItem(int position) {
    return position;
}

public long getItemId(int position) {
    return position;
}

@Override
public boolean areAllItemsEnabled() {
    return false;
}

@Override
public boolean isEnabled(int position) {
    return !data.get(position).get(UtiliShare.KEY_TITLE).startsWith("-");
}

@Override
public int getItemViewType(int position) {
    return data.get(position).get(UtiliShare.KEY_TITLE).startsWith("-") ? TYPE_SEPARATOR : TYPE_ITEM;
}

@Override
public int getViewTypeCount() {
    return TYPE_MAX_COUNT;
}

@Override
public View getView(int position, View convertView, ViewGroup parent) {
    ViewHolder holder = null;        
    ViewHolder2 holder2 = null;
    int type = getItemViewType(position);

    if (convertView == null) {
        //holder = new ViewHolder();
        switch (type) {
            case TYPE_ITEM:
                holder = new ViewHolder();
                convertView = mInflater.inflate(R.layout.reader_list_item, null);
                holder.textView = (TextView)convertView.findViewById(R.id.readerTitle);
                holder.textView.setTypeface(UtiliShare.getTf());
                convertView.setTag(holder);
                break;
            case TYPE_SEPARATOR:
                holder2 = new ViewHolder2();
                convertView = mInflater.inflate(R.layout.reader_list_devider, null);
                holder2.textView = (TextView)convertView.findViewById(R.id.readerTitle);
                holder2.textView.setTypeface(UtiliShare.getTf());
                convertView.setTag(holder2);
                break;
        }
        //convertView.setTag(holder);
    } else {
        if(type==TYPE_ITEM) holder = (ViewHolder)convertView.getTag();
        else holder2 = (ViewHolder2)convertView.getTag();
    }
    HashMap<String, String> curdata = new HashMap<String, String>();
    curdata = data.get(position);
    String txt = curdata.get(UtiliShare.KEY_TITLE);
    if(type == TYPE_SEPARATOR){
        txt = txt.replace("-", "");
        holder2.textView.setText(txt);
    }else{
        //if(position <= 100) System.out.println("getView " + position + " " + convertView + " type = " + type);
        holder.star = (CheckBox) convertView.findViewById(R.id.btn_Fav);
        holder.star.setOnCheckedChangeListener(((ReadersListActivity) this.activity).mStarCheckedChanceChangeListener);

        holder.textView.setText(txt);
    }
    //holder.textView.setText(txt);
    return convertView;
}



private static class ViewHolder {
    public CheckBox star;
    public TextView textView;
}

private static class ViewHolder2 {
    public TextView textView;
}

}

And this OnCheckedChangeListener in activity

public OnCheckedChangeListener mStarCheckedChanceChangeListener = new OnCheckedChangeListener() {
        @Override
        public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
         // TODO Cyril: Not implemented yet!
            final int position = list.getPositionForView(buttonView);
            Toast.makeText(ReadersListActivity.this, ""+position, Toast.LENGTH_SHORT).show();
        }
    };

How Can I fix this???

Thanks.

回答1:

As you may know the layouts are reused as you scroll, so you need to track whether each row should be checked yourself. Let's add a new member to your Adapter:

private SparseBooleanArray checked = new SparseBooleanArray();

And a new method:

public void toggleCheck(int position) {
    boolean state = expanded.get(position, false); 
    expanded.put(!state);
}

Now call our new method in onCheckedChanged():

final int position = list.getPositionForView(buttonView);
adapter.toggleCheck(position);

Lastly let's tweak getView() to change the checked state (and a few other things):

// Don't create a new HashMap that you'll never use
//HashMap<String, String> curdata = new HashMap<String, String>();

HashMap<String, String> curdata = data.get(position);
String txt = curdata.get(UtiliShare.KEY_TITLE);
if(type == TYPE_SEPARATOR){
    txt = txt.replace("-", "");
    holder2.textView.setText(txt);
}else{
    //if(position <= 100) System.out.println("getView " + position + " " + convertView + " type = " + type);

    // These two lines should be in `if(convertView == null)` with the others
    holder.star = (CheckBox) convertView.findViewById(R.id.btn_Fav);
    holder.star.setOnCheckedChangeListener(((ReadersListActivity) this.activity).mStarCheckedChanceChangeListener);

    // Set the appropriate values
    holder.textView.setText(txt);
    holder.star.setChecked(expanded.get(position, false));
}

来源：https://stackoverflow.com/questions/15187795/checkbox-issue-in-list-view-with-different-layout-for-rows