セットした情報が上書きされてしまう

こんにちは。
現在DAOの学習をしているものです。

現在、DAOに下記のようにデータをセットしてbookList.jspにてゲットし画面に表示させようとしています。

ところが、複数のデータをセットしてしまうとデータが上書きされてしまい前のデータが残りません。

今回の例でいうと、GenreDAOに

            //一つ目のメインジャンル情報をDTOにセット
            info.setMainGenresId("111");
            info.setMainGenresName("ああああああああああ");
            genre.add(info);

            //2つ目のメインジャンル情報をDTOにセット
            info.setMainGenresId("222");
            info.setMainGenresName("いいいいいいいいいい");

とデータをセットしているにもかかわらず、bookList.jspを実行すると出力結果は

となってしまいます。

何か気になる箇所などございましたらご教授いただければ幸いです。

どうぞよろしくお願いいたします。

###私のソースコード

bookList
1<%@ page import="java.util.ArrayList" %>
2<%@ page import="dto.GenreDTO" %>
3<%@ page import="javax.servlet.ServletContext" %>
4<%@ page import="javax.servlet.ServletContextEvent" %>
5<%@ page import="javax.servlet.ServletContextListener" %>
6<%@ page import="mutual.InitializationListener"%>
7<%@ page language="java" contentType="text/html; charset=UTF-8"
8    pageEncoding="UTF-8"%>
9<!DOCTYPE html>
10<html>
11<head>
12<meta charset="UTF-8">
13<title>書籍情報一覧画面</title>
14</head>
15<body>
16
17<%
18	ArrayList<GenreDTO> maingenreList = (ArrayList<GenreDTO>)application.getAttribute("maingenreList");
19%>
20
21<table>
22	<tr>
23		<th>メインジャンルID</th>
24		<th>メインジャンル名</th>
25		<th>サブジャンルID</th>
26		<th>サブジャンル名</th>
27		<th>メインジャンルID</th>
28
29	</tr>
30
31	<%	for(int i=0;i<maingenreList.size();++i){
32			GenreDTO info = maingenreList.get(i);
33	%>
34	<tr>
35		<td>
36		<%= info.getMainGenresId() %>
37		</td>
38		<td>
39		<%= info.getMainGenresName() %>
40		</td>
41		<td>
42		<%= info.getSubGenresId() %>
43		</td>
44		<td>
45		<%= info.getSubGenresName() %>
46		</td>
47		<td>
48		<%= info.getMainGenresId() %>
49		</td>
50	</tr>
51
52	<%}%>
53</table>
54</body>
55</html>

GenreDAO
1package dao;
2import java.io.Serializable;
3import java.sql.SQLException;
4import java.util.ArrayList;
5
6import javax.naming.NamingException;
7
8import dto.GenreDTO;
9
10public class GenreDAO implements Serializable {
11
12    //セットしたメインジャンル情報を取得
13    public  ArrayList<GenreDTO> searchAllMain()
14            throws NamingException,SQLException{
15
16
17            ArrayList<GenreDTO> genre = new ArrayList<GenreDTO>();
18            GenreDTO info = new GenreDTO();
19
20            //一つ目のメインジャンル情報をDTOにセット
21            info.setMainGenresId("111");
22            info.setMainGenresName("ああああああああああ");
23            genre.add(info);
24
25            //2つ目のメインジャンル情報をDTOにセット
26            info.setMainGenresId("222");
27            info.setMainGenresName("いいいいいいいいいい");
28            genre.add(info);
29
30            // 戻り値の設定
31            return genre;
32
33
34    }
35
36    //セットしたサブジャンル情報を取得
37    public  ArrayList<GenreDTO> searchAllSub()
38            throws ClassNotFoundException,SQLException{
39
40
41            ArrayList<GenreDTO> genre = new ArrayList<GenreDTO>();
42            GenreDTO info = new GenreDTO();
43
44            //一つ目のサブジャンル情報をDTOにセット
45            info.setSubGenresId("");
46            info.setSubGenresName("");
47            info.setMainGenresId("");
48            genre.add(info);
49
50            //2つ目のサブジャンル情報をDTOにセット
51            info.setSubGenresId("");
52            info.setSubGenresName("");
53            info.setMainGenresId("");
54            genre.add(info);
55
56            // 戻り値の設定
57            return genre;
58
59
60    }
61
62}

GenreDTO
1package dto;
2
3import javax.servlet.annotation.WebServlet;
4/**
5 * Servlet implementation class GenreDTO
6 */
7@WebServlet("/dto/GenreDTO")
8// シリアライズを行う
9public class GenreDTO implements java.io.Serializable{
10
11	//メンバ変数の定義
12	private String main_genres_id;
13	private String main_genres_name;
14	private String sub_genres_id;
15	private String sub_genres_name;
16
17
18	//---------------------以下getter処理-------------------------------------------------
19
20	//呼び出し元へメインジャンルIDを返す
21	public String getMainGenresId() {
22		return main_genres_id;
23	}
24
25	//呼び出し元へメインジャンル名を返す
26	public String getMainGenresName() {
27		return main_genres_name;
28	}
29
30	//呼び出し元へサブジャンルIDを返す
31	public String getSubGenresId() {
32		return sub_genres_id;
33	}
34
35	//呼び出し元へサブジャンル名を返す
36	public String getSubGenresName() {
37		return sub_genres_name;
38	}
39
40	//---------------------以下setter処理-------------------------------------------------
41
42	//メインジャンルIDに呼び出し元から受け取った値を設定する
43	public void setMainGenresId(String main_genres_id) {
44		this.main_genres_id = main_genres_id;
45	}
46
47	//メインジャンル名に呼び出し元から受け取った値を設定する
48	public void setMainGenresName(String main_genres_name) {
49		this.main_genres_name = main_genres_name;
50	}
51
52	//サブジャンルIDに呼び出し元から受け取った値を設定する
53	public void setSubGenresId(String sub_genres_id) {
54		this.sub_genres_id = sub_genres_id;
55	}
56
57	//サブジャンル名に呼び出し元から受け取った値を設定する
58	public void setSubGenresName(String sub_genres_name) {
59		this.sub_genres_name = sub_genres_name;
60	}
61
62}

InitializationListener
1package mutual;
2import java.util.ArrayList;
3
4import javax.servlet.ServletContext;
5import javax.servlet.ServletContextEvent;
6import javax.servlet.ServletContextListener;
7import javax.servlet.annotation.WebListener;
8
9import common.LogManager;
10import dao.GenreDAO;
11import dto.GenreDTO;
12
13@WebListener
14public class InitializationListener implements ServletContextListener {
15    public void contextInitialized(ServletContextEvent event) {
16        // ログの出力
17        LogManager lg = new LogManager();
18        lg.outMethodLog("", null);
19        ServletContext application = event.getServletContext();
20        try {
21            GenreDAO bn = new GenreDAO();
22
23            ArrayList<GenreDTO> maingenreList = bn.searchAllMain();
24
25            // genreListデータをapplicationスコープで保存
26            application.setAttribute("maingenreList", maingenreList);
27
28            ArrayList<GenreDTO> subGenreList = bn.searchAllSub();
29
30            // genreListデータをapplicationスコープで保存
31            application.setAttribute("subGenreList", subGenreList);
32
33            // 投げた例外をException型の変数e1で受取る
34        } catch (Exception e1) {
35            // e1の中には例外に関する情報が詰まっている。それをログで出力してあげる
36            lg.outExceptionLog(e1, null, null);
37        }
38    }
39
40    public void contextDestoryed(ServletContextEvent event) {
41        ServletContext application = event.getServletContext();
42        application.removeAttribute("maingenreList"); // 削除処理
43        application.removeAttribute("subGenreList");
44    }
45}

行動規範の内容に同意します

回答1件

ベストアンサー

一つ目の info を genre に add しても、実際には info のコピーが genre に add されているわけではありません。info の参照が genre に add されているだけです。
ですから二つ目の info として書き換えた瞬間に、参照している genre の中の info も（当然）書き換わったものとなります。

これを回避するためには、一つ一つ info を新規に作らねばなりません。

Java
1            ArrayList<GenreDTO> genre = new ArrayList<GenreDTO>();
2            GenreDTO info = new GenreDTO();
3
4            //一つ目のメインジャンル情報をDTOにセット
5            info.setMainGenresId("111");
6            info.setMainGenresName("ああああああああああ");
7            genre.add(info);
8
9            // 別の info を作る
10            info = new GenreDTO();
11
12            //2つ目のメインジャンル情報をDTOにセット
13            info.setMainGenresId("222");
14            info.setMainGenresName("いいいいいいいいいい");
15            genre.add(info);