Утечки памяти Image-Gallery Android - Как другие приложения могут справиться с этим?

Question

Я пытаюсь реализовать галерею изображений, которая должна показывать ~ 5-15 меньших изображений и одно "текущее выбранное" большее изображение.

Это выглядит так: http://www.mobisoftinfotech.com/blog/wp-content/uploads/2012/06/galleryDemo.png

Я просмотрел много источников и теперь решил использовать растровый кеш (lru-cache) (спасибо человеку с этого форума!).

У меня нет утечек памяти в данный момент, но я не доволен этим решением, потому что каждый раз, когда я выполняю прокрутку, некоторые изображения удаляются из кеша, и мне приходится их перезагружать... так что пользователь должен ждать перезагрузка изображений... Это действительно раздражает ждать 0,5 - 1 секунду каждый раз, когда я перехожу на другую сторону...

public static WeakReference<Bitmap> getBitmap(String imageName, int width,
        int height) {
    if (!imageName.contains(".jpg")){
        imageName += ".jpg";
    }
    String pathToImage = getPathToImage(imageName);
    Options options = new BitmapFactory.Options();
    options.inJustDecodeBounds = true;
    BitmapFactory.decodeFile(pathToImage, options);

    /*
     * Calculate inSampleSize
     */
    options.inSampleSize = calculateInSampleSize(options, width, height);

    /*
     * Removes the alpha-channel from bitmap (not necessary)
     */
    options.inPreferredConfig = Bitmap.Config.RGB_565;

    /*
     * Decode bitmap with inSampleSize set
     */
    options.inJustDecodeBounds = false;

    WeakReference<Bitmap> scaledBitmap = new WeakReference<Bitmap>(
            BitmapFactory.decodeFile(pathToImage, options));
    return scaledBitmap;

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

Действительно странным является тот факт, что не имеет значения, насколько велики разрешения изображений, lru-cache все равно удалит некоторые изображения, даже если я выберу только изображения размером 64x64... Возможно, я что-то не так делаю с кеш?

Следующий код показывает мою реализацию:

    /*
     * Bitmaps which should be shown
     */
    mMemoryCache = (BitmapCache) getLastNonConfigurationInstance();
    if (mMemoryCache == null) {
        // Get max available VM memory, exceeding this amount will throw an
        // OutOfMemory exception. Stored in kilobytes as LruCache takes an
        // int in its constructor.
        final int maxMemory = (int) (Runtime.getRuntime().maxMemory() / 1024);

        // Use 1/4th of the available memory for this memory cache.
        final int cacheSize = maxMemory / 2; // TODO default 8

        mMemoryCache = new BitmapCache(cacheSize);
    }

И Класс BitmapCache:

public BitmapCache(int maxSize, Resources resources) {
    super(maxSize);
    this.resources = resources;
}

@Override
protected int sizeOf(String key, AsyncDrawable value) {
    if (value.getBitmap() != null) {
        /* 
         * The cache size will be measured in kilobytes rather than
         * number of items.
         */
        return value.getBitmap().getRowBytes()
                * value.getBitmap().getHeight() / 1024;
    }
    return super.sizeOf(key, value);
}

public void loadBitmap(Picture picture, ImageView mImageView,
        ImageResolution resolution) {
    if (cancelPotentialWork(picture.getFileName(), //  + resolution.name()
            mImageView)) {
        final BitmapWorkerTask task = new BitmapWorkerTask(this,
                mImageView, picture, resolution);
        final AsyncDrawable asyncDrawable = new AsyncDrawable(resources,
                null, task);
        mImageView.setImageDrawable(asyncDrawable);
        task.execute();
    }
}

Большое спасибо:(

Еще немного кода: BitmapWorkerTask:

@Override
protected Bitmap doInBackground(Void... params) {
    /*
     *  Decode image in background.
     */
    final Bitmap bitmap = FileHandler.getBitmap(
            picture.getPictureId() + "", resolution).get();
    return bitmap;
}

@Override
protected void onPostExecute(Bitmap bitmap) {
    /*
     *  Once complete, see if ImageView is still around and set bitmap.
     */
    if (isCancelled()) {
        bitmap = null;
    }

    if (imageViewReference != null && bitmap != null) {
        final ImageView imageView = imageViewReference.get();

        if (imageView != null) {
            final Drawable drawable = imageView.getDrawable();
            if (drawable instanceof AsyncDrawable) {
                final AsyncDrawable asyncDrawable = (AsyncDrawable) drawable;
                final BitmapWorkerTask bitmapWorkerTask = asyncDrawable
                        .getBitmapWorkerTask();
                if (this == bitmapWorkerTask && imageView != null) {
                    imageView.setImageBitmap(bitmap);
                    bitmapCache.addBitmapToMemoryCache(
                            picture.getFileName(), // + resolution.name()
                            asyncDrawable);
                }
            }
        }
    }
}

Answer 1

КЛАССНО! Я нашел ошибку, почему это не сработало. Ошибка была в моем LruCache, который хранил AsyncDrawables, что совершенно глупо. Каждый раз, когда вызывался sizeOf в LruCache, растровое изображение внутри asyncdrawable было нулевым, поэтому он просто возвращал super.sizeOf(key, value).

Теперь это работает как шарм!:)

Спасибо за вашу помощь!