ArrayList对象引用的问题

老问题了，之所以贴出来是给大家看看，希望不要出现类似的错误：

this.item=new ArrayList(); 
　　 books = new Books(); 
　　 ResultSet rs = new DBoperate().getResultSet("select * from tblbooks"); 
　　 int i=0; 
　　 while (rs.next()) { 
　　 books.setBm(rs.getString("bm")); 
　　 books.setMc(rs.getString("mc")); 
　　 books.setAuthor(rs.getString("author")); 
　　 books.setPublisher(rs.getString("publisher")); 
　　 books.setPubdate(rs.getString("pubdate")); 
　　 books.setIsbn(rs.getString("isbn")); 
　　 books.setPrice(rs.getString("price")); 
　　 books.setLxbm(rs.getString("lxbm")); 
　　 books.setYm(rs.getString("ym")); 
　　 books.setPicpath(rs.getString("picpath")); 
　　 books.setContent(rs.getString("content")); 
　　 this.item.add(books); 
　　 } 

　　
　　上面的代码导致ArrayList里面都是同样的值，而且是数据库返回的最后一条记录的值；
　　曾经一个朋友遇到这样的问题，那我们来分析一下：
　　books在循环外部定义，那么在循环里面每次都同一个books，循环体对其进行赋值，然后将books的引用放入ArrayList里面；每次循环都在不停的改变其引用的值，那么ArrayList里面也会跟着改变其指引的books的值，那么最后停止循环的时候，ArrayList每个books的值都会被改变成最后一个值；
　　 所以也就不难理解，为啥都是一样的值，并且是最后一个。

或者说这么来解释：
只定义了一个books对象，循环都是在不停的改变其值，并且放入ArrayList里面的对象都是同一个，比如循环了20次，那么也就是加入了20个一模一样的books对象；当books的值最后一次被改变的时候，ArrayList里面由于都是同一个books对象，随之也会全部会改变成最后一次的值。

评论

2 楼 scholers 2009-09-18  

pangbuddy 写道

应该在循环中每次都new一个books就好了。
还有个小建议 item 改叫items， books改叫book，这样应该更合理些吧。

这位朋友说的有道理，确实是这样

1 楼 pangbuddy 2009-09-18  

应该在循环中每次都new一个books就好了。
还有个小建议 item 改叫items， books改叫book，这样应该更合理些吧。