RecyclerView устанавливает данные последнего элемента ArrayList на первый элемент - PullRequest
6 голосов
/ 24 марта 2020

У меня есть RecyclerView и адаптер с моделью ArrayList of Person. Когда приложение запускается, я набираю имена в EditText и нажимаю кнопку:

1) enter image description here

2) enter image description here

3) enter image description here

4) enter image description here

Но когда я регистрирую данные personList в действии, результат:

personList: Item 0 -----> Alireza
personList: Item 1 -----> Tohid
personList: Item 2 -----> Alireza

Первый элемент удаляется, а последний элемент повторяется в первом. (Мой адаптер более сложный, но я сократил код и представления, чтобы сосредоточиться на основной проблеме)

Расположение адаптера:

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:orientation="vertical">

<TextView
    android:id="@+id/tv_name"
    android:layout_width="match_parent"
    android:layout_height="wrap_content" />

<EditText
    android:id="@+id/et_name"
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:hint="Name" />

<Button
    android:id="@+id/button"
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:text="Add" />

</LinearLayout>

Класс адаптера:

public class TestAdapter extends RecyclerView.Adapter<RecyclerView.ViewHolder> {

private ArrayList<Person> mList;
private View.OnClickListener listener;

public TestAdapter(ArrayList<Person> mList, View.OnClickListener listener) {
    this.mList = mList;
    this.listener = listener;
}

@NonNull
@Override
public RecyclerView.ViewHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {
    View itemView = LayoutInflater.from(parent.getContext()).inflate(R.layout.test_adapter_item, parent, false);

    if (viewType == Person.State.FIRST.getValue()) {
        return new FirstViewHolder(itemView);
    } else { // if (viewType == Person.State.SECOND.getValue())
        return new SecondViewHolder(itemView);
    }
}

@Override
public void onBindViewHolder(@NonNull RecyclerView.ViewHolder holder, int position) {

    final int pos = holder.getAdapterPosition();
    final Person person = mList.get(pos);

    if (holder instanceof FirstViewHolder) {

        final FirstViewHolder h = (FirstViewHolder) holder;
        h.tvName.setVisibility(View.GONE);
        h.etName.setVisibility(View.VISIBLE);
        h.etName.setText("");
        h.etName.addTextChangedListener(new TextWatcher() {
            @Override
            public void beforeTextChanged(CharSequence s, int start, int count, int after) {

            }

            @Override
            public void onTextChanged(CharSequence s, int start, int before, int count) {
                person.setFirstName(s.toString());
            }

            @Override
            public void afterTextChanged(Editable s) {

            }
        });

        h.btnAccept.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                person.setState(Person.State.SECOND);
                notifyItemChanged(pos);

                listener.onClick(v);
            }
        });

    } else { // if (holder instanceof SecondViewHolder)

        final SecondViewHolder h = (SecondViewHolder) holder;
        h.tvName.setVisibility(View.VISIBLE);
        h.tvName.setText(person.getFirstName());
        h.etName.setVisibility(View.GONE);
        h.btnAccept.setVisibility(View.GONE);
    }
}

private static class FirstViewHolder extends RecyclerView.ViewHolder {

    private TextView tvName;
    private EditText etName;
    private Button btnAccept;

    private FirstViewHolder(@NonNull View itemView) {
        super(itemView);

        tvName = itemView.findViewById(R.id.tv_name);
        etName = itemView.findViewById(R.id.et_name);
        btnAccept = itemView.findViewById(R.id.button);
    }
}

private static class SecondViewHolder extends RecyclerView.ViewHolder {

    private TextView tvName;
    private EditText etName;
    private Button btnAccept;

    SecondViewHolder(@NonNull View itemView) {
        super(itemView);

        tvName = itemView.findViewById(R.id.tv_name);
        etName = itemView.findViewById(R.id.et_name);
        btnAccept = itemView.findViewById(R.id.button);
    }
}

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

@Override
public int getItemViewType(int position) {
    return mList.get(position).getState().getValue();
}
}

Связанные части упражнения:

ArrayList<Person> personList = new ArrayList<>();
personList.add(new Person("", Person.State.FIRST));
adapter = new TestAdapter(personList, new View.OnClickListener() {
        @Override
        public void onClick(View v) {
            if (personList.size() < 3) {
                personList.add(new Person("", Person.State.FIRST));
                adapter.notifyItemInserted(personList.size());
            } else {
                for (int i = 0; i < personList.size(); i++) {
                    Log.i("personList", "Item " + i + " -----> " + personList.get(i).getName());
                }
            }
        }
    });

RecyclerView recyclerView = findViewById(R.id.recycler_view);
recyclerView.setLayoutManager(new LinearLayoutManager(this));
recyclerView.setAdapter(adapter);

В чем проблема? И как мне получить правильные и реальные данные ArrayList?

1 Ответ

4 голосов
/ 28 марта 2020

Ваша проблема связана с addTextChangedListener, когда новые элементы добавляются в представление рециркулятора, старые TextChangedListeners не удаляются. и они продолжают изменять значения элементов списка. Вы должны удалить этих слушателей, когда состояние изменяется на ВТОРОЕ состояние (что удивительно сложно посмотреть на этот Как удалить всех слушателей, добавленных с помощью addTextChangedListener ). или вы должны добавить эту проверку при срабатывании onTextChanged. if (person.getState() == Person.State.FIRST) person.setFirstName(s.toString());

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...