Maintaining checkbox states in listview with CursorAdapter

Question

For my Android project, I have a listview which has a checkbox for every item. The data is loaded from an SQLite database by using a CursorAdapter class. However, whenever I

Answer 1

Reason : ListView re-uses the views.

Solution :

class VocabCursorAdapter extends CursorAdapter {
    List<Integer> selectedItemsPositions;//to store all selected items position

    public VocabCursorAdapter(Context context, Cursor c,int flags) {
        super(context, c,0);
        selectedItemsPositions = new ArrayList<>();
    }

    @Override
    public View newView(Context context, Cursor cursor, ViewGroup viewGroup) {
        View view = LayoutInflater.from(context).inflate(R.layout.item_vocab, viewGroup, false);
        CheckBox box = (CheckBox) view.findViewById(R.id.editCheckbox);
        box.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
            @Override
            public void onCheckedChanged(CompoundButton compoundButton, boolean b) {
                int position = (int) compoundButton.getTag();
                if (b) {
                    //check whether its already selected or not
                    if (!selectedItemsPositions.contains(position))
                        selectedItemsPositions.add(position);
                } else {
                    //remove position if unchecked checked item
                    selectedItemsPositions.remove((Object) position);
                }
            }
        });
        return view;
    }

    @Override
    public void bindView(View view, Context context, Cursor cursor) {

        //your other stuff

        CheckBox box = (CheckBox) view.findViewById(R.id.editCheckbox);
        box.setTag(cursor.getPosition());

        if (selectedItemsPositions.contains(cursor.getPosition()))
            box.setChecked(true);
        else
            box.setChecked(false);
    }
}

Answer 2

public class ObservationselectattributeFragment extends Fragment {

DatabaseHandler mDBHandler;
ListView mListView;
SimpleCursorAdapter mSCA;
Cursor mCsr;
ArrayList<String> attributeItems = new ArrayList<>();

public ObservationselectattributeFragment() {
    // Required empty public constructor
}


@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container,
                         Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    // Inflate the layout for this fragment
    View view1=inflater.inflate(R.layout.fragment_observationselectattribute, container, false);
   //Bundle bundle2 = getArguments();

    Bundle bundle1 = getArguments();
    final int firsttext= bundle1.getInt("TotalCount");
    final String selectedtreatment= bundle1.getString("SelectedTreatment");
   Toast.makeText(getActivity(),"value \n"+firsttext+"\n"+"treatment \n"+selectedtreatment, Toast.LENGTH_SHORT).show();
   // Toast.makeText(getActivity(),"SelectedTreatment \n"+selectedtreatment, Toast.LENGTH_SHORT).show();
    mListView = (ListView)view1.findViewById(R.id.lv001);
    mListView.setChoiceMode(ListView.CHOICE_MODE_MULTIPLE);
    Button addattribute = (Button)view1.findViewById(R.id.addattribute);
    addattribute.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            String items1="";
            Integer tcount1=0;
            for(String item1:attributeItems){
               items1+="-"+item1+"\n";
                tcount1++;
            }
            Toast.makeText(getActivity(),"you have selected \n"+items1,Toast.LENGTH_LONG).show();
            Toast.makeText(getActivity(),"you have selected \n"+tcount1,Toast.LENGTH_LONG).show();
           /*FragmentTransaction fr= getFragmentManager().beginTransaction();
            fr.replace(R.id.main_container, new ShowObservationDataRecordingFragment()).addToBackStack("ObservationselectattributeFragment");
            fr.commit();*/
            Bundle bundle = new Bundle();
            bundle.putInt("TotalCount2",firsttext);
            bundle.putInt("TotalCount1", tcount1);
            bundle.putString("SelectedTreatment", selectedtreatment);
            Fragment showobservationdatarecordingfragment = new ShowObservationDataRecordingFragment();
            showobservationdatarecordingfragment.setArguments(bundle);
            FragmentManager fragmentManager = getFragmentManager();

            fragmentManager.beginTransaction().replace(R.id.main_container, showobservationdatarecordingfragment).addToBackStack("ObservationselectattributeFragment").commit();
        }

    });
    mDBHandler = new DatabaseHandler(this.getActivity());
    mCsr = mDBHandler.getAllRecords();
    // Prepare a list of the columns to get the data from, for the ListViewt
    String[] columns_to_get_data_from = new String[]{
            DatabaseHandler.KEY_IDS,
            DatabaseHandler.KEY_NAMES,
            DatabaseHandler.KEY_FNAME,
            DatabaseHandler.KEY_MONAME,
            DatabaseHandler.KEY_SNAME
    };

    // Prepare a list of the Views into which to place the data
    int[] itemviews_to_place_data_in = new int[]{
            R.id.euserid,
            R.id.eusername,
            R.id.efname,
            R.id.emoname,
            R.id.esname
    };

    // get and instance of SimpleCursorAdapter
    mSCA = new SimpleCursorAdapter(getActivity(),
            R.layout.listviewitem_record,
            mCsr,
            columns_to_get_data_from,
            itemviews_to_place_data_in,
            0);
    // Save the ListView state (= includes scroll position) as a Parceble
    Parcelable state = mListView.onSaveInstanceState();
    // get and instance of SimpleCursorAdapter the listviewitem_record layout
    mListView.setAdapter(mSCA);
    // Restore previous state (including selected item index and scroll position)
    mListView.onRestoreInstanceState(state);
    mListView.setOnItemClickListener(new AdapterView.OnItemClickListener() {
        @Override
        public void onItemClick(AdapterView<?> parent, View view, int position, long id) {
            String attributeItem1 = ((TextView)view.findViewById(R.id.euserid)).getText().toString();
            String attributeItem2 = ((TextView)view.findViewById(R.id.eusername)).getText().toString();
            String attributeItem3 = ((TextView)view.findViewById(R.id.efname)).getText().toString();
            String attributeItem4 = ((TextView)view.findViewById(R.id.emoname)).getText().toString();
            String attributeItem5 = ((TextView)view.findViewById(R.id.esname)).getText().toString();
            String attributeItem = attributeItem1 + attributeItem2 + attributeItem3 + attributeItem4 + attributeItem5;
           // CheckedTextView box = (CheckedTextView) view.findViewById(R.id.record_checkbox);
         //  box.setChecked(true);
            CheckedTextView checkedTextView = (CheckedTextView) view.findViewById(R.id.record_checkbox);
            if(checkedTextView.isChecked()) {
                checkedTextView.setChecked(false);
            } else {
                checkedTextView.setChecked(true);
            }
            if(attributeItems.contains(attributeItem)){
                attributeItems.remove(attributeItem);//uncheck item

            }
            else
            {
                attributeItems.add(attributeItem);
            }

            Toast.makeText(getActivity(), "Item1 = " + attributeItem1 +"\n"+ "Item2 ="+attributeItem2 +"\n"+"Item3 ="+attributeItem3+"\n"+"Item4 ="+attributeItem4+"\n"+"Item5 ="+attributeItem5, Toast.LENGTH_SHORT).show();


        }
    });
    ((HomeActivity) getActivity())
            .setActionBarTitle("Select Attribute");
    return view1;
}

}

Answer 3

Try this

public class VocabCursorAdapter extends CursorAdapter { 

   private ArrayList<Boolean> itemChecked = new ArrayList<Boolean>(); // array list for store state of each checkbox

   public VocabCursorAdapter(Context context, Cursor c, int flags) {

       for (int i = 0; i < c.getCount(); i++) { // c.getCount() return total number of your Cursor
            itemChecked.add(i, false); // initializes all items value with false
       }
   }

    @Override
    public void bindView(View view, Context context, Cursor cursor) {

        ...
        final int position = cursor.getPosition(); // get position by cursor

        CheckBox checkBox = (CheckBox) view.findViewById(R.id.editCheckbox);
        checkBox.setOnClickListener(new OnClickListener() {
                public void onClick(View v) {

                    if (itemChecked.get(position) == true) { // if current checkbox is checked, when you click -> change it to false
                        itemChecked.set(position, false);
                    } else {
                        itemChecked.set(position, true);
                    }
                }
       });

       checkBox.setChecked(itemChecked.get(position)); // set the checkbox state base on arraylist object state
       Log.i("In VocabCursorAdapter","position: "+position+" - checkbox state: "+itemChecked.get(position));
    }
}