Index out of bound: ArrayList

Question

Problem

I have implemented multi-selection behavior for recycler view using an external library. Now, to remove items from recycler view, I implemented a two for loops. First for loop deleted items from SQLite and the second for loop removes correspondent views from the adapter. However, the problem arises when removing views from the adapter.

         for (i in selectedCardItems!!.indices)  //selectedCardItems stores selected card position.
            {
                val index = selectedCardItems!![i]
                val noteRowID = listItems!![index]  //list items contains references to items in SQLite and is fed to recyclerview.setadapter = myAdapter(context,listitems)


                dbHandler!!.deleteNote(noteRowID.noteID!!)
            }




            for(i in selectedCardItems!!.indices)
            {
                val index = selectedCardItems!![i]
                listItems!!.removeAt(i)  //problem starts here, due to mismatched indexes.

                adapter!!.notifyItemRemoved(i)
            }

            if(dbHandler!!.trashedNotesCount() == 0)
            {
                trashedRecyclerView!!.visibility = View.GONE
                emptyTrashImg!!.visibility = View.VISIBLE
                emptyTrashMsg!!.visibility = View.VISIBLE
            }

            selectedCardItems!!.clear()  //once all operation is done,remove card positions from this ArrayList.
        }

Both listitems and selectedCardPosition are of ArrayList type. I know that once an item from an ArrayList is removed from an index, then higher index item index is automatically moved to a lower index. What is an efficient way to fix this problem?

What I tried: A BAD fix is to basically remove the second for loop that removes views and replace it with adapter.notifyDataSetChanged() which is also removes deleting animations.

In selectedCardItems you save the indices of the items to be deleted? — forpas
– forpas, Commented Oct 21, 2018 at 13:32

forpas · Accepted Answer · 2018-10-21 13:43:30Z

2

If I understand your question correctly, the problem arises because deletions from the list ruin the initial indexing.If this is the case you have to start deletions from the higher index and then go down. So sort the list descending before the loop:

selectedCardItems!!.sortDescending()

answered Oct 21, 2018 at 13:43

forpas

165k10 gold badges51 silver badges85 bronze badges

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

Collectives™ on Stack Overflow

Index out of bound: ArrayList

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related