LeakCanary内存泄漏优化

接手了一个屎坑代码 在有限的改动下修改泄漏问题
以下是我收集的一下泄漏问题及修改办法
常见的内存泄漏主要有以下4种
1.“静态实例” 造成内存泄漏

┬───                                                                                                     
│ GC Root: System class                                                                                  
│                                                                                                        
├─ android.net.ConnectivityManager class                                                                 
│    Leaking: NO (a class is never leaking)                                                              
│    ↓ static ConnectivityManager.sInstance                                                              
│                                 ~~~~~~~~~                                                              
├─ android.net.ConnectivityManager instance                                                              
│    Leaking: UNKNOWN                                                                                    
│    Retaining 114 B in 5 objects                                                                        
│    mContext instance of com.MainActivity with mDestroyed = true                      
│    ↓ ConnectivityManager.mContext                                                                      
│                          ~~~~~~~~                                                                      
╰→ com.MainActivity instance                                                           
     Leaking: YES (ObjectWatcher was watching this because com.MainActivity received  
     Activity#onDestroy() callback and Activity#mDestroyed is true)                                     
     Retaining 27.8 MB in 17166 objects                                                                 
     key = ecff8e2d-70c4-4cbf-99f1-d02005ddd4f2                                                         
     watchDurationMillis = 7209                                                                         
     retainedDurationMillis = 2207                                                                      
     mApplication instance of com.Application                                   
     mBase instance of androidx.appcompat.view.ContextThemeWrapper                                      
实际代码和修改办法

这种的改法其实和很简单只要getApplicationContext即可
原因你这个方法都是静态的如果你传入的Context是非Application的那么它将会因为这个静态而不会释放 所以只要改成Application Context即可

另外一种DialogFragment引起的泄漏


DialogFragment泄漏

这种就极其脑抽 解决也很简单 首先DialogFragment这玩意是个Fragment 也就是说他有自己的Context


写法和改法

正常是Dialog里面持有外部的Context 比如 有个静态的Context常量 这是我们正常的理解
但是这东西就反过来这是外部持有了另外一个伪装的Fragment 所以解决办法就是给他搞个弱应用或者匿名对象

GC Root: Input or output parameters in native code 

├─ dalvik.system.PathClassLoader instance
 │    Leaking: NO (m↓ is not leaking and A ClassLoader is never leaking)
 │    ↓ ClassLoader.runtimeInternalObjects
 ┬───                                                                                                                          
 │ GC Root: Global variable in native code                                                                                     
 │                                                                                                                             
 ├─ dalvik.system.PathClassLoader instance                                                                                     
 │    Leaking: NO (IntentUtil↓ is not leaking and A ClassLoader is never leaking)                                              
 │    ↓ ClassLoader.runtimeInternalObjects                                                                                     
 ├─ java.lang.Object[] array                                                                                                   
 │    Leaking: NO (IntentUtil↓ is not leaking)                                                                                 
 │    ↓ Object[4169]                                                                                                           
 ├─ com.IntentUtil class                                                                          
 │    Leaking: NO (a class is never leaking)                                                                                   
 │    ↓ static IntentUtil.context                                                                                              
 │                        ~~~~~~~                                                                                              
 ╰→ com.MineMdActivity instance                                                  
      Leaking: YES (ObjectWatcher was watching this because com.miaodou.                
      MineMdActivity received Activity#onDestroy() callback and Activity#mDestroyed is true)                                  
      Retaining 9.7 MB in 2450 objects                                                                                        
      key = 366658d5-9533-4aa5-85ed-1397537a9089                                                                              
      watchDurationMillis = 31774                                                                                             
      retainedDurationMillis = 26773                                                                                          
      mContractView instance of com.miaodou.MineMdActivity with mDestroyed = true       
      context instance of com.MineMdActivity with mDestroyed = true             
      mApplication instance of com.sbds.secondvoice.MiaoyinApplication                                                        
      mBase instance of androidx.appcompat.view.ContextThemeWrapper                                                           
                                                                                                                               
这就是大聪明

为了静态初始化 为了个单例 写成这样 另外我也实在不懂一个Intent也要写个单例工具类干嘛 警告都不看
很多程序员有一个坏习惯 警告都觉得无所谓 实际上很多问题很难解决找不到原因 就是警告没解决
解决办法也简单 不要单例 不要静态即可

┬───                                                                                                                    
│ GC Root: Thread object                                                                                                
│                                                                                                                       
├─ java.lang.Thread instance                                                                                            
│    Leaking: NO (the main thread always runs)                                                                          
│    Thread name: 'main'                                                                                                
│    ↓ Thread.threadLocals                                                                                              
│             ~~~~~~~~~~~~                                                                                              
├─ java.lang.ThreadLocal$ThreadLocalMap instance                                                                        
│    Leaking: UNKNOWN                                                                                                   
│    Retaining 7.4 kB in 74 objects                                                                                     
│    ↓ ThreadLocal$ThreadLocalMap.table                                                                                 
│                                 ~~~~~                                                                                 
├─ java.lang.ThreadLocal$ThreadLocalMap$Entry[] array                                                                   
│    Leaking: UNKNOWN                                                                                                   
│    Retaining 7.3 kB in 73 objects                                                                                     
│    ↓ ThreadLocal$ThreadLocalMap$Entry[18]                                                                             
│                                      ~~~~                                                                             
├─ java.lang.ThreadLocal$ThreadLocalMap$Entry instance                                                                  
│    Leaking: UNKNOWN                                                                                                   
│    Retaining 28 B in 1 objects                                                                                        
│    ↓ ThreadLocal$ThreadLocalMap$Entry.value                                                                           
│                                       ~~~~~                                                                           
├─ android.animation.AnimationHandler instance                                                                          
│    Leaking: UNKNOWN                                                                                                   
│    Retaining 5.0 MB in 7251 objects                                                                                   
│    ↓ AnimationHandler.mAnimationCallbacks                                                                             
│                       ~~~~~~~~~~~~~~~~~~~                                                                             
├─ java.util.ArrayList instance                                                                                         
│    Leaking: UNKNOWN                                                                                                   
│    Retaining 5.0 MB in 7247 objects                                                                                   
│    ↓ ArrayList[0]                                                                                                     
│               ~~~                                                                                                     
├─ android.animation.ObjectAnimator instance                                                                            
│    Leaking: UNKNOWN                                                                                                   
│    Retaining 5.0 MB in 7245 objects                                                                                   
│    mListeners = null                                                                                                  
│    mPropertyName = rotation                                                                                           
│    mProperty = null                                                                                                   
│    mInitialized = true                                                                                                
│    mStarted = true                                                                                                    
│    mRunning = true                                                                                                    
│    mAnimationEndRequested = false                                                                                     
│    mDuration = 3000                                                                                                   
│    mStartDelay = 0                                                                                                    
│    mRepeatCount = INFINITE (-1)                                                                                       
│    mRepeatMode = RESTART (1)                                                                                          
│    ↓ ObjectAnimator.mTarget                                                                                           
│                     ~~~~~~~                                                                                           
├─ androidx.appcompat.widget.AppCompatImageView instance                                                                
│    Leaking: YES (View.mContext references a destroyed activity)                                                       
│    Retaining 5.0 MB in 7231 objects                                                                                   
│    View not part of a window view hierarchy                                                                           
│    View.mAttachInfo is null (view detached)                                                                           
│    View.mID = R.id.iv_record                                                                                          
│    View.mWindowAttachCount = 0                                                                                        
│    mContext instance of com.ShortVideoMainActivity with mDestroyed = true             
│    ↓ View.mContext                                                                                                    
╰→ com.ShortVideoMainActivity instance                                                  
     Leaking: YES (ObjectWatcher was watching this because com.ShortVideoMainActivity  
     received Activity#onDestroy() callback and Activity#mDestroyed is true)                                           
     Retaining 64.8 kB in 1152 objects                                                                                 
     key = e14565c2-166a-4e0a-9dd3-07f981f9f18d                                                                        
     watchDurationMillis = 5820                                                                                        
     retainedDurationMillis = 819                                                                                      
     context instance of com.ShortVideoMainActivity with mDestroyed = true             
     mApplication instance of com.Application                                                  
     mBase instance of androidx.appcompat.view.ContextThemeWrapper                                                     

查找问题发生位置

我们就能很快速的找到那个类 根据 View.mID = R.id.iv_record 知道那个控件泄漏并定位到代码

        rotation = ObjectAnimator.ofFloat(ivRecord, "rotation", 0, 360f);
        rotation.setRepeatCount(ValueAnimator.INFINITE);
        rotation.setRepeatMode(ValueAnimator.RESTART);
        rotation.setInterpolator(new LinearInterpolator());
        rotation.setDuration(3000).start();

解决方案 :

然后看代码 这个ObjectAnimator是一直在播放所以我们应该在页面销毁的时候cancel掉他
但是好巧不巧这玩意是在adapter里面的一个自定义view 那么怎么释放呢
应该这样重写onDetachedFromWindow方法然后释放

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

推荐阅读更多精彩内容