JiafanZhou wrote:
This is a severe bug.

cxp108 wrote:
版主可能没有注意看程序....

EntityDocument[] newArray = new EntityDocument[old.length];
声明时，数组每个元素都是指向null的reference
而当其在循环中对一个EntityDocument d进行赋值
EntityDocument d = new EntityDocument();
for(int i=0;i<old.length;i++){
d.content = old[i].getContent();
d.key = old[i].getKey();
尽管这段代码仅仅生成了一个d对象，很好的重用了d对象，但是
后面
newArray[i] = d ;
这段代码却大错特错，因为其将newArray的所有reference指向
唯一的一个d对象，这也就导致整个newArray数组都由一个对象组成。
与程序意图完全不符。

ditty wrote:
其实，说白了就是在循环体内，忘记了初始化对象d而已。
既然是资深，还是需要得到尊敬，lz有些借题发挥的味道。

ditty wrote:
嗬嗬，用代码说话吧：

EntityDocument d = new EntityDocument();
for(...){
....
newArray[i] = d;
}

=>

EntityDocument d;
for(...){
d = new EntityDocument();
....
newArray[i] = d;
}

首先要清楚人家为什么要把变量定义放在循环体外面，然后再去指责。

cxp108 wrote:
你这么写就不叫对象重用了，因为循环内每次都要重新构建一个EntityDocument
对象。尽管它符合程序的意图。

Document [] old ......//这是数据源
EntityDocument[] newArray = new EntityDocument[old.length];//自定义的类，为了把Document里数据保留下来避免Document被关联对象关闭而导致无法取出数据。
EntityDocument d = new EntityDocument();

for(int i=0;i<old.length;i++)
{
d.content = old[i].getContent();
d.key = old[i].getKey();
d......................
newArray[i] = d.clone();//如此对象重用..... // N.B. If we do this, then we create a new reference and assign the new reference to the array element.
} 

// still not the best way, but much more comprehensible than the original one...
Document[] old = ...
ArrayList<EntityDocument> newArray = new ArrayList<EntityDocument>();
EntityDocument document = new EntityDocument();

for (Document oldDoc : old)
{
//    EntityDocument document = new EntityDocument(); // remove this!!
    document.content = oldDoc.getContent();
    document.key = oldDoc.getKey();
    newArray.add(document.clone());
}
// deal with the newArray cache...

九佰 wrote:
for (int i = 0; i < 10000; ++i) {

Object obj = new Object();

System.out.println("obj= "+ obj);

}

… …

上面代码的书写方式相信对你来说不会陌生，也许在以前的应用开发中你也这样做过，尤其是在枚举一个Vector对象中的对象元素的操作中经常会这样书写，但这却违反了上述规则（1），因为这样会浪费较大的内存空间，正确的方法如下所示：

… …

Object obj = null;

for (int i = 0; i < 10000; ++i) {

obj = new Object();

System.out.println("obj= "+ obj);

}

… …

采用上面的第二种编写方式，仅在内存中保存一份对该对象的引用，而不像上面的第一种编写方式中代码会在内存中产生大量的对象应用，浪费大量的内存空间，而且增大了系统做垃圾回收的负荷。因此在循环体中声明创建对象的编写方式应该尽量避免。