如果是您的个人资料，则新闻提要中的帖子会重复出现，其他用户的帖子只会出现一次

JMB 发表于 Java

229

JMB：

我有以下问题。其他用户的帖子也只有一篇很好。但是，如果这是您的个人资料，则在HomeFragment中帖子重复出现，我不确定为什么会这样。

用我的readPosts();方法，如果我摆脱了这一行，} else if (mFirebaseUser.getUid().equals(post.getPublisher())) {mPostLists.add(post);那很好，但是我的帖子不会只显示我关注的人的帖子。我需要我的消息也出现在新闻提要中，但一次不能重复。我所有的帖子都被复制了。同一则帖子在我的供稿上出现两次。有人可以告诉我我在做什么错吗？

我的readPosts();方法做错了，但是我不确定在玩了一堆之后怎么办。

家庭碎片

public class HomeTabLayoutFragment extends Fragment {

    private ProgressBar mProgressBar;
    private PostAdapter mPostAdapter;
    private List<Post> mPostLists;

    private FirebaseAuth mFirebaseAuth;
    private FirebaseUser mFirebaseUser;

    private List<String> mFollowingList;

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        View v = inflater.inflate(R.layout.fragment_home_tab_layout, container, false);

        mFirebaseAuth = FirebaseAuth.getInstance();
        mFirebaseUser = FirebaseAuth.getInstance().getCurrentUser();

        mProgressBar = v.findViewById(R.id.progress_circular);

        RecyclerView recyclerView = v.findViewById(R.id.recycler_view);
        recyclerView.setHasFixedSize(true);
        LinearLayoutManager linearLayoutManager = new LinearLayoutManager(getContext());
        linearLayoutManager.setReverseLayout(true);
        linearLayoutManager.setStackFromEnd(true);
        recyclerView.setLayoutManager(linearLayoutManager);
        mPostLists = new ArrayList<>();
        mPostAdapter = new PostAdapter(getContext(), mPostLists);
        recyclerView.setAdapter(mPostAdapter);

        SwipeRefreshLayout refreshLayout = v.findViewById(R.id.refresh);
        refreshLayout.setOnRefreshListener(new SwipeRefreshLayout.OnRefreshListener() {
            @Override
            public void onRefresh() {
                refreshLayout.setRefreshing(false);
            }
        });

        checkIfUserExists();
        checkFollowing();

        return v;
    }

    private void checkIfUserExists() {
        if (mFirebaseAuth == null) {
            Intent intent = new Intent(getContext(), RegisterActivity.class);
            startActivity(intent);
        }
    }

    private void checkFollowing() {
        mFollowingList = new ArrayList<>();
        DatabaseReference reference = FirebaseDatabase.getInstance().getReference("Follow").child(mFirebaseAuth.getCurrentUser().getUid()).child("Following");
        reference.addValueEventListener(new ValueEventListener() {
            @Override
            public void onDataChange(@NonNull DataSnapshot dataSnapshot) {
                mFollowingList.clear();
                for (DataSnapshot snapshot : dataSnapshot.getChildren()) {
                    mFollowingList.add(snapshot.getKey());
                }

                readPosts();
            }

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

            }
        });
    }

    private void readPosts() {
        DatabaseReference reference = FirebaseDatabase.getInstance().getReference("Posts");
        reference.addValueEventListener(new ValueEventListener() {
            @Override
            public void onDataChange(@NonNull DataSnapshot dataSnapshot) {
                mPostLists.clear();
                for (DataSnapshot snapshot : dataSnapshot.getChildren()) {
                    Post post = snapshot.getValue(Post.class);
                    for (String id : mFollowingList) {
                        if (post != null) {
                            if (post.getPublisher().equals(id)) {
                                mPostLists.add(post);
                            } else if (mFirebaseUser.getUid().equals(post.getPublisher())) {
                                mPostLists.add(post);
                            }
                        }
                    }

                    mPostAdapter.notifyDataSetChanged();
                    mProgressBar.setVisibility(View.GONE);
                }
            }

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

            }
        });
    }
}

斯坦：

在readPosts（）中检查您的内部循环。该循环基于mFollowingList的元素。这种情况与此无关：

if (mFirebaseUser.getUid().equals(post.getPublisher()))

对于内部循环的每次执行，它都是true或false。因此，它可能会添加比您想要的更多的元素。您可能希望将其放在外部循环中，而不是内部循环中-每个帖子只执行一次。

不过，我还要进行一些更改。可以在循环外部进行检查（发布！= null）。无需无缘无故地使其多次循环。notifyDataSetChanged（）也不需要在任何循环内，它需要被调用一次。

关于代码样式的额外说明：-)-您使用了太多的缩进。将内容提取到单独的方法中以提高可读性是有意义的，最好使用描述性名称来使方法简短。

private void readPosts() {
    DatabaseReference reference = FirebaseDatabase.getInstance().getReference("Posts");
    reference.addValueEventListener(new ValueEventListener() {
        @Override
        public void onDataChange(@NonNull DataSnapshot dataSnapshot) {
            mPostLists.clear();
            for (DataSnapshot snapshot : dataSnapshot.getChildren()) {
                Post post = snapshot.getValue(Post.class);

                if (post != null && shouldAddPost(post)) {
                    mPostLists.add(post);
                }
            }

            mPostAdapter.notifyDataSetChanged();
            mProgressBar.setVisibility(View.GONE);
        }

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

        }
    });
}

private boolean shouldAddPost(@NotNull Post post) {
    boolean isFollowingPublisher = false;
    for (String id : mFollowingList) {
        if (post.getPublisher().equals(id)) {
            isFollowingPublisher = true;
        }
    }

    boolean isPublisher = post.getPublisher().equals(mFirebaseUser.getUid());

    return isFollowingPublisher || isPublisher;
}

本文收集自互联网，转载请注明来源。

如有侵权，请联系 [email protected] 删除。