Использование AsyncTask для загрузки изображений в ListView

У меня есть один ListView, который может содержать изображение. Это зависит от того, существует ли изображение в SDCARD или нет.

Вот мой пример кода:

public class MainActivity extends Activity  {

    ListView mListView;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        mListView = new ListView(this);
        setContentView(mListView);

        String[] arr = new String[] { 
                "/example/images/1.jpg", "/example/images/2.jpg",  
                "/example/images/3.jpg", "/example/images/4.jpg",  
                "/example/images/5.jpg", "/example/images/6.jpg", 
                "/example/images/7.jpg", "/example/images/8.jpg",  
                "/example/images/9.jpg", "/example/images/1.jpg", 
                "/example/images/2.jpg", "/example/images/3.jpg",  
                "/example/images/4.jpg", "/example/images/5.jpg",  
                "/example/images/6.jpg", "/example/images/7.jpg",  
                "/example/images/8.jpg", "/example/images/9.jpg", 
                "/example/images/1.jpg", "/example/images/2.jpg",  
                "/example/images/3.jpg", "/example/images/4.jpg",  
                "/example/images/5.jpg", "/example/images/6.jpg", 
                "/example/images/7.jpg", "/example/images/8.jpg",  
                "/example/images/9.jpg", "/example/images/1.jpg", 
                "/example/images/2.jpg", "/example/images/3.jpg",  
                "/example/images/4.jpg", "/example/images/5.jpg",  
                "/example/images/6.jpg", "/example/images/7.jpg",  
                "/example/images/8.jpg", "/example/images/9.jpg"}; 

        List<String> list = Arrays.asList(arr);

        MyAdapter adapter = new MyAdapter(this, R.layout.listitem_imv, list);

        mListView.setAdapter(adapter);
    }

    class MyAdapter extends ArrayAdapter<String>{

        List<String> mList;
        LayoutInflater mInflater;
        int mResource;

        public MyAdapter(Context context, int resource,
                List<String> objects) {
            super(context, resource, objects);

            mResource = resource;
            mInflater = getLayoutInflater();
            mList = objects;
        }

        @Override
        public View getView(int position, View convertView, ViewGroup parent) {
            View view;

            if(convertView == null){
                view = mInflater.inflate(mResource, null);
            }else{
                view = convertView;
            }

            ImageView imageView = (ImageView) view.findViewById(R.id.imv);
            TextView textView = (TextView) view.findViewById(R.id.txv);

                            imageView.setTag(mList.get(position));//tag of imageView == path to image
            new LoadImage().execute(imageView);
            textView.setText(mList.get(position).toString());

            return view;
        }       
    }

    class LoadImage extends AsyncTask<Object, Void, Bitmap>{

        private ImageView imv;
        private String path;


        @Override
        protected Bitmap doInBackground(Object... params) {
            imv = (ImageView)   params[0];

            path = imv.getTag().toString();

            Bitmap bitmap = null;
            File file = new File( 
                    Environment.getExternalStorageDirectory().getAbsolutePath() + path);

            if(file.exists()){
                bitmap = BitmapFactory.decodeFile(file.getAbsolutePath());
            }

            return bitmap;
        }
        @Override
        protected void onPostExecute(Bitmap result) {
            if(result != null && imv != null){
                imv.setVisibility(View.VISIBLE);
                imv.setImageBitmap(result);
            }else{
                imv.setVisibility(View.GONE);
            }
        }
    }
}

Каталог «sdcard/example/images» содержит изображения: 1.jpg, 2.jpg, 3.jpg, 4.jpg, 6.jpg, 7.jpg и 9.jpg. ожидаемый результат: example

Но если я быстро прокручиваю список, некоторые изображения вставляются не в те элементы. Это происходит из-за использования convertView в методе getView().

Если я использую следующий код, код работает нормально:

        //if(convertView == null){
        //  view = mInflater.inflate(mResource, null);
        //}else{
        //  view = convertView;
        //}
        view = mInflater.inflate(mResource, null);

При быстрой прокрутке списка две асинхронные задачи могут ссылаться на одно и то же представление из-за использования convertView. Как я могу отменить AsyncTask, когда представление больше не отображается? (и используется другим элементом ListView)

изменить

            @Override
    protected void onPostExecute(Bitmap result) {
        if(result != null && imv != null){

            if(imv.getTag().equals(path)){
                imv.setVisibility(View.VISIBLE);
                imv.setImageBitmap(result);
            }else{
                imv.setVisibility(View.GONE);
            }

        }else{
            imv.setVisibility(View.GONE);
        }
    }

person Rodrigo    schedule 11.10.2011    source источник


Ответы (6)


Вы можете отправить ImageView в конструктор задач и сохранить там ссылку на путь к изображению. Теперь в onPostExecute проверьте, совпадает ли текущий тег ImageView с тем, с которого вы начали. Если да, то установите образ. Если нет, ничего не делайте.

Однако это означает, что образ будет загружен в любом случае. Вы просто не установите неправильное изображение на вид.

EDIT: сначала передайте ImageView конструктору задач:

new LoadImage(imageView).execute()

Затем сохраните ссылку на ImageView и путь к изображению в конструкторе LoadImage. Важно сохранить путь в конструкторе, а не в doInBackground, чтобы избежать проблем с многопоточностью. Затем в onPostExecute мы проверяем текущий путь.

class LoadImage extends AsyncTask<Object, Void, Bitmap>{

        private ImageView imv;
        private String path;

        public LoadImage(ImageView imv) {
             this.imv = imv;
             this.path = imv.getTag().toString();
        }

    @Override
    protected Bitmap doInBackground(Object... params) {
        Bitmap bitmap = null;
        File file = new File( 
                Environment.getExternalStorageDirectory().getAbsolutePath() + path);

        if(file.exists()){
            bitmap = BitmapFactory.decodeFile(file.getAbsolutePath());
        }

        return bitmap;
    }
    @Override
    protected void onPostExecute(Bitmap result) {
        if (!imv.getTag().toString().equals(path)) {
               /* The path is not same. This means that this
                  image view is handled by some other async task. 
                  We don't do anything and return. */
               return;
        }

        if(result != null && imv != null){
            imv.setVisibility(View.VISIBLE);
            imv.setImageBitmap(result);
        }else{
            imv.setVisibility(View.GONE);
        }
    }

}
person shabyasachi    schedule 11.10.2011
comment
Я попробовал это. см. редактирование. Но изображения по-прежнему помещаются в неправильные элементы. - person Rodrigo; 11.10.2011
comment
@Rodrigo Также, если вы кешируете загруженное изображение перед проверкой правильности, это означает, что это не напрасная загрузка, поскольку ее можно использовать для других ваших задач. - person Blundell; 19.08.2012
comment
Да, @Droider, но посмотрите на дату сообщения. - person Rodrigo; 25.02.2013
comment
@shabyasachi это решение неверно, я не понимаю голосов. ListView работает иначе, он всегда использует одни и те же элементы представления для отображения содержимого, а при прокрутке использует переработанные представления для отображения новых элементов. Это решение ВООБЩЕ НЕПРАВИЛЬНО. - person Didac Perez Parera; 28.08.2013
comment
@DídacPérez: посмотрите большой комментарий в onPostExecute. Этот комментарий объясняет, как обрабатывается повторное использование представлений. - person shabyasachi; 28.05.2014
comment
Получилось.. Спасибо, теперь список загружается быстрее. - person TPG; 21.12.2014
comment
Если вы используете многоразовые представления, вы можете взять WeakReference в свой ImageView и проверить тег в ImageView перед загрузкой в ​​него растровых изображений. Я надеюсь, что это поможет всем, кто использует многоразовые представления. - person Salman Khakwani; 19.10.2016

Этот запись в блоге разработчиков Android содержит полную справочную информацию. проект для этого в комплекте с кэшированием. Просто замените код доступа Http на чтение файла SD-карты.

person Jeff Axelrod    schedule 30.07.2012

Надеюсь, это поможет. После долгих поисков у меня есть это рабочее решение.

public class CustomAdapter extends ArrayAdapter<String>{
    /*
    public CustomAdapter(Context context , String[] video) {
        super(context,R.layout.custom_row, video);
    }
*/

private final Activity context;
private final String[] video;

static class ViewHolder {
    public TextView videoTitle;
    public ImageView videoThumbnail;
    public int position;
    public String path;
}

public CustomAdapter(Activity context, String[] video) {
    super(context, R.layout.custom_row, video);
    this.context = context;
    this.video = video;
}
@Override
public View getView(final int position, View convertView, ViewGroup parent) {

    LayoutInflater videoInflator = LayoutInflater.from(getContext());
    View customView = videoInflator.inflate(R.layout.custom_row, parent, false);
        ViewHolder viewHolder = new ViewHolder();
        viewHolder.position = position;
        viewHolder.path = video[position];
        viewHolder.videoTitle = (TextView) customView.findViewById(R.id.videoTitle);
        viewHolder.videoThumbnail = (ImageView) customView.findViewById(R.id.videoThumbnail);
        //rowView.setTag(viewHolder);
    //}
    customView.setTag(viewHolder);


    final String videoItem = video[position];
    int index=videoItem.lastIndexOf('/');
    String lastString=(videoItem.substring(index +1));
    index = lastString.indexOf(".mp4");
    lastString=(lastString.substring(0,index));
    viewHolder.videoTitle.setText(lastString);

    new AsyncTask<ViewHolder, Void, Bitmap>() {
        private ViewHolder v;

        @Override
        protected Bitmap doInBackground(ViewHolder... params) {
            v = params[0];
            Bitmap thumb = ThumbnailUtils.createVideoThumbnail(videoItem, MediaStore.Images.Thumbnails.MINI_KIND);
            return thumb;
        }

        @Override
        protected void onPostExecute(Bitmap result) {
            super.onPostExecute(result);
            if (v.position == position) {
                // If this item hasn't been recycled already, hide the
                // progress and set and show the image
                v.videoThumbnail.setImageBitmap(result);
            }
        }
    }.execute(viewHolder);
    return customView;
}

}

person Amit    schedule 30.05.2016

Возможно, вам стоит попробовать:

view = mInflater.inflate(mResource,parent,null);

Проверьте этот блог, он объясняет аналогичную проблему:

http://www.doubleencore.com/2013/05/layout-inflation-as-intended/

person Munawwar Hussain Shelia    schedule 03.07.2014

Что бы я сделал (если у вас нет тысяч изображений): 1. создать структуру данных - простой класс, содержащий отображаемое имя String и растровое изображение 2. создать для него адаптер 3. в методе getView назначить правильное растровое изображение к правильному ImageView.

В вашем случае вы можете создать аналогичную структуру данных, но не растровое изображение, а AsyncTask. В любом случае вам нужно связать асинтаск со строкой в ​​один элемент. Массив (или массив) таких элементов будет передан вашему адаптеру. Отображается изображение и текст.

AsyncTask можно отменить с помощью cancel().

person Yar    schedule 11.10.2011

Эй, я нашел решение этой проблемы, просто используйте следующую функцию вместо вашей функции

 @Override
protected void onPostExecute(Bitmap result) {

    if (!imv.getTag().toString().equals(rec_id)) {
           return;
    }

    if(result != null && imv != null){
        int index = id.indexOf(imv.getTag().toString());
        if(list.getFirstVisiblePosition()<=index && index<=list.getLastVisiblePosition())
        {
            imv.setVisibility(View.VISIBLE);
            imv.setImageBitmap(result);
        }
    }else{
        imv.setImageBitmap(icon);
        imv.setVisibility(View.GONE);
    }
}

Здесь список является объектом просмотра списка. Просто передайте объект представления списка вашему адаптеру и вставьте эту функцию вместо функции onPostExecute.

person Sandeep    schedule 09.11.2012