1

У меня в приложении есть диалог в котором отображается список прикрепленных к письму файлов. Список заполняется данными из массива в котором находятся элементы в формате json. Дальше я пытаюсь сделать удаление по свайпу. Если я правильно понял, то удалятся будет не только элемент из массива но и элемент из списка. Все вроде хорошо, но есть несколько проблем. При удалении элемента у меня в списке остается пустое место и список не обновляется. А после повторного вызова диалога со списком у меня вообще список ничем не заполняется хотя в логах показывает что массив не пустой. Вот как я вызываю диалог со списком прикрепленных файлов:

attachment.setOnLongClickListener(new View.OnLongClickListener() {
            @Override
            public boolean onLongClick(View view) {
                if (array.size() > 0) {
                    if (movieList.size() > 0) {
                        movieList.clear();
                        showDialog(SHOW_ATTACHED_FILES);
                        prepareList();
                    } else {
                        showDialog(SHOW_ATTACHED_FILES);
                        prepareList();
                    }


                } else {
                    Toast.makeText(WriteResponseMess.this, "NO attached files", Toast.LENGTH_SHORT).show();
                }
                return true;
            }
        });

и вот построение диалога:

dialog = new Dialog(WriteResponseMess.this);
dialog.setContentView(R.layout.show_attachment);
dialog.setCanceledOnTouchOutside(true);

final RecyclerView recyclerView = dialog.findViewById(R.id.lv);

final AttachmentAdapter mAdapter = new AttachmentAdapter(movieList);
RecyclerView.LayoutManager mLayoutManager = new LinearLayoutManager(getBaseContext());
recyclerView.setLayoutManager(mLayoutManager);
recyclerView.setItemAnimator(new DefaultItemAnimator());
recyclerView.setAdapter(mAdapter);

mAdapter.notifyDataSetChanged();

ItemTouchHelper.SimpleCallback simpleCallback = new ItemTouchHelper.SimpleCallback(0, ItemTouchHelper.LEFT) {
@Override
public boolean onMove(@NonNull RecyclerView recyclerView, @NonNull RecyclerView.ViewHolder viewHolder, @NonNull RecyclerView.ViewHolder viewHolder1) {
return false;
}

@Override
public void onSwiped(@NonNull RecyclerView.ViewHolder viewHolder, int position) {
for (int i = 0; i < array.size(); i++) {
array.remove(i);
}
}
};

new ItemTouchHelper(simpleCallback).attachToRecyclerView(recyclerView);

вот функция которая заполняет список:

public void prepareList() {
        for (int i = 0; i < array.size(); i++) {
            JsonObject object = array.get(i).getAsJsonObject();
            Log.w("MY_TAG", String.valueOf(object.get("filename")));
            Log.w("MY_TAG", String.valueOf(object.get("data")));

            Integer data = object.get("data").toString().getBytes().length;
            movieList.add(new FileList(String.valueOf(object.get("filename")), String.valueOf(data)));
        }
    }

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

update

   @Override
public void onSwiped(@NonNull RecyclerView.ViewHolder viewHolder, int position) {
for (int i = 0; i < array.size(); i++) {
array.remove(i); // удаляем запись из источника (как я понял)
mAdapter.notifyItemRemoved(position); // уведомляем адаптер об удалении
movieList.remove(position); // удаляем запись из коллекции адаптера ---- здесь ошибка 
}
}

код ошибки:

java.lang.IndexOutOfBoundsException: Index: 4, Size: 4
6
  • Цикл-то зачем? Вы же хотите удалить один элемент а не все разом.
    – woesss
    21 ноя 2018 в 13:58
  • да но я так понял что индексы в списке не соответствуют индексам массива, или это не так? просто если цикл не вставлять то другая ошибка
    – Andrew
    21 ноя 2018 в 13:58
  • Вы скопируйте мой код. У Вас в цикле удаляется одна позиция (position) и когда в списке остаётся меньше элементов чем position и выстреливает ошибка.
    – woesss
    21 ноя 2018 в 14:02
  • @woesss, я скопировал, цикл убрал, все сделал так как у вас, но все-равно ошибку кидает - java.lang.IndexOutOfBoundsException: Index: 4, Size: 4 хотя все как у вас
    – Andrew
    21 ноя 2018 в 14:08
  • 1
    Вы ввели меня в заблуждение именем параметра - это ж не позиция а направление свайпа. Обновил код.
    – woesss
    21 ноя 2018 в 14:18

1 ответ 1

1

После удаления элемента наверное надо уведомить адаптер, чтобы он перерисовался: mAdapter.notifyItemRemoved(position). Только непонятно что Вы делаете в onSwiped...
Может имелось ввиду это:

@Override
public void onSwiped(@NonNull RecyclerView.ViewHolder viewHolder, int direction) {
    int position = viewHolder.getAdapterPosition();
    array.remove(position); // удаляем запись из источника (как я понял)
    movieList.remove(position); // удаляем запись из коллекции адаптера
    mAdapter.notifyItemRemoved(position); // уведомляем адаптер об удалении
}

Update: Если я правильно понял смысл Вашего кода, то так должно работать:

attachment.setOnLongClickListener(new View.OnLongClickListener() {
            @Override
            public boolean onLongClick(View view) {
                if (array.size() > 0) {
                    showDialog(SHOW_ATTACHED_FILES);
                } else {
                    Toast.makeText(WriteResponseMess.this, "NO attached files", Toast.LENGTH_SHORT).show();
                }
                return true;
            }
        });

final RecyclerView recyclerView = dialog.findViewById(R.id.lv);
prepareList();
final AttachmentAdapter mAdapter = new AttachmentAdapter(movieList);
RecyclerView.LayoutManager mLayoutManager = new LinearLayoutManager(getBaseContext());
recyclerView.setLayoutManager(mLayoutManager);
recyclerView.setItemAnimator(new DefaultItemAnimator());
recyclerView.setAdapter(mAdapter);

public void prepareList() {
    movieList.clear();
    for (int i = 0; i < array.size(); i++) {
        JsonObject object = array.get(i).getAsJsonObject();
        Log.w("MY_TAG", String.valueOf(object.get("filename")));
        Log.w("MY_TAG", String.valueOf(object.get("data")));

        Integer data = object.get("data").toString().getBytes().length;
        movieList.add(new FileList(String.valueOf(object.get("filename")), String.valueOf(data)));
    }
}
6
  • не работает ваш метод, кидает fatal_exception - java.lang.IndexOutOfBoundsException: Index: 4, Size: 4 на строку array.remove(position)
    – Andrew
    21 ноя 2018 в 13:07
  • А что за условие в attachment.setOnLongClickListener? Из-за него адаптер может отображать не только то что есть в array - отсюда и выход за пределы массива.
    – woesss
    21 ноя 2018 в 13:26
  • если этого условия не писать, то при каждом длительном нажатии у меня добавляется в список все элементы по новой, в итоге каждого элемента там по несколько штук может быть
    – Andrew
    21 ноя 2018 в 13:27
  • Если Вам нужно показать только содержимое array - уберите условие if (movieList.size() > 0) и очищайте movieList.clear() безусловно. Хотя похоже, что нарушена последовательность действий и диалог показывает не актуальное содержимое.
    – woesss
    21 ноя 2018 в 13:38
  • да это решило часть проблем, но приложение крашится все-равно, закинул удаление элемента в цикл, но теперь обновление адаптера выбрасывает меня
    – Andrew
    21 ноя 2018 в 13:44

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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