android - RecyclerView 绑定(bind)到 FirebaseRecyclerAdapter，选中复选框时删除了错误的行

Question

我将 recyclerView 绑定(bind)到 FirebaseRecyclerAdapter，并为 ViewHolder 中的复选框设置 OnClickListener。

我的想法:
当一行的复选框被选中时，适配器将 getRef(clickedPosition) 并调用 removeValue 从 Firebase 数据库中删除该行。之后，调用 populateViewHolder，应用程序将显示数据库中的最新数据。

问题:如果我从列表底部向上删除，没有问题。但是，如果随机选中复选框，似乎会删除错误的行，有时甚至会出现 NULL 位置并导致 Android 应用程序崩溃。我有 Log the position，似乎复选框的位置在删除一行后没有正确更新。

这是我的代码:

public Firebase mRef;

// UI
RecyclerView mRecyclerView;

FirebaseRecyclerAdapter<Task, MessageViewHolder> adapter;

@Override
protected void onCreate(Bundle savedInstanceState) {
   super.onCreate(savedInstanceState);
   setContentView(R.layout.activity_my_task);
   ...
   ...
   Firebase.setAndroidContext(this);
   mRef = new Firebase("https://what-to-do-list.firebaseio.com/todoItems");

   mRecyclerView = (RecyclerView) findViewById(R.id.recyclerView);
   mRecyclerView.setHasFixedSize(true);
   mRecyclerView.setLayoutManager(new LinearLayoutManager(this));
}


@Override
protected void onStart() {
    super.onStart();

    adapter =
            new FirebaseRecyclerAdapter<Task, MessageViewHolder>(
                    Task.class,
                    R.layout.custom_row_view,
                    MessageViewHolder.class,
                    mRef
            ) {
                @Override
                protected void populateViewHolder(MessageViewHolder messageViewHolder, Task t, int i) {

                    messageViewHolder.mText.setText(t.getName());
                    messageViewHolder.chkBox.setTag(i);

                    messageViewHolder.chkBox.setOnClickListener(new View.OnClickListener() {
                        @Override
                        public void onClick(View v) {
                            CheckBox cb = (CheckBox) v;
                            int clickedPos = ((Integer) cb.getTag()).intValue();


                            Log.v("POSITION: ", String.valueOf(clickedPos));
                            adapter.getRef(clickedPos).removeValue();
                        }
                    });
                }      
            };
    mRecyclerView.setAdapter(adapter);
}

@JsonIgnoreProperties(ignoreUnknown = true)
public static class MessageViewHolder
        extends RecyclerView.ViewHolder {
    TextView mText;
    CheckBox chkBox;

    public MessageViewHolder(View v) {
        super(v);
        mText = (TextView) v.findViewById(R.id.list_item_name);
        chkBox = (CheckBox) v.findViewById(R.id.task_chkbox);
    }
}

Answer 1

将您的 populateViewHolder 更改为:

@Override
protected void populateViewHolder(MessageViewHolder messageViewHolder, Task t, int i) {

    final DatabaseReference ref = getRef(i);
    messageViewHolder.mText.setText(t.getName());
    messageViewHolder.chkBox.setTag(ref);

    messageViewHolder.chkBox.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            CheckBox cb = (CheckBox) v;
            String key = cb.getTag().getKey();

            adapter.getRef(key).removeValue();
         }
     });