Как устранить некорректное отображение избранных элементов в recyclerview?

Question

[Орхан Гасанлы]

Доброго времени суток!

Ниже указан код, который я использую для добавления или удаления из избраного

imageButton = (ImageButton) findViewById(R.id.favorite_button);
//                if(db.isFavorite(valueOf(medicationList.get(position).get_id())))
//                    imageButton.setImageResource(R.drawable.ic_add_to_fav);

                // Ловим клик по кнопке "Добавить в избранное"
                holder.itemView.findViewById(R.id.favorite_button).setOnClickListener(new View.OnClickListener() {
                    @Override
                    public void onClick(View v) {

                        if(!db.isFavorite(valueOf(medicationList.get(position).get_id()))) {
                            db.addToFavorites(valueOf(medicationList.get(position).get_id()));
                            imageButton.setImageResource(R.drawable.ic_add_to_fav);
                            
                        } else {
                            db.removeFromFavorites(valueOf(medicationList.get(position).get_id()));
                            imageButton.setImageResource(R.drawable.ic_remove_from_fav);
                            
                        }

                    }
                });

Сама функция записи и удаления ид выбранного элемента в БД отрабатывает корректно, но проблема с ImageButton. Независимо, от того какой элемент был выбран всегда кнопка выделяется "избранной" только для первого элемента)))))

Пробовал, как medicationList.get(position).get_id()так и

medicationList.get(holder.getAdapterPosition()).get_id()

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

И еще попутный вопрос:
имеется небольшая строка кода, которая по идее, должна в методе onCreate отметить все элементы, которые являются избранными.

//                if(db.isFavorite(valueOf(medicationList.get(position).get_id())))
//                    imageButton.setImageResource(R.drawable.ic_add_to_fav);

Но тоже как-то не отрабатывает(

Буду признателен за помощь или за подсказку

Comments

[Модератор]

azerphoenix дорогой пользователь, настоятельно рекомендуем еще раз обратить самое пристальное внимание на п. 3.1 регламента работы сервиса (и, в особенности, на его последний абзац). В противном случае, ваши вопросы будут удаляться по причине тег-спама, а систематические нарушения приведут к блокировке учетной записи.

[Орхан Гасанлы]

Модератор, благодарю за информацию. Что-то я вчера не нашел тега "android development" вот и выбрал вместо него android studio)

Answer 1

[Алексей Перелыгин]

хм, ребята, не работайте с установкой onClickListener в onBindViewHolder это раз. Во вторых, если уж хотите работать - выносите его в холдер, а данные, по которым должен работать клик меняйте просто в onBindViewHolder. В третьих, холдер предназначен для того, в том числе, чтобы единожды в конструкторе найти все значимые поля и связать их с версткой, а не вызывать каждый раз в onBindViewHodler findViewById. Не нужно работать в RecyclerView с позицией. Нужно работать с данными в холдере.

Comments

[Орхан Гасанлы]

Здравствуйте! Благодарю за информацию)
В данном случае слушатель находится не в методе onBindViewHolder. Правда, раньше он действительно там располагался.
На данный момент я создал интерфейс ClickListener и класс RecyclerTouchListener. Не скажу насколько правильно я это сделал))

RecyclerTouchListener

public class RecyclerTouchListener implements RecyclerView.OnItemTouchListener {

    private GestureDetector gestureDetector;
    private ClickListener clickListener;

    public RecyclerTouchListener(Context context, final RecyclerView recyclerView, final ClickListener clickListener) {
        this.clickListener = clickListener;
        gestureDetector = new GestureDetector(context, new GestureDetector.SimpleOnGestureListener() {
//            @Override
//            public boolean onSingleTapUp(MotionEvent e) {
//                return true;
//            }

            @Override
            public boolean onDown(MotionEvent e) {
                return true;
            }


            @Override
            public void onLongPress(MotionEvent e) {
                View child = recyclerView.findChildViewUnder(e.getX(), e.getY());
                if (child != null && clickListener != null) {
                    clickListener.onLongClick(child, recyclerView.getChildAdapterPosition(child));
                }
            }
        });
    }

    @Override
    public boolean onInterceptTouchEvent(RecyclerView rv, MotionEvent e) {

        View child = rv.findChildViewUnder(e.getX(), e.getY());
        if (child != null && clickListener != null && gestureDetector.onTouchEvent(e)) {
            clickListener.onClick(child, rv.getChildPosition(child));
        }
        return false;
    }

    @Override
    public void onTouchEvent(RecyclerView rv, MotionEvent e) {
    }

    @Override
    public void onRequestDisallowInterceptTouchEvent(boolean disallowIntercept) {

    }

    public interface ClickListener {
        void onClick(View view, int position);

        void onLongClick(View view, int position);
    }
}

Первоисточник:
https://medium.com/@harivigneshjayapalan/android-r...

Код, который я указал выше в вопросе располагается в методе onCreate MainActivity

[Алексей Перелыгин]

ну норм. Но пару принципов SOLID нарушено. И снова вы не работаете с данными а с представлениями. Представление должно изменять состояние в onBindViewHolder а меняться должны данные с последующим вызовом onNotifyItemChanged

Answer 2

[Юрий]

хм, я бы посоветовал вам изучить как анонимные классы захватывают переменные - например тут.
причина ошибки - вы ипользуете position, которые передается параметром метода onBindViewHolder. тут стоит использовать holder.getAdapterPosition() внутри анонимного класса-обработчика нажатия.

Comments

[Орхан Гасанлы]

Большое спасибо! Надо будет изучить)

Answer 3

[davidnum95]

notifyDataSetChanged или notifyItemChanged ?

Comments

[Орхан Гасанлы]

Благодарю за ответ. Надо будет изучить, но по моему это не совсем то.