0

Использую Adapter для RecyclerView. И вроде все работает и показывает, но есть странность, на 15 элементе когда листаешь вниз он меняет текст(со следующих элементов), и все последующие элементы начинают повторяться по имени. То есть 16 элемент уже имеет значение 0 элемента, семнадцатый равен первому элементу. С чем это может быть связано?

Использую во фрагменте

    mRecyclerView = (RecyclerView) view.findViewById(R.id.resurs);
    mRecyclerView.setHasFixedSize(true);
    mRecyclerView.setLayoutManager(new LinearLayoutManager(getActivity()));
    mAdapter = new Adapter(locationTanki.getTexnika_pokaz().getList_texnika(), getActivity());
    mRecyclerView.setAdapter(mAdapter);
    mAdapter.notifyDataSetChanged();

Сам Adapter

private class Adapter extends RecyclerView.Adapter<ViewHolder> {

    public List<Texnika> Texnika_list;
    public Context mcontext;
    ViewHolder viewHolder;

    public Adapter(List<Texnika> Texnika_list, Context context) {
        this.Texnika_list = Texnika_list;
        this.mcontext = context;
    }

    @Override
    public ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
        // create a layout
        View itemLayoutView = LayoutInflater.from(parent.getContext()).inflate(
                R.layout.items_texnika_pokaz_content, null);

        viewHolder = new ViewHolder(itemLayoutView);
        return viewHolder;
    }

    @Override
    public void onBindViewHolder(ViewHolder holder, final int position) {
        viewHolder.checkBox.setText(Texnika_list.get(position).getName());
        viewHolder.checkBox.setChecked(Texnika_list.get(position).isVisible());

        viewHolder.checkBox.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
            @Override
            public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
                if(isChecked)
                    Texnika_list.get(position).setVisible(true);
                else {
                    Texnika_list.get(position).setVisible(false);
                }
            }
        });
    }

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

    public List<Texnika> getTexnika_list() {
        return Texnika_list;
    }
}

Сам ViewHolder

 public static class ViewHolder extends RecyclerView.ViewHolder {

    public CheckBox checkBox;

    public ViewHolder(View itemLayoutView) {
        super(itemLayoutView);

        checkBox = (CheckBox) itemLayoutView.findViewById(R.id.checkBox);
    }
}
6
  • Очень советую использовать стиль при написании кода и придерживаться правил, очень тяжело читать. Прошу прощения за вопрос, но вы уверены, что в списке все правильно приходит? 14 дек 2017 в 12:28
  • @iamthevoid, хорошо учту насчет правил, вы правы. Насчет списка проверял, список правильный приходит. Допустим я снимаю флажок на первом элементе, то это дублируется и на 16 элементе 14 дек 2017 в 13:39
  • По приведенному тут коду все должно работать. Нет возможности выложить куда нибудь проект? 14 дек 2017 в 13:50
  • @iamthevoid, ru.files.fm/u/jjezkqga вот добавил 14 дек 2017 в 17:35
  • !!! Не надо так делать. Вы знаете для чего нужен ресайклер вью? Не нужно хранить вью холдер в адаптере. Мне это даже в голову не пришло, пока я адаптер не переписал. Условно ресайклер - ресайклит, то есть используется одна вью для всего, она и возвращается в onBind. В памяти их хранится в момент времени столько, сколько на экране, а не по количеству элементов списка. и в новые в onBind сетятся параметры. Не надо сохранять, возвращайте в onCreateViewHolder new и все будет хорошо. Можете поискать туториалы, как раньше это делали с лист вью и посмотреть видео, тогда станет понятнее 15 дек 2017 в 6:38

1 ответ 1

-1

Текст меняется на элементах при прокрутке, потому что в некоторых из них у вас не указан никакой текст. А так как RecyclerView использует элементы повторно, то старый текст бывает сохраняется. То же самое касается и флажков.

Чтобы решить проблему, перед биндом элементов выставляйте состояние полей на дефолтное.

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.