On scroll recyclerview item Background color changed when recylcerview scroll

Question

I want to change Recyclerview item background color on my condition but when I scroll background color automatically changed I know holder.setIsRecyclable(false); but I don't want to set holder.setIsRecyclable(false);

and I know

 if(item.value == 1){
    // do something
    } else{
      // do something
   }

but i want to do using nested if else

           if (feetInt > 0 && feetInt < 4) {
                ((PatientViewHolder) holder).rlClientItemMain.setBackgroundColor(ContextCompat.getColor(mContext, R.color.player_list_green));
            } else if (feetInt >= 4 && feetInt < 6) {
                ((PatientViewHolder) holder).rlClientItemMain.setBackgroundColor(ContextCompat.getColor(mContext, R.color.player_list_yellow));
            } else if (feetInt == 0 || feetInt >= 6) {
                ((PatientViewHolder) holder).rlClientItemMain.setBackgroundColor(ContextCompat.getColor(mContext, R.color.player_list_red));
            }else{
                ((PatientViewHolder) holder).rlClientItemMain.setBackgroundColor(ContextCompat.getColor(mContext, R.color.player_list_grey));
            }

please share your complete adapter – yogesh lokhande Dec 08 '17 at 06:05 — yogesh lokhande, Dec 08 '17 at 06:05

score 2 · Answer 1 · answered Dec 08 '17 at 06:06

2

Instead of changing color on feetInt maintain a flag in your object class and on the basis of particular flag change your background color

answered Dec 08 '17 at 06:06

yogesh lokhande

1,245
1
11
20

score 0 · Answer 2 · edited Dec 22 '17 at 09:27

0

It looks like you are not updating the value of feetInt for every single object in list. That's why you are facing this issue.

Update feet int value in list for different different position, you will get the desire result.

Also, remove getItemViewCount() and setIsRecyclable(). These are not required at all. Also, share your code for more specific answers.

edited Dec 22 '17 at 09:27

Pang

9,564
146
81
122

answered Dec 08 '17 at 06:29

Rishabh Saxena

1,765
15
26

score 0 · Answer 3 · edited Jul 05 '19 at 06:51

0

holder.setIsRecyclable(false);

edited Jul 05 '19 at 06:51

barbsan

3,418
11
21
28

answered Jul 04 '19 at 09:51

sanjay sawan

37
2

It will be great if you could explain your solution and how it solves the OP's problem – ammportal Jul 04 '19 at 10:12
RecyclerView uses one view multiple times. user Scroll the list then remaining off screen item will display. for stop the Recycle item you can use setIsRecyclable(false) – sanjay sawan Jul 04 '19 at 10:25
If you read the question though, the OP does not want to use this specific property – ammportal Jul 04 '19 at 10:28

On scroll recyclerview item Background color changed when recylcerview scroll

3 Answers3