我们在android上发消息的时候,使用
Message msg=new Message()
后,发现整夜播放系统提示OOM,程序重启。将new Message()换成
Message.obtain();
后,内存泄露消失。
查了下android的source code,除了测试代码外,基本上都是使用后者方法,没有用前面的方法,确定是new Message泄露了。
继续追了下源码,终于看到问题了。就在Message.java中,可以看到问题的症结:
Message是通过一个message pool(消息池)来存储消息资源的,默认这个池子的大小是10个消息资源。代码片段:
private static Object mPoolSync = new Object(); private static Message mPool; private static int mPoolSize = 0; private static final int MAX_POOL_SIZE = 10; /** * Return a new Message instance from the global pool. Allows us to * avoid allocating new objects in many cases. */ public static Message obtain() { synchronized (mPoolSync) { if (mPool != null) { Message m = mPool; mPool = m.next; m.next = null; return m; } } return new Message(); }
这里可以看到,使用obtain()方法,会从池子里获取一个消息。 当没有可用的空余资源时,new一个。
再看一下,当looper处理完一个消息时,调用的recyle:
/** * Return a Message instance to the global pool. You MUST NOT touch * the Message after calling this function -- it has effectively been * freed. */ public void recycle() { synchronized (mPoolSync) { if (mPoolSize < MAX_POOL_SIZE) { clearForRecycle(); next = mPool; mPool = this; } } }
这里的意思就是,当looper用完一个消息后,调用recycle,当当前的pool size没达到MAX_POOL_SIZE的时候,将此消息归还给消息池子。
这里看起来也没啥问题,但通篇搜索了google的源代码,也没看到在哪里修改了mPoolSize。如果不改,这个值一直为0,那么不管是通过new Message()还是通过Message.obtain()方法获取的Message,都归还给Message Pool了。通过new出来的message就无法被GC掉,这样就是内存泄露!!
一个可行的修改方法(没测试)
public static Message obtain() { synchronized (mPoolSync) { if (mPool != null) { Message m = mPool; mPool = m.next; m.next = null; mPoolSize --; return m; } } return new Message(); }
public void recycle() { synchronized (mPoolSync) { if (mPoolSize < MAX_POOL_SIZE) { mPoolSize++; clearForRecycle(); next = mPool; mPool = this; } } }
按理说这是一个很小的问题,不知道google为何没修,而是所有的使用上都是通过obtain的方式来实现。是否有何讲究没?