在 ExpandableListView 上滚动时选择复选框重复

Posted 2023-05-07

【中文标题】在 ExpandableListView 上滚动时选择复选框重复

【英文标题】：Select checkbox is duplicated when scrolling on an ExpandableListView

【发布时间】：2018-02-23 05:12:49

【问题描述】：

我有一个 BaseExpandableListAdapter（下面的代码），它操纵一个 ExpandableListView 和一个名为 Checklist 的模型。

清单模型包含一个类别列表。 类别是检查列表。 支票是低点列表。 Low 包含一个布尔项，由视图中的复选框表示。

我使用 ViewHolder 模式来处理我的模型：CategoryViewHolder 用于我的 GroupView 和 CheckViewHolder 用于我的 ChildView。

我遇到的问题是在 getChildView 方法中，当我选择 Low 中的复选框（在 Check 中）并向下滚动时，我发现另一个复选框被选中。 当我在 Check 中的 edittext 对象中写一些东西并滚动时，也会发生同样的事情，另一个 edittext 被编辑。

我怀疑我的 vue 回收不能正常工作，但我没有找到任何解决方案。

有什么想法吗？

public class ChecklistAdapter extends BaseExpandableListAdapter 
private Context mContext;
private Checklist mChecklist;
private boolean isEditable;
private final int ID_LOW_ADDED = -1;


public ChecklistAdapter(Context mContext, Checklist mChecklist, boolean editable) 
    this.mContext = mContext;
    this.mChecklist = mChecklist;
    this.isEditable = editable;


@Override
public Object getChild(int groupPosition, int childPosition) 
    return mChecklist.getCategories().get(groupPosition).getChecks().get(childPosition);



@Override
public long getChildId(int groupPosition, int childPosition) 
    return childPosition;


// ChildView is the Check item with a label and buttons actions
@Override
public View getChildView(final int groupPosition, final int childPosition, boolean isLastChild, View convertView, final ViewGroup parent) 
    final CheckViewHolder holder;

    final List<Category> mCategories = mChecklist.getCategories();
    final Category mCategory = (Category) getGroup(groupPosition);
    final List<Check> mChecks = mCategory.getChecks();
    final Check mCheck = (Check) getChild(groupPosition, childPosition);
    final List<Low> mLows = mCheck.getLows();


    if (convertView != null)
        holder = (CheckViewHolder) convertView.getTag();
    
    else 
        LayoutInflater inflater = (LayoutInflater) this.mContext.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
        convertView = inflater.inflate(R.layout.item_check, parent, false);
        holder = new CheckViewHolder(convertView);
        convertView.setTag(holder);
    

    Utils.logK("Holder = "+holder);

    // Prepare checking lows elements
    addLowsToView(mLows, convertView, holder);


    // Set Checks elements
    String label = mCheck.getLabel();
    holder.tvCheckLabel.setText(label);

    // Check the state of button
    final View finalConvertView = convertView;
    final ViewGroup finalParent = parent;
    holder.rgActions.setOnCheckedChangeListener(new RadioGroup.OnCheckedChangeListener() 
        @Override
        public void onCheckedChanged(RadioGroup radioGroup, @IdRes int rbId) 

            switch (rbId)
                case R.id.rb_ko:
                    holder.llLow.setVisibility(View.VISIBLE);
                    mCheck.setState(Check.KO);

                    //finalParent.scrollTo(0,finalConvertView.getTop());

                    //((ExpandableListView) parent).smoothScrollToPosition(finalConvertView.getTop());
                    //((ExpandableListView) parent).setSelection(0);
                    //Utils.logK("Scroll to top = "+finalConvertView.getTop());
                    break;
                case R.id.rb_nc:
                    holder.llLow.setVisibility(View.GONE);
                    mCheck.setState(Check.NC);
                    break;
                case R.id.rb_ok:
                    holder.llLow.setVisibility(View.GONE);
                    mCheck.setState(Check.OK);
                    break;
                default:
                    //TODO
            

            // Prepere updated checklist object to send
            mChecks.set(childPosition, mCheck);
            mCategory.setChecks(mChecks);
            mCategories.set(groupPosition, mCategory);
            mChecklist.setCategories(mCategories);

        
    );

    /**
     * Save the state of radioButtons (selection)
     */
    if(mCheck.getState() != null) 
        switch (mCheck.getState()) 
            case Check.KO:
                holder.rbKo.setChecked(true);
                holder.rbKo.setSelected(true);
                break;
            case Check.NC:
                holder.rbNc.setChecked(true);
                holder.rbNc.setSelected(true);
                break;
            case Check.OK:
                holder.rbOk.setChecked(true);
                holder.rbOk.setSelected(true);
                break;
        
    

    holder.cbOther.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() 

        int lowsSize = mLows.size();
        @Override
        public void onCheckedChanged(CompoundButton compoundButton, boolean isChecked) 
            if (isChecked)
                holder.etOther.setEnabled(true);
            
            else
                holder.etOther.setEnabled(false);
            

            // Add the new low and updated checklist object to send
            final Low mOtherLow = new Low();
            mOtherLow.setId(ID_LOW_ADDED);
            mOtherLow.setSelected(true);


            // TODO save checklist when text is added
            holder.etOther.addTextChangedListener(new TextWatcher() 
                @Override
                public void beforeTextChanged(CharSequence charSequence, int i, int i1, int i2) 

                

                @Override
                public void onTextChanged(CharSequence charSequence, int i, int i1, int i2) 

                

                @Override
                public void afterTextChanged(Editable editable) 
                    mOtherLow.setLabel(holder.etOther.getText().toString());

                
            );

            Utils.logK("otherLow added = "+mOtherLow.getLabel());

               //long idLastLowAdded = mLows.get(mLows.size()-1).getId();
                Utils.logK("Low added = "+mOtherLow.getLabel());
                mLows.add(mOtherLow);
                mCheck.setLows(mLows);
                mChecks.set(childPosition, mCheck);
                mCategory.setChecks(mChecks);
                mCategories.set(groupPosition, mCategory);
                mChecklist.setCategories(mCategories);
         

        


    );

    holder.setViewsEnabled(isEditable);


    return convertView;


@Override
public int getChildrenCount(int groupPosition) 
    return mChecklist.getCategories().get(groupPosition).getChecks().size();


@Override
public Object getGroup(int groupPosition) 
    return mChecklist.getCategories().get(groupPosition);



@Override
public int getGroupCount() 
    return mChecklist.getCategories().size();


@Override
public long getGroupId(int groupPosition) 
    return groupPosition;



@Override
public View getGroupView(int groupPosition, boolean isExpanded, View convertView, ViewGroup parent) 
    final CategoryViewHolder groupHolder;
    final Category mCategory = (Category) getGroup(groupPosition);


    if (convertView != null)
        groupHolder = (CategoryViewHolder) convertView.getTag();
    
    else 
        LayoutInflater inflater = (LayoutInflater) this.mContext
                .getSystemService(Context.LAYOUT_INFLATER_SERVICE);
        convertView = inflater.inflate(R.layout.item_category, parent, false);
        groupHolder = new CategoryViewHolder(convertView);

        convertView.setTag(groupHolder);
    

    groupHolder.tvCategoryTitle.setText(mCategory.getLabel());

    return convertView;



@Override
public boolean hasStableIds() 
    return false;


@Override
public boolean isChildSelectable(int groupPosition, int childPosition) 
    return true;



/**
 * Add the lines of works to my check item (checkboxes)
 * @param lows List of line of work
 * @param v The view will be attached this new view
 */
private void addLowsToView(final List<Low> lows, View v, final CheckViewHolder holder) 
    LinearLayout llLowPosition = v.findViewById(R.id.ll_low_position);
    if(((LinearLayout) llLowPosition).getChildCount() > 0)
        ((LinearLayout) llLowPosition).removeAllViews();

    for (int i = 0; i < lows.size(); i++) 

        final int position = i;
        final Low low = lows.get(i);
        // Ignore the "others" lows (id of an other low start at -1 and decrement for each time the
        // user click on "other" checkbox
        if (low.getId()<0)
            if (isEditable)
                break;
            
            else
                holder.cbOther.setSelected(true);
                holder.etOther.setText(low.getLabel());
            
        
        else 

            CheckBox cbLow = new CheckBox(mContext);
            cbLow.setText(lows.get(i).getLabel());
            cbLow.setChecked(lows.get(i).isSelected());
            //cbLow.setButtonDrawable(ContextCompat.getColor(mContext,R.color.accent));
            cbLow.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() 
                @Override
                public void onCheckedChanged(CompoundButton compoundButton, boolean isChecked) 
                    low.setSelected(isChecked);
                    lows.set(position, low);
                
            );
            llLowPosition.addView(cbLow, 0, new ViewGroup.LayoutParams(ViewGroup.LayoutParams.MATCH_PARENT, ViewGroup.LayoutParams.MATCH_PARENT));
            // Disable if it's not editable mode
            if (!isEditable)
                cbLow.setEnabled(false);
            
        



    



/**
 * Allow activity to get the updated checklist
 * @return The updated checklist
 */
public Checklist getChecklist()
    return mChecklist;



/**
 * Class that define the view holder of a child : the check model
 */
static class CheckViewHolder
    @BindView(R.id.tv_check_label)
    TextView tvCheckLabel;

    @BindView(R.id.rg_actions)
    RadioGroup rgActions;

    @BindView(R.id.ll_low)
    LinearLayout llLow;

    @BindView(R.id.ll_low_position)
    LinearLayout llLowPosition;

    @BindView(R.id.cb_other)
    CheckBox cbOther;

    @BindView(R.id.et_other)
    EditText etOther;

    @BindView(R.id.rb_ko)
    RadioButton rbKo;

    @BindView(R.id.rb_nc)
    RadioButton rbNc;

    @BindView(R.id.rb_ok)
    RadioButton rbOk;

    @BindView(R.id.cv_item_check)
    CardView cvItemCheck;


    public CheckViewHolder(View view)
        ButterKnife.bind(this, view);
    

    /**
     * Set actions on the view. If it's not editable, disable all actions.
     * @param editable : Mode of view : consulting or editing
     */
    public void setViewsEnabled(boolean editable)
        rgActions.setEnabled(editable);
        llLowPosition.setEnabled(editable);
        rbKo.setEnabled(editable);
        rbNc.setEnabled(editable);
        rbOk.setEnabled(editable);
        cbOther.setEnabled(editable);
        //etOther.setEnabled(editable);
    


/**
 * Class that define the view holder of a parent : the category model
 */
static class CategoryViewHolder
    @BindView(R.id.tv_category_title)
    TextView tvCategoryTitle;

    public CategoryViewHolder(View v)
        ButterKnife.bind(this, v);
    

【问题讨论】：

你能添加你的布局/布局吗

【参考方案1】：

如果我理解正确，你检查你的复选框，然后滚动，然后一些随机复选框被选中？ 我在 ExpandableListView 中遇到了类似的问题，您需要在问题中实现类似的东西：

Checking a checkbox in listview makes other random checkboxes checked too

【参考方案2】：

在回收器视图中，视图被回收，因此，一些复选框已经被选中（在滚动之前被选中，现在被回收）。这是正常行为，您需要取消设置复选框的状态并编辑文本。

【参考方案3】：

在 Activity 中声明与 mChecklist 大小相同的 ArrayList

喜欢：- ArrayList mChecklistBool = new ArrayList();

 for (int i = 0;i<Checklist.size();i++)
    
        mChecklistBool.add(false);
    


 When you call adapter send this ArrayList of Boolean 

  new ChecklistAdapter (mContext,mChecklist,mChecklistBool,editable);



      In Adapter class declare 

    ArrayList<Boolean> mChecklistBool;

public ChecklistAdapter(Context mContext, Checklist mChecklist, ArrayList<Boolean> mChecklistBool,boolean editable) 
this.mContext = mContext;
this.mChecklist = mChecklist;
this.isEditable = editable;
this.mChecklistBool =mChecklistBool;

然后，您将视图与视图持有者绑定，然后获取选中的位置，然后您可以设置或重置任何位置。

new View.OnClickListener() 
           public void onClick(View v) 
           CheckBox cb = (CheckBox) v; 
           if (mChecklist.contains(v.getTag())) 
                 int ChkPos = mChecklist.indexOf(v.getTag());
                 mChecklistBool.set(ChkPos, cb.isChecked());