Удаление элемента в recyclerview с помощью OnClick. Firebase Android

Я получаю данные в firebase и хочу удалить их, если пользователь щелкает элемент в recyclerview, но когда я обращаюсь к 2 или 3 элементам в моем recyclerview, я получаю трассировку стека, подобную этой. И иногда элемент, который я нажал, неверен. Как это исправить?

 E/AndroidRuntime: FATAL EXCEPTION: main 
java.lang.IndexOutOfBoundsException: Invalid index 3, size is 2
at java.util.ArrayList.throwIndexOutOfBoundsException(ArrayList.java:251)
at java.util.ArrayList.get(ArrayList.java:304)
at com.firebase.ui.database.FirebaseArray.getItem(FirebaseArray.java:52)
at com.firebase.ui.database.FirebaseRecyclerAdapter.getRef(FirebaseRecyclerAdapter.java:150)
at com.myapp.maecea.ipon4.Expenses$3$1$2.onClick(Expenses.java:274)
at android.support.v7.app.AlertController$ButtonHandler.handleMessage(AlertController.java:157)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:137)
at android.app.ActivityThread.main(ActivityThread.java:4745)
at java.lang.reflect.Method.invokeNative(Native Method)
at java.lang.reflect.Method.invoke(Method.java:511)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:786)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:553)
at dalvik.system.NativeStart.main(Native Method)

Это мой код:

adapter = new FirebaseRecyclerAdapter < ExpenseData, myViewHolder > (
     ExpenseData.class, R.layout.listitem, myViewHolder.class, mRef.orderByChild("date").equalTo(date)) {

     @Override
     protected void populateViewHolder(final myViewHolder viewHolder, ExpenseData model, final int position) {
         viewHolder.DateText.setText(model.getDate());
         viewHolder.ExpenseTitleText.setText(model.getExpensetitle());
         viewHolder.AmountText.setText(String.valueOf(String.format("%.2f", model.getAmount())));

         viewHolder.mView.setOnClickListener(new View.OnClickListener() {@
             Override
             public void onClick(final View v) {
                 AlertDialog.Builder builder = new AlertDialog.Builder(Expenses.this);
                 builder.setMessage("Gusto mo ba talaga burahin ito, katulad ng pagkalimot niya sayo?").setCancelable(false)
                     .setPositiveButton("Yes", new DialogInterface.OnClickListener() {
                         @Override
                         public void onClick(DialogInterface dialog, int which) {
                             int selectedItems = position;
                             for (int i = selectedItems; i >= 1; i++) {
                                 adapter.getRef(i).removeValue();
                                 adapter.notifyItemRemoved(i);
                             }
                         }
                     })
                     .setNegativeButton("No", new DialogInterface.OnClickListener() {
                         @Override
                         public void onClick(DialogInterface dialog, int which) {
                             dialog.cancel();
                         }
                     });
                 AlertDialog dialog = builder.create();
                 dialog.setTitle("Expenses");
                 dialog.show();
             }
         });
     }
 };    

 recyclerView.setAdapter(adapter);

firebase android android-recyclerview firebase-realtime-database firebaseui
person Kmmp    schedule 03.12.2016    source источник

Ответы (2)


arrow_upward
0
arrow_downward

IndexOutOfBoundsException, поскольку вы увеличиваете значение индекса. Измените i++ на i-- в цикле for. Этот код удаляет элемент из выбранных элементов в selectedItems >= 1.

for (int i = selectedItems; i >= 1; i--) {
    adapter.getRef(i).removeValue();
    adapter.notifyItemRemoved(i);
}

Если вы хотите удалить только один элемент (выбранный элемент в режиме просмотра корзины), это означает простое использование цикла для удаления.

adapter.getRef(i).removeValue();
adapter.notifyItemRemoved(i);
person sasikumar    schedule 03.12.2016
comment
если я добавлю это в свой код, я также получу indexOutOfBoundsException - person Kmmp; 03.12.2016
comment
У вас уже есть решения? @Kmmp - person Animesh Jena; 21.03.2017

arrow_upward
0
arrow_downward

рассмотрите возможность использования viewHolder.getAdapterPosition() вместо position

person madim    schedule 12.04.2017