Recyclerview Adapter не работает должным образом - PullRequest
Новая
       8

Recyclerview Adapter не работает должным образом

0 голосов
/ 01 апреля 2020

В моем адаптере я проверяю, больше ли значение или равно нулю, я изменяю цвет макета на зеленый, если нет, он остается красным. Проблема заключается в том, что в представлении реселлера перестает изменяться множество элементов или все они отображаются зелеными (даже если значение меньше нуля). Что является причиной проблемы и как я могу это исправить? Заранее спасибо. это мой адаптер: 


import android.content.Context;
import android.content.res.ColorStateList;
import android.content.res.Resources;

import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.view.animation.AnimationUtils;
import android.widget.LinearLayout;
import android.widget.TextView;

import androidx.cardview.widget.CardView;
import androidx.recyclerview.widget.RecyclerView;

import java.text.DecimalFormat;
import java.util.ArrayList;

public class RVAdapter extends RecyclerView.Adapter<RVAdapter.RVViewHolder> {

    private Context mContext;

    private ArrayList<Item> itemList;

    public RVAdapter(Context context, ArrayList<Item> arrayList) {
        mContext = context;
        itemList = arrayList;

    }

    public static class RVViewHolder extends RecyclerView.ViewHolder {

        TextView valueTv, memoTv, categoryTv, dateTv;

        LinearLayout indicatorLL, infoLL;
        CardView itemLL;

        public RVViewHolder(View itemView) {
            super(itemView);

            // Calling FindViewByID()
            valueTv = itemView.findViewById(R.id.tv_value);
            memoTv = itemView.findViewById(R.id.tv_memo);
            categoryTv = itemView.findViewById(R.id.tv_category);
            dateTv = itemView.findViewById(R.id.tv_date);
            indicatorLL = itemView.findViewById(R.id.layout_indicator);
            infoLL = itemView.findViewById(R.id.layout_item_info);
            itemLL = itemView.findViewById(R.id.item_layout);

        }

    }

    @Override
    public RVViewHolder onCreateViewHolder(ViewGroup parent, int i) {
        LayoutInflater inflater = LayoutInflater.from(mContext);
        View view = inflater.inflate(R.layout.item_layout, parent, false);
        return new RVViewHolder(view);

    }

    @Override
    public void onBindViewHolder(RVViewHolder holder, int position) {
        //holder.infoLL.setAnimation(AnimationUtils.loadAnimation(mContext, R.anim.fade_transition_anim));
        //holder.indicatorLL.setAnimation(AnimationUtils.loadAnimation(mContext, R.anim.fade_transition_anim));

        Item item = itemList.get(position);
        DecimalFormat format = new DecimalFormat("#,###,###,###.##");
        Double value = item.getValue();
        holder.valueTv.setText(format.format(value));
        if (value > 0 || value == 0) {
            Resources res = mContext.getResources();
            //holder.indicatorLL.setBackground(res.getDrawable(R.drawable.item_income_indicator_bg, mContext.getTheme()));
            holder.indicatorLL.setBackgroundTintList(ColorStateList.valueOf(res.getColor(R.color.green, mContext.getTheme())));
        }
        holder.memoTv.setText(item.getMemo().toString());
        holder.categoryTv.setText(item.getCategory().toString());
        holder.dateTv.setText(item.getDate().toString());

        holder.itemLL.setAnimation(AnimationUtils.loadAnimation(mContext, R.anim.fade_transition_anim));

    }

    @Override
    public int getItemCount() {
        return itemList.size();
    }

}

Ответы [ 2 ]

1 голос
/ 01 апреля 2020

Это может быть связано с тем, что, будучи представителем программы повторного просмотра, держатели просмотра перерабатываются, и ваш код меняет цвет только в одном направлении, т.е. на зеленый

try 


Resources res = mContext.getResources();
if (value > 0 || value == 0) {
holder.indicatorLL.setBackgroundTintList(ColorStateList.valueOf(res.getColor(R.color.green, mContext.getTheme())));
} else {
holder.indicatorLL.setBackgroundTintList(ColorStateList.valueOf(res.getColor(R.color.red, mContext.getTheme())));
}
1 голос
/ 01 апреля 2020

Вы должны добавить else statement, потому что представления перезапускаются при прокрутке 

  if (value > 0 || value == 0) {

 holder.indicatorLL.setBackgroundTintList(ColorStateList.valueOf(res.getColor(R.color.green, mContext.getTheme())));
  } else {
   // Set default color
holder.indicatorLL.setBackgroundTintList(ColorStateList.valueOf(res.getColor(R.color.red, mContext.getTheme())));
  }
...