【问题标题】:What's wrong with my DiffUtil implementation?我的 DiffUtil 实现有什么问题?
【发布时间】:2019-06-07 07:05:23
【问题描述】:

更新:问题之一已解决:现在updateList 已解决,问题是我将mAdapter 定义为RecyclerView.Adapter 而不是MyAdapter。但是现在即使我正在获取数据,列表上也没有显示任何内容,它是空的

--------原帖------ --

我想使用DiffUtil 更新我的RecyclerView 以防止重复。

我有 4 个类:User 类、我设置数据的 Activity 类、Adapter 类和 DiffUtil 类。我不确定我是否正确地结合了所有这 4 个。

这是用户类:

public class User {

    private String mUserId;
    private Uri mImageUrl;


    public User(String userId, String imageUrl) {
        mUserId = userId;
        mImageUrl = Uri.parse(imageUrl);
    }


    public String getUserId() {
        return mUserId;
    }

    public Uri getImageUrl() {
        return mImageUrl;
    }
}

这就是我动态设置数据的方式(我不断从服务器获取包含要显示的用户 ID 的新 Json 数组,然后我从 Firebase 存储设置用户图像):(这是一个由 onClick 调用的函数听众:)

这是来自片段的方法调用:

button.setOnClickListener(new View.OnClickListener() {
    public void onClick(View v) {               
        updateUsersList();
    }
});

这是函数:

   private void updateUsersList() {
        @Override
        public void onResponse(JSONArray response) { // the JSON ARRAY response of user ids ["uid1", "uid334", "uid1123"]
            myDataset.clear(); // clear dataset to prevent duplicates
            for (int i = 0; i < response.length(); i++) {
                try {
                    String userKey = response.get(i).toString(); // the currently iterated user id
                    final DatabaseReference rootRef = FirebaseDatabase.getInstance().getReference();
                    DatabaseReference userKeyRef = rootRef.child("users").child(userKey); // reference to currently iterated user
                    ValueEventListener listener = new ValueEventListener() {
                    @Override
                    public void onDataChange(DataSnapshot dataSnapshot) {
                        myDataset.add(new User(dataSnapshot.getKey(), dataSnapshot.child("imageUrl").getValue().toString())); //add new user: id and image url
                        mAdapter.updateList(myDataset); // cannot resolve this method, why?
                   }
                   @Override
                   public void onCancelled(@NonNull DatabaseError databaseError) {
                   Log.d(TAG, databaseError.getMessage());
                   }
                  };
                  userKeyRef.addListenerForSingleValueEvent(listener);
              }
              catch (JSONException e) { Log.d(TAG, "message " + e); }
           }
   }

这就是我的DiffUtil 班级的样子:

public class MyDiffUtilCallBack extends DiffUtil.Callback{

    ArrayList<User> oldUsers;
    ArrayList<User> newUsers;

    public MyDiffUtilCallBack(ArrayList<User> newUsers, ArrayList<User> oldUsers) {
        this.newUsers = newUsers;
        this.oldUsers = oldUsers;
    }

    @Override
    public int getOldListSize() {
        return oldUsers.size();
    }

    @Override
    public int getNewListSize() {
        return newUsers.size();
    }

    @Override
    public boolean areItemsTheSame(int oldItemPosition, int newItemPosition) {
        return oldUsers.get(oldItemPosition).getUserId().equals( newUsers.get(newItemPosition).getUserId());
    }

    @Override
    public boolean areContentsTheSame(int oldItemPosition, int newItemPosition) {
        return oldUsers.get(oldItemPosition).equals(newUsers.get(newItemPosition));
    }

    @Nullable
    @Override
    public Object getChangePayload(int oldItemPosition, int newItemPosition) {
        //you can return particular field for changed item.
        return super.getChangePayload(oldItemPosition, newItemPosition);
    }
}

这是我的适配器:

public class MyAdapter extends RecyclerView.Adapter<MyAdapter.MyViewHolder> {

    private ArrayList<User> mDataset;
    private MyViewHolder myHolder;
    private User user;

    public static class MyViewHolder extends RecyclerView.ViewHolder {

        public TextView singleItemTextView;
        public ImageView singleItemImage;
        public View layout;
        public ConstraintLayout constraintLayout;
        public MyViewHolder(View v) {
            super(v);
            layout = v;
            singleItemImage = (ImageView) v.findViewById(R.id.icon);
            singleItemTextView = (TextView) v.findViewById(R.id.singleitemtv);
            constraintLayout = (ConstraintLayout) v.findViewById(R.id.nbConstraintLayout);
        }
    }

    // Provide a suitable constructor (depends on the kind of dataset)
    public MyAdapter(ArrayList<User> myDataset) {
        mDataset = myDataset;
    }

    // Create new views (invoked by the layout manager)
    @Override
    public MyAdapter.MyViewHolder onCreateViewHolder(ViewGroup parent,
                                                     int viewType) {


        View v =  LayoutInflater.from(parent.getContext())
                .inflate(R.layout.nb_image_view, parent, false);

        MyViewHolder vh = new MyViewHolder(v);
        return vh;
    }

    @Override
    public void onBindViewHolder(final MyViewHolder holder, final int position) {
        myHolder = holder;


        user = mDataset.get(position);
        Uri userImage = user.getImageUrl();       
        myHolder.singleItemTextView.setText(user.getUserId());

        Glide.with(myHolder.itemView.getContext() /* context */)
                .load(userImage)
                .into(myHolder.singleItemImage);
        myHolder.constraintLayout.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {               
                 Context context = v.getContext();                
                Intent intent = new Intent(v.getContext(), DisplayUserActivity.class);
              context.startActivity(intent);
            }
        });

    }
    public void updateList(ArrayList<User> newList) {
        DiffUtil.DiffResult diffResult = DiffUtil.calculateDiff(new MyDiffUtilCallBack(this.mDataset, newList));
        diffResult.dispatchUpdatesTo(this);
    }
}

我不确定我是否正确组合了所有类(我第一次使用DiffUtil),我也得到了cannot resolve method updateList(?)

我做错了什么?

这就是我在 Fragment 中定义 mAdapter 的方式:

public class MyFragment extends Fragment {
    private ArrayList<User> myDataset;
    private RecyclerView.Adapter mAdapter;

    public View onCreateView(LayoutInflater inflater, ViewGroup container,
                             Bundle savedInstanceState) {

        // Inflate the layout for this fragment
        rootView = inflater.inflate(R.layout.fragment_lks, container, false);

        mRecyclerView = (RecyclerView) rootView.findViewById(R.id.my_recycler_view);
        myDataset = new ArrayList<User>();
        mAdapter = new MyAdapter(myDataset);

【问题讨论】:

  • 似乎是正确的。请贴出mAdapter的定义?
  • 已添加。顺便说一句,获取数据是在一个由 onClick 事件调用的方法中,我将编辑帖子来解释。由于某种原因,updateList 在 mAdapter 上不可见:/
  • 你为你的User 类重新定义了equals 吗?没有它,areContentsTheSame 将无法正常工作。
  • 你是对的,什么都没有显示,你认为是因为等于?你能更具体一点吗?我是 Android 新手,所以我不知道如何重新定义它
  • @ThibaultSeisel 你能补充一下我需要如何改变它吗?我不能让它工作

标签: java android android-recyclerview android-diffutils


【解决方案1】:

问题来自mAdapter的定义。您将其定义为RecyclerView.Adapter,它是您的MyAdapter 的超类,它不包含updateList()。您应该将其更改如下:

private MyAdapter mAdapter;

2019 年 1 月 13 日更新:

我已经用AsyncListDiffer 修改了您的适配器,它异步计算差异然后将其应用于适配器。

MyAdapter.java

import android.content.Context;
import android.content.Intent;
import android.net.Uri;
import android.support.annotation.NonNull;
import android.support.constraint.ConstraintLayout;
import android.support.v7.recyclerview.extensions.AsyncListDiffer;
import android.support.v7.util.DiffUtil;
import android.support.v7.widget.RecyclerView;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.ImageView;
import android.widget.TextView;   
import com.bumptech.glide.Glide;    
import java.util.List;


public class MyAdapter extends RecyclerView.Adapter<MyAdapter.MyViewHolder> {

    private AsyncListDiffer<User> mAsyncListDiffer;

    public static class MyViewHolder extends RecyclerView.ViewHolder {

        public TextView singleItemTextView;
        public ImageView singleItemImage;
        public View layout;
        public ConstraintLayout constraintLayout;

        public MyViewHolder(View v) {
            super(v);
            layout = v;
            singleItemImage = (ImageView) v.findViewById(R.id.icon);
            singleItemTextView = (TextView) v.findViewById(R.id.singleitemtv);
            constraintLayout = (ConstraintLayout) v.findViewById(R.id.nbConstraintLayout);
        }
    }

    // Provide a suitable constructor (depends on the kind of dataset)
    public MyAdapter() {
        DiffUtil.ItemCallback<User> diffUtilCallback = new DiffUtil.ItemCallback<User>() {

            @Override
            public boolean areItemsTheSame(@NonNull User newUser, @NonNull User oldUser) {
                return newUser.getUserId().equals(oldUser.getUserId());
            }

            @Override
            public boolean areContentsTheSame(@NonNull User newUser, @NonNull User oldUser) {
                return newUser.equals(oldUser);
            }
        };
        mAsyncListDiffer = new AsyncListDiffer<>(this, diffUtilCallback);
    }

    // Create new views (invoked by the layout manager)
    @Override
    public MyAdapter.MyViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
        View v = LayoutInflater.from(parent.getContext()).inflate(R.layout.nb_image_view, parent, false);
        MyViewHolder vh = new MyViewHolder(v);
        return vh;
    }

    @Override
    public void onBindViewHolder(final MyViewHolder holder, final int position) {
        User user = mAsyncListDiffer.getCurrentList().get(position);
        Uri userImage = user.getImageUrl();
        holder.singleItemTextView.setText(user.getUserId());

        Glide.with(holder.itemView.getContext() /* context */)
                .load(userImage)
                .into(holder.singleItemImage);

        holder.constraintLayout.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                Context context = v.getContext();
                Intent intent = new Intent(v.getContext(), DisplayUserActivity.class);
                context.startActivity(intent);
            }
        });
    }

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

    public void updateList(List<User> newList) {
        mAsyncListDiffer.submitList(newList);
    }

}

User.java

public class User {

    private String mUserId;
    private Uri mImageUrl;

    public User(String userId, String imageUrl) {
        mUserId = userId;
        mImageUrl = Uri.parse(imageUrl);
    }

    public String getUserId() {
        return mUserId;
    }

    public Uri getImageUrl() {
        return mImageUrl;
    }

    @Override
    public boolean equals(Object other) {
        if (other instanceof User) {
            User user = (User) other;
            return mUserId.equals(user.getUserId()) && mImageUrl.equals(user.getImageUrl());
        } else {
            return false;
        }
    }

}

【讨论】:

  • 不客气。我认为最好定义public MyAdapter() 而不是public MyAdapter(ArrayList&lt;User&gt; myDataset) 并在MyAdapter 中将mDataset 定义和实例化为private ArrayList&lt;User&gt; mDataset = new ArrayList&lt;User&gt;();。它可能会解决问题。
  • 是的,我的意思是它不会将用户列表传递给它。它第一次导致空mDataset,然后当从数据库中检索到列表时,DiffUtil 计算空列表和检索到的列表之间的差异,从而显示所有新数据。当然,我会编辑它。
  • 好的,我确实做到了,但没有任何显示,我们遗漏了一些东西。 cmets 中的某个人提到了一些关于在 User 类中覆盖 equals 的内容,我没有这样做,也不知道该怎么做。但我不确定这是否会解决它。我们缺少什么? updateList 应该完全取代 notifyDataSetChanged
  • 是的,areItemsTheSame 中有一个错误。当您将mUserId 定义为String 时,您应该使用equals 而不是== 检查相等性。所以我们有:return oldUsers.get(oldItemPosition).getUserId().equals(newUsers.get(newItemPosition).getUserId()); 我用更新的User 类编辑答案。如您所料,它可能会解决问题。
  • 已更新 updateList 不等于 notifyDataSetChanged。因为更新整个项目的重要部分是将所有值重新分配给视图。它只是替换数据ArrayList 并使用DiffUtil 更新唯一需要的Views。
【解决方案2】:

除了@aminography 的回答,我建议您使用ListAdapter,这是一个RecyclerView.Adapter 实现,可以更轻松地使用正确的动画更新您RecyclerView。此类包含在 recyclerview 支持库中。

以下是基于您的用例的使用示例:

public class MyAdapter extends ListAdapter<User, UserViewHolder> {
    public MyAdapter() {
        super(new UserDiffCallback());
    }

    public UserViewHolder onCreateViewHolder(int position, int viewType) { ... }

    public void onBindViewHolder(UserViewModel holder, int position) {
        User userAtPosition = getItem(position); // getItem is a protected method from ListAdapter
        // Bind user data to your holder...
    }
}

public class UserDiffCallback extends DiffUtil.ItemCallback<User> {

    @Override
    public boolean areItemsTheSame(@NonNull User oldUser, @NonNull User newUser) {
        return oldUser.getUserId().equals(newUser.getUserId());
    }

    @Override
    public boolean areContentsTheSame(@NonNull User oldUser, @NonNull User newUser) {
        // No need to check the equality for all User fields ; just check the equality for fields that change the display of your item.
        // In your case, both impact the display.
        return oldUser.getUserId().equals(newUser.getUserId()) 
                && (oldUser.getImageUrl() == null) ? newUser.getImageUrl() == null : oldUser.getImageUrl().equals(newUser.getImageUrl());
    }
}

然后,当您需要用新用户更新列表时,请致电myAdapter.submitList(newList)。就像AsyncListDiffer 一样,两个列表之间的差异是在后台线程上计算的。

【讨论】:

  • 我注意到,每当我使用 submitList 向我的 listadapter 添加更多列表时,新列表都会覆盖旧列表。这是正常的还是我做错了什么?
  • 是的,这是正常行为。当您调用submitList 时,将在后台线程上计算差异,然后将旧列表替换为新列表。
  • 感谢您的回复,这意味着 ListAdapter 可以实现无休止的滚动,因为它不会添加到上一个列表中。我一直在努力解决问题....非常感谢您的反馈
  • 如果我理解得很好,你想实现一个RecyclerView.Adapter,在你滚动时加载新项目? ListAdapter 仍然可以:将前一个列表中的项目与新列表合并,然后使用合并列表调用 submitList
  • 谢谢,我从来没想过。
【解决方案3】:

看起来
public void onBindViewHolder(@NonNull ViewHolder holder, int position, @NonNull List&lt;Object&gt; payloads) 未实现 实现这一点以正确使用 DiffUtils,因为将调用此方法进行更改,并且根据有效负载,您可以更新您的 recyclerview 项目,而不是调用 notifyDataSetChanged()

【讨论】:

    【解决方案4】:

    修改你的方法:

        public void updateList(ArrayList<User> newList) {
            DiffUtil.DiffResult diffResult = DiffUtil.calculateDiff(new MyDiffUtilCallBack(this.mDataset, newList));
            this.mDataSet.clear()
            this.mDataSet.addAll(newList)
            diffResult.dispatchUpdatesTo(this);
        }
    

    【讨论】:

      猜你喜欢
      • 2021-07-25
      • 2011-09-08
      • 2016-05-28
      • 1970-01-01
      • 1970-01-01
      • 2012-06-14
      • 1970-01-01
      • 1970-01-01
      • 1970-01-01
      相关资源
      最近更新 更多