Проблема с флажком в представлении списка с другим макетом для строк

Я попытался сделать представление списка TextView и CheckBox. Он работает нормально, но при выборе CheckBox в строке также выбирается другой CheckBox в другой строке. Пример: если я проверил первую строку и прокрутил вниз, я нашел еще одну выбранную строку.

Это мой код адаптера

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;
}

}

И это OnCheckedChangeListener в действии

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();
        }
    };

Как я могу это исправить???

Спасибо.


person AwadKab    schedule 03.03.2013    source источник


Ответы (1)


Как вы, возможно, знаете, макеты повторно используются при прокрутке, поэтому вам нужно отслеживать, следует ли проверять каждую строку самостоятельно. Давайте добавим нового члена в ваш адаптер:

private SparseBooleanArray checked = new SparseBooleanArray();

И новый метод:

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

Теперь вызовите наш новый метод в onCheckedChanged():

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

Наконец, давайте настроим getView(), чтобы изменить отмеченное состояние (и несколько других вещей):

// 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));
}
person Sam    schedule 03.03.2013