DBから複数レコードを取り出すプログラミングについて

Question

以下のクラスがあります。
・Empクラス
⇨社員ID、社員名、所属フィールドがある
・EmpDAOクラス
⇨DBに接続してSQL文を実行、取得した複数のレコードをリストで返す
・Mainクラス
⇨main()実行、取り出したレコードを表示

多くの場合、EmpDAOのメソッドで、
①SQL文実行、ResultSetに格納
②next()でレコードを一行ずつlistに追加
という処理をすると思います(だいぶ省いてますが)

今回、while文を使って②の処理を行ったのですが、取り出したi行目のレコードを一旦格納するEmp型変数empのオブジェクトを、EmpDAOクラス内のどこで作ったかによって、最終的な表示結果が変わってしまいました。

ケース①
・処理を行うメソッド(searchAllという名前に今回はしました)内の一番最初で行う
結果
・rsに取り出した複数レコードのうち、一番最後の行のレコードが、取り出した行分表示された

例) 001、鈴木、営業
002、高橋、経理
003、田中、企画　が取り出したレコードだとすると、
003、田中、企画
003、田中、企画
003、田中、企画
と表示されてしまった

ケース②
・while(rs.next())内で行う
結果
・取り出したレコードが一行ずつ正しく表示された

何故このように処理が変わってしまうのか、わかる方はどうかご教示下さい！

言語 Java ver.8
環境 eclipse

以下searchAll()メソッドです。

public List<Emp> searchAll() {
List<Emp> list = new ArrayList<Emp>();
String sql = "SELECT empno, ename, job "
+ "FROM emp;";

try(Connection conn = DriverManager.getConnection(＊パスを記述しています);
PreparedStatement pstmt = conn.prepareStatement(sql);) {

ResultSet rs = pstmt.executeQuery();

while(rs.next()) {
Emp emp = new Emp();
int empno = rs.getInt("empno");
String ename = rs.getString("ename");
String job = rs.getString("job");

emp.setEmpno(empno);
emp.setEname(ename);
emp.setJob(job);

list.add(emp);
}
} catch(SQLException e) {
e.printStackTrace();
}
return list;
} //メソッド終了

こんな感じです。
上記は正しく表示された記述です。

ケース①のようになってしまったときは、
Emp emp = new Emp();
を while(rs.next()) の中ではなく、
public List<Emp> searchAll() {
のすぐ下に記述しておりました。

宜しくお願い致します！

Comments

[shiena]

> ②next()でレコードを一行ずつlistに追加

この処理がうまくいく方法といかない方法で違ってるような気がします。ResultSet取得後のにlistに追加する処理とsearchAllのコードを載せられませんか？

[m.ts10806]

一通りコードを提示された方が良いかと思います。

[Josh_Jan_J]

コメント有難うございます！　以下searchAll()メソッドです。

public List<Emp> searchAll() {
List<Emp> list = new ArrayList<Emp>();
String sql = "SELECT empno, ename, job "
+ "FROM emp;";

try(Connection conn = DriverManager.getConnection(＊パスを記述しています);
PreparedStatement pstmt = conn.prepareStatement(sql);) {

ResultSet rs = pstmt.executeQuery();

while(rs.next()) {
Emp emp = new Emp();
int empno = rs.getInt("empno");
String ename = rs.getString("ename");
String job = rs.getString("job");

emp.setEmpno(empno);
emp.setEname(ename);
emp.setJob(job);

list.add(emp);
}
} catch(SQLException e) {
e.printStackTrace();
}
return list;
} //メソッド終了

こんな感じです。
上記は正しく表示された記述です。

ケース①のようになってしまったときは、
Emp emp = new Emp();
を while(rs.next()) の中ではなく、
public List<Emp> searchAll() {
のすぐ下に記述しておりました。

宜しくお願い致します！

[m.ts10806]

質問は編集できますので。
こちらに提示されても目につきませんしマークダウンが使えません。

[Josh_Jan_J]

失礼致しました、先程質問本文を編集致しました。ご教示有難うございます！

[m.ts10806]

コードはマークダウンのcode機能を利用してご提示ください。

[Josh_Jan_J]

このような方法があったのですね、有難うございます！

Answer 1

ケース①のようになってしまったときは、
Emp emp = new Emp();
を while(rs.next()) の中ではなく、
public List<Emp> searchAll() {
のすぐ下に記述しておりました。

正にこれが原因です。
そのように記述するとlistに追加しているのは同じインスタンスなのでwhileで毎回上書きされて最後にsetした値になります。なのでwhileの中でnew Emp()するとそれぞれ異なったインスタンスになり各ループでsetした値になります。

図解すると以下のようになります。

Comments

[Josh_Jan_J]

ご回答ありがとうございます！　原因、理解できました。同じオブジェクトでリストを繰り返し作っていたので、一番最後にsetした値が、全リストに反映されてしまったんですね。レコード一つ一つ、別のオブジェクトを作ってあげないとだめなのか...　勉強になりました！

[shiena]

listに追加したものが何なのか分かりやすいように図を追加してみました

[Josh_Jan_J]

図解の追加、有難うございます！とても分かりやすいです。