从适配器中删除项目后，Recyclerview 获取现有数据并且未正确更新

Question

我尝试使用带有 firebase 数据库的自定义适配器在片段中实现好友请求功能。问题是当用户接受或删除某人的请求时，它会从 firebase 中删除，但不会在 RecyclerView 中正确更新。此问题仅发生在运行时。如果我刷新页面，我的问题就会消失。

让我有两个好友请求。如果我删除第二个数据，那么第二个数据将从RecyclerView 消失，但问题是RecyclerView 显示第一个数据加倍。如果我删除第一个数据，那么第一个数据进入第二行，第二个数据进入第一行。

这是我的数据库截图

片段类-

public class NotificationFragment extends Fragment {

private RecyclerView NotificationRecyclerView;
private NotificationAdapter adapter;
private List<Friend> friendList;

public NotificationFragment() {
    // Required empty public constructor
}


@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container,
                         Bundle savedInstanceState) {
    // Inflate the layout for this fragment
    View view = inflater.inflate(R.layout.fragment_notification, container, false);

    NotificationRecyclerView = view.findViewById(R.id.NotificationRecyclerView);
    NotificationRecyclerView.setHasFixedSize(true);
    LinearLayoutManager LayoutManager = new LinearLayoutManager(getContext());
    NotificationRecyclerView.setLayoutManager(LayoutManager);
    friendList = new ArrayList<>();
    adapter = new NotificationAdapter(getContext(), friendList);
    NotificationRecyclerView.setAdapter(adapter);

    readAllNotification();

    return view;
}

private void readAllNotification() {

    final FirebaseUser firebaseUser = FirebaseAuth.getInstance().getCurrentUser();

    DatabaseReference reference = FirebaseDatabase.getInstance().getReference("FriendRequest");
    reference.addValueEventListener(new ValueEventListener() {
        @Override
        public void onDataChange(@NonNull DataSnapshot dataSnapshot) {
            for (DataSnapshot snapshot : dataSnapshot.getChildren()) {
                Friend friend = snapshot.getValue(Friend.class);
                if (firebaseUser.getUid().equals(friend.getReceiverID())) {
                    friendList.add(friend);
                }
            }

            Collections.reverse(friendList);
            adapter.notifyDataSetChanged();
        }

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

        }
    });
}

}

自定义适配器 -

public class NotificationAdapter extends RecyclerView.Adapter<NotificationAdapter.NotificationViewHolder> {

private Context context;
private List<Friend> friendList;

public NotificationAdapter(Context context, List<Friend> friendList) {
    this.context = context;
    this.friendList = friendList;
}

@NonNull
@Override
public NotificationViewHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {

    View view = LayoutInflater.from(context).inflate(R.layout.single_notification_item, parent, false);
    return new NotificationAdapter.NotificationViewHolder(view);

}

@Override
public void onBindViewHolder(@NonNull final NotificationViewHolder holder, final int position) {

    final Friend friend = friendList.get(position);

    getUserInfo(holder.profileImage, holder.NotificationUserName, friend.getSenderID());

    holder.cancelRequestButton.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            FirebaseDatabase.getInstance().getReference("FriendRequest")
                    .child(friend.getRequestID()).removeValue().addOnCompleteListener(new OnCompleteListener<Void>() {
                @Override
                public void onComplete(@NonNull Task<Void> task) {
                    removeItem(position);
                    Toast.makeText(context, "removed", Toast.LENGTH_SHORT).show();
                }
            });
        }
    });

}

public void removeItem(int position) {
    friendList.remove(position);
    notifyDataSetChanged();
}

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

private void getUserInfo(final CircleImageView prfileImage, final TextView NotificationUserName, String senderID) {

    DatabaseReference reference = FirebaseDatabase.getInstance().getReference("Users").child(senderID);
    reference.addValueEventListener(new ValueEventListener() {
        @Override
        public void onDataChange(@NonNull DataSnapshot dataSnapshot) {
            Users users = dataSnapshot.getValue(Users.class);

            NotificationUserName.setText(users.getUserName());
            Picasso.with(context).load(users.getImageUrl()).into(prfileImage);
        }

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

        }
    });
}

public class NotificationViewHolder extends RecyclerView.ViewHolder {

    private TextView NotificationUserName;
    private Button cancelRequestButton;
    private CircleImageView profileImage;

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

        NotificationUserName = itemView.findViewById(R.id.NotificationUserName);
        cancelRequestButton = itemView.findViewById(R.id.cancelRequestBtn);
        profileImage = itemView.findViewById(R.id.profileImage);
    }
}

}

我的 APP 问题截图 - 让我有两个要求 1）如果我删除第二个数据第一个数据显示双打：

2）如果我删除第一个数据，第一个数据进入第二行，第二个数据进入第一行：

Answer 1

替换

 removeItem(position);

与

 removeItem(holder.getAdapterPosition());

Answer 2

您在onCreateView 中初始化您的recyclerView 和adapter，这是不合适的。您必须重写方法onViewCreated，然后初始化您的recyclerView 和adapter。像这样尝试

public class NotificationFragment extends Fragment {

private RecyclerView NotificationRecyclerView;
private NotificationAdapter adapter;
private List<Friend> friendList;

public NotificationFragment() {
// Required empty public constructor
}


@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container,
                     Bundle savedInstanceState) {
// Inflate the layout for this fragment
View view = inflater.inflate(R.layout.fragment_notification, container, false);


return view;
}
@Override
public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceState) {
    super.onViewCreated(view, savedInstanceState);

  NotificationRecyclerView = view.findViewById(R.id.NotificationRecyclerView);
NotificationRecyclerView.setHasFixedSize(true);
LinearLayoutManager LayoutManager = new LinearLayoutManager(getContext());
NotificationRecyclerView.setLayoutManager(LayoutManager);
friendList = new ArrayList<>();
adapter = new NotificationAdapter(getContext(), friendList);
NotificationRecyclerView.setAdapter(adapter);

readAllNotification();

}

private void readAllNotification() {

final FirebaseUser firebaseUser = FirebaseAuth.getInstance().getCurrentUser();

DatabaseReference reference = FirebaseDatabase.getInstance().getReference("FriendRequest");
reference.addValueEventListener(new ValueEventListener() {
    @Override
    public void onDataChange(@NonNull DataSnapshot dataSnapshot) {
        for (DataSnapshot snapshot : dataSnapshot.getChildren()) {
            Friend friend = snapshot.getValue(Friend.class);
            if (firebaseUser.getUid().equals(friend.getReceiverID())) {
                friendList.add(friend);
            }
        }

        Collections.reverse(friendList);
        adapter.notifyDataSetChanged();
    }

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

    }
});
}

好的，我刚刚注意到您使用 holder.getAdapterPosition() 在 removeItem 方法中传递了一个参数，这导致了您的问题。尝试传递由 public void onBindViewHolder(@NonNull final NotificationViewHolder holder, final int position) 提供的位置。所以基本错误是当您在onBindViewHolder 你不需要使用holder.getAdapterPosition() 因为onBindViewHolder 已经给了你职位

在您的 removeItem 方法中使用 notifyDataSetChanged 而不是 notifyItemRemoved(position) 试试这样

    @Override
public void onBindViewHolder(@NonNull final NotificationViewHolder holder, final int position) {
  final Friend friend = friendList.get(position);
  holder.cancelRequestButton.setOnClickListener(new View.OnClickListener() {
     @Override
     public void onClick(View v) {
        FirebaseDatabase.getInstance().getReference("FriendRequest").child(friend.getRequestID()).removeValue().addOnCompleteListener(new OnCompleteListener<Void>() {
            @Override
            public void onComplete(@NonNull Task<Void> task) {
                removeItem(position);
                Toast.makeText(context, "removed", Toast.LENGTH_SHORT).show();
            }
        });
    }
});
}
  public void removeItem(int position) {
friendList.remove(position);
notifyDataSetChanged();
}