2

Using this Expandable List checkbox example code as an baseline, I am trying to save and maintain the checkbox state. However, random checkboxes are being checked and unchecked ( triggering my OnCheckedChangeListener with the new values ) when I scroll them out of sight, minimize their group, or even minimize/maximize a nearby group!

public Object getChild(int groupPosition, int childPosition) {
    return colors.get( groupPosition ).get( childPosition );
}

public long getChildId(int groupPosition, int childPosition) {
    return (long)( groupPosition*1024+childPosition );  // Max 1024 children per group
}

public View getChildView(final int groupPosition, final int childPosition, 
        boolean isLastChild, View convertView, ViewGroup parent) {

    View v = null;
    if( convertView != null ) {
        v = convertView;
    } else {
        v = inflater.inflate(R.layout.child_row, parent, false); 
    }

    Color c = (Color)getChild( groupPosition, childPosition );

    TextView color = (TextView)v.findViewById( R.id.childname );
    if( color != null ) {
        color.setText( c.getColor() );
    }

    TextView rgb = (TextView)v.findViewById( R.id.rgb );
    if( rgb != null ) {
        rgb.setText( c.getRgb() );
    }

    CheckBox cb = (CheckBox)v.findViewById( R.id.check1 );
    cb.setChecked( c.getState() );
    cb.setOnCheckedChangeListener(new OnCheckedChangeListener()
    {
        public void onCheckedChanged(CompoundButton buttonView, boolean isChecked)
        {
            colors.get(groupPosition).get(childPosition).setState(isChecked);
            context.setColorBool(groupPosition, childPosition, isChecked);
            Log.d("ElistCBox2", "listitem position: " +groupPosition+"/"+childPosition+" "+isChecked);
        }
    });

    return v;
}

I don't know what piece of code could be responsible for this, so any suggestions on what to include here are welcome. My code only differs from the original in my attempt to save the values.

Robert Kuykendall
  • 300
  • 2
  • 13
  • I have this exact issue with CheckBoxes in ExpandableListViews. Been looking for an answer forever. Hope your question gets more response than mine. – jmease Jul 02 '12 at 13:38
  • @jmease Today I commented out my convertView code, and the problem went away. I'm still a little uncertain that everything is working as it should, and I feel like I should be rewriting the UI using something besides ExpandableListView, but the code is working so I will probably leave it as-is for now. I highly recommend you try ignoring convertview and see if it works for you. – Robert Kuykendall Jul 02 '12 at 19:13

2 Answers2

0

my guess is that as your adapter is creating views, the check listener is being called as the checkbox view is initialized. a lot of widgets in android work like this ... the listener is called when the view is initialized.

i don't know why things work like this, but it might be to allow the client code to initialize itself in a consistent way. e.g., whether the checkbox is checked by the user or whether it is initialized as checked, run the same code.

to counteract this, you can try doing something like setting a flag in your listener class impl to allow you to ignore the first click, something like,

cb.setOnCheckedChangeListener(new OnCheckedChangeListener()
    {
        private void first = true;

        public void onCheckedChanged(CompoundButton buttonView, boolean isChecked)
        {
            if (first) {
              first = false;
              return;
            }

            colors.get(groupPosition).get(childPosition).setState(isChecked);
            context.setColorBool(groupPosition, childPosition, isChecked);
            Log.d("ElistCBox2", "listitem position: " +groupPosition+"/"+childPosition+" "+isChecked);
        }
    });

also, ensure that you are correctly re-using convertView in your implementation of getView() in your adapter. e.g.,

@Override
public View getView(int position, View convertView, ViewGroup parent) {
    View view = convertView;
    if (view == null) {
        view = inflater.inflate(R.layout.applications_item, null);
    }
Jeffrey Blattman
  • 22,176
  • 9
  • 79
  • 134
  • I added more source code. I believe I'm using convertView correctly. I don't think that `setOnCheckedChangeListener` flag would do anything to stop the checkbox from being changed randomly, it just wouldn't save the change. When I add the code, the "random changes" become each checkbox just reverting to its original state. – Robert Kuykendall Jun 27 '12 at 19:59
  • put breakpoints in your `getView()` method to understand if the converrtView is being used, if you are re-creating the checkboxes more than you think, and if there's a correlation between creating checkboxes and the listener being called. – Jeffrey Blattman Jun 27 '12 at 20:02
0

This is a very old question, but I struggled with the same problem so here is my answer for anybody looking:

The simplest way is to use CheckBox.onClickListener instead of onCheckedChangeListener.

This is only mildly annoying in terms of rearranging your logic, but will ensure that when the boxes are unchecked randomly (by, e.g. expanding an adjacent group) the event will not fire.

Honestly I think this should be considered a bug, even though I'm sure the behaviour can be explained from the Android source.

D2TheC
  • 2,203
  • 20
  • 23