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

以下のクラスがあります。
・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()メソッドです。

Java
1public List<Emp> searchAll() {
2List<Emp> list = new ArrayList<Emp>();
3String sql = "SELECT empno, ename, job "
4+ "FROM emp;";
5
6try(Connection conn = DriverManager.getConnection(＊パスを記述しています);
7PreparedStatement pstmt = conn.prepareStatement(sql);) {
8
9ResultSet rs = pstmt.executeQuery();
10
11while(rs.next()) {
12Emp emp = new Emp();
13int empno = rs.getInt("empno");
14String ename = rs.getString("ename");
15String job = rs.getString("job");
16
17emp.setEmpno(empno);
18emp.setEname(ename);
19emp.setJob(job);
20
21list.add(emp);
22}
23} catch(SQLException e) {
24e.printStackTrace();
25}
26return list;
27} //メソッド終了

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

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

宜しくお願い致します！

shiena

2020/08/29 09:33

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

m.ts10806

2020/08/29 09:51

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

Josh_Jan_J

2020/08/29 09:53

コメント有難うございます！　以下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

2020/08/29 09:54

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

Josh_Jan_J

2020/08/29 09:57

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

m.ts10806

2020/08/29 09:59

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

Josh_Jan_J

2020/08/29 10:03

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