session管理で作るスケジュール管理がわかりません。

前提・実現したいこと

スケジュール管理ができるカレンダーを作成中です。
カレンダーjsp画面にある予定入力フォームに予定を入力すると処理サーブレットを通して
カレンダーjsp画面に予定が反映されるというシステムを作成中です。

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

現在初回の入力は反映されるのですが、2回目を送信すると
1回目の予定が消去される問題が発生しています。

### 該当のソースコード

			if(session.getAttribute("map") == null) {
				HashMap<Integer,String> map = new HashMap<Integer,String>();
				map.put(tb.getDay(), tb.getTask());
				session.setAttribute("map",map);
				getServletContext().getRequestDispatcher("/calendar.jsp").forward(req, res);
			}else {
				map.put(tb.getDay(), tb.getTask());
				session.setAttribute("map",map);
				getServletContext().getRequestDispatcher("/calendar.jsp").forward(req, res);

ソースコード

		// カレンダーに予定を記入する部分
		else if (a.equals("2")) {

			// beanの宣言
			TaskBean tb = new TaskBean();

			String task = req.getParameter("task");	// 予定
			tb.setTask(task);
			String d = req.getParameter("day");	// 日
			int day = Integer.parseInt(d);
			tb.setDay(day);

			// mapの宣言
			HashMap<Integer,String> map = new HashMap<Integer,String>();

			HttpSession session = req.getSession();

			if(session.getAttribute("map") == null) {
				map.put(tb.getDay(), tb.getTask());
				session.setAttribute("yotei",tb);
				session.setAttribute("map",map);
				getServletContext().getRequestDispatcher("/calendar.jsp").forward(req, res);
			}else  {
				map.put(tb.getDay(), tb.getTask());
				session.setAttribute("yotei",tb);
				session.setAttribute("map",map);
				getServletContext().getRequestDispatcher("/calendar.jsp").forward(req, res);
			}
		}


### 試したこと
2回目に画面入力されると同じ処理を通るので新しくmapが作られ保持が消えるのかと考えて
hashmapをif文の中に入れてみたのですが、else{}内で値が取れない状態になったりと、
どうしていいのかわからない状態です。

2回目に通るときは既存のmapに取得内容をputして画面遷移ができればいいと思うのですが
どうすればいいのでしょう…


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

エクリプス

momon-ga

2020/11/26 05:07 編集

ご提示のソースコードは、コンパイルエラーにならないですか？修正する前の動いているコードを載せてください

wakewakame1

2020/11/26 05:15

大変失礼致しました。動いているコードを掲載致しました。掲載したコードですとhashmapを通過してしまい以前のデータが失われてしまうと考えています。

m.ts10806

2020/11/26 05:16

なぜセッション？半永久的にスケジュールのデータをもちたいのならDBでは？

wakewakame1

2020/11/26 05:23

m.ts10806様 dbを使うと問題なく作れます。ですが、セッション管理に苦手意識があるので作ってみました、じゃ駄目でしょうか

m.ts10806

2020/11/26 05:29

前提や背景は書かれてないとわかりません。ただ、「本来DBでやること」をセッションでやったとして、それは身にならないのではと思います。

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

回答2件

まず。。提示いただいたソースでは

Java
1　　　　　　　// mapの宣言
2            HashMap<Integer,String> map = new HashMap<Integer,String>();

毎度、mapの初期化してますよね。
で。。

Java
1            if (session.getAttribute("map") == null) {
2                map.put(tb.getDay(), tb.getTask());
3                session.setAttribute("yotei",tb);
4                session.setAttribute("map",map);
5                getServletContext().getRequestDispatcher("/calendar.jsp").forward(req, res);
6            } else {
7                map.put(tb.getDay(), tb.getTask());
8                session.setAttribute("yotei",tb);
9                session.setAttribute("map",map);
10                getServletContext().getRequestDispatcher("/calendar.jsp").forward(req, res);
11            }

if,elseどちらとも

Java
1
2
3                map.put(tb.getDay(), tb.getTask());
4                session.setAttribute("yotei",tb);
5                session.setAttribute("map",map);
6                getServletContext().getRequestDispatcher("/calendar.jsp").forward(req, res);

してるので、その時画面から入力しか値がmapに格納していないからです。（原因）

** sessionからmapを取り出さないといけません**

Java
1            if (session.getAttribute("map") == null) {
2                map.put(tb.getDay(), tb.getTask());
3                session.setAttribute("yotei",tb);
4                session.setAttribute("map",map);
5                getServletContext().getRequestDispatcher("/calendar.jsp").forward(req, res);
6            } else {
7                map = (Map) session.getAttribute("map"); //追加
8                map.put(tb.getDay(), tb.getTask());
9                session.setAttribute("yotei",tb);
10                session.setAttribute("map",map);
11                getServletContext().getRequestDispatcher("/calendar.jsp").forward(req, res);
12            }

これでどうですかね。

投稿2020/11/26 05:54

Takeshi_Ueda

総合スコア183

wakewakame1

2020/11/26 06:03

!!!!! こ、こちらの回答も問題なく動きました。 newしていても2回目は以前のデータを取り出して下でsetしてるからOKという解釈いいのでしょうか… すごいです。どちらも参考にさせていただきます。

Takeshi_Ueda

2020/11/26 06:40

そうですね。ですが、momon-gaさん提示のソースの方がJavaの実装しては良い方だと思います。私のは、あくまで何の処理が足りないかを補足しているだけなので。 momon-gaさんの方は、初めにsessionのmapを取り出して、無ければHashMapの map生成しているので処理の流れ的にはあちらの方が良いですよ。原因が把握できたのでしたら幸いです。頑張ってくださいね。

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

ベストアンサー

掲載したコードですとhashmapを通過してしまい以前のデータが失われてしまうと考えています。

その通りです。
毎回新規にHashMapをnewしているので、それを参照する場合、既存の情報は失われます。

なので、いったん作ったHashMapがあれば、それを使いましょう。

java
1            // mapの宣言
2            HashMap<Integer,String> map = new HashMap<Integer,String>();
3
4            HttpSession session = req.getSession();
5
6            if(session.getAttribute("map") == null) {
7//省略
8            }else  {
9                map = (HashMap<Integer,String>)session.getAttribute("map");
10
11                map.put(tb.getDay(), tb.getTask());
12                session.setAttribute("yotei",tb);
13                session.setAttribute("map",map);
14                getServletContext().getRequestDispatcher("/calendar.jsp").forward(req, res);
15            }

最初からHashMapを取得してNullなら新規に作るという方が自然かなと思います。

java
1HttpSession session = req.getSession();
2HashMap<Integer,String> map = (HashMap<Integer,String>)session.getAttribute("map")
3if(map == null) {
4    map = new HashMap<Integer,String>();
5}