Recycling ListView rows with different row layouts

Question

I have a ListView adapter which is set with different layouts according to the type of the row item.

If I recycle my rows using a holder, as shown in the code below, I think I'll get some errors because for a recycled row I'll get the layout of the latest non-recycled row, which might not be of the same type.

Should I avoid recycling the rows in that case? What other options do I have?

@Override
public View getView(int position, View convertView, ViewGroup parent) {
    View row = convertView;
    MyHolder holder = null;

    int type = getItemViewType(position);

    if(row == null)
    {           
        holder = new MyHolder();

        LayoutInflater inflater = ((Activity)mContext).getLayoutInflater();

        if(type == TYPE_1){
            row = inflater.inflate(R.layout.layout_type_1, parent, false);
        }
        else if(type == TYPE_2){
            row = inflater.inflate(R.layout.layout_type_2, parent, false);
        }               
        else {
            row = inflater.inflate(R.layout.layout_type_3, parent, false);
        }

        row.setTag(holder);
    }
    else
    {
        holder = (MyHolder)row.getTag();
    }

    /* ... */
}

Recycling with RecyclerView and multiple row layout sample demo http://code2concept.blogspot.in/2015/10/android-multiple-row-layout-using.html where recycling is handled recycler itself — Nitesh Tiwari, Oct 15 '15 at 05:26
Use the recycleView instead of the listview, Check below link it inflates the different rows in the recyclview http://stackoverflow.com/a/39972276/3946958 — Ravindra Kushwaha, Oct 18 '16 at 10:45

score 5 · Accepted Answer · answered Mar 04 '13 at 16:17

5

Always recycle rows! Your implementation of getItemViewType (and getViewTypeCount) instructs the ListView when to recycle what types of views so that they won't get mixed up.

Your code snippet looks good. So, it all should work. The only slightly odd part is the if(type == TYPE_3). Did you mean if(type == TYPE_2)?

The question is: Do you have problem/exception in your code right now?

answered Mar 04 '13 at 16:17

Streets Of Boston

12,576
2
25
28

Yes if(type == TYPE_3) was a typo. Thanks. – jul Mar 04 '13 at 16:21
@Streets Of Boston, sorry for pinging this way. It's very urgent for me. Pls help me out here https://stackoverflow.com/questions/47897770/issues-in-using-edittext-in-row-of-recyclerview-and-the-textwatcher-on-edittext?noredirect=1#comment82874592_47897770 – Shambhu Dec 23 '17 at 13:27

score 2 · Answer 2 · answered Mar 04 '13 at 16:18

If I recycle my rows using a holder, as shown in the code below, I think I'll get some errors because for a recycled row I'll get the layout of the latest non-recycled row, which might not be of the same type.

If you are referring to the row == null check, then there is no problem as the ListView will give you the right convertView according to the getItemViewType() method.

If you are referring to the MyHolder class this again is not a problem because the holder is set as the tag for the row view and if the ListView gives you the right recycled row view then the holder tag will also have the proper fields initialized in it for that type of row. And, as you should handle the MyHolder according to the row type this isn't a problem as you'll access the initialized fields. Your MyHolder class should have references for all the views that your three type of rows expose so you have where to put them.

So keep the recycling mechanism.

Yes, I was referring to the row == null check. Thanks for the explanation. I'll accept Streets Of Boston's answer because he was first. — jul, Mar 04 '13 at 16:24
@Luksprog, sorry for pinging this way. It's very urgent for me. Pls help me out here https://stackoverflow.com/questions/47897770/issues-in-using-edittext-in-row-of-recyclerview-and-the-textwatcher-on-edittext?noredirect=1#comment82874592_47897770 — Shambhu, Dec 23 '17 at 13:30

Recycling ListView rows with different row layouts

2 Answers2