Как повесить обработчик нажатия на элемент RecyclerView?

Question

[Константин Липатов]

Всем привет, у меня есть NavigationDraweerActivity в которой выводиться Фрагмент с ViewPager, в котором я вывожу фрагмент с recycler view + card view. В адаптере для recyclerView у viewholder'а есть метод onClick, в самом адаптере реализован интерфейс OnItemClickListener с методом onItemClick. Также у адаптера есть метод SetOnItemClickListener, чтобы повесить обработчик на cardview.

class myViewHolder extends RecyclerView.ViewHolder implements View.OnClickListener {

        private CardView cv;
        private ImageView commandImageView;
        private TextView commandTextView;

        public myViewHolder(View itemView) {
            super(itemView);
            cv = itemView.findViewById(R.id.cv);
            commandImageView = itemView.findViewById(R.id.commandImageView);
            commandTextView = itemView.findViewById(R.id.commandTextView);

            cv.setOnClickListener(this);
        }
        @Override
        public void onClick(View v) {
            if (mItemClickListener != null) {
                mItemClickListener.onItemClick(v, getAdapterPosition());
            }
        }
    }
public interface OnItemClickListener {
        public void onItemClick(View view , int position);
    }
    public void SetOnItemClickListener(final OnItemClickListener mItemClickListener) {
        this.mItemClickListener = mItemClickListener;
    }

В коде, когда я создаю recyclerVIew и адаптер для него, я реализую метод onClick

recyclerView = view.findViewById(R.id.recyclerView);
        recyclerView.setLayoutManager(new LinearLayoutManager(getContext()));

        recViewAdapter = new RecViewAdapter(get_data(), getContext());
        recyclerView.setAdapter(recViewAdapter);

        recViewAdapter.SetOnItemClickListener(new RecViewAdapter.OnItemClickListener() {
            @Override
            public void onItemClick(View view, int position) {
                //Мое действие
            }
        });

Но из-за особенности view pager (он создает текущую страницу и следующую) при создании следующей страницы переопределяется обработчик для текущей страницы. Как я понял, это потому что элементы PageVIew ссылаются на один и тот же recycler view. Как это можно исправить?

Answer 1

[Евгений]

public class TransportAdapter extends RecyclerView.Adapter<TransportAdapter.TransportViewHolder> {

    public interface onClickListener{
        void onVariantClick(TransportModel model);
    }
    
    private onClickListener listener;
   
   .....
    @Override
    public void onBindViewHolder(@NonNull TransportViewHolder holder, int position) {
        final TransportModel transportModel = transportVariants.get(position);
        ....
        if(listener != null){
            holder.itemView.setOnClickListener(new View.OnClickListener() {
                @Override
                public void onClick(View view) {
                    listener.onVariantClick(transportModel);
                }
            });
        }
    }
   .... бла бла бла

    public void setListener(onClickListener listener){
        this.listener = listener;
    }

    class TransportViewHolder extends RecyclerView.ViewHolder {
       .....бла бла бла
        public TransportViewHolder(View itemView) {
            super(itemView);
            ButterKnife.bind(this, itemView);
        }
    }
}

Answer 2

[Ivan0206]

Как то сложно ты делаешь.
Тебе надо обрабатывать нажатие card view?
Если да, то имплементируешь OnClickListener
реализуешь метод OnClick в экземпляре RecyclerView.ViewHolder и ловишь нажатие card view через его id.
Соответственно в конструкторе Холдера инициализируешь сам card view

Comments

[Константин Липатов]

Совершенно верно, мне нужно обрабатывать нажатие на card view. Получается нечто подобное? И подойдет ли данный способ, если мне нужно открывать активити по нажатию на card view?

class myViewHolder extends RecyclerView.ViewHolder implements 
        View.OnClickListener {

        private CardView cv;
        private ImageView commandImageView;
        private TextView commandTextView;

        public myViewHolder(View itemView) {
            super(itemView);
            cv = itemView.findViewById(R.id.cv);
            commandImageView = itemView.findViewById(R.id.commandImageView);
            commandTextView = itemView.findViewById(R.id.commandTextView);


            cv.setOnClickListener(this);
        }

        @Override
        public void onClick(View v) {
            //мое действие
        }
    }

[Ivan0206]

Да, у меня сделано так. Активити через Intent запускаешь. Единственное, cv.setOnClickListener(this), тут надо context передавать, я его передавал через конструктор, если не путаю. Сейчас нет возможности свои код открыть. Попробуй getActivity().getContext() или через конструктор Адаптера прокидывать

[Константин Липатов]

Ivan0206, Ок, сейчас попробую, спасибо!

[Денис Загаевский]

Константин Липатов, идея в целом правильная, но никакого твоего кода внутри быть не должно. Ниже тебе написали более нормальное решение

[Константин Липатов]

Денис Загаевский, В таком случае, если я не ошибаюсь то мой текущий вариант реализует именно способ описанный ниже, но проблема в другом. PageView создает сразу 2 экземпляра recycler view(для текущей страницы и следующей), при создании следующей страницы переопределяется метод onClick для первой страницы и в итоге клик на элемент первой страницы отображает контент, который должен быть показан по клику на элемент второй страницы.

[Евгений]

Константин Липатов, PageView сам по себе ничего не создает. он просто возвращает фрагменты в зависимости от позиции. у нас на всех позициях один и тот же экземпляр франгмента?

[Константин Липатов]

Евгений, Адаптер PageView создает сразу 2 страницы*
В PageView я поместил recycler view, и в зависимости от страницы он заполняется соответствующими данными. Под позицией вы подразумеваете страницу pageview?

[Евгений]

Константин Липатов, давайте код пэйджвьюера

[Константин Липатов]

Евгений, Вот код фрагмента с ViewPager

public class ViewPagerFragment extends Fragment{

    static final String TAG = "myLogs";
    ViewPager pager;
    PagerAdapter pagerAdapter;
    int cardview_position;

    @Override
    public void onCreate(@Nullable Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
                             Bundle savedInstanceState) {
        // Inflate the layout for this fragment
        View view = inflater.inflate(R.layout.fragment_view_pager, null);
        Log.d("myLogs", "onCreateView");
        pager = view.findViewById(R.id.pager);

        switch (getArguments().getString("flag")) {
            case "flag_1":pagerAdapter = new MyFragmentPagerAdapter(getActivity().getSupportFragmentManager(), new String[] {
                    "СТРАНИЦА 1",
                    "СТРАНИЦА 2",
                    "СТРАНИЦА 3",
                    "СТРАНИЦА 4"
            },
                    "flag 1");
                break;
            case "flag_2":pagerAdapter = new MyFragmentPagerAdapter(getActivity().getSupportFragmentManager(), new String[] {
                    "СТРАНИЦА 1"
            },
                    "flag_2");
                break;
        }

        pager.setAdapter(pagerAdapter);

        pager.addOnPageChangeListener(new ViewPager.OnPageChangeListener() {

            @Override
            public void onPageSelected(int position) {

            }

            @Override
            public void onPageScrolled(int position, float positionOffset,
                                       int positionOffsetPixels) {
            }

            @Override
            public void onPageScrollStateChanged(int state) {
            }
        });

        return view;
    }

    public int getResFromId (int i, String str_res, String type) {
        int res_id = getResources().getIdentifier(str_res + i, type, getContext().getPackageName());

        return res_id;
    }


    private class MyFragmentPagerAdapter extends FragmentStatePagerAdapter {

        String items[];
        String flag_content;

        public MyFragmentPagerAdapter(FragmentManager fm, String items[], String flag_content) {
            super(fm);
            this.items = items;
            this.flag_content = flag_content;

        }

        @Override
        public Fragment getItem(int position) {
            Log.d("myLogs", "newInstance   pageNumber = " + position);
            return mcMapsFragment.newInstance(position, flag_content);
        }

        @Override
        public int getCount() {
            return items.length;
        }

        @Nullable
        @Override
        public CharSequence getPageTitle(int position) {
            return items[position];
        }
    }

    public void setPosition (int position) {
        this.cardview_position = position;
    }

}

[Константин Липатов]

Евгений, Только мне кажется, что я что-то неправильно делаю, когда вешаю слушателя на элемент recycler view. По идее я должен создавать новый экземпляр recycler view и адаптера для каждой страницы, чтобы у каждой страницы был свой слушатель. Но почему-то слушатель получается один и тот же на все страницы, т.е. при нажатии на элемент 1 страницы и 2, происходит одно и тоже действие. Может я где-то пропустил слово new?

RecyclerView recyclerView = new RecyclerView(getContext());
                recyclerView =  view.findViewById(R.id.recyclerView);
                recyclerView.setLayoutManager(new LinearLayoutManager(getContext()));
                RecViewAdapter recViewAdapter = new RecViewAdapter(get_data(flag_content), getContext());
                recyclerView.setAdapter(recViewAdapter);

                recViewAdapter.SetOnItemClickListener(new RecViewAdapter.OnItemClickListener() {
                    @Override
                    public void onItemClick(View view, int position) {

                        try {
                            Intent intent;
                            intent = new Intent("com.weys.app.mobile.ReadActivity");
                            intent.putExtra("element_number", position + 1);
                            int image_res_id = getResFromId(position + 1, "image_" + (pageNumber + 1) + "_", "drawable");
                            int text_res_id = getResFromId(position + 1, "main_par_" + (pageNumber + 1) + "_", "string");
                            int text_res_id2 = getResFromId(position + 1, "sub_par_" + (pageNumber + 1) + "_", "string");

                            intent.putExtra("image_id", image_res_id);
                            intent.putExtra("text_title", getString(text_res_id));
                            intent.putExtra("text_article", getString(text_res_id2));
                            startActivity(intent);
                        } catch (Exception error) {
                            Log.d("myLogs", "error message: " + error);
                           
                        }
                    }

[Константин Липатов]

Я нашел ошибку, у меня переменная mItemClickListener типа OnItemClickListener была static (не спрашивайте почему :D) и из-за этого на все элементы вешался один и тот же обработчик...