リスト内が毎回初期化される

Question

今javaの勉強をしていて商品管理のシステムを作っています。追加ボタンを押すと商品がリストに追加され、それが表示されるという処理をしています。追加ボタンを押すたびにリストが初期化されてしまう状況です。何かアドバイスいただけると幸いです。よろしくお願いします。サーブレット ``` package Section3; import java.io.IOException; import java.util.ArrayList; import java.util.List; import DAO.SalesSearchDAO; import bean.Product; import jakarta.servlet.RequestDispatcher; import jakarta.servlet.ServletContext; import jakarta.servlet.ServletException; import jakarta.servlet.annotation.WebServlet; import jakarta.servlet.http.HttpServlet; import jakarta.servlet.http.HttpServletRequest; import jakarta.servlet.http.HttpServletResponse; import jakarta.servlet.http.HttpSession; @WebServlet("/AddSalesServlet") public class AddSalesServlet extends HttpServlet{ protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException { String code = request.getParameter("product_inf"); int quantity = Integer.parseInt(request.getParameter("quantity")); SalesSearchDAO dao = new SalesSearchDAO(); List saleslist = (dao.search(quantity , code)); HttpSession session = request.getSession(); session.setAttribute("saleslist", saleslist); RequestDispatcher rd = request.getRequestDispatcher("./sales.jsp"); rd.forward(request, response); } } ``` DAO public List search(int quantity , String code){ int product_code = Integer.parseInt(code); List saleslist = new ArrayList<>(); Connection con = null; Statement smt = null; String sql = null; try { con = getConnection(); smt = con.createStatement(); sql = "SELECT product_name FROM m_product WHERE product_code = " + product_code + " "; rs = smt.executeQuery(sql); while (rs.next()) { String name = rs.getString("product_name"); Product product = new Product(quantity , name , code); saleslist.add(product); } }catch (Exception e) { throw new IllegalStateException(e); }finally { if(smt != null) { try{smt.close();}catch(SQLException ignore){} } if(con != null) { try{con.close();}catch(SQLException ignore){} } } return saleslist; } JSP

売上登録

売上日 <% GregorianCalendar cal = new GregorianCalendar(); SimpleDateFormat format = new SimpleDateFormat("yyyy/M/d"); String datestr = format.format(cal.getTime()); out.println(datestr); %>

<% %>

<% for (Product product : list) { %> <%}%> <% if(saleslist!=null) { for (Product product : saleslist) { %> <% } } %>

商品名	数量
<%=product.getproduct_name()%>	<%=product.getquantity()%>
<%=product.getproduct_name()%>	<%=product.getquantity()%>

Accepted Answer

おそらく、 AddSalesServlet の doPost で DAO.search をご提示のコードのように用いるのは間違いです。
また、マスターデータ(プロダクト) と入力されたデータ(売上?)を混在させてしまっているのも問題に思います。

※以下のコードは切り貼り・直書きしたもので、コンパイル・動作確認はしておりません。

Product クラスは例えば

java
1public class Product {
2  final String code;
3  final String name;
4  Product(String code, String name) {
5    this.code = code;
6    this.name = name;
7  }
8}

とし、本システムで入力したデータは SaleData クラスで表すとします。

java
1public class SaleData {
2  final Product product;
3  final int quantity;
4  SaleData(Product product, int quantity) {
5    this.product = product;
6    this.quantity = quantity;
7  }
8}

"プロダクトコードから Product オブジェクト (1 件 ) を返す" ようなメソッドを作ります。(もしかしたら既にあるのかもしれませんので仮に) ProductDAO クラスとします。

java
1public class ProductDAO {
2  public Product getProduct(String code) throws SQLException {
3    String sql = "SELECT product_name FROM m_product WHERE product_code = ?";
4
5    try(Connection con = getConnection();
6        PreparedStatement pstmt = con.prepareStatement(sql);) {
7      pstmt.setString(1, code);
8      try(ResultSet rs = pstmt.executeQuery();) {
9        if(rs.next()) {
10          String name = rs.getString("product_name");
11          return new Product(code, name);
12        }
13      }
14    }
15    return null;
16  }
17  //getConnection メソッドとか他に必要/ありそうですが省略
18}

AddSalesServlet の doPost メソッドでは以下のようにできるのではないでしょうか。

java
1String code = request.getParameter("product_inf");
2int quantity = Integer.parseInt(request.getParameter("quantity"));
3
4Product product = new ProductDAO().getProduct(code);
5//product が null だったり search の例外の処理は省略
6
7HttpSession session = request.getSession();
8List<SaleData> saleslist = session.getAttribute("saleslist");
9if(saleslist == null) {
10  saleslist = new ArrayList<SaleData>();
11  session.setAttribute("saleslist", saleslist);
12}
13saleslist.add(new SaleData(product, quantity));
14
15RequestDispatcher rd = request.getRequestDispatcher("./sales.jsp");
16rd.forward(request, response);

jsp のほうの table 部分も直すとこんなでしょうか。(jsp は良く知らないので変かもしれません。)

jsp
1<table>
2<tr>
3	<th style="width: 20%">商品名</th>
4	<th style="width: 5%">数量</th>
5</tr>
6<%
7if(saleslist!=null) {
8for(SaleData saledata : saleslist) {
9%>
10<tr>
11	<td><%=saledata.product.name%></td>
12	<td><%=saledata.quantity%></td>
13</tr>
14<%
15}
16}
17%>
18</table>

商品名	数量
<%=saledata.product.name%>	<%=saledata.quantity%>

```

Answer

ありがとうございます。
助かりました。
今回ご指摘いただいたことを含めて勉強していきます。

関連した質問