2

В приложении используется recyclerwiew для отображения списка контактов. В старой версии приложения один раз и на всю жизнь качались 300мб фотографий пользователей и потом в OnBindViewHolder просто доставались из базы данных. Никаких проблем в отображении (в recyclerview) не возникало.

Сейчас же фотографии скачиваются из сети по мере необходимости (если ранее не были записаны в БД) их отображения (естественно асинхронно), записываются в БД и результат отображается в нужном айтеме. Всё выглядит прекрасно, но как только происходит scroll списка контактов, их фотографии дублируются и только когда scroll остановится, они исчезают, пересоздаются уже корректно отображаются. То есть recyclerview в момент остановки мигает как новогодняя ёлка. Понятно, что это происходит от того, что recyclerview переиспользует видимые itemview. Проблема решается, если перейти с recyclerview на listview и обязательно установить ему setHasFixedSize(true). Однако, мне просто необходимо использовать recyclerview, а не listview.

Код адаптера:

@Override
public void onBindViewHolderCursor(ContactViewHolder holder, Cursor cursor) {

    String firstName = cursor.getString(cursor.getColumnIndex(DatabaseSchemas.Profile.FIRST_NAME));
    String lastName = cursor.getString(cursor.getColumnIndex(DatabaseSchemas.Profile.LAST_NAME));
    holder.mName.setText(String.format("%s %s", lastName, firstName));

    long profileId = cursor.getInt(cursor.getColumnIndex(DatabaseSchemas.Profile.ID));
    boolean gender = cursor.getString(cursor.getColumnIndex(DatabaseSchemas.Profile.SEX)).equals("M");
    holder.mPhoto.setEmploeePhoto(mContext, profileId, gender, holder.mPhoto);
    //ВНИМАНИЕ ниже старый метод отображения изображений, который работает без проблем
    //ImageUtils.displayUserPhoto(mContext, profileId, gender, holder.mPhoto);

    holder.setHighlightQuery(mQuery);
}

Теперь подробно о новом методе holder.mPhoto.setEmploeePhoto, который как раз либо показывает изображение из базы данных, либо качает из сети, сохраняет в БД и показывает.

public void setEmploeePhoto(final Context context, final long mEmployee_id, final boolean gender, final ImageView imageView) {

    if (ImageUtils.userPhotoIsExist(context, mEmployee_id)){

        ImageUtils.displayUserPhoto(context, mEmployee_id, gender, imageView);
//ВНИМАНИЕ, это тот самый метод отображения фото, 
//который был закомментирован в OnBindViewHolder, 
//и ещё ниже он снова используется уже после окончания загрузки и записи в БД. 
//Но только пока он работал в OnBindViewHolder, фото прекрасно отображались при скроллинге, 
//а вот здесь появляется проблема с дублированием и перемигиванием новогодней ёлкой

    }
    else {

        JSONObject filter = new JSONObject();
        JSONArray values = new JSONArray();
        values.put(String.valueOf(mEmployee_id));
        try {
            filter.put("values", values);
            filter.put("field", "user_id");
        } catch (JSONException e) {
            e.printStackTrace();
            return;
        }
        new SapApi(getContext()).getPhoto(filter)
                .subscribeOn(Schedulers.io())
                .observeOn(AndroidSchedulers.mainThread())
                .subscribe(new Action1<InputStream>() {
                    @Override
                    public void call(final InputStream inputStream) {
//Если фото успешно скачалось из сети
                        new LoadAndWritePhoto(context,mEmployee_id, gender, imageView, inputStream).execute();

                    }
                }, new Action1<Throwable>() {
                    @Override
                    public void call(Throwable throwable) {
                        Toast.makeText(getContext(), throwable.getMessage(), Toast.LENGTH_LONG).show();
                    }
                });

    }

}

public class LoadAndWritePhoto extends AsyncTask<Void, Void, Void> {//Потому что нельзя записывать данные в БД в главном потоке
    Context mmContext;
    long profileId;
    boolean gender;
    ImageView mImageView;
    InputStream inputStream;

    public LoadAndWritePhoto(Context mmContext, long profileId, boolean gender, ImageView mImageView, InputStream inputStream) {
        this.mmContext = mmContext;
        this.profileId = profileId;
        this.gender = gender;
        this.mImageView = mImageView;
        this.inputStream = inputStream;
    }

    @Override
    protected void onPreExecute() {
        super.onPreExecute();
    }

    @Override
    protected Void doInBackground(Void... params) {//фоновая логика
        {
            try {
                PhotosSdCardWriter.writePhotosFromJson(mmContext, inputStream);
            } catch (Exception e){
                e.printStackTrace();
            }
        }
        return null;
    }

    @Override
    protected void onProgressUpdate(Void... values) {//обновление интерфейса до окончания выполнения потока (не требуется)
        super.onProgressUpdate(values);
    }

    @Override
    protected void onPostExecute(Void aVoid) {//результат фоновой логики
        super.onPostExecute(aVoid);
        try {
            ImageUtils.displayUserPhoto(mmContext, profileId, gender, mImageView);
        }catch (Exception e){
            e.printStackTrace();
        }
    }
}
5
  • а можете код показать onBindViewHolder
    – miha_dev
    10 ноя 2016 в 8:40
  • @miha_dev добавил код и комментарии внутри него, посмотрите, пожалуйста 10 ноя 2016 в 10:08
  • 1
    Попробуйте занулить изображение ImageView перед добавлением в него нового изображения - mImageView.setImageDrawable(null).
    – Вадим
    10 ноя 2016 в 13:15
  • @Вадим сбой в отображении стал происходить немного реже, но это не решило проблему =( 10 ноя 2016 в 14:37
  • А метод onBindViewHolder сколько раз вызывается на каждое представление? Один?
    – Вадим
    10 ноя 2016 в 15:29

1 ответ 1

2

Вам нужно переопределить метод getItemId(int position) в адаптере.

public long getItemId(int position) {
    return position;
}

После этого указать адаптеру что у него для каждой ячейки используется свой ID так:

adapter.setHasStableIds(true);

Теперь ячейки не будут переиспользоваться

3
  • Переопределил этот метод, указав свой список объектов. Указал сразу после инициализации адаптера setHasStableIds(true); Но проблему это не решило =( Мне помогает только holder.setIsRecyclable(false); но это плохое решение проблемы, потому что нарушает концепцию RecyclerView и тормозит скролл. 10 ноя 2016 в 14:21
  • @VadimKravchenko, попробуйте отображать картинки не самописным методом, а чрез либы типа Picasso/Glide
    – ЮрийСПб
    10 ноя 2016 в 17:30
  • пробовал обе эти библиотеки, но тоже не дало никаких результатов =( Причём, у меня два разных проекта с этой же проблемой (один писался не мной, а второй соответственно мной со стандартным Recycler Adapter и стандартным View Holder), и я на каждом пробовал всё, что мне посоветовали в комментариях. 11 ноя 2016 в 8:41

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.