JiafanZhou wrote:
This is a severe bug.

Biubiu wrote:
只是个bug而已。谁都会犯错误。

cxp108 wrote:
我觉得这是一个很基础的错误，一个资深工程师
不应该犯这种错，而且还和我争了半天说他没错
，最后是他自己写代码验证才不说话。

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

for (int i = 0; i < old.length; i++)
{
    EntityDocument document = new EntityDocument();
    document.content = old[i].getContent();
    document.key = old[i].getKey();
    newArray.add(document);
}
// deal with the newArray cache...

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
对象。尽管它符合程序的意图。

ditty wrote:
应用的规则参见2楼九佰贴的那部分，每个人都有自己的代码风格，前提是要服从项目中的具体规定。这里就不和你较真的，其实我要说的就是，你认为有问题的代码其实仅仅是一个bug，至于是否符合你的所说的对象重用，那就不好说了。赫赫。

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...

System.arraycopy(sourceArray, 0, destinationArray, 0, sourceArray.length);

cxp108 wrote:
new 和clone都能解决这个bug，但我们现在考虑另外两条路：对象池 和 类似javaNIO里
的buffer机制。

九佰 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);

}

… …

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