戻り値がarraylistの時の受け取りがわからない。

Question

前提・実現したいこと

飲み物を検索し、select文でDBに引っかかった飲み物の種類だけ表示させる。

①list.jspから飲料水番号と飲料水名を取得し、ListController.javaで受け取る。
②受け取った2つの値を引数として別クラスT001ItemDao.javaのserchItemメソッドを呼び出し
③arraylistを宣言。
④select文を実行。
⑤別クラスItemBean.javaでゲッターセッターを宣言。
⑥select文実行結果をItemBeanのセッターに代入。
⑦セッターに代入したオブジェクトをlist.addでリストに個数分追加
⑧リストをListController.javaに返しsetAttribute()でjspに渡し、jspでリストの個数分だけループして表示。

発生している問題・エラーメッセージ

発生している問題として、T001ItemDao.javaのserchItemメソッドの戻り値として、ItemBean型のarraylistがListcontrollerクラスで正しく受け取れていない？

該当のソースコード

list.jsp（値取得部分省略）

<table cellspacing="1" cellpadding="8" border="0" bgcolor="#999999">
  <tr bgcolor="#EBEBEB">
    <th> </th>
    <th> </th>
    <th> </th>
    <th align="center">商品コード</th>
    <th align="center">商品名</th>
    <th align="center">金額</th>
    <th align="center">数量</th>
  </tr>
<%	List<ItemBean> list = (List<ItemBean>)request.getAttribute("Item"); %>
<%
	if(list != null){
		for(int i = 0; i < list.size(); i++){
			//取得したlistから1件ずつ商品データ（ItemBean）を取り出す
			ItemBean selectedItm = list.get(i);
%>

  <tr bgcolor="#FFFFFF">
    <td><a href="./view.html?{{ person.key }}">詳細</a></td>
    <td><a href="./edit.html?/{{ item.key }}">編集</a></td>
    <td><a href="./delete.html?{{ item.key }}">削除</a></td>

	<!-- 各セル（tdタグ）に検索結果（商品コード等）入れます -->
	<td align="center"><%=selectedItm.getCode()%></td>
	<td align="center"><%=selectedItm.getName()%></td>
	<td align="center"><%=selectedItm.getPrice()%></td>
	<td align="center"><%=selectedItm.getCount()%></td>
  </tr>
<% }}%>
</table>

コード

ListController.java（Postメソッド部分のみ）

	protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		//文字コードをUTF-8にセット
		request.setCharacterEncoding("UTF-8");
		response.setContentType("text/html:charset=UTF-8");
		response.setCharacterEncoding("UTF-8");

		String code = request.getParameter("code");
		String name = request.getParameter("name");
		System.out.println(code + "" + name);

		T001ItemDao itemDao = null;
			try{
				ItemBean item = new ItemBean();
				List<ItemBean> list = new ArrayList<ItemBean>();//リストで受け取るために用意
				itemDao = new T001ItemDao();

				list = itemDao.serchItem(code, name);//jspにリストを送る

				request.getSession().setAttribute("Item", list);

				ServletContext application = getServletContext();
				application.getRequestDispatcher("/list.jsp").forward(request,response);

			} catch(ClassNotFoundException e){
				e.printStackTrace();
			} catch(SQLException e){
				e.printStackTrace();
			} finally{
				//データベース切断
				itemDao.close();
			}
	}

コード

T001ItemDao.java（serchItemメソッドのみ）

//の商品検索
	public List<ItemBean> serchItem(String code, String name){
		//リスト作成
		List<ItemBean> list = new ArrayList<ItemBean>();
		try{
			//sql発行
			String sql2 = "select ITEM_NO, ITEM_NM, UNIT_PRICE, STOCK_COUNT from T001_ITEM "
						 + "where ITEM_NO like '" +code+ "%'"
						 + "and ITEM_NM like '%" +name+ "%'";
			
			pstmt = conn.prepareStatement(sql2);
			//実行
			ResultSet resultSet = pstmt.executeQuery(sql2);

			while(resultSet.next()){
				//1件分のデータを格納するItemBeanを宣言する
				ItemBean selectedItem = new ItemBean();

				selectedItem.setCode(resultSet.getString("ITEM_NO"));
				selectedItem.setName(resultSet.getString("ITEM_NM"));
				selectedItem.setPrice(resultSet.getString("UNIT_PRICE"));
				selectedItem.setCount(resultSet.getString("STOCK_COUNT"));

				//宣言したリストに１件ずつwhile分追加する
				list.add(selectedItem);
			}
			System.out.println("商品検索結果"+list.size()+"件ヒットしました");
		}catch (SQLException e){
			e.printStackTrace();
		}
		return list;
	}
コード

試したこと

T001ItemDao.javaでselectで返ってきた値をセッターに代入できている。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

説明不足が多いですがよろしくお願いします。

Comments

[unz.hori]

ソース(java/JSP)がインデントなしになっていて見にくいのでコードを使ってください

[退会済みユーザー]

すいません。コードの使い方がわからなくて反映されません

[unz.hori]

編集時のアイコンで</>を押すと```ここに言語を入力 コード ```と出てきますので```の間にコードを貼ってください。

[unz.hori]

例えば```Java(改行)int i = 1;```てな感じで。(改行)は実際の改行コードです。

[退会済みユーザー]

プレビューでコードは反映されているようなのですが、更新を押しても反映がなかなかされません・・・

[unz.hori]

なんかエラーが出てないですか？更新理由を入れろみたいな

[退会済みユーザー]

できたようです。ありがとうございます。

[unz.hori]

見易くなりましたね。回答者側の事も考えましょう。

Answer 1

細かいですが、ListControllerクラスで
List<ItemBean> list = new ArrayList<ItemBean>();
としていますが、戻り値としてlistを受け取るのでnewする必要はないです。
List<ItemBean> list = itemDao.serchItem(code, name);
でいいです。ぱっと見た目は特に問題なさそうですがjspでListとかはimportされてますか？

Comments

[退会済みユーザー]

java.util.*と記述しておりますのでおそらく大丈夫かと思います。念のためにjava.util.ArrayListと追加で記述させてもらいました。newを消してList<ItemBean> list = itemDao.serchItem(code, name);と記述しましたがあっていますか？補足として、修正した後実行しましたが商品はまだ表示されておらず、直接の問題の解決にはなっていないようです。

[unz.hori]

そもそもDBからデータはちゃんと取得できていますか？内容を確認しましたか？

[unz.hori]

ソース修正は合っています。

[退会済みユーザー]

select文の結果は適当な値が入っていることを確認しました。そして、System.out.println(selectedItem.getCode());という感じでセッターで値が代入されているかを確認し、selectで必要なデータを取り出し、適当な値をセッターにセットするというところまではできているかと思います。

[退会済みユーザー]

修正します。今確認をしたのですが、T001ItemDao.javaのselectでの結果はちゃんと適当な値が返ってきているようです。が、セッターにセットしlist.addをしたあとにlist.size()をsysoutで表示すると0になりました。以前はちゃんと適当な値が入っていたのですが、どういうことなのか自分でもわかりません・・・

[unz.hori]

T001ItemDao.java#serchItemは変更していませんか？またはデータに変更がありませんでしたか？

[退会済みユーザー]

T001ItemDao.javaにゴミが入っていたようです。list.size()の結果が正しい結果になりました。お騒がせしました。

[unz.hori]

気になる点といえば、セッションに設定している時は
request.getSession().setAttribute("Item", list);
取得している方は
request.getAttribute("Item");
これを
request.getSession().getAttribute("Item");
としたらどうでしょう？

[退会済みユーザー]

関係ないかもしれませんが、list.jspの最後当たりのarraylist宣言のところに気になるところがありました。<% List<ItemBean> list = (List<ItemBean>)request.getAttribute("Item"); %>と宣言してあるのですが、型の安全性: Object から List<ItemBean> への未検査キャストと警告？黄色のビックリマークがついています。何か関係があるとよいのですが、、、

[unz.hori]

それは警告レベルなので問題ないです

[退会済みユーザー]

request.getSession().getAttribute("Item");にしたところ、正しく表示されました！！！
Attribute()にSessionをつけると受け取る側もSessionをつけないとプログラムの中で違うものとして区別されるということでしょうか？

[unz.hori]

違うところを見に行っていたんでしょうね。requestオブジェクトのattributeを取得しようとして取得できなかったと想像できます。

[退会済みユーザー]

なるほど。。。
勉強になりました！！！ベストアンサー？にさせてもらいましたが正しく反映してますでしょうか？プログラミング初心者なのでこれからも質問等していくと思います。またお見えする機会があればどうぞよろしくお願いします。

[unz.hori]

ベストアンサーいただきましたw。今回のようなケースでは大体がどこに設定した？どこから持ってきた？という観点をじっくり見てみれば結構解決するものです。これからも頑張ってください。