java.lang.ArrayIndexOutOfBoundsException: length = 10; index = -1 из базы данных OnChildRemoved Firebase

Question

Я пытаюсь удалить данные из моего маклера и обновить их в мобильном интерфейсе. Это приложение android . Я могу удалить данные в порядке, но мне всегда нужно покидать эту страницу и возвращаться после внесения изменений. Я попытался внести некоторые изменения в метод onChildRemoved , но я начал получать сбои после внесения этого изменения. Что я делаю? Ниже мой код.

MainActivity.java

DatabaseReference databaseReference;
List<DataSnapshot> listData;
RecyclerView recyclerView;
MyCart.MyAdapter adapter;


recyclerView = findViewById(R.id.recyclerview);
recyclerView.setLayoutManager(new LinearLayoutManager(this));
recyclerView.setHasFixedSize(true);

listData = new ArrayList<>();
adapter = new MyCart.MyAdapter(listData);
adapter.setHasStableIds(true);

GetDataFirebase();

private void GetDataFirebase() {
    databaseReference = FirebaseDatabase.getInstance().getReference()
            .child("Customers")
            .child(FirebaseAuth.getInstance().getCurrentUser().getUid()).child("My Cart");
    databaseReference.addChildEventListener(new ChildEventListener() {
        @Override
        public void onChildAdded(DataSnapshot dataSnapshot, String s) {

            Items students = dataSnapshot.getValue(Items.class);

            listData.add(dataSnapshot);

            recyclerView.setAdapter(adapter);

        }

        @Override
        public void onChildChanged(@NonNull DataSnapshot dataSnapshot, String s) {

        }

        @Override
        public void onChildRemoved(@NonNull DataSnapshot dataSnapshot) {
            int index = listData.indexOf(dataSnapshot);
            listData.remove(index);
            adapter.notifyDataSetChanged();
        }

        @Override
        public void onChildMoved(@NonNull DataSnapshot dataSnapshot, String s) {

        }

        @Override
        public void onCancelled(@NonNull DatabaseError databaseError) {

        }
    });

Адаптер класса

public class MyAdapter extends RecyclerView.Adapter<MyCart.MyAdapter.ViewHolder> {

    List<DataSnapshot> list;

    public MyAdapter(List<DataSnapshot> List) {
        this.list = List;
    }

    @Override
    public void onBindViewHolder(MyCart.MyAdapter.ViewHolder holder, int position) {
        final DataSnapshot studentSnapshot = list.get(position);

        final Items students = studentSnapshot.getValue(Items.class);

        final String list_user_id = studentSnapshot.getKey();

        holder.item_name.setText(students.getItem_name());
        holder.item_price.setText(students.getItem_price());

        Picasso.with(holder.item_image.getContext()).load(students.getItem_image()).placeholder(R.drawable.no_image_two).into(holder.item_image);

        holder.delete.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                DatabaseReference prod_ref = FirebaseDatabase.getInstance().getReference().child("Customers").child(FirebaseAuth.getInstance().getCurrentUser().getUid()).child("My Cart").child(list_user_id);
                prod_ref.removeValue();
                adapter.notifyDataSetChanged();
                Toast.makeText(MyCart.this, "Item removed from cart", Toast.LENGTH_SHORT).show();
            }
        });

    }

    @NonNull
    @Override
    public MyCart.MyAdapter.ViewHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {

        View view = LayoutInflater.from(parent.getContext()).inflate(R.layout.cart_item_layout, parent, false);

        return new MyCart.MyAdapter.ViewHolder(view);
    }

    public class ViewHolder extends RecyclerView.ViewHolder {

        TextView item_name, item_price;
        ImageView item_image;
        ImageView delete;

        public ViewHolder(View itemView) {
            super(itemView);

            item_name = itemView.findViewById(R.id.item_name);
            item_price = itemView.findViewById(R.id.item_price);

            item_image = itemView.findViewById(R.id.item_image);

            delete = itemView.findViewById(R.id.delete_item);

        }

    }

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

    @Override
    public long getItemId(int position) {
        return position;

    }

    @Override
    public int getItemViewType(int position) {
        return position;
    }
}

Answer 1

holder.delete.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            DatabaseReference prod_ref = FirebaseDatabase.getInstance().getReference().child("Customers").child(FirebaseAuth.getInstance().getCurrentUser().getUid()).child("My Cart").child(list_user_id);
            prod_ref.removeValue();

            //add/modify these two lines here 
            list.remove(position);
            notifyDataSetChanged();


        }
    });

И еще одно изменение, которое вам нужно сделать // переместить этот тост в метод onChildRemoved и очистить другой код оттуда

Toast.makeText(MyCart.this, "Item removed from cart", Toast.LENGTH_SHORT).show();

как

@Override
    public void onChildRemoved(@NonNull DataSnapshot dataSnapshot) {
        //remove these line these will raise the exception and Toast here 
        int index = listData.indexOf(dataSnapshot);
        listData.remove(index);
        adapter.notifyDataSetChanged();
    }

Answer 2

Если dataSnapshot не существует в списке, возвращается -1.Индекс списка начинается с 0.Поэтому вы получаете эту ошибку.Вы должны добавить охранную проверку здесь как:

        @Override
        public void onChildRemoved(@NonNull DataSnapshot dataSnapshot) {

            int index = listData.indexOf(dataSnapshot);
            if(index > 0) {
               listData.remove(index);
            }
            adapter.notifyDataSetChanged();
        }