无法从 Recycler View 中删除条目的问题。

3

这个问题让我很疯狂。我已经尝试了20种不同的方法,现在让我们看看是否有人可以帮助我。

我正在使用RecyclerView创建一个ToDo列表应用程序,它将数据存储在Firebase中。

该应用程序具有一个TasksActivity,其中所有任务都出现在RecyclerView中。我有一个按钮可以进入任务创建对话框。我可以创建任务,然后它们会出现在RecyclerView(Tasks Activity)中,并且在Firebase中更新没有任何问题。我可以关闭APP,稍后再回来时一切正常,加载应用程序时所有条目也会再次出现。我还可以滑动删除一个条目,该条目也将从Firebase数据库中消除。

问题是,当我创建一个任务并且没有关闭应用程序时,我尝试删除刚刚创建的任务。它不允许我这样做。当我创建一个新任务,并在关闭应用程序之前立即将其删除时,它再次出现。但是,如果我关闭应用程序,然后再次加载它,那个条目就可以正常删除了,但如果我在创建该条目的同一会话中,就没有办法将其删除。

我正在使用一些Log.d参数来查看它如何变化。我认为由于各种原因,问题出现在OnDataChange()中。但到目前为止,我还没有能够找到问题的根源。这是TaskActivity类,接下来我会粘贴TasksCreation(我认为不需要粘贴Adapter)。

public class TasksActivity extends AppCompatActivity  {

    DatabaseReference reference;
    RecyclerView myTasks;
    ArrayList<TaskItems> myTasksList;
    TasksAdapter tasksAdapter;


    @Override
    protected void onCreate(@Nullable Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_tasks);
        myTasks = findViewById(R.id.my_tasks);   // RecyclerView that I defined as part of the layout. This is the id of it

        myTasks.setLayoutManager(new LinearLayoutManager(this,LinearLayoutManager.VERTICAL,false));
        myTasksList = new ArrayList<>();
        Button openCreateTask = findViewById(R.id.openCreateTask);
        tasksAdapter = new TasksAdapter(this,myTasksList); 
        myTasks.setAdapter(tasksAdapter);
        new ItemTouchHelper(itemTouchHelper).attachToRecyclerView(myTasks);


        openCreateTask.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                Intent intent_task = new Intent(getApplicationContext(), TasksCreation.class);
                startActivity(intent_task);
            }
        });


        reference = FirebaseDatabase.getInstance().getReference().child("MotApp"); // Name of the App in the database .child("MotApp")

        reference.addValueEventListener(new ValueEventListener() {


            @Override
            public void onDataChange(@NonNull DataSnapshot dataSnapshot) {   // It gets the info from the database
                Log.d("data Changed called", "onDataChange: is called");
                Log.d("whatever", "onDataChange BEGIN Array of myTasksList size is "+myTasksList.size());
                myTasksList.clear();                                            // Added later to avoid duplication




                for(DataSnapshot elements: dataSnapshot.getChildren()){


                    TaskItems p = elements.getValue(TaskItems.class);
                    myTasksList.add(p);

                }


                tasksAdapter.notifyDataSetChanged(); // If this is put outside of onDataChange, it displays a blank list.
                Log.d("whatever", "onDataChange END Array of myTasksList size is "+myTasksList.size());
            }

            @Override
            public void onCancelled(@NonNull DatabaseError databaseError) {
                Toast.makeText(getApplicationContext(), "No data", Toast.LENGTH_SHORT).show();
            }


        });

    }


    ItemTouchHelper.SimpleCallback itemTouchHelper = new ItemTouchHelper.SimpleCallback(0,ItemTouchHelper.RIGHT) {
        @Override
        public boolean onMove(@NonNull RecyclerView recyclerView, @NonNull RecyclerView.ViewHolder viewHolder, @NonNull RecyclerView.ViewHolder target) {
            return false;
        }

        @Override
        public void onSwiped(@NonNull RecyclerView.ViewHolder viewHolder, int direction) {
            int position = viewHolder.getAdapterPosition();
            Log.d("ARRAY SIZE", "onSwiped BEGIN Array of myTasksList size is "+myTasksList.size());
            String key =    myTasksList.get(position).getKey();
            reference= FirebaseDatabase.getInstance().getReference().child("MotApp").child(key);


        Toast.makeText(getApplicationContext(),"This is key "+key,Toast.LENGTH_LONG).show();
         reference.removeValue();
        myTasksList.remove(position);
        tasksAdapter.notifyItemRemoved(position);

            Log.d("ARRAY SIZE", "onSwiped END Array of myTasksList size is "+myTasksList.size());

        }
    };

}

以下是关于TasksCreation活动的内容:

public class TasksCreation extends AppCompatActivity {

    DatabaseReference referenceCreation;
    ArrayList<String> list;
    EditText taskName;
    EditText taskDescr;
    Button selectDates;
    TextView taskDate;
    Button createTask;
    Button cancel;

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

        taskName = findViewById(R.id.et_TaskName);
        taskDescr = findViewById(R.id.et_TaskDescr);
        selectDates = findViewById(R.id.selectDates);
        taskDate= findViewById(R.id.tv_Dates);
        createTask = findViewById(R.id.createTask);
        cancel = findViewById(R.id.cancelButton);


        createTask.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {

                if (!TextUtils.isEmpty(taskName.getText()) && !TextUtils.isEmpty(taskDate.getText())) {

                    referenceCreation = FirebaseDatabase.getInstance().getReference().child("MotApp").push(); //saves it with custom key created by Firebase
                    final String key = referenceCreation.getKey();

                    referenceCreation.addValueEventListener(new ValueEventListener() {
                        @Override
                        public void onDataChange(@NonNull DataSnapshot dataSnapshot) {       
                            dataSnapshot.getRef().child("key").setValue(key);

                            dataSnapshot.getRef().child("taskTitle").setValue(taskName.getText().toString());
                            dataSnapshot.getRef().child("taskDescription").setValue(taskDescr.getText().toString());
                            dataSnapshot.getRef().child("taskDate").setValue(taskDate.getText().toString());

                        }


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

                        }
                    });


                    Intent intent = new Intent(getApplicationContext(), TasksActivity.class);

                    startActivity(intent);

                } else{
                    Toast.makeText(getApplicationContext(), "You haven't filled all the fields", Toast.LENGTH_SHORT).show();
                }
            }
        });

        cancel.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                Intent intent = new Intent(getApplicationContext(), TasksActivity.class);

                startActivity(intent);
            }
        });

        selectDates.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                createAlertDialogue();
            }
        });

    }

    private void createAlertDialogue(){
        list = new ArrayList<String>();

        AlertDialog.Builder builder = new AlertDialog.Builder(this,R.style.MyDialogTheme);


        builder.setTitle("Select days");
        builder.setMultiChoiceItems(R.array.Days, null, new DialogInterface.OnMultiChoiceClickListener() {
            @Override
            public void onClick(DialogInterface dialog, int which, boolean isChecked) {

                String arr[] = getResources().getStringArray(R.array.Days);

                if(isChecked){
                    list.add(arr[which]);
                }else if(list.contains(arr[which])){
                    list.remove(arr[which]);

                }

            }
        });

         builder.setPositiveButton("Save", new DialogInterface.OnClickListener() {

            String data = "";
            @Override
            public void onClick(DialogInterface dialog, int which) {
            for(String elements: list){
                elements = elements.substring(0,3);
                data= elements+" "+data;

            }
                taskDate.setText(data);

            }
        });


        builder.create();

        builder.show();

    }

}

非常感谢您对这个问题的帮助。

非常感谢。

注:这是TaskAdapter的代码(请忽略dayoftheweek部分,这只是我注释的代码一部分)。

public class TasksAdapter extends RecyclerView.Adapter<TasksAdapter.MyViewHolder> { // V 1.3 added OnClickListener


    Context context;
    ArrayList<TaskItems> tasks;
    DatabaseReference reference;
    SimpleDateFormat sdf = new SimpleDateFormat("EEEE");
    Date d = new Date();
    final String dayOfTheWeek = sdf.format(d).substring(0,3);


    public TasksAdapter(Context context, ArrayList<TaskItems> tasks) {
        this.context = context;
        this.tasks = tasks;
    }


    public class MyViewHolder extends RecyclerView.ViewHolder {  // V1.3 Added implements View.OnClickListener

        TextView taskTitle;
        TextView taskDate;
        TextView taskDescription;
        CheckBox taskCheckBox;
        ConstraintLayout constraintLayout;  // Added to change background  of each RecyclerView item.



        public MyViewHolder(@NonNull View itemView) {
            super(itemView);

            taskTitle = itemView.findViewById(R.id.taskTitle);
            taskDate = itemView.findViewById(R.id.taskDate);
            taskDescription = itemView.findViewById(R.id.taskDescription);
            taskCheckBox = itemView.findViewById(R.id.taskCheckBox);
            constraintLayout = (ConstraintLayout) itemView.findViewById(R.id.item_task_layout);


            itemView.setOnClickListener(new View.OnClickListener() {
                @Override
                public void onClick(View v) {
                    int pos = getAdapterPosition();

                    //            if(pos != RecyclerView.NO_POSITION){//Checks if item still exists
                    TaskItems clickedDataItem = tasks.get(pos);
                    Toast.makeText(v.getContext(), "You clicked " + clickedDataItem.getTaskTitle(), Toast.LENGTH_SHORT).show();
                    //          }
                }
            });

        }

    }

    @NonNull
    @Override
    public MyViewHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {                  // standard code for onCreateViewHolder

        return new MyViewHolder(LayoutInflater.from(context).inflate(R.layout.item_task,parent,false));
    }



    @Override
    public void onBindViewHolder(final @NonNull MyViewHolder holder, final int position) {  // This method is called once for each item on the list.

            holder.taskTitle.setText(tasks.get(position).getTaskTitle());
            holder.taskDescription.setText(tasks.get(position).getTaskDescription());
            holder.taskDate.setText(tasks.get(position).getTaskDate());
            holder.taskCheckBox.setChecked(tasks.get(position).isChecked());
            holder.taskCheckBox.setTag(tasks.get(position).getKey());


            holder.taskCheckBox.setOnClickListener(new View.OnClickListener() {
                @Override
                public void onClick(View v) {

                    FirebaseDatabase database = FirebaseDatabase.getInstance();


                    String post = (String) holder.taskCheckBox.getTag();
                    Toast.makeText(context,"This is checkbox number: "+post,Toast.LENGTH_SHORT).show();  // Working Well.        //change for new logic

                    reference =  database.getReference("MotApp").child(post);

                    boolean checkboxStatus =  holder.taskCheckBox.isChecked();
                    Log.d("Checked", "onClick: The taskcheckbox checked is "+holder.taskCheckBox.isChecked());

                    TaskItems value = new TaskItems(tasks.get(position).getKey(),tasks.get(position).getTaskTitle(), tasks.get(position).getTaskDate(),tasks.get(position).getTaskDescription(),checkboxStatus);
                    reference.setValue(value);
                   Toast.makeText(context,"This is a checkbox belonging to item "+tasks.get(position).getTaskTitle(),Toast.LENGTH_LONG).show();

                }
            });

    }

    @Override
    public int getItemCount() { //tells the adapter the size . if it's zero then it won't create anything
        return tasks.size();
    }

}

1
通常当我看到在Activity中将Adapter的数据作为引用时,那就是bug产生的地方。去掉对ArrayList<TaskItems> myTasksList的引用,直接使用Adapter进行添加和删除操作,这将更紧密地封装数据,从而提供更清晰的修改数据方式。 - Submersed
你能详细解释一下吗?可以举个例子吗?这是我做的第一个大型应用程序,所以我没有太多经验。到目前为止,我看过的所有教程都使用ArrayList作为存储值的方式。谢谢。 - SoulAndBone
你能否发布一下你的TasksAdapter代码,如果这个问题还没有被解决的话。 - Submersed
已完成。我编辑了帖子,它在消息的末尾。 - SoulAndBone
2个回答

1

我终于找到了解决方法。

首先,我在TaskActivity和TaskCreation中将所有的addValueEventListener都改成了addListenerforSingleValueEvent。通过使用addValueEventListener,我发现每次创建新任务时都会产生多个循环,这导致了各种问题。

我还改变了TaskCreation中startIntent的设置方式。最初我是在Listener块之后设置的。但是onDataChange的异步性质使得每次加载意图时更新的信息没有与刚刚添加的新任务一起设置。

将其放在onDataChange内部就可以解决问题了。我进行了多次测试,现在它完美地工作了。

我浪费了很多天来解决这个问题。但我在这个过程中学到了很多东西。所以我想它们并不是浪费的日子 :)

感谢所有帮助我解决这个问题的人。


1
很高兴你发现了它。这些日子会让你成为更强大的开发人员,并在未来节省大量时间。 - Zain

1

当您从 TasksCreation 返回时,需要在 TasksActivity 中重新构建 RecyclerView 数据;这是因为只有在打开应用程序时才会调用 TasksActivityonCreate() 回调函数。这也是为什么只有当您关闭应用程序并重新打开它时,删除操作才有效。

onCreate() 不会在从 TasksCreate 返回时被调用,因为当您从 TasksActivity 转移到 TasksCreation 时,TasksActivity 并没有被销毁,而只是停止了;因此,当您回到 TasksActivity 时,它将重新启动和恢复。所以将代码从 onCreate() 转移到 onResume(),以便允许列表更新最近的更改。

因此,请将您的 TasksActivity 更改为以下内容:

public class TasksActivity extends AppCompatActivity  {

    DatabaseReference reference;
    RecyclerView myTasks;
    ArrayList<TaskItems> myTasksList;
    TasksAdapter tasksAdapter;


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

    }

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

        myTasks = findViewById(R.id.my_tasks);   // RecyclerView that I defined as part of the layout. This is the id of it

        myTasks.setLayoutManager(new LinearLayoutManager(this,LinearLayoutManager.VERTICAL,false));
        myTasksList = new ArrayList<>();
        Button openCreateTask = findViewById(R.id.openCreateTask);
        tasksAdapter = new TasksAdapter(this,myTasksList); 
        myTasks.setAdapter(tasksAdapter);
        new ItemTouchHelper(itemTouchHelper).attachToRecyclerView(myTasks);


        openCreateTask.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                Intent intent_task = new Intent(getApplicationContext(), TasksCreation.class);
                startActivity(intent_task);
            }
        });


        reference = FirebaseDatabase.getInstance().getReference().child("MotApp"); // Name of the App in the database .child("MotApp")

        reference.addValueEventListener(new ValueEventListener() {


            @Override
            public void onDataChange(@NonNull DataSnapshot dataSnapshot) {   // It gets the info from the database
                Log.d("data Changed called", "onDataChange: is called");
                Log.d("whatever", "onDataChange BEGIN Array of myTasksList size is "+myTasksList.size());
                myTasksList.clear();                                            // Added later to avoid duplication

                for(DataSnapshot elements: dataSnapshot.getChildren()){


                    TaskItems p = elements.getValue(TaskItems.class);
                    myTasksList.add(p);

                }

                tasksAdapter.notifyDataSetChanged(); // If this is put outside of onDataChange, it displays a blank list.
                Log.d("whatever", "onDataChange END Array of myTasksList size is "+myTasksList.size());
            }

            @Override
            public void onCancelled(@NonNull DatabaseError databaseError) {
                Toast.makeText(getApplicationContext(), "No data", Toast.LENGTH_SHORT).show();
            }

        });

    }


    ItemTouchHelper.SimpleCallback itemTouchHelper = new ItemTouchHelper.SimpleCallback(0,ItemTouchHelper.RIGHT) {
        @Override
        public boolean onMove(@NonNull RecyclerView recyclerView, @NonNull RecyclerView.ViewHolder viewHolder, @NonNull RecyclerView.ViewHolder target) {
            return false;
        }

        @Override
        public void onSwiped(@NonNull RecyclerView.ViewHolder viewHolder, int direction) {
            int position = viewHolder.getAdapterPosition();
            Log.d("ARRAY SIZE", "onSwiped BEGIN Array of myTasksList size is "+myTasksList.size());
            String key =    myTasksList.get(position).getKey();
            reference= FirebaseDatabase.getInstance().getReference().child("MotApp").child(key);


        Toast.makeText(getApplicationContext(),"This is key "+key,Toast.LENGTH_LONG).show();
         reference.removeValue();
        myTasksList.remove(position);
        tasksAdapter.notifyItemRemoved(position);

            Log.d("ARRAY SIZE", "onSwiped END Array of myTasksList size is "+myTasksList.size());

        }

    };


}

谢谢你的帮助,Zain。但不幸的是,这并没有解决问题。我刚刚测试了你提出的建议,问题仍然存在。我怀疑问题在于“onDataChange”。但是你的想法至少给了我一些澄清。我也考虑过像你提出的那样做,但我缺乏技术专业知识,不知道该如何做。我们至少现在知道不是那个问题了。 - SoulAndBone
@SoulAndBone,感谢您的评论,我会尽力检查代码,如果我发现更多问题。 - Zain
@SoulAndBone,你在onDataChanged中有一个日志Log.d(“whatever”,“onDataChange BEGIN Array of myTasksList size is”+myTasksList.size()); 当你创建一个新的任务并返回到TasksActivity时,这个日志会增加一吗? - Zain
你好@Zain,再次感谢。这是日志。从滑动开始,在我有一个初始任务(表现正常)的情况下,创建了第二个任务而没有退出会话。imagehost.com.au//d7amw0Kd9r/Log_5e9c0adbdfec4.png。正如您所看到的,onDataChange被调用了多次。尽管我执行的唯一操作是滑动删除刚刚创建的任务。 - SoulAndBone
@SoulAndBone 再次感谢您提供的新信息.. 我会再次确认。 - Zain
我终于解决了。如果你想检查问题是什么,可以在这篇帖子中看到我的答案。无论如何,还是感谢提供的帮助 :) - SoulAndBone

网页内容由stack overflow 提供, 点击上面的
可以查看英文原文,
原文链接