String类substring方法导致的Java内存泄漏问题

此问题在项目中被发现，经查看JDK源码(JDK1.6)，String类的public String substring(int beginIndex, int endIndex)的实现让我很意外。

想重现这个场景很容易，请看代码。

 

 import java.util.ArrayList; import java.util.List; public class LeakTest { public static void main(String...args) { List<String> handler = new ArrayList<String>(); for(int i = 0; i < 100000; i++) { Huge h = new Huge(); handler.add(h.getSubString(1, 5)); } } } class Huge { private String str = new String(new char[100000]); public String getSubString(int begin, int end) { return str.substring(begin, end); } }

 

执行此代码结果：

Exception in thread "main" java.lang.OutOfMemoryError: Java heap space

 

问题就出在Huge类的 getSubString 方法，它调用了String类的substring方法。

来让我们看看 substring 类的实现吧，JDK源码如下：

public String substring(int beginIndex, int endIndex) { if (beginIndex < 0) { throw new StringIndexOutOfBoundsException(beginIndex); } if (endIndex > count) { throw new StringIndexOutOfBoundsException(endIndex); } if (beginIndex > endIndex) { throw new StringIndexOutOfBoundsException(endIndex - beginIndex); } return ((beginIndex == 0) && (endIndex == count)) ? this : new String(offset + beginIndex, endIndex - beginIndex, value); }

 

再让我们接下来看看 new String(offset + beginIndex, endIndex - beginIndex, value); 的实现：

// Package private constructor which shares value array for speed. String(int offset, int count, char value[]) { this.value = value; this.offset = offset; this.count = count; }

 

char[] value 数组被共享了。

 

在我们的main函数里的循环中，每循环一次后，我们希望Huge对象被回收，且释放它占有的内存。

但实际上 private String str = new String(new char[100000]); 占有的内存并不会被释放。

因为 我们通过 Huge 类的 getSubString 方法得到的 String 对象还存在(存在于handler的列表中)，

它虽然是 length 只有 4 的对象，却享有着 char[100000] 的空间。

 

解决方案：

可以修改Huge 类的 getSubString 方法如下：

public String getSubString(int begin, int end) { return new String(str.substring(begin, end)); }

只要再套一个String的构造方法即可。

 

至于为什么，看看JDK源码，一看便知了。这里就不贴出来了。

 

 

唉，以后写代码得多多小心啊。