二つの配列を一つのfor文で回す事はできますでしょうか。

Question

package controller;

import java.io.IOException;
import java.util.ArrayList;
import java.util.HashMap;

import javax.servlet.RequestDispatcher;
import javax.servlet.ServletException;
import javax.servlet.annotation.WebServlet;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import javax.servlet.http.HttpSession;

import dao.ProductDao;
import dao.SalesDao;
import model.CartBean;

/**
 * Servlet implementation class ConfirmationServlet
 */
@WebServlet("/ConfirmationServlet")
public class ConfirmationServlet extends HttpServlet {

	protected void doPost(HttpServletRequest request, HttpServletResponse response)
			throws ServletException, IOException {
		// TODO Auto-generated method stub

		//購入確認機能制御用クラス

		HttpSession session = request.getSession(false);
		@SuppressWarnings("unchecked")
		ArrayList<CartBean> product_select_list = (ArrayList<CartBean>) session.getAttribute("product_select_list");

		ProductDao pro_dao = new ProductDao();

		ArrayList<Integer> product_cd_list = new ArrayList<Integer>();

		for (CartBean i : product_select_list) {

			product_cd_list.add(i.getPro_cd());
		}

		HashMap<Integer, Integer> new_stock_no = pro_dao.get_new_stock_no(product_cd_list);

		System.out.println("map在庫" + new_stock_no.get(product_cd_list.get(0)));

		for (CartBean i : product_select_list) {
			int newStockNo = (int) (new_stock_no.get(i.getPro_cd()));
			i.setStock_no(newStockNo);
		}

		//購入後の在庫更新のためのArrayList
		ArrayList<Integer> remainingStockNo = new ArrayList<Integer>();

	    //salesテーブルに更新するため。合計金額を入れるArrayList。
		ArrayList<Integer> salesPrice = new ArrayList<Integer>();

		//--------------------------------------------------------------------------------------------------------------
		for (CartBean i : product_select_list) {

			System.out.println("ConfirmationServlet:各商品の在庫数。　" + i.getPro_name() + ": " + i.getStock_no());
			System.out.println("ConfirmationServlet:各商品の購買数。　" + i.getPro_name() + ": " + i.getProduct_buy_no());

			if (i.getStock_no() < i.getProduct_buy_no()) {
				
				session.setAttribute("product_select_list", product_select_list);
				
				System.out.println("ConfirmationServlet:在庫がない状態。エラー文表示");
				request.setAttribute("error", "お買い求め商品の残り在庫がありません。");
				RequestDispatcher rd = request.getRequestDispatcher("/jsp/Confirmation.jsp");
				rd.forward(request, response);
				//returnはこれ以上、下の処理に進まないようにするための処理。メモ　return文　戻り値なしで検索
				//returnは処理を1ブロックではなく全て終了します。
				return;

			} else {
				//-----------------------------------------------------------------------------------------------------------------
				//⒉残り在庫計算(productテーブル)
				System.out.println("ConfirmationServlet:在庫数に余裕がある");


				int remaining_stock_no = i.getStock_no() - i.getProduct_buy_no();
				System.out
						.println("ConfirmationServlet:各商品の在庫数-購買数=残りの在庫数。　" + i.getPro_name() + ": "
								+ remaining_stock_no);

				//データベースに一括で更新するために、remainingStockNoに追加していく。
				remainingStockNo.add(remaining_stock_no);

				//---------------------------------------------------------------------------------------------------------------
                //3.Salesテーブル更新

				//④商品ごとの合計金額。
				//商品ごとの合計金額（購入商品の値段　*　購入個数）
				int sales_price = i.getPro_price() * i.getProduct_buy_no();
				System.out.println("ConfirmationServlet:商品ごとの合計金額：" + sales_price);

				salesPrice.add(sales_price);

			} //if分括弧
		} //for文括弧
		//-------------------------------------------------------------------------------------------------------------------
---
		//2.購入後のproductテーブル更新
		//④購入後の在庫更新(SalesDao.javaのメソッドを呼び、在庫数を更新する。)
		pro_dao.remaining_stock_no_update(product_cd_list, remainingStockNo);
---



		//3.Salesテーブル更新
		int user_id = (int) session.getAttribute("user_id");
		System.out.println("ConfirmationServlet:ユーザIDの番号:　" + user_id);

		//SalesDao.javaのメソッドを使いためのインスタン生成。
		SalesDao sal_db = new SalesDao();

		//⑤Sales（購入テーブル）に格納する。
		//ユーザーID(user_id),商品コード( pro_cd),購入金額(product_sum_price)を入れる
---
		sal_db.sales_db_insert(user_id, product_cd_list, salesPrice);
---
		//-------------------------------------------------------------------------------------------------------------------
		//product_select_listを消す←購入を続けるとき、履歴を消すため。
		product_select_list = new ArrayList<CartBean>();
		session.setAttribute("product_select_list", product_select_list);

		//Complete.jsp(購入画面)に行く。
		RequestDispatcher rd = request.getRequestDispatcher("/jsp/Complete.jsp");
		rd.forward(request, response);
	}
}

//ConfirmationServletより、商品を判別するための商品コードと、残り在庫数を取ってくる。
	public void remaining_stock_no_update(ArrayList<Integer> product_cd_list, ArrayList<Integer> remainingStockNo) {

		String url = "jdbc:mysql://localhost/ecsite?allowPublicKeyRetrieval=true&useSSL=false&useUnicode=true&"
				+ "useJDBCCompliantTimezoneShift=true&"
				+ "useLegacyDatetimeCode=false&"
				+ "serverTimezone=UTC";
		String id = "root";
		String pass = "password";

		Connection con = null;
		PreparedStatement pstmt = null;

		try {
			Class.forName("com.mysql.jdbc.Driver");
			con = DriverManager.getConnection(url, id, pass);

---
			//SQL文--------------------------------------------------------------------------------------------
　　　　　　 for() {
			  pstmt = con.prepareStatement("update product set  stock_no = ? where pro_cd = ?");
			  //残り在庫
			  pstmt.setInt(1, remainingStockNo.get());
			  //商品コード
			  pstmt.setInt(2, pro_cd_list.get());
			
			  pstmt.executeUpdate();
             }
---
		} catch (ClassNotFoundException ex) {
			//例外の文をコンソールに出力。何行目にエラーが出てるかの説明。
			ex.printStackTrace();
		} catch (SQLException ex) {
			ex.printStackTrace();
		} finally {
			try {
				if (pstmt != null) {
					//後片付け
					pstmt.close();
				}
				if (con != null) {
					//後片付け
					con.close();
				}
			} catch (Exception ex) {
			}
		}
	}

　PoductDaoに引数としてArrayListを二つ持ってきたのですがそれをSQL文プリペアドステートメントに活用したいです。
しかし、二つのArrayListの中身を活用するにはネストしか思いつかず、ArrayListの二次元配列を使い、二つの配列をネストせずに回したいです。
二つの配列をネストせずに二つの配列を回す方法はありますでしょうか？

Comments

[m.ts10806]

コードの中に実際のコードにないのものは入れないようにしてください。
forの中にupdate文入れてしまうとまずいんでは。
executeUpdateはSQLを１つだけ実行するもののはずです。

あと、「どういう配列にするか」イメージはできてますか？

[mikannn]

返信頂き誠にありがとうございます。
コードの中にいらないものをいれてしまいすみませんでした。ご指摘頂きありがとうございます。
「forの中にupdate文入れてしまうとまずいんでは。」という質問に対しては、確かに、ArrayListのサイズ分だけ更新処理したいので、間違っていました。ありがとうございます。
質問の意図にあっていなければ申し訳ありません。

「「どういう配列にするか」イメージはできてますか？」に関しては、よくわかっていません。

[m.ts10806]

**

とか、入ってますか？
質問のコードを削るのではなく、お手元のコードをコピペしてください。

[mikannn]

なるほど申し訳ありませんでした。修正させて頂きます。

[Zuishin]

この場合のネストが何を意味するのかわからないけど、ネストしたらまずいんですか？

[mikannn]

返信頂きありがとうございます。
SQL文のfor文の所をネストにして考えた場合です。ややこしくて申し訳ありません。
ネストにしてやってみたのですが、外側のfor文が回っている間に、内側のfor文が回ってしまうので、入れたい値が入らないため、ネストでかけないです。

for(int i = 0; i <= product_cd_list.size(); i++ ){
for(int j = 0; j <= remainingStockNo.size();j++){
pstmt = con.prepareStatement("update product set stock_no = ? where pro_cd = ?");
//残り在庫
pstmt.setInt(1, remainingStockNo.get(j));
//商品コード
pstmt.setInt(2, pro_cd_list.get(i));

pstmt.executeUpdate();
}
}
ネストの場合、上記のように考えてます。

[Zuishin]

product_cd_list が商品リストで、remainingStockNo が在庫数ですよね？
だとしたら remainingStockNo を回しちゃいけません。
product_cd_list[i] から在庫数を計算できるようにする必要があります。

在庫数だけからリストを作っているのが間違いです。
商品をキーに、在庫数を値にしたハッシュマップを作り、商品から在庫数を得られるようにしてみてください。

[mikannn]

HashMapで考え直して作らせて頂きます。
助言誠にありがとうございます。

Answer 1

ArrayListを２つ持つのではなくMapのArrayListを持てば、keyとvalueのリストになるので１つにまとめられると思いますが、いかがでしょうか。

Comments

[mikannn]

返信頂き誠にありがとうございます。
SQL文を更新するためには各商品コードリストと、各在庫数リストが必要なのですが、そのためにリストを二つ必要としています。
HashMapにコードをKey、valueを在庫数にした場合、在庫数は取ってこれると思うのですが、keyの値となっている商品コードをvalueとしてどう取り出すのかが自分にはわかっていません。
ですので、今回は二つリストを呼びだした形を取らさせて頂いております。助言の意図にあっていない場合は申し訳ありません。
最後に、助言誠にありがとうございます。

Answer 2

package controller;

import java.io.IOException;
import java.util.ArrayList;
import java.util.HashMap;

import javax.servlet.RequestDispatcher;
import javax.servlet.ServletException;
import javax.servlet.annotation.WebServlet;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import javax.servlet.http.HttpSession;

import dao.ProductDao;
import dao.SalesDao;
import model.CartBean;

/**
 * Servlet implementation class ConfirmationServlet
 */
@WebServlet("/ConfirmationServlet")
public class ConfirmationServlet extends HttpServlet {

	protected void doPost(HttpServletRequest request, HttpServletResponse response)
			throws ServletException, IOException {
		// TODO Auto-generated method stub

		//購入確認機能制御用クラス

		HttpSession session = request.getSession(false);
		@SuppressWarnings("unchecked")
		ArrayList<CartBean> product_select_list = (ArrayList<CartBean>) session.getAttribute("product_select_list");

		ProductDao pro_dao = new ProductDao();

		ArrayList<Integer> product_cd_list = new ArrayList<Integer>();

		for (CartBean i : product_select_list) {

			product_cd_list.add(i.getPro_cd());
		}


		HashMap<Integer, Integer> new_stock_no = pro_dao.get_new_stock_no(product_cd_list);

		System.out.println("map在庫" + new_stock_no.get(product_cd_list.get(0)));

		//在庫刷新（HashMapに格納した在庫を新しく入れ替えていく。）
		for (CartBean i : product_select_list) {
			int newStockNo = (int) (new_stock_no.get(i.getPro_cd()));
			i.setStock_no(newStockNo);
		}

		//購入後の在庫更新のためのArrayList
		HashMap<Integer, Integer> remainingStockNo = new HashMap<Integer, Integer>();

		//salesテーブルに更新するため。合計金額を入れるArrayList。
		ArrayList<Integer> salesPrice = new ArrayList<Integer>();

		//--------------------------------------------------------------------------------------------------------------
		int j = 0;
		for (CartBean i : product_select_list) {


			System.out.println("ConfirmationServlet:各商品の在庫数。　" + i.getPro_name() + ": " + i.getStock_no());
			//②product_buy_listから各商品の購買数を取り出す。(CartServlet.javaの商品価格の取り方参考)
			System.out.println("ConfirmationServlet:各商品の購買数。　" + i.getPro_name() + ": " + i.getProduct_buy_no());

			if (i.getStock_no() < i.getProduct_buy_no()) {
				
				session.setAttribute("product_select_list", product_select_list);
				System.out.println("ConfirmationServlet:在庫がない状態。エラー文表示");
				request.setAttribute("error", "お買い求め商品の残り在庫がありません。");
				RequestDispatcher rd = request.getRequestDispatcher("/jsp/Confirmation.jsp");
				rd.forward(request, response);
				return;

			} else {
				//-----------------------------------------------------------------------------------------------------------------
				//⒉残り在庫計算(productテーブル)
				System.out.println("ConfirmationServlet:在庫数に余裕がある");

				Integer remaining_stock_no = i.getStock_no() - i.getProduct_buy_no();
				System.out
						.println("ConfirmationServlet:各商品の在庫数-購買数=残りの在庫数。　" + i.getPro_name() + ": "
								+ remaining_stock_no);

				Integer productSelectListIndex = (Integer)(j);
				System.out.println("productSelectListIndexの中身" + productSelectListIndex);

				//データベースに一括で更新するために、remainingStockNoに追加していく。
				remainingStockNo.put(productSelectListIndex, remaining_stock_no);

				//---------------------------------------------------------------------------------------------------------------
				//3.Salesテーブル更新
				int sales_price = i.getPro_price() * i.getProduct_buy_no();
				System.out.println("ConfirmationServlet:商品ごとの合計金額：" + sales_price);

				salesPrice.add(sales_price);

			} //if分括弧

			j++;
			System.out.println(j);

		} //for文括弧
			//-------------------------------------------------------------------------------------------------------------------
			//2.購入後のproductテーブル更新
		pro_dao.remaining_stock_no_update(product_cd_list, remainingStockNo);

		//3.Salesテーブル更新
		int user_id = (int) session.getAttribute("user_id");
		System.out.println("ConfirmationServlet:ユーザIDの番号:　" + user_id);

		SalesDao sal_db = new SalesDao();

		//⑤Sales（購入テーブル）に格納する。
		//ユーザーID(user_id),商品コード( pro_cd),購入金額(product_sum_price)を入れる
		sal_db.sales_db_insert(user_id, product_cd_list, salesPrice);
		//-------------------------------------------------------------------------------------------------------------------

		//product_select_listを消す←購入を続けるとき、履歴を消すため。
		product_select_list = new ArrayList<CartBean>();
		session.setAttribute("product_select_list", product_select_list);

		//Complete.jsp(購入画面)に行く。
		RequestDispatcher rd = request.getRequestDispatcher("/jsp/Complete.jsp");
		rd.forward(request, response);
	}
}

//ConfirmationServletより、商品を判別するための商品コードと、残り在庫数を取ってくる。
	public void remaining_stock_no_update(ArrayList<Integer> product_cd_list, HashMap<Integer, Integer> remainingStockNo) {

		String url = "jdbc:mysql://localhost/ecsite?allowPublicKeyRetrieval=true&useSSL=false&useUnicode=true&"
				+ "useJDBCCompliantTimezoneShift=true&"
				+ "useLegacyDatetimeCode=false&"
				+ "serverTimezone=UTC";
		String id = "root";
		String pass = "password";

		Connection con = null;
		PreparedStatement pstmt = null;

		try {
			Class.forName("com.mysql.jdbc.Driver");
			con = DriverManager.getConnection(url, id, pass);

			//SQL文--------------------------------------------------------------------------------------------
			for (int i = 0; i < product_cd_list.size(); i++) {
				pstmt = con.prepareStatement("update product set  stock_no = ? where pro_cd = ?");
				//残り在庫
				pstmt.setInt(1, remainingStockNo.get(i));
				//商品コード
				pstmt.setInt(2, product_cd_list.get(i));

				pstmt.executeUpdate();
			}

			//-----------------------------------------------------------------------------------------------

		} catch (ClassNotFoundException ex) {
			//例外の文をコンソールに出力。何行目にエラーが出てるかの説明。
			ex.printStackTrace();
			//データベースに関する不具合。
		} catch (SQLException ex) {
			ex.printStackTrace();
		} finally {
			//tryで囲む理由：ここでもエラーが起こる可能性があるため。
			try {
				//リザルトセットがヌルでなかった場合閉じる。
				if (pstmt != null) {
					//後片付け
					pstmt.close();
				}
				if (con != null) {
					//後片付け
					con.close();
				}
			} catch (Exception ex) {
			}
		}
	}

HashMapで送信し、在庫数を商品コードキーでとれるようにしました。
一つ目のコードでInteger productSelectListIndex = (Integer)(j);となっている点がおかしいですが、そこを除けばうまくいきました。