关于Java中String的Memory问题

mooncui

浏览: 72481 次

最近访客更多访客>>

mtx_2010

zc5510670

guxingleisos

nikymaco

博主相关

博客

微博

相册

留言

关于我

文章分类

社区版块

存档分类

博客分类：

Java

Java JDK Blog

昨天看到一个blog，用一个极端例子来说明Java中的内存问题：

http://blog.xebia.com/2007/10/04/leaking-memory-in-java/

测试代码如下：

public class TestGC {
private String large = new String(new char[100000]);
public String getSubString() {
return this.large.substring(0,2);
}
public static void main(String[] args) {
ArrayList<string></string> subStrings = new ArrayList<string></string>();
for (int i = 0; i <1000000; i++) {
TestGC testGC = new TestGC();
subStrings.add(testGC.getSubString());
}
}
}

如果要改掉这个bug，只要改写一下getSubString的方法就好了。

public String getSubString() {
return new String(this.large.substring(0,2)); // <-- fixes leak!
}

我测试了一下，确实如此。然后我看了一下JDK1.5.0_06代码中String类

from JDK1.5.0_06, String class:

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);
}

这里用到的String的构造函数是private的

private String(int offset, int count, char value[]) {
this.value = value;
this.offset = offset;
this.count = count;
}

上面那个例子程序改写的部分其实就是增加调用public的String构造函数另外创建一个String对象，我们再来看看这个public的构造函数，已经另外一个类似上面这个private的构造函数的public构造函数：

两个public的构造函数

public String(String original) {
int size = original.count;
char[] originalValue = original.value;
char[] v;
if (originalValue.length > size) {
// The array representing the String is bigger than the new
// String itself. Perhaps this constructor is being called
// in order to trim the baggage, so make a copy of the array.
v = new char[size];
System.arraycopy(originalValue, original.offset, v, 0, size);
} else {
// The array representing the String is the same
// size as the String, so no point in making a copy.
v = originalValue;
}
this.offset = 0;
this.count = size;
this.value = v;
}
public String(char value[], int offset, int count) {
if (offset < 0) {
throw new StringIndexOutOfBoundsException(offset);
}
if (count < 0) {
throw new StringIndexOutOfBoundsException(count);
}
// Note: offset or count might be near -1>>>1.
if (offset > value.length - count) {
throw new StringIndexOutOfBoundsException(offset + count);
}
char[] v = new char[count];
System.arraycopy(value, offset, v, 0, count);
this.offset = 0;
this.count = count;
this.value = v;
}

上面这两个public的构造函数，另外创建出来的String对象和原来的String对象并不共享其char数组的。而前面那个private的构造函数没有作arraycopy，是共享char其数组的。我想JDK这个实现有待商讨，内部使用的private的构造函数中不需要copy数组，可以共享数组。但private的构造函数被public的subString方法调用后，被外部使用时，就有问题了。

以后写相关代码时要小心了！

分享到：

JAVA中怎样测private方法? | 《C++沉思录》笔记--一个面向对象设计的例 ...

2007-10-11 10:48
浏览 1446
评论(1)
论坛回复 / 浏览 (0 / 4400)
查看更多

1 楼 zhangsheng79 2007-10-11

java也太傻了!

发表评论

您还没有登录,请您登录后再发表评论

最近访客更多访客>>

博主相关

文章分类

社区版块

存档分类

最新评论