Что не так с моей реализацией DiffUtil? - PullRequest
0 голосов
/ 12 января 2019

Обновление : Одна из проблем решена: теперь updateList решена, проблема заключалась в том, что я определил mAdapter как RecyclerView.Adapter вместо MyAdapter. Но теперь, хотя я получаю данные, в списке ничего не отображается, он пуст

-------------------- ОРИГИНАЛЬНЫЙ ПОЧТА ------------------- -

Я хочу обновить RecyclerView, используя DiffUtil, чтобы избежать дублирования.

У меня есть 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;
    }
}

Это - то, как я устанавливаю данные динамически (я продолжаю получать новые 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(?)

Что я делаю не так?

Вот как я определяю 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);

Ответы [ 2 ]

0 голосов
/ 13 января 2019

В дополнение к ответу @ 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, разница между двумя списками рассчитывается на фоне темы.

0 голосов
/ 12 января 2019

Проблема возникает из определения mAdapter. Вы определили его как RecyclerView.Adapter, который является суперклассом вашего MyAdapter, и он не содержит updateList(). Вы должны изменить его следующим образом:

private MyAdapter mAdapter;

Обновлено 13.01.2009:

Я переписал ваш адаптер с помощью 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;
        }
    }

}
...