Кеширование изображений на SD карте

В методе fetchImage Вы только что изобрели AsyncTask.

> Activity activity — активити приложения, нужно для определения пути для хранения кешированных файлов

Так все таки нужно Activity или достаточно Context?
http://developer.android.com/reference/android/content/Context.html#getExternalCacheDir()

Еще можно сделать второй уровень: HashMap
В качестве ключей — URL картинок.

Мне кажется есть некоторые нестыковки.
1. getExternalCacheDir — вернет null если, например, телефон подключен через usb как девайс или если места недостаточно — думаю приложение упадет в данном случае
2. Нет проверки размера файла и ресайза — думаю приложение упадет по переполнению буфера если попытается отобразить картинку 2048*1024
3. Как то странно (если не сказать тупо) организовано время хранения файла. Ожидал чего то вроде чтения хидеров из http заголовков и выставления времени на основании expired cashe time или как там.
4. Нет таймаута на запрос — зависнет?
5. Как справедливо замечено выше — нет HashMap c ссылками для отображения изображений, например в listview, а ведь именно там и начинаются все пляски с onLowMemoryReceived, очистки кеша, recycle и т.д.

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

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

} catch (Exception e) {
e.printStackTrace();
}

Не айс…

Дело в том, что я писал статью "Пишем функцию сохранения картинок на SD-карту", а не «Качаем картинки из Инета и потом куда-нибудь их сохраняем при этом кэшируя». Вероятно по этому ваши ожидания не оправдались.
С другой стороны — это же хорошо. Неоправдавшиеся ожидания привели к новой статье с дополнительными деталями и объяснениями некоторых неохваченных доселе вопросов.

Что же до моего проекта, я в нем не мудрствую лукаво, и для скачивания картинок из интернета пользуюсь специальным довольно стандартным классом (AsyncTask, да) такого вот содержания:

public class ImageDownloader extends AsyncTask<String, Void, Bitmap> {

    private String url;
    private final WeakReference<ImageView> imageViewReference;
   
    public ImageDownloader(ImageView imageView) {
        imageViewReference = new WeakReference<ImageView>(imageView);
    }

// эта функция качает файло
    @Override
    protected Bitmap doInBackground(String... params) {
    	url = params[0];
    	URL myFileUrl = null;
    	HttpURLConnection conn;
    	InputStream is;
    	
 		try {
 			myFileUrl= new URL(url);
 		} catch (MalformedURLException e) { 			
 			e.printStackTrace();
 			return null;
 		}
 		try {
 			conn = (HttpURLConnection)myFileUrl.openConnection();
 			conn.setDoInput(true);
 			conn.connect();
 			is = conn.getInputStream();
 			return BitmapFactory.decodeStream(is); 							 		
 		catch (Exception e)
 		{
 			e.printStackTrace(); 			
 			is = null;
			conn = null; 		
 			return null;
 		}
    }

// эта функция показывает скачанный файл в ImageView
    @Override
    protected void onPostExecute(Bitmap result) {
        if (isCancelled()) {
            result = null;
        }
        if (imageViewReference != null) {
            ImageView imageView = imageViewReference.get();
            if (imageView != null) {
                imageView.setImageBitmap(result);
            }
        }
    }

// Эта функция вызывается для того, чтобы показать пользователю что-то, пока грузится картинка.
// Например песочные часы или лучше прогрессбар
    @Override
    protected void onPreExecute() {
        if (imageViewReference != null) {
            ImageView imageView = imageViewReference.get();
            if (imageView != null) {
                imageView.setImageResource(R.drawable.icon);
            }
        }
    }
}

Этого вполне достаточно, чтобы асинхронно грузить картинки из интернета не тормозя UI.
Нужно просто создать новый класс, записать в него вышеприведенный код и сохранить его как отдельный файл в проекте. А вызывается он всего тремя строчками в коде:

ImageView iv;
ImageDownloader downloader = new ImageDownloader(iv);
downloader.execute(pictureURL);

У вас проблемы с многопоточностью, и то что Vector засинхронизирован вас не спасёт. findObject никак не защищён от того, что во время цикла элемент может быть удылён другим потоком. И почему список downloaded это список ImageView (т.е. куда качаем), а не собственно того что мы качаем? Одна и та же картинка для разных ImageView будет качаться отдельно?

Использовал ваш код у себя. У меня >300 изображений грузится во время обновления. В принципе они не большие, не более 500кб. Но весь процесс происходит до ужаса долго. Каждая картинка льется по 2-3 минуты. Не в курсе что могло послужить этому причиной?