Массив объекта, переопределяющего все индексы в обратном вызове AsyncTask

Question

Прочтите пару вопросов здесь о переполнении стека, но не смогли найти рабочее решение.

Мой сценарий => В методе onActivityResult - получение списка, который отправляется в Async Task для некоторой фоновой работы. После выполнения некоторой работы, он отправляет обратно список, скажем output . После получения output обратно в асинхронном обратном вызове регистрируется список output , и он работает как положено.

Теперь я хочу поместить его в Arraylist, скажем, addProductModelListTemp , чтобы я мог добавлять другие объекты. Но при добавлении других объектов в фоновом режиме, как показано в предыдущем сценарии, addProductModelListTemp получает переопределение всех своих индексов с последним списком output , полученным в асинхронном обратном вызове. Ниже приведен код, как я обновляю addProductModelListTemp . currentRowIdToInsertImages - это индекс для вставки объекта в addProductModelListTemp , который работает нормально (не включен здесь для простоты)

new CompressFileAsyncTask(new AsyncResponse() {
                                @Override
                                public void processFinish(List<Uri> output) {
                                    for (int i = 0; i < output.size(); i++) {
                                        Timber.e("Received from Async Class contains => " + output.get(i));
                                    }

                                    AddProductModel tempProductModel = new AddProductModel("", "", "", dummyList);// Dummy list is an empty list you can assume ... Tried setting it directly as well with the **output**
                                    addProductModelListTemp.add(tempProductModel);
                                    addProductModelListTemp.get(currentRowIdToInsertImages).setImagePathList(output);
                                    for (int i = 0; i < addProductModelListTemp.size(); i++) {
                                        Timber.e("addProductModelListTemp index " + i);
                                        for (int k = 0; k < addProductModelListTemp.get(i).getImagePathList().size(); k++) {
                                            Timber.e("addProductModelListTemp.get(i).getImagePathList() index " + k + " contains " + addProductModelListTemp.get(i).getImagePathList().get(k)); 
                                        }
                                    }
                                }
                            }).execute(imageURIList); // imageURIList is the list received in onActivityResult method 

Ниже приведен класс моделей, с помощью которых я создаю и обновляю объекты.

   public AddProductModel(String size, String color, String quantity, List<Uri> imagePath) {

    this.size = size;
    this.color = color;
    this.quantity = quantity;
    this.imagePath = imagePath;
}

public void setImagePathList(List<Uri> imagePath) {
    if (this.imagePath.isEmpty()) {
        Timber.e("Image Path List not exists, adding");
        this.imagePath = imagePath;
    } else {
        Timber.e("Image Path List exists, appending");
        this.imagePath.addAll(this.imagePath.size(), imagePath);
    }
} // Getter setter not attached for clarity}

Класс AsyncTask равен

class CompressFileAsyncTask extends AsyncTask<List<Uri>, Void, List<Uri>> {
    public AsyncResponse asyncResponse = null;

    public CompressFileAsyncTask(AsyncResponse asyncResponse) {
        this.asyncResponse = asyncResponse;
    }

    @Override
    protected List<Uri> doInBackground(List<Uri>... imageUriList) {
        List<Uri> fileList = imageUriList[0];

        for (int i = 0; i < fileList.size(); i++) {
            filesAdded.add(Uri.parse(compressImage(String.valueOf(fileList.get(i)))));
        }
        return filesAdded;
    }


    @Override
    protected void onPostExecute(List<Uri> uriArrayList) {
        asyncResponse.processFinish(uriArrayList);
    }
}

и сам интерфейс

public interface AsyncResponse {
public void processFinish(List<Uri> output);

}

Answer 1

Проблема в том, что вы изменяете один и тот же объект списка и сохраняете его в каждом индексе addProductModelListTemp.Вам нужно сделать filesAdded локальной переменной.

@Override
    protected List<Uri> doInBackground(List<Uri>... imageUriList) {
        List<Uri> fileList = imageUriList[0];
        List<Uri> filesAdded = new ArrayList<>();
        for (int i = 0; i < fileList.size(); i++) {
            filesAdded.add(Uri.parse(compressImage(String.valueOf(fileList.get(i)))));
        }
        return filesAdded;
    }