Listview checkbox array index out of bound exception

Question

i have one listview with checkbox,whenever user pressed the checkbox i'm getting some user information(for example say profile id of facebook) and stored in arraylist.Now what's the problem is whenever user scrolled the listview some other profile id's is also added or removed additionally eventhough it is not marked.so it provide

array index out of bound exception

I tried to sort out the problem but i cant able to do it,please provide me some solution.

Here is my complete listview

public class MyComposeListItemAdapter extends ArrayAdapter {

public Context mContext;
int row;
public List<MyComposeObject> arrayList;     
public ImageLoader imageLoader;     
private static LayoutInflater inflater=null;    

/*
 * Create an ArrayList of Boolean Object to store the state of the each CheckBox
 * Initializes the ArrayList items to default value false, means no CheckBox is checked yet.
 * When you click on CheckBox. Set a check against Checked/Unchecked state and store that value in ArrayList.
 * Now set that position to CheckBox using setChecked() method.*/

private ArrayList<Boolean> itemChecked = new ArrayList<Boolean>();  

// for radio button 
private RadioButton mCurrentlyCheckedRB;
private boolean userSelected = false;
private int mResourceId = 0;
private RadioButton mSelectedRB;
private int mSelectedPosition = -1;//

public static List<String> selected_profileid =new ArrayList<String>();
public static List<String> selected_profiletype =new ArrayList<String>();

public MyComposeListItemAdapter(Context context, int resource) {
    super(context, resource);
    // TODO Auto-generated constructor stub

    this.mContext = context;
    this.row = resource;
    arrayList = (ArrayList<MyComposeObject>) Woosuite_Login.composeObjectslist;
//  System.out.println("arrayList2"+arrayList.size());      
    imageLoader=new ImageLoader(context);       
    for (int i = 0; i < this.getCount(); i++) {
        itemChecked.add(i, false); // initializes all items value with false
        }       
}   
@Override
public Context getContext() {
    // TODO Auto-generated method stub
    return mContext;
}
@Override
public int getCount() {
    // TODO Auto-generated method stub
    return arrayList.size();
}
@Override
public View getView(final int position, View convertView, ViewGroup parent) {
    // TODO Auto-generated method stub
    ViewHolder holder;

    if (convertView == null) {
        LayoutInflater inflater = (LayoutInflater) mContext
                .getSystemService(Activity.LAYOUT_INFLATER_SERVICE);
        convertView = inflater.inflate(row, null);
        holder = new ViewHolder();
        convertView.setTag(holder);
    } else
        holder = (ViewHolder) convertView.getTag();

    holder.userimage_ImageView=(ImageView) convertView.findViewById(R.id.imageView666);
    holder.username_TextView=(TextView) convertView.findViewById(R.id.textView666);
    holder.checkBox=(CheckBox) convertView.findViewById(R.id.checkBox666);
    holder.radioButton=(RadioButton) convertView.findViewById(R.id.radio666);

        holder.checkBox.setOnCheckedChangeListener(new OnCheckedChangeListener() {

            @Override
            public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
                // TODO Auto-generated method stub

                if(isChecked){                                                                  
                    itemChecked.set(position, true);

                    selected_profileid.add(arrayList.get(position).getCom_profileId());
                    System.out.println("add selected_profileid "+selected_profileid);

                    selected_profiletype.add(arrayList.get(position).getCom_profileType());
                    System.out.println("add selected_profiletype.size(); "+selected_profiletype);

                }else{

                    selected_profiletype.remove(arrayList.get(mposition).getCom_profileType());
                    System.out.println("remove selected_profileid "+selected_profileid);

                    selected_profileid.remove(arrayList.get(mposition).getCom_profileId());
                    System.out.println("remove selected_profiletype.size(); "+selected_profiletype);
                }
            }
        });     

    if(arrayList.get(position).getCom_ProfileImgUrl()!=null){                       
        imageLoader.DisplayImage(arrayList.get(position).getCom_ProfileImgUrl(), holder.userimage_ImageView);           

    }else{
        holder.userimage_ImageView.setImageResource(R.drawable.fb);
    }

    if(arrayList.get(position).getCom_ProfileName()!=null){
        holder.username_TextView.setText(arrayList.get(position).getCom_ProfileName());
    }

    holder.checkBox.setChecked(itemChecked.get(position));
    return convertView;
}

private static class ViewHolder {

    ImageView userimage_ImageView;
    TextView username_TextView;
    CheckBox checkBox;
    RadioButton radioButton;
}
}

its already there know , please check it in boolean true part — Androidcuckoo
– Androidcuckoo, Commented Feb 27, 2014 at 7:46
i want to add it profile id and profile type if checkbox is clicked and need to remove the same it checkbox is unchecked — Androidcuckoo
– Androidcuckoo, Commented Feb 27, 2014 at 7:47

alex · Accepted Answer · 2014-02-27 08:52:40Z

1

Please add the following code and run it again and show me the logcat output:

 [...]
 @Override
        public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
            // TODO Auto-generated method stub
             if(position >= getCount()) {
                System.out.println("Error: position '"+position+"' is out of bounds in 'arrayList'");
                return;
             }
 [...]

Prevent duplicate list entries:

if(!myList.contains(objectToInsert) {
    myList.add(objectToInsert);
}

edited Feb 27, 2014 at 8:52

answered Feb 27, 2014 at 8:09

alex

5,7142 gold badges39 silver badges62 bronze badges

Sign up to request clarification or add additional context in comments.

9 Comments

Androidcuckoo Over a year ago

hai here problem is i'm adding profile id's inside checkbox,which means when ever it clicks then only it will add but here whats happening is whenever i scroll listview it s adding,how it will enter checkbox listener without clicking checkbox

alex Over a year ago

So try my code, with your example above, its not possible to help because we can't see whats happening outside the class

Androidcuckoo Over a year ago

Its not printing in logcat because condition always getting false

alex Over a year ago

ok that means there is no problem with 'arrayList'. the please post the full stacktrace of your exception

Androidcuckoo Over a year ago

ya that problem is solved , and exception in atacktrace too, the problem suppose consider i select first checkbox which means one profile id is added,now whenever i scroll down and comming to first checkbox it again adding the same,that mean duplicate of same profile id is added in the arraylist

|

Collectives™ on Stack Overflow

Listview checkbox array index out of bound exception

1 Answer 1

9 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

9 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related