CheckBox in ListView setonclicklistener not working properly

Question

I have a ListView with CheckBox and a TextView . In my adapters getView() method I implemented this listener on checkbox.

holder.check.setOnClickListener(new OnClickListener() {

        @Override
        public void onClick(View v) {
            HashMap<String, String> localMap = (HashMap<String, String>) holder.check
                    .getTag();
            if (holder.check.isChecked()) {
                trackinglist.add(localMap.get("taskid"));
                checkedlist.add(localMap.get("taskid"));
            } else {
                if (trackinglist.contains(localMap.get("taskid"))) {
                    trackinglist.remove(localMap.get("taskid"));
                }
                if (alreadycheckedlist.contains(localMap.get("taskid"))) {
                    undonelist.add(localMap.get("taskid"));
                    alreadycheckedlist.remove(localMap.get("taskid"));
                } else {
                    checkedlist.remove(localMap.get("taskid"));
                }
            }

        }
    });

Now my problem is that the holder.check.isChecked() always returns false even when the CheckBox is clicked and it is checked . What might be causing this behavior ? and yes I dont want to use setoncheckchangelistener . Please Help.

When OnClick, you will get the view which is clicked. You can check if the holder.check is the same as v. If not, maybe holder.check is already past (because of orientation, etc). It is better to use View v directly, or findviewbyid. — e7fendy, Nov 22 '12 at 10:11

score 1 · Answer 1 · answered Nov 22 '12 at 09:36

1

you may use

holder.check.setOnCheckedChangeListener(new OnCheckedChangeListener(){
public void onCheckedChanged(CompoundButton buttonView, boolean isChecked)
    {
        if ( isChecked )
        {
            // perform logic
        }

    }
});

answered Nov 22 '12 at 09:36

mini

855
4
15
22

please read the last line of the question. This method is called every time the checked value changes . I want it to be fired only when the user clicks on the check. – Ashwani Nov 22 '12 at 09:38

CheckBox in ListView setonclicklistener not working properly

1 Answers1