ViewPager не обнаруживает onclicklistener

Question

Я использую видовой пейджер в качестве слайдера изображений.

Моя проблема в том, что onclicklistener не определяет, когда пользователь нажимает на изображение на каждой странице.

Это мой адаптер:

public class SlidingImage_Adapter extends PagerAdapter {


    private String[] urls;
    private LayoutInflater inflater;
    private Context context;


    public SlidingImage_Adapter(Context context, String[] urls) {
        this.context = context;
        this.urls = urls;
        inflater = LayoutInflater.from(context);
    }

    @Override
    public void destroyItem(ViewGroup container, int position, Object object) {
        container.removeView((View) object);
    }

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



    @Override
    public Object instantiateItem(ViewGroup view, final int position) {
        final View imageLayout = inflater.inflate(R.layout.slidingimages_layout, view, false);

        assert imageLayout != null;
        final ImageView imageView = (ImageView) imageLayout
                .findViewById(R.id.image);

        imageLayout.setOnClickListener(new View.OnClickListener(){
            public void onClick(View v){
                //this will log the page number that was click

                Log.d("position","position "+position);//not working
            }
        });


        Glide.with(context)
                .load(urls[position])
                .into(imageView);

        view.addView(imageLayout, 0);

        return imageLayout;
    }

    @Override
    public boolean isViewFromObject(View view, Object object) {
        return view.equals(object);
    }

    @Override
    public void restoreState(Parcelable state, ClassLoader loader) {
    }

    @Override
    public Parcelable saveState() {
        return null;
    }


}

EDIT Код для viewPager в программе recyclerView

   holder.mPager.setAdapter(new SlidingImage_Adapter(context,urls));


        holder.indicator.setViewPager(holder.mPager);



//Set circle indicator radius


        NUM_PAGES = urls.length;

        // Auto start of viewpager
        final Handler handler = new Handler();
        final Runnable Update = new Runnable() {
            public void run() {
                if (currentPage == urls.length) {
                    currentPage = 0;
                }
                holder.mPager.setCurrentItem(currentPage++, true);
            }
        };
        Timer swipeTimer = new Timer();
        swipeTimer.schedule(new TimerTask() {
            @Override
            public void run() {
                handler.post(Update);
            }
        }, 3000, 3000);

        // Pager listener over indicator
        holder.indicator.setOnPageChangeListener(new ViewPager.OnPageChangeListener() {

            @Override
            public void onPageSelected(int position) {
                currentPage = position;


            }

            @Override
            public void onPageScrolled(int pos, float arg1, int arg2) {

            }

            @Override
            public void onPageScrollStateChanged(int pos) {

            }
        });

Answer 1

Хорошо, поэтому в вашем SlidingImage_Adapter мы внесем некоторые изменения для вашего instantiateItem метода.

Попробуйте изменить ваш instantiateItem метод следующим образом:

@Override
public Object instantiateItem(ViewGroup view, final int position) {

//this is the inflater
LayoutInflater inflater = (LayoutInflater) context.getSystemService(Context.LAYOUT_INFLATER_SERVICE);

//this is the view
final View imageLayout = inflater.inflate(R.layout.slidingimages_layout, null);

//this is the imageview in the view 
final ImageView imageView = (ImageView) imageLayout.findViewById(R.id.image);

//use glide to load images into the image views
Glide.with(context)
            .load(urls[position])
            .into(imageView);

//click listener of each image
imageLayout.setOnClickListener(new View.OnClickListener(){
        public void onClick(View v){
            //this will log the page number that was click

            Log.d("position","position "+position); //should work
        }
    });

 //do this now
 ViewPager viewpager = (ViewPager) view;
 viewpager.addView(imageLayout, 0);

 //return the view
 return imageLayout;
}

Также изменитеметод isViewFromObject для этого:

@Override
public boolean isViewFromObject(View view, Object object) {
    return view == object;
}

Также измените метод destroyItem на этот:

@Override
public void destroyItem(ViewGroup container, int position, Object object) {

    ViewPager viewpager = (ViewPager) container;
    viewpager.removeView((View) object);
}

Answer 2

Нет метода обратного вызова OnItemClick для ViewPager. Если вы хотите нажимать на события на каждой странице, вам придется встроить слушателя в содержимое страницы в вашем адаптере. Создайте Inflater экземпляр внутри instantiateItem метода переопределения.

как то так:

    @Override
        public Object instantiateItem(ViewGroup view, final int position) {
LayoutInflater inflater = (LayoutInflater) view.getContext()
            .getSystemService(Context.LAYOUT_INFLATER_SERVICE);

final View imageLayout = inflater.inflate(R.layout.slidingimages_layout, view, false);

            assert imageLayout != null;
            final ImageView imageView = (ImageView) imageLayout
                    .findViewById(R.id.image);

            imageLayout.setOnClickListener(new View.OnClickListener(){
                public void onClick(View v){
                    //this will log the page number that was click

                    Log.d("position","position "+position);//not working
                }
            });


            Glide.with(context)
                    .load(urls[position])
                    .into(imageView);

            view.addView(imageLayout, 0);

            return imageLayout;
        }