Множественные таймеры обратного отсчета во время мерцания RecyclerView при прокрутке
Я реализовал таймер обратного отсчета для каждого элемента RecyclerView, который находится в активности фрагмента. Таймер обратного отсчета показывает время, оставшееся до истечения. Таймер обратного отсчета работает нормально, но при прокрутке вверх начинает мигать. Много искал, но не получил хорошую ссылку. Может кто-нибудь мне помочь?
Это мой адаптер RecyclerView
public class MyOfferAdapter extends RecyclerView.Adapter<MyOfferAdapter.FeedViewHolder>{
private final Context mContext;
private final LayoutInflater mLayoutInflater;
private ArrayList<Transactions> mItems = new ArrayList<>();
private ImageLoader mImageLoader;
private String imageURL;
private View mView;
private String mUserEmail;
public MyOfferAdapter(Context context) {
mContext = context;
mLayoutInflater = LayoutInflater.from(context);
VolleySingleton mVolley = VolleySingleton.getInstance(mContext);
mImageLoader = mVolley.getImageLoader();
}
public void addItems(ArrayList<Transactions> items,String userEmail) {
int count = mItems.size();
mItems.addAll(items);
mUserEmail = userEmail;
notifyItemRangeChanged(count, items.size());
}
@Override
public FeedViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
mView = mLayoutInflater.inflate(R.layout.my_feed_item_layout, parent, false);
return new FeedViewHolder(mView);
}
@Override
public void onBindViewHolder(final FeedViewHolder holder, final int position) {
holder.desc.setText(mItems.get(position).getDescription());//replace by title
holder.scratchDes.setText(mItems.get(position).getScratchDescription());
long timer = mItems.get(position).getTimerExpiryTimeStamp();
Date today = new Date();
final long currentTime = today.getTime();
long expiryTime = timer - currentTime;
new CountDownTimer(expiryTime, 500) {
public void onTick(long millisUntilFinished) {
long seconds = millisUntilFinished / 1000;
long minutes = seconds / 60;
long hours = minutes / 60;
long days = hours / 24;
String time = days+" "+"days" +" :" +hours % 24 + ":" + minutes % 60 + ":" + seconds % 60;
holder.timerValueTimeStamp.setText(time);
}
public void onFinish() {
holder.timerValueTimeStamp.setText("Time up!");
}
}.start();
}
@Override
public int getItemCount() {
return mItems.size();
}
public static class FeedViewHolder extends RecyclerView.ViewHolder {
TextView desc;
TextView scratchDes;
TextView timerValueTimeStamp;
ImageView feedImage;
CardView mCv;
public FeedViewHolder(View itemView) {
super(itemView);
mCv = (CardView) itemView.findViewById(R.id.cv_fil);
desc = (TextView) itemView.findViewById(R.id.desc_tv_fil);
feedImage = (ImageView) itemView.findViewById(R.id.feed_iv_fil);
scratchDes = (TextView) itemView.findViewById(R.id.tv_scratch_description);
timerValueTimeStamp = (TextView) itemView.findViewById(R.id.tv_timer_value_time_stamp);
}
}
И это мой XML файл, используемый в адаптере
<LinearLayout
xmlns:android="http://schemas.android.com/apk/res/android"
android:layout_width="match_parent"
android:layout_height="wrap_content">
<android.support.v7.widget.CardView xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:app="http://schemas.android.com/apk/res-auto"
android:id="@+id/cv_fil"
android:layout_width="match_parent"
android:layout_height="match_parent"
android:layout_margin="@dimen/card_margin"
android:layout_gravity="center"
app:cardUseCompatPadding="true"
app:cardElevation="4dp"
android:elevation="6dp">
<RelativeLayout
android:layout_width="match_parent"
android:layout_height="match_parent">
<ImageView
android:id="@+id/feed_iv_fil"
android:layout_width="match_parent"
android:layout_height="200dp"
android:layout_alignParentTop="true"
android:scaleType="fitXY"
android:tint="@color/grey_tint_color" />
<TextView
android:id="@+id/tv_scratch_description"
style="@style/ListItemText"
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:gravity="center"
android:text="casul shoes"
android:fontFamily="sans-serif-light"
android:padding="10dp" />
<TextView
android:id="@+id/tv_timer_value_time_stamp"
style="@style/CardTitle"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_centerInParent="true"
/>
<TextView
android:id="@+id/desc_tv_fil"
style="@style/VendorNameText"
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:layout_below="@id/feed_iv_fil"
android:textColor="#3f3e3f"
android:padding="10dp"
/>
</RelativeLayout>
</android.support.v7.widget.CardView>
А это скриншот моего RecyclerView ![enter image description here]()
Ответы
Ответ 1
Эта проблема проста.
RecyclerView
повторно использует держатели, каждый раз вызывая bind для обновления данных в них.
Поскольку вы создаете CountDownTimer
каждый раз, когда какие-либо данные связаны, у вас будет несколько таймеров, обновляющих один и тот же ViewHolder
.
Лучше всего было бы переместить CountDownTimer
в FeedViewHolder
в качестве ссылки, отменить его перед привязкой данных (если они начались) и перенастройкой на желаемую продолжительность.
public void onBindViewHolder(final FeedViewHolder holder, final int position) {
...
if (holder.timer != null) {
holder.timer.cancel();
}
holder.timer = new CountDownTimer(expiryTime, 500) {
...
}.start();
}
public static class FeedViewHolder extends RecyclerView.ViewHolder {
...
CountDownTimer timer;
public FeedViewHolder(View itemView) {
...
}
}
Таким образом, вы отмените любой текущий экземпляр таймера для этого ViewHolder
до запуска другого таймера.
Ответ 2
Как сказал Андрей Лупса, вы должны держать ссылку CountDownTimer в вашем ViewHolder, если вы не хотите сбрасывать таймер при прокрутке (onBindViewHolder), вы должны проверить, является ли ссылка CountDownTimer нулевой или нет в onBindViewHolder:
public void onBindViewHolder(final FeedViewHolder holder, final int position) {
...
if (holder.timer == null) {
holder.timer = new CountDownTimer(expiryTime, 500) {
...
}.start();
}
}
public static class FeedViewHolder extends RecyclerView.ViewHolder {
...
CountDownTimer timer;
public FeedViewHolder(View itemView) {
....
}
}
Ответ 3
Есть два типа решений, о которых я могу думать, и они не используют класс CountDownTimer
- Сделайте Handle с методом
postDelayed
и в этом вызовите notifyDataSetChanged()
. В вашем адаптере сделайте расчет по времени. как ниже код.
в конструкторе класса адаптера
final Handler handler = new Handler();
handler.postDelayed(new Runnable() {
@Override
public void run() {
notifyDataSetChanged();
handler.postDelayed(this, 1000);
}
}, 1000);
а в тебе метод onBindViewHolder
public void onBindViewHolder(final FeedViewHolder holder, final int position) {
updateTimeRemaining(endTime, holder.yourTextView);
}
private void updateTimeRemaining(long endTime, TextView yourTextView) {
long timeDiff = endTime - System.currentTimeMillis();
if (timeDiff > 0) {
int seconds = (int) (timeDiff / 1000) % 60;
int minutes = (int) ((timeDiff / (1000 * 60)) % 60);
int hours = (int) ((timeDiff / (1000 * 60 * 60)) % 24);
yourTextView.setText(MessageFormat.format("{0}:{1}:{2}", hours, minutes, seconds));
} else {
yourTextView.setText("Expired!!");
}
}
-
если вы думаете, что notifyDataSetChanged()
каждую миллисекунду неверна, то здесь есть второй вариант. Создать класс с реализацией Runnable
и использовать его в адаптере с getTag()
setTag()
и getTag()
public class DownTimer implements Runnable {
private TextView yourTextView;
private long endTime;
private DateFormat formatter = new SimpleDateFormat("HH:mm:ss", Locale.getDefault());
private Handler handler = new Handler();
public DownTimer(long endTime, TextView textView) {
this.endTime = endTime;
yourTextView = textView;
formatter.setTimeZone(TimeZone.getTimeZone("UTC"));
}
public void setEndTime(long endTime) {
this.endTime = endTime;
}
public void start() {
if (handler != null)
handler.postDelayed(this, 0);
}
public void cancel() {
if (handler != null)
handler.removeCallbacks(this);
}
@Override
public void run() {
if (handler == null)
return;
if (yourTextView == null && endTime == 0)
return;
long timeDiff = endTime - System.currentTimeMillis();
try {
Date date = new Date(timeDiff);
yourTextView.setText(formatter.format(date));
}catch (Exception e){e.printStackTrace();}
}
}
и использовать в onBindViewHolder
как это
if (holder.yourTextView.getTag() != null) {
DownTimer downTimer = (DownTimer) holder.yourTextView.getTag();
downTimer.cancel();
downTimer.setEndTime(endTime);
downTimer.start();
} else {
DownTimer downTimer = new DownTimer(endTime, holder.yourTextView);
downTimer.start();
holder.yourTextView.setTag(downTimer);
}