RecyclerView绑定到FirebaseRecyclerAdapter,选中复选框时删除错误的行

时间:2016-04-08 12:47:54

标签: android firebase android-recyclerview recycler-adapter firebaseui

我将recyclerView绑定到FirebaseRecyclerAdapter,并为OnClickListener中的复选框设置ViewHolder

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

问题: 如果我从列表底部向上删除,则没有问题。 但是,如果随机选中复选框,似乎错误的行将被删除,有时甚至会有NULL位置并导致Android应用程序崩溃。 我有记录位置,似乎删除行后复选框的位置没有正确更新。

这是我的代码:

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);
    }
}

2 个答案:

答案 0 :(得分:4)

将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();
         }
     });

答案 1 :(得分:0)

问题是,在回收者视图中删除元素时,其余元素的位置不会更新。因此,使用“i”作为索引是错误的。相反,使用:

messageViewHolder.getAdapterPosition()