关于Glide出现trying to use a recycled bitmap错误的问题分析

老衲上个月在清理项目历史遗留下的疑难问题的时候,被分配了这么了个诡异的bug,当快速进入退出我们项目的时候,如果执行的次数量级够大,则会出现

Glide : trying to use a recycled bitmap balabala......

奇怪的是,其他同事在图片的使用时,已经按照Glide的要求在各个地方去释放内存了。如下

Glide.get(this).clearMemory();

但是,该问题依然如跗骨之蛆一般存在。没法办,只好去撸源码找原因。

clearMemory()方法的内部实现逻辑是

 // Engine asserts this anyway when removing resources, fail faster and consistently
Util.assertMainThread();
// memory cache needs to be cleared before bitmap pool to clear re-pooled Bitmaps too. See #687.
memoryCache.clearMemory();
bitmapPool.clearMemory();
arrayPool.clearMemory();

刚开始的思路是,LruCache内的清理逻辑出了问题。先看下它内部的实现逻辑

Map.Entry<T, Y> last;
Iterator<Map.Entry<T, Y>> cacheIterator;
while (currentSize > size) {
  cacheIterator  = cache.entrySet().iterator();
  last = cacheIterator.next();
  final Y toRemove = last.getValue();
  currentSize -= getSize(toRemove);
  final T key = last.getKey();
  cacheIterator.remove();
  onItemEvicted(key, toRemove);
}

Lru本质就是个LinkedHashMap,本质就是hashmap的对象清理逻辑,瞅了一顿也没发现有啥不对。于是把目标转移到了Glide的精髓BitmapPool

Glide的一个最重要的思想就是对象池的使用,占内存一样的两个bitmap会进行内存的复用,因为首页图片的尺寸和格式都是同样的,这里肯定会进行bitmap的复用。所以接下来重点BitmapPool的实现逻辑。

BitmapPool有三个实现类


image.png

不解释,直接看LruBitmapPool的clearMemory方法

@Override
public void clearMemory() {
if (Log.isLoggable(TAG, Log.DEBUG)) {
  Log.d(TAG, "clearMemory");
}
trimToSize(0);

}

直接找trimToSize(0)

private synchronized void trimToSize(long size) {
while (currentSize > size) {
  final Bitmap removed = strategy.removeLast();
  // TODO: This shouldn't ever happen, see #331.
  if (removed == null) {
    if (Log.isLoggable(TAG, Log.WARN)) {
      Log.w(TAG, "Size mismatch, resetting");
      dumpUnchecked();
    }
    currentSize = 0;
    return;
  }
  // 划重点!!!!!
  tracker.remove(removed);
  currentSize -= strategy.getSize(removed);
  evictions++;
  if (Log.isLoggable(TAG, Log.DEBUG)) {
    Log.d(TAG, "Evicting bitmap=" + strategy.logBitmap(removed));
  }
  dump();
  removed.recycle();
}

}

重点就在于 tracker.remove(removed)这句代码, tracker???这特么是个毛?继续跟

private interface BitmapTracker {
  void add(Bitmap bitmap);
  void remove(Bitmap bitmap);
}

这也简单,其实就是一个专门用于添加和删除bitmap的一个工具类,重点来了。。看下LruBitmapPool构造方法里BitmapTracker的实现类:

LruBitmapPool(long maxSize, LruPoolStrategy strategy, Set<Bitmap.Config> allowedConfigs) {
this.initialMaxSize = maxSize;
this.maxSize = maxSize;
this.strategy = strategy;
this.allowedConfigs = allowedConfigs;
this.tracker = new NullBitmapTracker();

}

继续跟NullBitmapTracker:

private static final class NullBitmapTracker implements BitmapTracker {

  @Synthetic
  NullBitmapTracker() { }

  @Override
  public void add(Bitmap bitmap) {
    // Do nothing.
  }

@Override
public void remove(Bitmap bitmap) {
    // Do nothing.
  }
}

Do nothing...
noting...
...

Emmm.....我特么....

所以可能的原因是bitmap依然在BitmapPool中,但是已经被recycled了,导致下次使用了复用的bitmap。

解决思路也简单,禁用掉BitmapPool对象池,或者给LruBitmapPool自己设置一个自定义Tracker的实现类。

对于常用的开源框架,还是建议同行们多了解下内部的实现逻辑,关键时候派得上用场。

©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 214,377评论 6 496
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 91,390评论 3 389
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 159,967评论 0 349
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 57,344评论 1 288
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 66,441评论 6 386
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 50,492评论 1 292
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 39,497评论 3 412
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 38,274评论 0 269
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,732评论 1 307
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 37,008评论 2 328
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 39,184评论 1 342
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 34,837评论 4 337
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 40,520评论 3 322
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 31,156评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,407评论 1 268
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 47,056评论 2 365
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 44,074评论 2 352

推荐阅读更多精彩内容

  • 我希望通过这篇文章能够把Android内存相关的基础和大部分内存相关问题如:溢出、泄漏、图片等等产生的都讲解清楚,...
    Cactus_b245阅读 7,129评论 6 82
  • Glide笔记 一、简介 在泰国举行的谷歌开发者论坛上,谷歌为我们介绍了一个名叫Glide的图片加载库,作者是bu...
    AndroidMaster阅读 3,891评论 0 27
  • 7.1 压缩图片 一、基础知识 1、图片的格式 jpg:最常见的图片格式。色彩还原度比较好,可以支持适当压缩后保持...
    AndroidMaster阅读 2,500评论 0 13
  • Glide的使用 Glide是google开发用于Android加载媒体的类库,包括图片,gif,video,已经...
    敖大胆阅读 7,786评论 2 17
  • 文/熠歆 材料:鸡胸肉,胡萝卜,香芹,青椒,葱头,黄瓜(苤纳充当),大蒜,姜,醋,生抽,耗油,小粉 腌制20分(大...
    熠歆阅读 297评论 4 2