记一次内存泄露排查

最后在实现一个无限循环的ViewPager，展示图片，功能实现了，但是运行一段时间之后会挂掉。

多亏了AndroidStudio的Memory Monitor，发现了内存一直在增长。

怎么触发gc内存都不会减少，确定了内存泄露了，但是不知哪里出问题了。

 

一时想到的排查内存泄露的工具，就是MAT了，但是没找到AndroidStudio的MAT插件。

只能先把java heap dump出来先，如下图所示

dump出来之后，hprof文件会保存在项目下captures目录，之前一直不知到，找了很久。。。

如果切换到Captures这个tab，是可以直接看到HeapSnapshot的，如下图所示

但是这个hprof文件mat不认，需要转换一下，点击hprof文件右键，转成标准.hprof文件即可

 

 

然后到eclipse用mat插件打开，如果没有安装mat插件请自行搜索解决

window->open perspective->memory analysis

在memory analysis界面下

File->Open Heap Dump->选中上面Android Studio转换之后的.hprof文件即可

在OverView下面点击Top Consumers, 如下图。byte占了大头，内存基本都给它用了

byte占用这么多内存，一想到的就是bitmap没被释放了。

回去研究我的代码

PagerAdapter代码如下

public class SlidePicPagerAdapter extends PagerAdapter {

    private List<SlidePicModel> mItems;
    private int itemSize;

    public void setItems(List<SlidePicModel> items) {
        mItems = items;
        itemSize = items.size();
    }

    @Override
    public int getCount() {
        return ListUtils.isEmpty(mItems) ? 0 : Integer.MAX_VALUE;
    }

    @Override
    public boolean isViewFromObject(View view, Object object) {
        return view == object;
    }

    @Override
    public Object instantiateItem(ViewGroup container, int position) {
        Context context = container.getContext();
        SlidePicModel item = mItems.get(getCurPos(position));
        ImageView iv = item.getImageView();
        if(iv == null){
            iv = new ImageView(context);
            item.setImageView(iv);
        }
        final String imgUrl = ImageUrlExtends.getImageUrl(item.getUrl());
        Picasso.with(context).load(imgUrl).into(iv);
        container.addView(iv);
        return iv;
    }

    @Override
    public void destroyItem(ViewGroup container, int position, Object object) {
        View imageView = (View) object;
        container.removeView(imageView);
    }


    private int getCurPos(int pos){
        return pos % mItems.size();
    }
}

 我用的是Picasso去加载图片。首先把picasso加载图片给屏蔽了，发现内存正常了，擦，以为我发现了个bug，先去给Picasso提Bug去呢。

再看看我代码，第28行，我把ImageView给缓存到数据源SlidePicModel的成员变量里了。

SlidePicModel代码如下

public class SlidePicModel {

    private ImageView imageView;
    private String url;

    public SlidePicModel(String url) {
        this.url = url;
    }

    public ImageView getImageView() {
        return imageView;
    }

    public void setImageView(ImageView imageView) {
        this.imageView = imageView;
    }

    public String getUrl() {
        return url;
    }

    public void setUrl(String url) {
        this.url = url;
    }
}

 

要想想，我这个数据源mItems里面代表好几百张图片，而我的应用是无限循环显示图片，也即是我的数据库mItems是不会回收的。

本来显示完图片，ImageView是要被回收的，但是，却被我数据源里面的model引用着，几百张图片的bitmap被引用着，不能回收，app肯定内存溢出。

解决方法如下，把ImageView这个成员变量设置成弱引用，当内存不足时，ImageView可以被内存回收。问题解决，代码如下

public class SlidePicModel {


    private String url;
    private WeakReference<ImageView> mImageViewRef;
    public SlidePicModel(String url) {
        this.url = url;
    }

    public ImageView getImageView() {
        return mImageViewRef == null ? null : mImageViewRef.get();
    }

    public void setImageView(ImageView imageView) {
//        this.imageView = imageView;
        mImageViewRef = new WeakReference<ImageView>(imageView);
    }

    public String getUrl() {
        return url;
    }

    public void setUrl(String url) {
        this.url = url;
    }
}

 内存回收正常。图片如下

当然了，更好的办法就是，这个ImageView成员变量完全没有存在的必要，但是当时一时把代码写错，出刚好让我研究了一下内存泄露，对内存泄露有更深的认识。

 

总结：

之前就有看到一些文章在建议，不要把Activity当成静态成员变量。

其实从我上面出现的问题里就可以发现一样的道理。上面是因为数据源一直存在，没有被内存回收，所以引用的ImageView也没有被回收，导致内存溢出。

同理，如果把Activity当成静态成员变量。那么它的生命周期就会和app的生命周期一样，Activity里所引用的对象都不会被释放，即使activity已经结束了，这就会导致内存举出。

使用context的时候，能使用ApplicationContext就使用AplicationContext，如果把Activity当成context传给别人，要注意内存泄露，具体看我另一篇博客

这里还说明另外一个问题，就是变量，能不用成员变量就尽量不要用，不然，一旦内存泄露，会把成员变量的内存也泄露了，我这里就是成员变量导致的内存泄露