[英]ArrayList in java printing last inserted value?
我有以下java类
package com.picvik.model;
import java.util.Date;
public class ViewAlbum {
private Integer albumid;
private String albumname;
private String description;
private String location;
private Date date;
private Integer uid;
public Integer getAlbumid() {
return albumid;
}
public void setAlbumid(Integer albumid) {
this.albumid = albumid;
}
public String getAlbumname() {
return albumname;
}
public void setAlbumname(String albumname) {
this.albumname = albumname;
}
public String getDescription() {
return description;
}
public void setDescription(String description) {
this.description = description;
}
public String getLocation() {
return location;
}
public void setLocation(String location) {
this.location = location;
}
public Date getDate() {
return date;
}
public void setDate(Date date) {
this.date = date;
}
public Integer getUid() {
return uid;
}
public void setUid(Integer uid) {
this.uid = uid;
}
}
我正在从数据库中检索数据并将其添加到我的数组列表中
public ArrayList getAllAlbums(Integer uid) {
ViewAlbum album = new ViewAlbum();
ArrayList<ViewAlbum>allAlbums = new ArrayList<ViewAlbum>();
try {
String qstring = "SELECT albumid, albumname, description, location," +
" date, uid FROM picvik_picture_album WHERE " +
"uid = '" + uid + "';";
System.out.println(qstring);
connection = com.picvik.util.MySqlConnection.getInstance().getConnection();
ptmt = connection.prepareStatement(qstring);
resultSet = ptmt.executeQuery();
while(resultSet.next()) {
//System.out.println(resultSet.getString("albumname"));
album.setAlbumid(resultSet.getInt("albumid"));
album.setAlbumname(resultSet.getString("albumname"));
album.setDescription(resultSet.getString("description"));
album.setLocation(resultSet.getString("location"));
album.setDate(resultSet.getDate("date"));
album.setUid(resultSet.getInt("uid"));
allAlbums.add(album);
}
resultSet.close();
ptmt.close();
connection.close();
} catch (Exception e) {
e.printStackTrace();
}
return allAlbums;
}
但是当我尝试打印存储在数组列表中的值时。 它总是给我最后插入的记录。
<div class="row">
<div class="span10">
<s:iterator value="allAlbums">
<s:property value="albumname"/>
</s:iterator>
</div>
</div>
这里,
ViewAlbum album = new ViewAlbum();
// ...
while (resultSet.next()) {
album.setAlbumid(resultSet.getInt("albumid"));
// ...
allAlbums.add(album);
}
您正在为所有记录重复使用相同的album
实例。 实例的数据每次在循环中都会被覆盖。 该列表不包含实例的副本,但包含对单个实例的引用的副本。 您知道,Java 是面向对象的。
您应该为每条记录创建一个新album
实例。 将实例化移动到循环内部。
// ...
while (resultSet.next()) {
ViewAlbum album = new ViewAlbum();
album.setAlbumid(resultSet.getInt("albumid"));
// ...
allAlbums.add(album);
}
与具体问题无关,你应该在finally
块中关闭JDBC资源,或者在try()
try-with-resources语句中打开它们,否则在执行查询或处理过程中出现异常时它们仍然会泄漏结果集。 您还应该将 JDBC 资源的声明移动到方法块内部,否则您也会遇到线程安全问题。 最后但并非最不重要的一点是,您应该使用PreparedStatement
的 setter 方法在 SQL 字符串中设置用户控制的变量。 如果它们是字符串,你就会有一个 SQL 注入攻击漏洞。
您只有一个 ViewAlbum 实例,并且您在整个循环中仅使用该单个实例进行播放(设置值)。 因此,在循环完成后,您只有一个对象插入到 ArrayList 中 N(结果集的大小)次数。
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.