Java ajax ショッピングカート

Question

現在、研修でショッピングカート機能を作っています。

商品一覧画面(JSP)があって
各商品があって、それぞれセレクトボックスで個数を選べるようにしています。
セレクとボックスで選んだ値をajaxで取得、サーブレットにリクエストして、値を渡しています。
サーブレット側で、貰った値からDB検索して、それをArrayListに格納してArrayListをセッションに格納して、
カート画面(JSP)で各商品ごと表示をする

という流れで実装したいと考えています。
現状としては、カートには商品は表示されるのですが、一つしか表示されません。最後にセレクトボックスで選択した商品しか表示されません。
ArrayListの要素をデバッグしても、やはり要素は１のままです。

自分が考えている原因としては、ajaxで非同期でリクエストするたびに上書きされているのかなと考えているのですが、どうでしょうか？
ずっと調べているんですが、わかりません。
なにかヒントでもあれば、是非教えていただきたいです。
よろしくお願いいたします。

以下、サーブレットのコードです。

public void doGet (HttpServletRequest request, HttpServletResponse response)
            throws ServletException, IOException {

    	 HttpSession session = request.getSession();	// セッション
    	 request.setCharacterEncoding("UTF-8");
    	 RequestDispatcher dispatcher;
    	 ServletContext sc = getServletContext();

         //セレクトボックスで選んだ個数と商品IDを取得、代入
         String purchase = request.getParameter("param1");
         String itemId = request.getParameter("param2");

　　　　　   　Shopping shopping = new Shopping();
　　　　　 //取得した値を元に検索、代入
　　　　　 　　　CartBean bean = shopping.purchaseItem(purchase, itemId);
　　　　　 　　　ArrayList<CartBean>cartList = new ArrayList<CartBean>();
         cartList.add(bean);
         session.setAttribute("cartList",cartList);

わかりにくくて、すいません。

Comments

[m.ts10806]

「わかりにくくて、すいません。 」というのでしたら質問内容を分かりやすくする努力をしてください。この一言があることで投げっぱなし感を与えてしまうこともあります。あくまで質問内容を充実することに注力し、本件と関係のないことは含まないようにしていただけると「わかりやすく」「伝わりやすく」なりますよ。

Answer 1

ajaxで非同期でリクエストするたびに上書きされているのかな

およそ合っていると思いますが正確な表現ではありません。
リクエストは1回1回単体で行われます。
つまり、前の値は何かで保持しておくようにしないとただ新しいリクエストが送られているだけになります。

セッションに保持しておいても良いですが、
ショッピングカートのようなものでしたら、商品コードと個数をDBに保存しておき、そちらに追記していく形が望ましいと思います。
ただ「買い物完了までいかないまま終了する」こともありえますので、その場合に買い物かごをどのような状態にするかは要件として決めなければなりませんし、
もし、ログイン機能など、今利用しているユーザと紐づけるような仕組みがないのでしたら、ワンタイムトークンなどで紐づける必要があります。
ログイン機能がありログインしていても、買い物かごの情報をどうするかによって作りは変わります。
※セッションに買い物かごの情報を保持しておく場合も同じです

Comments

[akamarukun]

いつもご丁寧にありがとうございます。
非常に助かります。
なるほど、リクエストとはそういうものなのですね。
一つお伺いしたいのですが、
コメントの//取得した値を元に検索、代入から下ののコードでセッションに保存しているつもりなんですが、上書きされてしまいます。リストに入れているつもりなんですが、何かおわかりでしたら教えていただきたいです。

[m.ts10806]

毎回ArrayList作ってるのでそれは当然かと。
セッション利用するならセッションから取り出したArrayListに追記する形にしてください。

[akamarukun]

言われてみればそうですね。
自分でもこんな事に気付かないとかびっくりです、大変失礼しました。
同時にありがとうございます。
勉強し直します。