java DAOを使って取得ができない

Question

実現したいこと

mysqlにて、userテーブル

field	type	null	key	Default	Extra
id	varchar(5)	NO		NULL
name	varchar(20)	YES		NULL
pass	varchar(20)	YES		NULL
dep	int	YES		NULL

depテーブル

Field	Type	Null	Key	Default	Extra
id	varchar(5)	NO		NULL
name	varchar(20)	YES		NULL

を作成し、
userテーブルには、

id	name	pass	dep
1	山田　太郎	pass1	1
2	山田　花子	pass2	2
3	山田　三郎	pass3	1

depテーブルには、

id	name
1	管理部
2	システム開発部
3	教育事業部

を登録しました。
login.jsp よりidとpassを入力しログインボタンを押すと、LoginUserServletConf.javaからUserDao.javaへ行き、userテーブルとdepテーブルから参照取得してLoginUserServletConf.javaに戻り、 menu.jspで”部署：○○　氏名：○○”と表示させたい。

発生している問題・分からないこと

遷移はするが、部署がNULLで返ってきます。

エラーメッセージ

遷移はされる為、エラーメッセージはありません。

該当のソースコード

import java.io.IOException;
import javax.servlet.RequestDispatcher;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import javax.servlet.http.HttpSession;

	public class UserDao extends HttpServlet {
	       
		public void doGet (HttpServletRequest req, HttpServletResponse res)
		        throws ServletException, IOException {
					doPost( req, res );
			}

			public void doPost(HttpServletRequest req, HttpServletResponse res)
		        throws ServletException, IOException {

				res.setContentType("text/html;charset=UTF-8");
				req.setCharacterEncoding("UTF-8");
			
				//セッションの取得
		        HttpSession session = req.getSession(true);
				
				String forward=null;
	
	//ログインがおされた
				if(req.getParameter("submit").equals("ログイン")){

	      			String uid=req.getParameter("uid");
	      			String upass=req.getParameter("upass");
	      		
	      			//DAOをインスタンス化
	    			UserDao logindao= new UserDao();
	    			//DAOを使用してID・パスワードをチェックする
	    			UserBean bean=logindao.loginCheck(uid, upass);
	    			DepBean bean2=logindao.loginCheck2(uid,upass);
	    			//もし該当者がいれば
	    			if(bean != null){
	    				session.setAttribute("uname",bean.getName());
	    				session.setAttribute("dname",bean2.getName());
	    				forward="/web/menu.jsp";
	    			}else{
	    				req.setAttribute("errMsg","※ログインID・パスワードが違います");
	    				forward="/web/login.jsp";
	    			}
	    		}
				RequestDispatcher rd = req.getRequestDispatcher(forward);
				rd.forward(req, res);
				}
			}

import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;

public class  extends DBAccess{
	
	//ユーザーIDとパスワードを使用し、情報を取得し返却する
		public  UserBean loginCheck(String uid ,String upass) {
			
			UserBean user=null;
			
			String sql1 = "select * from user where id=? && pass=?";
			try {
				connect();
				// ステートメントの作成
				PreparedStatement ps = getConnection().prepareStatement(sql1);
				ps.setString(1, uid);
				ps.setString(2, upass);
				//select文の実行（値を取得）
				ResultSet rs=ps.executeQuery();
				//値が取得してこれたら
				if(rs.next()){
					//UserBeanを実体化
					user= new UserBean();
					//値の格納
					user.setName(rs.getString("name"));
					user.setId(rs.getString("id"));
					user.setPass(rs.getString("pass"));
					user.setDep(rs.getInt("dep"));
				}
			} catch (SQLException e) {
				e.printStackTrace();
			} finally {
				disconnect();
			}		
		return user;
	}
			
		//ユーザーIDとパスワードを使用し、情報を取得し返却する
				public  DepBean loginCheck2(String uid , String upass) {					
					UserBean user=null;
					DepBean dep =null;					
					String sql2 ="SELECT dep.name FROM user INNER JOIN dep ON user.dep = dep.id WHERE user.id = ? AND user.pass = ?"; 							
				try {
					connect();
					// ステートメントの作成
					PreparedStatement ps2 = getConnection().prepareStatement(sql2);
					ps2.setString(1, uid);
					ps2.setString(2, upass);
					//select文の実行（値を取得）
					ResultSet rs2=ps2.executeQuery();
					//値が取得してこれたら
					if(rs2.next()){
						//DepBeanを実体化
						dep= new DepBean();
						//値の格納
						dep.setName(rs2.getString("id"));
						dep.setId(rs2.getString("name"));
					}
				} catch (SQLException e) {
					e.printStackTrace();
				} finally {
					disconnect();
				}
				return dep;
}

import java.io.Serializable;
public class DepBean implements Serializable {
		private String id;
		private String name;
		
		
		public String getId() {
			return id;
		}
		public void setId(String id) {
			this.id = id;
		}
		
		
		public String getName() {
			return name;
		}
		public void setName(String name) {
			this.name = name;
		}
}

import java.io.Serializable;
public class UserBean implements Serializable{
	private String id;
	private String name;
	private String pass;
	private int dep;
	

	public String getName() {
		return name;
	}
	public void setName(String name) {
		this.name = name;
	}
	
	
	public String getId() {
		return id;
	}
	public void setId(String id) {
		this.id = id;
	}
	
	
	public String getPass() {
		return pass;
	}
	public void setPass(String pass) {
		this.pass = pass;
	}
	
	
	public int getDep() {
		return dep;
	}
	public void setDep(int dep) {
		this.dep = dep;
	}

}

<%@ page contentType="text/html;charset=UTF-8" language="java"%>
<html>
<head>
<title>aaa</title>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
	<div align="center">
		<strong>部署：<%=session.getAttribute("dname") %></strong><br>
		<strong>氏名：<%=session.getAttribute("uname") %></strong>
	</div>
</body>
</html>

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

int型とvarchar型だから？と思い、CAST(user.dep AS CHAR)にしました。
結果、同じく部署のみNULLでした。

補足

値を取得後、depbeanのことしか書いていないのも変だとは思うのですが、どのように変更すればいいのかもわからない状態です。
初心者で、そもそも構成が間違っているかもしれませんが、よろしくお願いいたします。

Comments

[jimbe]

テーブルの構造を提示されるのでしたら、マークダウンの "表" をご利用されればちゃんと矩形で表示出来て見易いと思います。

[対応しているMarkdownの記法を知りたい]
https://teratail.com/help#about-markdown

質問は編集でき、リアルタイムにプレビューが表示されているはずですので、確認しながら弄ってみてください。

また、「該当のソースコード」でコードのマークダウン(```)のファイル名を書かれている個所は言語名(java や html 等)を書く所です。
例えば

```LoginUserServletConf
(コード)
```

は

LoginUserServletConf.java
```java
(コード)
```

のように ``` の後は言語名とし、ファイル名は ``` の囲みの外に書くようにしてください。そうすることで、コード内の言語的キーワードに色が付くようになります。

[退会済みユーザー]

UserDaoの61行目で例外は出力されていませんか？
コードを見ると
43行目、`SELECT dep.name FROM user ~~~`でnameだけ取得していて、
57行目、`dep.setName(rs2.getString("id"));`でidも参照しようとしているようです。
65行目で返されるのは55行目でコンストラクタが呼び出されただけのDepBeanになるような気がしました。
見当違いでしたらごめんなさい・・

[jimbe]

>試したこと・調べたこと
>上記の詳細・結果
>遷移はするが、NULLで返ってきます。

「試したこと・調べたこと」の「上記の詳細・結果」というのは、試したり調べたことで得た情報をコードに反映した結果どういう結果が出たかを書く個所です。
「遷移はするが、NULLで返ってきます。」は、試したり調べたり以前に現状起こっていることですよね? それなら

>発生している問題・分からないこと
>部署がNULLで返ってきます。

の「部署がNULLで返ってきます。」を「遷移はするが、部署がNULLで返ってきます。」とするほうが結果をより端的に説明すると思うのですが如何でしょうか。

逆に「補足」とされている「int型とvarchar型だから？と思い、CAST(user.dep AS CHAR)にした」は「試したこと・調べたこと」であり、そうしたらどうなったのかを「上記の詳細・結果」に書かれると良いと思います。

[lovam-nm]

loginCheckでString sql1 = "select * from user where id=? && pass=?";
と記載していますが正しくユーザ情報を取得できていますか？
「&&」ではなく「AND」ではないでしょうか？

[jimbe]

エラーメッセージに「遷移はされる為、エラーメッセージはありません。」とありますが、遷移すればエラーが無いわけではありません。
実際コード内では例外が発生したら e.printStackTrace() をしているだけで処理は継続してしまいます。
問題が発生したらどのような動作をすると想定しているのでしょう。何もしなくてもエラーが表示されるものとお考えなら、それは間違いです。
例外が発生したら遷移先でエラーを表示するつもりなら、そのようにコードを組む必要があり、プログラムでシステムを作るということは正常な動作だけでなく異常が起きた場合の動作も作ることです。

[kyakya]

jimbe様
コメントありがとうございます。
質問を修正したのでご確認よろしくお願いいたします。
遷移すればエラーが無いわけではないとの事。とても勉強になりました。ありがとうございます。

yametai様
コメントありがとうございます。
例外出力はされませんでした。
65行目で返されるのは55行目でコンストラクタが呼び出されただけのDepBeanになるような気がしたとの事ですが、私も初心者ながらそのような気がしております。ですが、修正方法も見いだせない状態でございます。もう一度、この箇所に絞って調べていきたいと思います。ありがとうございます。

lovam-nm様
コメントありがとうございます。
ユーザー情報は取得できておりました。&&をANDに変更してもユーザー情報は取得できました。
ご指摘頂きありがとうございました。

[jimbe]

編集ありがとうございます。 java のコードは見易くなったと思います。
"表" ですが、2行目に "|:--|:--(中略)|" といった行を入れる必要があります。例えば user テーブルの所は(その前の "実現したいこと" に "|" が入っちゃってるのでそれから直すとして)

### 実現したいこと
mysqlにて、userテーブル
| field | type | null | key | Default | Extra |
|:--|:--|:--|:--|:--|:--|
| id | varchar(5) | NO | | NULL | |
| name | varchar(20)| YES | | NULL | |
| pass | varchar(20)| YES | | NULL | |
| dep | int | YES | | NULL | |

とすると、綺麗な表が出来ると思います。

また、コードの最後のは java でなく html ですね。ファイル名が無くなってしまうと何のファイルか分からなくなってしまいますので、コードのマークダウンの直前に普通のテキスト行としてファイル名は書くようにしてください。

Answer 1

とりあえず、

UserDAO.java (クラス名消えちゃってますけど) の 43 行目の SELECT 文には dep.id を付け足すべきでしょう。
before:

SELECT dep.name FROM ～

after:

SELECT dep.id, dep.name FROM ～

そして、 56 ～ 58 行目の取得する部分で name と id が逆です。
before:

//値の格納
dep.setName(rs2.getString("id"));
dep.setId(rs2.getString("name"));

after:

//値の格納
dep.setId(rs2.getString("id"));
dep.setName(rs2.getString("name"));

---

で、 user → dep は n:1 なのですから、 user だけを取り出すのと user の情報を使って dep を取り出す二つのメソッドにするより、 UserBean にオプション的に dep.name を持たせられるよにして loginCheck メソッドを getWithDep とか名前を変えて loginCheck2 みたいに dep テーブルと join して名前を入れちゃうほうが簡単じゃないですかね。(そうすれば loginCheck2 メソッドも DepBean クラスも不要ですし。ついでに SQLException は throws にして使う側で エラー表示に使用するとか…。)

UserDAO.java (loginCheck メソッドの修正案)

  public UserBean getWithDep(String uid, String upass) throws SQLException {
    String sql = "SELECT u.name, id, pass, dep, d.name AS dep_name" +
                 " FROM user AS u INNER JOIN dep AS d ON u.dep = d.id" +
                 " WHERE id=? AND pass=?";
    try {
      connect();
      PreparedStatement ps = getConnection().prepareStatement(sql);
      ps.setString(1, uid);
      ps.setString(2, upass);
      ResultSet rs = ps.executeQuery();
      if(rs.next()) {
        UserBean user = new UserBean();
        user.setName(rs.getString("name"));
        user.setId(rs.getString("id"));
        user.setPass(rs.getString("pass"));
        user.setDep(rs.getInt("dep"));
        user.setDepName(rs.getString("dep_name"));
        return user;
      }
    } finally {
      disconnect();
    }
    return null;
  }

UserBean.java (追加部分)

public class UserBean implements Serializable {
  :
  private String depName;

  :
  public String getDepName() {
    return depName;
  }
  public void setDepName(String name) {
    this.depName = name;
  }
}

使う側

  :
  UserDao userDao = new UserDao();
  try {
    UserBean user = userDao.getWithDep(uid, upass);
    if(user != null) {
      session.setAttribute("uname", user.getName());
      session.setAttribute("dname", user.getDepName());
    :
  } catch(SQLException e) {
    //例えば
    e.printStackTrace(); //例外の詳細はどこかに残るようにしておいて
    req.setAttribute("errMsg", "※処理に異常が発生しました。"); //利用者には問題が発生したことを知らせる?
    forward = "/web/login.jsp";
  }
  :

Comments

[kyakya]

jimbe様
ご丁寧に教えて頂き本当にありがとうございます。”user → dep は n:1”の意味がわからず、n:1の事が学べ、”SQLException は throws にして使う側で エラー表示に使用”などのテクニックやエラーメッセージの概念など、今回の質問で沢山の学びができ感謝の気持ちでいっぱいです。
教えて頂いたコードを学習後、訂正実行したのですが、エラーがでてしまいました。ですが、後もう一歩な気がますので、自身で修正してみたいと思います。また、解決・未解決にかかわらず、その後の報告は改めてさせて頂きます。
本当にありがとうございます。

[kyakya]

先日はありがとうございました。

ログインをクリックすると"※処理に異常が発生しました。"と表示され自力で解決できず、もしおわかりでしたらご教授いただけないでしょうか？以下詳細になります。

(UserDao.java)
```java
public class UserDao extends DBAccess{
public UserBean getWithDep(String uid, String upass) throws SQLException {
String sql = "SELECT u.name, id, pass, dep, d.name AS dep_name" +
" FROM user AS u INNER JOIN dep AS d ON u.dep = d.id" +
" WHERE id=? AND pass=?";
try {
connect();
PreparedStatement ps = getConnection().prepareStatement(sql);
ps.setString(1, uid);
ps.setString(2, upass);
ResultSet rs = ps.executeQuery();
if(rs.next()) {
UserBean user = new UserBean();
user.setName(rs.getString("name"));
user.setId(rs.getString("id"));
user.setPass(rs.getString("pass"));
user.setDep(rs.getInt("dep"));
user.setDepName(rs.getString("dep_name"));
return user;
}
} finally {
disconnect();
}
return null;
}
}
```

(UserBean.java)
```java
public class UserBean implements Serializable{
:
private String depName;

public UserBean () {
}
:
public String getDepName() {
return depName;
}
public void setDepName(String name) {
this.depName = name;
}
}
```
(使う側)
```java
//ログインがおされた
if(req.getParameter("submit").equals("ログイン")){

String uid=req.getParameter("uid");
String upass=req.getParameter("upass");

//DAOをインスタンス化
UserDao userDao = new UserDao();
try {
//DAOを使用してID・パスワードをチェックする
UserBean user = userDao.getWithDep(uid, upass);
//もし該当者がいれば
if(user != null) {
session.setAttribute("uname", user.getName());
session.setAttribute("dname", user.getDepName());
forward="/web/menu.jsp";
}else{
req.setAttribute("errMsg","※ログインID・パスワードが違います");
forward="/web/login.jsp";
}
} catch(SQLException e) {
e.printStackTrace(); //例外の詳細はどこかに残るようにしておいて
req.setAttribute("errMsg", "※処理に異常が発生しました。"); //利用者には問題が発生したことを知らせる。
forward = "/web/login.jsp";
:
```

と変更いたしました。
自分なりに調べたところ、データベース接続の確認、SQLクエリの確認、大文字と小文字の区別の確認との事でした。（確認しましたが、全て問題無いと思いました。）
又”dep テーブルに depName カラムが存在することも確認”とchatGPTで分かったので、depテーブルのnameをdepNameに変更して実行しましたが、同じくログインをクリックすると、"※処理に異常が発生しました。"

教科書を読み直したりしたのですが、その他、どのように修正をすればいいのか分からず苦戦しております。
自力で解決出来ず申し訳ございませんでした。
どうぞ宜しくお願い致します。

[jimbe]

問題が発生した時に真っ先に行うべきは『何が起きているのかの確認』です。闇雲に検索するとか何かに聞くとか解決策を探すことではありません。

コードのコメントに書いています通り、『※処理に異常が発生しました。』はシステムに関与できない利用者に向けての状況の説明であり、システムに関与できるプログラマが見るべき詳細情報は
『例外の詳細はどこかに残るようにしておいて』とコメントを付けた
e.printStackTrace();
による出力です。
printStackTrace は『標準出力』に出されますので、実行しているシステムにおいて標準出力が何処に保存されるのかを調べて状況の詳細を得てください。
もし標準出力が保存されないようなシステムでしたら、逆に何処に出力すれば保存されるのかを調べて問題発生時はそこに出力するようにコードを直した上で再度実行して、詳細を得てください。
全てはそれからです。

[kyakya]

回答ありがとうございます。
問題が解決しました!!
"『例外の詳細はどこかに残るようにしておいて』とコメントを付けたe.printStackTrace();による出力"
"標準出力が保存されないようなシステムでしたら、逆に何処に出力すれば保存されるのかを調べて問題発生時はそこに出力するようにコードを直した上で再度実行"
また勉強させていただきました。

私なりにコードを追加し、問題解決いたしました。
始めて質問をさせていただき、考え方まで学ぶ事ができ、jimbe様にご回答頂き嬉しく思います。
ありがとうございました。

ベストアンサーに選ばせていただきました。

[jimbe]

terastail でプログラミングに慣れていない人の質問を見ると大抵の場合原因よりも先に解決方法を探しています。(逆に言うと原因を探していない/調べようとしていない風な質問は大抵プログラミングに慣れていない人です。)
プログラムは100%"作られたモノ"であり、因果関係によって現象が発生します。従って原因を直さなければ結果は変わりません。
また、(私は行っていないので想像ですが)学校等でプログラミングを教える場合、"作り方"は教えても"問題が発生した場合にどうするか"は教えていないようです。(そもそも"問題が発生する"事自体を想定しているのか怪しいくらいですね。)

昔(c 言語だけでシステム全部を作っていた頃?)は「プログラムの 7～80% は問題に対応するためのコード」と言われたくらい、正常な動作の為の部分よりも労力が払われた処理です。
ちゃんとシステムが動作していれば動かないような部分だったりもしますので作成する段階ではヤル気が出難いのは確かですが^^; 「想定外」のことで長時間システムが止まって世間を騒がせるようなニュースが実はこのような処理が足りなかったこと(=想定して処理を入れていなければならなかった)を表していると考えれば、重要性は分かると思います。(入れていても尚複雑過ぎて時間が掛かってしまう事もあるでしょうが、なら入れてなかったら尚更…ってことです。)

お疲れさまでした。