RecyclerView перезагрузить те же данные при обновлении

Question

У меня проблема, когда я провожу пальцем, чтобы обновить данные, первый свип в порядке, но после этого каждый свип перезагружается и добавляет одни и те же данные снова и снова, в конце у меня есть список с одними и теми же элементами снова и снова... Я использую загрузчик. Я пытался очистить раньше, но я не понимаю, что не так с моим кодом, если кто-то может мне это объяснить. Благодарю вас.

Вот мой код:

public abstract class NewsFragment extends Fragment implements LoaderManager.LoaderCallbacks<ArrayList<Articles>> {

    protected ItemAdapter mArticleAdapter;
    protected RecyclerView mRecyclerView;
    protected NewsFragment.OnNewSelectedInterface mListener;
    protected RecyclerView.LayoutManager mManager;
    protected SwipeRefreshLayout mSwipeRefreshLayout;
    protected LoaderManager mLoaderManager;
    private boolean mStateSaved;

    private static final int NEWS_LOAD_ID = 1;
    public static final String KEY_LIST = "key_list";

    public interface OnNewSelectedInterface {
        void onListNewSelected(int index, ArrayList<Articles> articles);
    }


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

        View view = inflater.inflate(R.layout.list_present_news, container, false);

        mListener = (NewsFragment.OnNewSelectedInterface) getActivity();
        mSwipeRefreshLayout = (SwipeRefreshLayout) view.findViewById(R.id.swipeContainer);
        mRecyclerView = (RecyclerView) view.findViewById(R.id.recyclerview);
        mManager = new LinearLayoutManager(getActivity());
        mArticleAdapter = new ItemAdapter(getActivity(), new ArrayList<Articles>(), mListener);
        mLoaderManager = getLoaderManager();
        mStateSaved = mArticleAdapter.isStateSaved();

        mRecyclerView.setAdapter(mArticleAdapter);
        mRecyclerView.setLayoutManager(mManager);

        getData();
        refreshData();

        if(!isNetworkAvailable())alertUserAboutError();

        setDivider();

        return view;
    }

    private void setDivider() {
        DividerItemDecoration dividerItemDecoration = new DividerItemDecoration(mRecyclerView
                .getContext(), DividerItemDecoration.VERTICAL);
        mRecyclerView.addItemDecoration(dividerItemDecoration);
    }

    private void getData() {
        getLoaderManager().initLoader(NEWS_LOAD_ID, null, this).forceLoad();
    }

    private void alertUserAboutError() {
        AlertDialogFragment alertDialogFragment = new AlertDialogFragment();
        alertDialogFragment.show(getActivity().getFragmentManager(), "error_dialog");
    }

    protected abstract String[] getUrl();

    private boolean isNetworkAvailable() {
        ConnectivityManager manager = (ConnectivityManager)
                getActivity().getSystemService(Context.CONNECTIVITY_SERVICE);
        NetworkInfo networkInfo = manager.getActiveNetworkInfo();
        boolean isAvailable = false;
        if (networkInfo != null && networkInfo.isConnected()) {
            isAvailable = true;
        }
        return isAvailable;
    }

    private void refreshData() {
        mSwipeRefreshLayout.setOnRefreshListener(new SwipeRefreshLayout.OnRefreshListener() {
            @Override
            public void onRefresh() {
                mArticleAdapter.clear();
                mSwipeRefreshLayout.setRefreshing(false);

            }
        });

        mSwipeRefreshLayout.setColorSchemeResources(
                android.R.color.holo_orange_light,
                android.R.color.holo_red_light);
    }

    @Override
    public Loader<ArrayList<Articles>> onCreateLoader(int id, Bundle args) {
        return new NewsLoader(getActivity(), getUrl());
    }

    @Override
    public void onLoadFinished(Loader<ArrayList<Articles>> loader, ArrayList<Articles> data) {
        if (data != null && !data.isEmpty()) {
            mArticleAdapter.addAll(data);
        }
    }

    @Override
    public void onLoaderReset(Loader<ArrayList<Articles>> loader) {
        mArticleAdapter.clear();
    }
}

Мой класс погрузчиков:

public class NewsLoader extends AsyncTaskLoader<ArrayList<Articles>>{

    private ArrayList<Articles> mArticlesArrayList;
    private String[] mUrl;

    public NewsLoader(Context context, String[] url) {
        super(context);
        mUrl = url;
    }

    @Override
    public ArrayList<Articles> loadInBackground() {

        OkHttpClient mClient = new OkHttpClient();
        for (String aMUrl : mUrl) {
            Request mRequest = new Request.Builder().url(aMUrl).build();
            try {
                Response response = mClient.newCall(mRequest).execute();
                try {
                    if (response.isSuccessful()) {
                        String json = response.body().string();
                        getMultipleUrls(json);
                    }
                } catch (IOException | JSONException e) {
                    e.printStackTrace();
                }
            } catch (IOException e) {
                e.printStackTrace();
            }
        }
        return mArticlesArrayList;
    }

    private void getMultipleUrls(String jsonData) throws JSONException {

        if (mArticlesArrayList == null) {
            mArticlesArrayList = getArticleForecast(jsonData);
        } else {
            mArticlesArrayList.addAll(getArticleForecast(jsonData));
        }
    }

    private ArrayList<Articles> getArticleForecast(String jsonData) throws JSONException {
        JSONObject forecast = new JSONObject(jsonData);
        JSONArray articles = forecast.getJSONArray("articles");

        ArrayList<Articles> listArticles = new ArrayList<>(articles.length());

        for (int i = 0; i < articles.length(); i++) {
            JSONObject jsonArticle = articles.getJSONObject(i);
            Articles article = new Articles();

            String urlImage = jsonArticle.getString("urlToImage");

            article.setTitle(jsonArticle.getString("title"));
            article.setDescription(jsonArticle.getString("description"));
            article.setImageView(urlImage);
            article.setArticleUrl(jsonArticle.getString("url"));

            listArticles.add(i, article);
        }

        return listArticles;
    }
}

Мой класс адаптера:

public class ItemAdapter extends RecyclerView.Adapter<ItemAdapter.ArticleViewHolder> {

    private static final String TAGO = ItemAdapter.class.getSimpleName();
    private final NewsFragment.OnNewSelectedInterface mListener;
    private ArrayList<Articles> mArticlesList;
    private Context mContext;
    private int lastPosition = -1;
    private boolean mStateSaved = false;


    public boolean isStateSaved() {
        return mStateSaved;
    }

    public void setStateSaved(boolean stateSaved) {
        mStateSaved = stateSaved;
    }

    public ItemAdapter(Context context, ArrayList<Articles> articles, NewsFragment.OnNewSelectedInterface listener){
        mContext = context;
        mArticlesList = articles;
        mListener = listener;
    }

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

        View view = LayoutInflater.from(parent.getContext()).inflate(R.layout.item_card_view, parent, false);

        ArticleViewHolder articleViewHolder = new ArticleViewHolder(view);
        articleViewHolder.setIsRecyclable(false);
        return articleViewHolder;
    }

    @Override
    public void onBindViewHolder(ArticleViewHolder holder, int position) {

        holder.bindArticle(mArticlesList.get(holder.getAdapterPosition()));
        setAnimation(holder.itemView, holder.getAdapterPosition());
    }

    private void setAnimation(View viewToAnimate, int position) {
        if (position > lastPosition) {
            Animation animation = AnimationUtils.loadAnimation(viewToAnimate.getContext(), android.R.anim.slide_in_left);
            viewToAnimate.startAnimation(animation);
            lastPosition = position;
        }

    }

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

    public void clear() {
        mArticlesList.clear();
        notifyDataSetChanged();
    }

    public void addAll(ArrayList<Articles> articles) {
        mArticlesList.addAll(articles);
        notifyDataSetChanged();
    }


    protected class ArticleViewHolder extends RecyclerView.ViewHolder implements View.OnClickListener{

        private ImageView mImageView;
        private TextView mTitleTextView, mDescriptionTextView;
        private FloatingActionButton mSaveButton;

        private ArticleViewHolder(View itemView) {
            super(itemView);

            mImageView = (ImageView) itemView.findViewById(R.id.photoImageView);
            mTitleTextView = (TextView) itemView.findViewById(R.id.titleWithoutImage);
            mDescriptionTextView = (TextView) itemView.findViewById(R.id.descriptionTextView);
            mSaveButton = (FloatingActionButton) itemView.findViewById(R.id.floatingActionButton);

            itemView.setOnClickListener(this);
        }

        private void bindArticle(final Articles article) {

            Glide.with(mContext).load(article.getImageView()).into(mImageView);
            mTitleTextView.setText(article.getTitle());
            mDescriptionTextView.setText(article.getDescription());
            if(mDescriptionTextView.getText().equals("")){
                mDescriptionTextView.setVisibility(View.GONE);
            }

            mSaveButton.setOnClickListener(new View.OnClickListener() {
                @Override
                public void onClick(View view) {
                    insertArticle(article);
                    article.setStateSaved(true);
                }
            });

            Log.v(TAGO, "Item id : " + getItemId()
                    + "Item count : " + getItemCount()
                    + "Item position : " + getAdapterPosition()
                    + String.valueOf(article.isStateSaved()));
        }

        private void insertArticle(Articles articles) {

            String title = articles.getTitle();
            String description = articles.getDescription();
            String url = articles.getArticleUrl();

            ContentValues contentValues = new ContentValues();
            contentValues.put(ArticleContract.ArticleEntry.COLUMN_TITLE_ARTICLE, title);
            contentValues.put(ArticleContract.ArticleEntry.COLUMN_DESCRIPTION_ARTICLE, description);
            contentValues.put(ArticleContract.ArticleEntry.COLUMN_URL_ARTICLE, url);

            Uri uri = mContext.getContentResolver().insert(ArticleContract.ArticleEntry.CONTENT_URI, contentValues);

            if(uri == null) {
                Log.v(TAGO, "Error");
            } else Toast.makeText(mContext, "Article Saved", Toast.LENGTH_SHORT).show();
        }

        @Override
        public void onClick(View view) {
            mListener.onListNewSelected(getLayoutPosition(), mArticlesList);
        }

    }
}

Answer 1

Ты используешь ViewHolder#setIsRecyclable неправильно; этот метод предназначен для предотвращения ViewHolder от переработки только тогда, когда в него вносятся изменения. Согласно документации:

Звонки в setIsRecyclable() всегда должен быть в паре (один вызов setIsRecyclabe(false) всегда должны быть сопоставлены с последующим вызовом setIsRecyclable(true)).

Это означает, что ни один из ваших ViewHolder объекты будут переработаны, эффективно используя RecyclerView бесполезным и не позволяющим ему повторно использовать представления при попытке привязать новые объекты к вашему RecyclerView,

Короче, удалите эту строку кода.

---

Я заметил еще несколько небольших проблем с кодом вашего адаптера, которые могут вызвать множество головных болей в будущем; поэтому я позволил себе выделить некоторые изменения, которые я бы сделал.

Просто для моего здравомыслия, я буду ссылаться на ваш Articles класс как Article,

Обычно это не очень хорошая идея, чтобы обойти Context повсюду. View перешел к вашему ViewHolder уже есть ссылка на Context, так что вы можете использовать это вместо.

Для insertArticle() код, Activity должен справиться с этим в любом случае. Таким образом, вы можете передать Article назад к Activity передавая слушателя к вашему Adapter (и впоследствии каждый ViewHolder) вместо Context,

Вы также должны рассмотреть возможность использования DiffUtil класс вместо того, чтобы просто позвонить notifyDataSetChanged(); это намного эффективнее. Просто убедитесь, что ваш Article класс реализует equals() а также hashCode() или это не будет работать.

Я не включил код анимации (который можно легко добавить обратно) или код сохраненного состояния (в основном потому, что я не знаю, что вы пытались сделать).

public class ArticleAdapter extends RecyclerView.Adapter<Article> {

    private List<Article> mData;

    private ArticleViewHolder.OnSelectedListener mOnSelectedListener;
    private ArticleViewHolder.OnSaveListener mOnSaveListener;

    public ArticleAdapter(ArticleViewHolder.OnSelectedListener onSelectedListener, ArticleViewHolder.OnSaveListener onSaveListener) {
        mOnSelectedListener = onSelectedListener;
        mOnSaveListener = onSaveListener;
        mData = new ArrayList<>();
    }

    public void replaceData(final List<Article> data) {
        final List<Article> oldData = new ArrayList<>(mData);
        mData.clear();

        if (data != null) {
            mData.addAll(data);
        }

        DiffUtil.calculateDiff(new DiffUtil.Callback() {
            @Override
            public int getOldListSize() {
                return oldData.size();
            }

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

            @Override
            public int areItemsTheSame(int oldItemPosition, int newItemPosition) {
                return oldData.get(oldItemPosition).equals(mData.get(newItemPosition));
            }

            @Override
            public boolean areContentsTheSame(int oldItemPosition, int newItemPosition) {
                return oldData.get(oldItemPosition).equals(mData.get(newItemPosition));
            }
        }).dispatchUpdatesTo(this);
    }

    @Override
    public ArticleViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
        View view = LayoutInflater.from(parent.getContext()).inflate(R.layout.item_card_view, parent, false);
        return new SelectLocationViewHolder(view, mOnSelectedListener, mOnSaveListener);
    }

    @Override
    public void onBindViewHolder(ArticleViewHolder holder, int position) {
        holder.bind(mData.get(position));
    }

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

}

---

public class ArticleViewHolder extends RecyclerView.ViewHolder {

    public interface OnSelectedListener {
        void onSelected(Article article);
    }

    public interface OnSaveListener {
        void onSave(Article article);
    }

    private View mView;
    private Article mArticle;

    private OnSelectedListener mOnSelectedListener;
    private OnSaveListener mOnSaveListener;

    private ImageView mImageView;
    private TextView mTitleTextView, mDescriptionTextView;
    private FloatingActionButton mSaveButton;

    public ArticleViewHolder(View itemView, final OnSelectedListener onSelectedListener, final OnSaveListener onSaveListener) {
        super(itemView);

        mImageView = (ImageView) itemView.findViewById(R.id.photoImageView);
        mTitleTextView = (TextView) itemView.findViewById(R.id.titleWithoutImage);
        mDescriptionTextView = (TextView) itemView.findViewById(R.id.descriptionTextView);
        mSaveButton = (FloatingActionButton) itemView.findViewById(R.id.floatingActionButton);

        mView = itemView;
        mView.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                onSelectedListener.onSelected(mArticle);
            }
        });

        mSaveButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                onSaveListener.onSave(mArticle);
            }
        });
    }

    public void bind(Article article) {
        mArticle = article;
        mTitleTextView.setText(article.getTitle());

        mDescriptionTextView.setText(article.getDescription());
        if(TextUtils.isEmpty(article.getDescription())) {
            mDescriptionTextView.setVisibility(View.GONE);
        }

        Glide.with(mView.getContext()).load(article.getImage()).into(mImageView);
    }

}

---

редактировать

Фактическая проблема заключается в том, что ваш загрузчик использует тот же ArrayList каждый раз, и продолжает добавлять новые результаты к нему.

public class NewsLoader extends AsyncTaskLoader<List<Article>> {

    private final String[] mUrls;
    private final OkHttpClient mClient;

    public NewsLoader(Context context, OkHttpClient client, String... urls) {
        super(context);
        mClient = client;
        mUrls = urls;
    }

    @Override
    public List<Article> loadInBackground() {
        List<Article> articles = new ArrayList<>();

        for (String url : mUrls) {
            Request request = new Request.Builder().url(url).build();
            try {
                Response response = mClient.newCall(request).execute();
                if (response.isSuccessful()) {
                    parseData(response.body().string(), articles);
                }
            } catch (IOException | JSONException e) {
                e.printStackTrace();
            }
        }

         return articles;
    }

    private void parseData(List<Article> articles, String data) throws JSONException {
        JSONObject forecast = new JSONObject(data);
        JSONArray a = forecast.getJSONArray("articles");

        for (int i = 0; i < a.length(); i++) {
            JSONObject o = a.getJSONObject(i);
            Article article = new Article(
                    o.getString("title"),
                    o.getString("description"),
                    o.getString("url"),
                    o.getString("urlToImage"));
            articles.add(article);
        }
    }

}

Кроме того, вы могли заметить, я сделал небольшое изменение в вашем Article конструктор. Вы должны рассмотреть возможность сделать Article класс неизменяемый, так как это позволит вам избежать ошибок при работе с многопоточностью. Это должно выглядеть примерно так:

public class Article {

    private final String mTitle;
    private final String mDescription;
    private final String mUrl;
    private final String mImageUrl;


    public Article(String title, String description, String url, String imageUrl) {
        mTitle = title;
        mDescription = description;
        mUrl = url;
        mImageUrl = imageUrl;
    }

    public String title() {
        return mTitle;
    }

    public String description() {
        return mDescription;
    }

    public String url() {
        return mUrl;
    }

    public String imageUrl() {
        return mImageUrl;
    }


    @Override
    public boolean equals(Object o) {
        if (this == o) return true;
        if (o == null || getClass() != o.getClass()) return false;

        Article other = (Article) o;

        return mTitle != null && mTitle.equals(other.mTitle) &&
                mDescription != null && mDescription.equals(other.mDescription) &&
                mUrl != null && mUrl.equals(other.mUrl) &&
                mImageUrl != null && mImageUrl.equals(other.mImageUrl);
    }

    @Override
    public int hashCode() {
        int result = mTitle != null ? mTitle.hashCode() : 0;
        result = 31 * result + (mDescription != null ? mDescription.hashCode() : 0);
        result = 31 * result + (mUrl != null ? mUrl.hashCode() : 0);
        result = 31 * result + (mImageUrl != null ? mImageUrl.hashCode() : 0);
        return result;
    }

}

Answer 2

@Override
public void onBindViewHolder(ArticleViewHolder holder, int position) {
    holder.bindArticle(mArticlesList.get(position));
    setAnimation(holder.itemView, position);
}


public void addAll(ArrayList<Articles> articles) {
    mArticlesList.clear();
    mArticlesList.addAll(articles);
    notifyDataSetChanged();
}

Если это не работает, то я думаю, что ваш API дает вам избыточные данные. Почему вы используете articleViewHolder.setIsRecyclable(false);

Еще одно место, которое может вызвать проблему

private void getMultipleUrls(String jsonData) throws JSONException {

    if (mArticlesArrayList == null) {
         mArticlesArrayList = getArticleForecast(jsonData);
    } else {
        mArticlesArrayList.addAll(getArticleForecast(jsonData));
    }
}

Вы вызываете его из цикла, добавляя данные в ваш массив. Там как-то несколько данных могут быть вставлены в ваш ArrayList