记一次内存泄露优化

前言

dialog引起的内存泄露

最近在优化项目发现好几个地方出现了内存泄露

我封装了一个TipDialog，用在网络请求时候显示，请求结束时消息。上代码：

    private void initDialog(Activity activity) {

        dialog = (TipDialog) new TipDialog(activity)
                .build()
                .setTipText("正在加载")
                .setIconId(R.drawable.icon_load)
                .setOrientationHorizontal(false)
                .setIsLoading(true)
                .show();

    }

    public DialogCallback(Activity activity) {
        super();
        initDialog(activity);
    }

    @Override
    public void onStart(Request<T, ? extends Request> request) {
        super.onStart(request);
    }

    @Override
    public void onFinish() {
        //网络请求结束后关闭对话框
        if (dialog != null ) {
            dialog.dismissDialog();
        }
        
    }

TipDialog:

package com.yeqiu.hydrautils.view.dialog;

import android.animation.ObjectAnimator;
import android.app.Activity;
import android.content.DialogInterface;
import android.support.v4.content.ContextCompat;
import android.text.TextUtils;
import android.view.Gravity;
import android.view.View;
import android.view.WindowManager;
import android.view.animation.LinearInterpolator;
import android.widget.ImageView;
import android.widget.LinearLayout;
import android.widget.RelativeLayout;

import com.yeqiu.hydrautils.R;
import com.yeqiu.hydrautils.common.DensityUtils;
import com.yeqiu.hydrautils.ui.widget.marquee.MarqueeTextView;
import com.yeqiu.hydrautils.view.dialog.base.BaseDialog;

/**
 * @project：HailHydra
 * @author：小卷子
 * @date 2018/9/10
 * @describe：
 * @fix：
 */
public class TipDialog extends BaseDialog {

    private ObjectAnimator animator;
    private ImageView imageView;
    private LinearLayout llRoot;

    public TipDialog(Activity context) {
        super(context);
    }


    @Override
    protected int getstyle() {

        return R.style.TipDialog;
    }

    @Override
    protected Object getDiaologlayoutIdOrView() {

        return R.layout.layout_tip_dialog;
    }

    @Override
    protected void initView(View view) {

        llRoot = (LinearLayout) view.findViewById(R.id.ll_tip_dialog_root);
        if (dialogBuilder.getOrientationHorizontal()) {
            llRoot.setOrientation(LinearLayout.HORIZONTAL);
            RelativeLayout.LayoutParams rootLayoutParams = (RelativeLayout.LayoutParams) llRoot
                    .getLayoutParams();
            rootLayoutParams.width = WindowManager.LayoutParams.WRAP_CONTENT;
            rootLayoutParams.height = WindowManager.LayoutParams.WRAP_CONTENT;
            llRoot.setLayoutParams(rootLayoutParams);

        } else {
            llRoot.setOrientation(LinearLayout.VERTICAL);
            RelativeLayout.LayoutParams rootLayoutParams = (RelativeLayout.LayoutParams) llRoot
                    .getLayoutParams();
            rootLayoutParams.width = DensityUtils.dp2px(100);
            rootLayoutParams.height = DensityUtils.dp2px(100);
            llRoot.setLayoutParams(rootLayoutParams);
        }
        if (dialogBuilder.getIconId() != -999) {
            imageView = new ImageView(context);
            LinearLayout.LayoutParams imageViewLP = new LinearLayout.LayoutParams(DensityUtils
                    .dp2px(30), DensityUtils.dp2px(30));
            imageView.setLayoutParams(imageViewLP);
            imageView.setImageResource(dialogBuilder.getIconId());
            llRoot.addView(imageView);
        }
        if (!TextUtils.isEmpty(dialogBuilder.getTipText())) {
            MarqueeTextView textView = new MarqueeTextView(context);
            LinearLayout.LayoutParams tipViewLP = new LinearLayout.LayoutParams(LinearLayout
                    .LayoutParams.WRAP_CONTENT, LinearLayout.LayoutParams.WRAP_CONTENT);
            //垂直布局
            tipViewLP.topMargin = dialogBuilder.getOrientationHorizontal() ? 0 : DensityUtils.dp2px(10);

            //水平布局
            tipViewLP.leftMargin = dialogBuilder.getOrientationHorizontal() ? DensityUtils.dp2px  (10) : 0;
            textView.setLayoutParams(tipViewLP);
            textView.setEllipsize(TextUtils.TruncateAt.MARQUEE);
            textView.setSingleLine();
            textView.setGravity(Gravity.CENTER);
            textView.setTextColor(ContextCompat.getColor(context, R.color.color_white));
            textView.setTextSize(14);
            textView.setText(dialogBuilder.getTipText());

            llRoot.addView(textView);
        }

    }

    public void startAnimate() {

        if (imageView != null) {
            animator = ObjectAnimator.ofFloat(imageView, "rotation",
                    0f, 360f);
            animator.setDuration(1000);
            animator.setInterpolator(new LinearInterpolator());
            animator.setRepeatCount(ObjectAnimator.INFINITE);
            animator.setRepeatMode(ObjectAnimator.RESTART);
        }
        animator.start();

    }


    @Override
    public void show() {

        if (context == null || context.isFinishing()) {
            return;
        }

        super.show();

        if (dialogBuilder.isLoading()) {
            startAnimate();
        }

        int dismissTime = dialogBuilder.getDismissTime();

        if (dismissTime != 0 && llRoot != null) {
            llRoot.postDelayed(new Runnable() {
                @Override
                public void run() {
                    dismiss();
                }
            }, dismissTime);
        }


        dialog.setOnDismissListener(new DialogInterface.OnDismissListener() {
            @Override
            public void onDismiss(DialogInterface dialog) {
                if (animator != null) {
                    animator.end();
                }
            }
        });
    }

    public void dismiss() {
        if (dialog != null && context != null && !context.isFinishing()) {
            dialog.dismiss();
        }
    }
}

泄露的信息：

img

看上去似乎没问题，但是在我快速进入退出页面的时候，leakcanary就报出了泄露的地方，TipDialog.context。提示消息很好理解，当页面消失的时候 TipDialog持有了context，这里的context就是当前Activity。TipDialog没有销毁导致Activity无法回收。

知道问题就开始分析，我想在dialog消失或者网络请求结束的时候把context置为null。再次运行，并没有效果。猜想可能是我封装的dialog有问题，仔细检查好像似乎并没有问题。我尝试用系统的ProgressDialog。

private void initDialog(Activity activity) {
        dialog = new ProgressDialog(activity);
        dialog.requestWindowFeature(Window.FEATURE_NO_TITLE);
        dialog.setCanceledOnTouchOutside(false);
        dialog.setProgressStyle(ProgressDialog.STYLE_SPINNER);
        dialog.setMessage("请求网络中...");
    }

    public DialogCallback(Activity activity) {
        super();
        initDialog(activity);
    }

    @Override
    public void onStart(Request<T, ? extends Request> request) {
        if (dialog != null && !dialog.isShowing()) {
            dialog.show();
        }
    }

    @Override
    public void onFinish() {
        //网络请求结束后关闭对话框
        if (dialog != null && dialog.isShowing()) {
            dialog.dismiss();
        }
    }

点击运行，嗯 果然没有泄露。继续看泄露信息，message.obj 这个似乎好像是和Handler有关。可是我的代码里都没有用过Handler。最后在网上看到一篇博文介绍了dialog监听的实现方式。系统内部通过Handler来回调事件。博文里使用的是 DialogInterface.OnClickListener()造成内存泄露，最后的解决办法是做了一个包裹类，在dialog消失的时把 DialogInterface.OnClickListener()置为null。分析一波后就知道我这里泄露的原因，因为我设置dialog消失的监听，在消失后停止转圈的动画。

 dialog.setOnDismissListener(new DialogInterface.OnDismissListener() {
            @Override
            public void onDismiss(DialogInterface dialog) {
                if (animator != null) {
                    animator.end();
                }
            }
        });
    }

我尝试在dialog的onDismiss监听里把OnDismissListener置为null，但是还是没有解决。这里贴出我最后的解决办法

    public void clearOnDetach(final Dialog dialog) {

        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.JELLY_BEAN_MR2) {
            dialog.getWindow()
                    .getDecorView()
                    .getViewTreeObserver()
                    .addOnWindowAttachListener(new ViewTreeObserver.OnWindowAttachListener() {
                        @Override
                        public void onWindowAttached() {
                        }

                        @Override
                        public void onWindowDetached() {

                            onDialogDismiss();
                            
                        }
                    });
        }

    }

    protected void onDialogDismiss() {
        if (dialogBuilder.getDialogListener() != null) {
            dialogBuilder.getDialogListener().onDialogDismiss();
        }

    }

这里不再设置dialog消失的监听，但是对dialog的window做了脱离视图的监听。重新测试，没有在出现泄漏。代码已经加入HailHydr豪华午餐。

相关资料：

一个内存泄漏引发的血案-Square

HailHydra

最后编辑于 ：
©著作权归作者所有,转载或内容合作请联系作者
  • 人面猴
    序言：七十年代末，一起剥皮案震惊了整个滨河市，随后出现的几起案子，更是在滨河造成了极大的恐慌，老刑警刘岩，带你破解...
    沈念sama阅读 220,295评论 6赞 512
  • 死咒
    序言：滨河连续发生了三起死亡事件，死亡现场离奇诡异，居然都是意外死亡，警方通过查阅死者的电脑和手机，发现死者居然都...
    沈念sama阅读 93,928评论 3赞 396
  • 救了他两次的神仙让他今天三更去死
    文/潘晓璐 我一进店门，熙熙楼的掌柜王于贵愁眉苦脸地迎上来，“玉大人，你说我怎么就摊上这事。” “怎么了？”我有些...
    开封第一讲书人阅读 166,682评论 0赞 357
  • 道士缉凶录：失踪的卖姜人
    文/不坏的土叔 我叫张陵，是天一观的道长。 经常有香客问我，道长，这世上最难降的妖魔是什么？ 我笑而不...
    开封第一讲书人阅读 59,209评论 1赞 295
  • 港岛之恋（遗憾婚礼）
    正文 为了忘掉前任，我火速办了婚礼，结果婚礼上，老公的妹妹穿的比我还像新娘。我一直安慰自己，他们只是感情好，可当我...
    茶点故事阅读 68,237评论 6赞 397
  • 恶毒庶女顶嫁案：这布局不是一般人想出来的
    文/花漫 我一把揭开白布。 她就那样静静地躺着，像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上，一...
    开封第一讲书人阅读 51,965评论 1赞 308
  • 城市分裂传说
    那天，我揣着相机与录音，去河边找鬼。 笑死，一个胖子当着我的面吹牛，可吹牛的内容都是我干的。 我是一名探鬼主播，决...
    沈念sama阅读 40,586评论 3赞 420
  • 双鸳鸯连环套：你想象不到人心有多黑
    文/苍兰香墨 我猛地睁开眼，长吁一口气：“原来是场噩梦啊……” “哼！你这毒妇竟也来了？” 一声冷哼从身侧响起，我...
    开封第一讲书人阅读 39,487评论 0赞 276
  • 万荣杀人案实录
    序言：老挝万荣一对情侣失踪，失踪者是张志新（化名）和其女友刘颖，没想到半个月后，有当地人在树林里发现了一具尸体，经...
    沈念sama阅读 46,016评论 1赞 319
  • 护林员之死
    正文 独居荒郊野岭守林人离奇死亡，尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 38,136评论 3赞 340
  • 白月光启示录
    正文 我和宋清朗相恋三年，在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 40,271评论 1赞 352
  • 活死人
    序言：一个原本活蹦乱跳的男人离奇死亡，死状恐怖，灵堂内的尸体忽然破棺而出，到底是诈尸还是另有隐情，我是刑警宁泽，带...
    沈念sama阅读 35,948评论 5赞 347
  • 日本核电站爆炸内幕
    正文 年R本政府宣布，位于F岛的核电站，受9级特大地震影响，放射性物质发生泄漏。R本人自食恶果不足惜，却给世界环境...
    茶点故事阅读 41,619评论 3赞 331
  • 男人毒药：我在死后第九天来索命
    文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹，春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 32,139评论 0赞 23
  • 一桩弑父案，背后竟有这般阴谋
    文/苍兰香墨 我抬头看了看天上的太阳。三九已至，却和暖如春，着一层夹袄步出监牢的瞬间，已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 33,252评论 1赞 272
  • 情欲美人皮
    我被黑心中介骗来泰国打工， 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留，地道东北人。 一个月前我还...
    沈念sama阅读 48,598评论 3赞 375
  • 代替公主和亲
    正文 我出身青楼，却偏偏与公主长得像，于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子，可洞房花烛夜当晚...
    茶点故事阅读 45,267评论 2赞 358