为什么 recyclerview 返回错误的项目？

Question

我尝试在单击 itemview 时打开视频。有时它可以正常工作，但有时当我单击第一个视频时它会打开第二个视频，反之亦然。我试图实现的是当我点击第一个 recyclerview 项目时打开第一个视频等等。

UserHomeVideoAdapter.java:

public class UserHomeVideoAdapter extends FirestoreRecyclerAdapter<FollowList, UserHomeVideoAdapter.UserVideoHolder> {

    Context context;
    final FirebaseFirestore db = FirebaseFirestore.getInstance();

    String thumbUrl, videoTitle, videoUrl, videoDesc, videoId, publisherId;

    ArrayList<String> videoIdArrayList = new ArrayList<>();

    public UserHomeVideoAdapter(@NonNull @NotNull FirestoreRecyclerOptions<FollowList> options, Context context) {
        super(options);
        this.context = context;
    }

    @Override
    protected void onBindViewHolder(@NonNull @NotNull UserVideoHolder holder, int position, @NonNull @NotNull FollowList model) {

        Query query = db.collection("Videos").whereEqualTo("publisherId", model.getUserId());

        query.get().addOnCompleteListener(task ->  {
            if (task.isSuccessful()) {
                if (task.getResult() != null) {
                    for (QueryDocumentSnapshot documentSnapshot : task.getResult()) {
                        Video video = documentSnapshot.toObject(Video.class);

                        thumbUrl = video.getThumbUrl();
                        videoTitle = video.getVideoTitle();
                        videoUrl = video.getVideoUrl();
                        videoDesc = video.getVideoDesc();
                        videoId = video.getVideoId();
                        publisherId = video.getPublisherId();

                        videoIdArrayList.add(videoId);
                    }

                    if (task.getResult().size() > 0) {
                        db.collection("Users").document(model.getUserId()).get().addOnSuccessListener(documentSnapshot -> {
                            if (documentSnapshot != null) {
                                final User user = documentSnapshot.toObject(User.class);

                                if (user != null) {
                                    if (user.getUserImageUrl() == null) {
                                        holder.userProfileImage.setImageResource(R.drawable.ic_launcher_background);
                                    } else {
                                        Glide.with(context).load(Uri.parse(user.getUserImageUrl())).into(holder.userProfileImage);
                                    }
                                }
                            }
                        }).addOnFailureListener(e -> Toast.makeText(context, e.getLocalizedMessage(), Toast.LENGTH_SHORT).show());

                        Glide.with(context).load(thumbUrl).into(holder.videoImageView);
                        holder.videoTitle.setText(videoTitle);
                        holder.mainContainerVideo.setVisibility(View.VISIBLE);
                    } else if (task.getResult().size() == 0) {
                        holder.mainContainerVideo.getLayoutParams().height = 0;
                        holder.mainContainerVideo.getLayoutParams().width = 0;
                    }
                }
            } else {
                Toast.makeText(context, String.valueOf(task.getException()), Toast.LENGTH_SHORT).show();
            }
        }).addOnFailureListener(e -> Toast.makeText(context, e.getLocalizedMessage(), Toast.LENGTH_SHORT).show());

        holder.itemView.setOnClickListener(view -> {
            Intent intent = new Intent(context, VideoActivity.class);
            intent.putExtra("videoPublisherUserName", model.getUserName());
            intent.putExtra("thumbUrl", thumbUrl);
            intent.putExtra("videoPublisherEmail", model.getUserEmail());
            intent.putExtra("videoUrl", videoUrl);
            intent.putExtra("videoId", videoIdArrayList.get(position));
            intent.putExtra("videoPublisherFullName", model.getUserFullName());
            intent.putExtra("videoPublisherId", publisherId);
            context.startActivity(intent);

            Log.d("Bax", videoIdArrayList.get(position));
        });
    }

    @NonNull
    @NotNull
    @Override
    public UserVideoHolder onCreateViewHolder(@NonNull @NotNull ViewGroup parent, int viewType) {
        View v = LayoutInflater.from(context).inflate(R.layout.video_cell, parent, false);
        return new UserVideoHolder(v);
    }

    public static class UserVideoHolder extends RecyclerView.ViewHolder {
        RelativeLayout mainContainerVideo;
        CircleImageView userProfileImage;
        TextView videoTitle;
        ImageView videoImageView;

        public UserVideoHolder(@NonNull @NotNull View itemView) {
            super(itemView);
            mainContainerVideo = itemView.findViewById(R.id.mainContainerVideo);
            userProfileImage = itemView.findViewById(R.id.userProfileImage);
            videoTitle = itemView.findViewById(R.id.videoTitle);
            videoImageView = itemView.findViewById(R.id.videoImageView);
        }
    }
}

我不明白我的代码存在的问题，因此感谢您提供任何帮助。谢谢

Answer 1

这里的问题是您的videoIdArrayList.add(videoId) 没有被重置。每次您的 recyclerview 加载时，它都会查询数据库并在 videoIdArrayList 中添加 videoId 并不断添加。

这会弄乱要发送到下一个活动的 videoId。 因此，请清除您的 videoIdArrayList 每次 recyclerview 加载并在此之后添加 Id。

因此，在 firebase 查询中的 for 循环之前，只需清除列表即可。 videoIdArrayList.clear()

应该可以完美运行。

Answer 2

onBindViewHolder 对RecyclerView 中的每个项目执行。一次都没有

Query query = db.collection("Videos").whereEqualTo("publisherId", model.getUserId()); 返回 List，而不是项目。

videoIdArrayList.add(videoId); 可以多次执行第一项。

String thumbUrl, videoTitle, videoUrl, videoDesc, videoId, publisherId; 在 Query 中设置为 onBindViewHolder。再次为RecyclerView 中的每个项目执行onBindViewHolder。一次都没有。

你必须重构你的适配器。

1. 通过您的适配器执行查询。
2. 目前尚不清楚，哪些物品会收到持有人。 （对我来说。如果你很清楚 -> 跳过这一步）
3. 而不是 holder.videoTitle 在 holder 中创建一个方法（例如 void configView(YourItem item)），并在 holder 中执行与该项目有关的一切。