Невозможно передать данные из адаптера RecyclerView в адаптер Viewpager - PullRequest
Новая
       18

Невозможно передать данные из адаптера RecyclerView в адаптер Viewpager

1 голос
/ 18 апреля 2019

Я создаю простое приложение для создания галереи изображений, в котором изображения отображаются в MainActivity внутри RecyclerView, и по щелчку на любом изображении оно приведет нас к SlideActivity, в котором я хочу показать изображения в ViewPager, а затем проведем пальцем по всем изображениям, которые должны быть показаны на ViewPager. Для получения изображений я использую Glide. Изображения прекрасно загружаются в RecyclerView, но когда я нажимаю на любое изображение, я получаю это исключение «java.lang.ArrayIndexOutOfBoundsException: length = 15; index = -1». Я знаю, что есть много таких же вопросов, как этот, я видел все коды, но не понял, поэтому, наконец, решил задать вопрос: я вставляю код, что бы я ни пытался до сих пор. Любые предложения или советы будут полезны, потому что я работаю один, у меня нет наставника, и я учусь. Спасибо!

Ошибка: 

java.lang.ArrayIndexOutOfBoundsException: length=15; index=-1
        at java.util.ArrayList.get(ArrayList.java:439)
        at com.starmoonsolutions.imagegallery.RecycleViewAdapter$ViewHolder$1.onClick(RecycleViewAdapter.java:63)
        at android.view.View.performClick(View.java:6256)
        at android.view.View$PerformClick.run(View.java:24697)
        at android.os.Handler.handleCallback(Handler.java:789)
        at android.os.Handler.dispatchMessage(Handler.java:98)
        at android.os.Looper.loop(Looper.java:164)
        at android.app.ActivityThread.main(ActivityThread.java:6541)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.Zygote$MethodAndArgsCaller.run(Zygote.java:240)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:767)

ImageUrl.java 

public class ImageUrl {
    String imageUrl;

    public String getImageUrl() {
        return imageUrl;
    }

    public void setImageUrl(String imageUrl) {
        this.imageUrl = imageUrl;
    }
}

MainActivity: 

public class MainActivity extends AppCompatActivity {

    private ImageView imageView;
    RecyclerView recyclerView;
    GridLayoutManager gridLayoutManager;
    ArrayList<ImageUrl> imageUrlList;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        imageView = findViewById(R.id.imageView);
        recyclerView = findViewById(R.id.recyclerView);
        gridLayoutManager = new GridLayoutManager(getApplicationContext(), 2);
        recyclerView.setLayoutManager(gridLayoutManager);

        imageUrlList = prepareData();
        RecycleViewAdapter galleryAdapter = new RecycleViewAdapter(getApplicationContext(), imageUrlList);
        recyclerView.setAdapter(galleryAdapter);
    }
    private ArrayList prepareData(){
        // here you should give your image URLs and that can be a link from the Internet
        String imageUrls[] = {
                "https://images.pexels.com/photos/358457/pexels-photo-358457.jpeg?auto=compress&cs=tinysrgb&dpr=2&h=750&w=1260",
                "https://images.pexels.com/photos/934964/pexels-photo-934964.jpeg?auto=compress&cs=tinysrgb&dpr=2&h=650&w=940",
                "https://images.pexels.com/photos/1202887/pexels-photo-1202887.jpeg?auto=compress&cs=tinysrgb&h=650&w=940",
                "https://images.pexels.com/photos/337909/pexels-photo-337909.jpeg?auto=compress&cs=tinysrgb&dpr=2&h=650&w=940",
                "https://images.pexels.com/photos/894443/pexels-photo-894443.jpeg?auto=compress&cs=tinysrgb&dpr=2&h=650&w=940",
                "https://images.pexels.com/photos/951325/pexels-photo-951325.jpeg?auto=compress&cs=tinysrgb&dpr=2&h=650&w=940",
                "https://images.pexels.com/photos/1534411/pexels-photo-1534411.jpeg?auto=compress&cs=tinysrgb&dpr=2&h=650&w=940",
                "https://images.pexels.com/photos/266436/pexels-photo-266436.jpeg?auto=compress&cs=tinysrgb&dpr=2&h=650&w=940",
                "https://images.pexels.com/photos/1894350/pexels-photo-1894350.jpeg?auto=compress&cs=tinysrgb&dpr=2&h=650&w=940",
                "https://images.pexels.com/photos/87009/earth-soil-creep-moon-lunar-surface-87009.jpeg?auto=compress&cs=tinysrgb&dpr=2&h=650&w=940",
                "https://images.pexels.com/photos/1820563/pexels-photo-1820563.jpeg?auto=compress&cs=tinysrgb&dpr=2&h=650&w=940",
                "https://images.pexels.com/photos/1274260/pexels-photo-1274260.jpeg?auto=compress&cs=tinysrgb&dpr=2&h=650&w=940"
        };
        imageUrlList = new ArrayList<>();
        for (int i = 0; i < imageUrls.length; i++) {
            ImageUrl imageUrl = new ImageUrl();
            imageUrl.setImageUrl(imageUrls[i]);
            imageUrlList.add(imageUrl);
        }
        Log.d("MainActivity", "List count: " + imageUrlList.size());
        return  imageUrlList;
    }
}

RecyclerViewAdapter: 

    public class RecycleViewAdapter extends RecyclerView.Adapter<RecycleViewAdapter.ViewHolder>{

        private ArrayList<ImageUrl> imageUrls;
        private Context context;

        public RecycleViewAdapter(Context context, ArrayList<ImageUrl> imageUrls) {
            this.imageUrls = imageUrls;
            this.context = context;
        }

        @NonNull
        @Override
        public ViewHolder onCreateViewHolder(@NonNull ViewGroup viewGroup, int i) {
            View view = LayoutInflater.from(viewGroup.getContext()).inflate(R.layout.image_layout, viewGroup, false);
            return new ViewHolder(view);
        }

        /**
         * gets the image url from adapter and passes to Glide API to load the image
         *
         * @param viewHolder
         * @param i
         */
        @Override
        public void onBindViewHolder(@NonNull ViewHolder viewHolder, int i) {
            Glide.with(context).load(imageUrls.get(i).getImageUrl()).into(viewHolder.img);
        }

        @Override
        public int getItemCount() {
            return imageUrls.size();
        }

        public class ViewHolder extends RecyclerView.ViewHolder{

            RecyclerView recyclerView;
            ImageView img;

            public ViewHolder(@NonNull final View itemView) {
                super(itemView);
                img = itemView.findViewById(R.id.imageView);
                recyclerView = itemView.findViewById(R.id.recyclerView);
                final int pos = getAdapterPosition();
                itemView.setOnClickListener(new View.OnClickListener() {
                    @Override
                    public void onClick(View v) {
                        Intent intent = new Intent(context, SlideActivity.class);
error point here->      intent.putExtra("image", String.valueOf(imageUrls.get(pos).getImageUrl())); <- error point here 
                        intent.putExtra("position", pos);
                        intent.putExtra("list", imageUrls.get(pos).imageUrl);
                        context.startActivity(intent);
                    }
                });
            }
        }
    }

SlideActivity (ViewPager): 

public class SlideActivity extends AppCompatActivity {
    private ViewPager viewPager;
    private Context context = SlideActivity.this;
    private ViewPagerAdapter adapter;
    private ArrayList<String> list;
    String image;
    private int position;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.slide_activity);

        viewPager = findViewById(R.id.viewPager);
        getIntentInfo();

        adapter = new ViewPagerAdapter(list,context);
        viewPager.setAdapter(adapter);
        viewPager.setCurrentItem((position));
    }

    private void getIntentInfo() {
        image = getIntent().getStringExtra("image");
        list  = getIntent().getStringArrayListExtra("list");
        position = getIntent().getIntExtra("position",0);
    }
}

ViewPagerAdapter: 

public class ViewPagerAdapter extends PagerAdapter {
    private ArrayList<String> imageUrls;
    private android.content.Context context;
    private LayoutInflater inflater;
    private ImageView wallpaper;

    public ViewPagerAdapter() {

    }

    public ViewPagerAdapter(ArrayList<String> imageUrls, android.content.Context context) {
        this.imageUrls = imageUrls;
        this.context = context;
    }

    @Override
    public int getCount() {
        return imageUrls.size();
    }

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

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

        inflater = LayoutInflater.from(container.getContext());
        View view = inflater.inflate(R.layout.pager_item, container, false);
        wallpaper = view.findViewById(R.id.walpaperImage);
        Glide.with(context).asBitmap().load(imageUrls.get(position)).into(wallpaper);
        container.addView(view, 0);
        return view;
    }

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

1 Ответ

0 голосов
/ 18 апреля 2019

Проблема на самом деле в получении позиции адаптера. 

public ViewHolder(@NonNull final View itemView) {
                super(itemView);
                img = itemView.findViewById(R.id.imageView);
                recyclerView = itemView.findViewById(R.id.recyclerView);

                // remove the below line
                //final int pos = getAdapterPosition();

                itemView.setOnClickListener(new View.OnClickListener() {
                    @Override
                    public void onClick(View v) {
                        Intent intent = new Intent(context, SlideActivity.class);

                        // you need to freshly get the adapter position here.
                        int pos = getAdapterPosition();

error point here->      intent.putExtra("image", String.valueOf(imageUrls.get(pos).getImageUrl())); <- error point here 
                        intent.putExtra("position", pos);
                        intent.putExtra("list", imageUrls.get(pos).imageUrl);
                        context.startActivity(intent);
                    }
                });
            }

Поскольку ViewHolder перерабатывается, положение адаптера может меняться, поэтому вам необходимо установить самое последнее положение адаптера в прослушивателе щелчков.

А что касается данных Bundle, передайте список URL-адресов типа String вместо пользовательского объекта, 

public void onClick(View v) { 
Intent intent = new Intent(context, SlideActivity.class); 
error point here-> intent.putExtra("image", String.valueOf(imageUrls.get(pos).getImageUrl())); <- error point here 
intent.putExtra("position", pos); 

// imageUrls is of type ArrayList<String>
intent.putStringArrayListExtra("list", imageUrls);

context.startActivity(intent); }

или используйте библиотеку Parcelable для сериализации пользовательского объекта.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.

Похожие темы

...