Servlet.service() for servlet [servlet.loginservlet] in context with path

###前提・実現したいこと
書籍貸出システムを作成しているのですが、
ログインしたユーザが借りている書籍だけ表示させたいのです。
すかしSQL文が間違っていてエラーが出てしまいます。
上手く行くにはどこを改善すれば良いかご指摘お願いいたします。
###発生している問題・エラーメッセージ

重大: サーブレット jsp のServlet.service()が例外を投げました [水 6 15 15:55:16 JST 2016]
重大: Servlet.service() for servlet [servlet.loginservlet] in context with path [/Library] threw exception [org.apache.jasper.JasperException: An exception occurred processing JSP page /user_mypage.jsp at line 59

56: 				 		+" from lend left join book on book.book_no = lend.book_no "
57: 				 		+"where user_no = ?"; //"検索したいuser_id"
58: 				ps = con.prepareStatement(sql);
59: 				rs = ps.executeQuery();
60: 		%>
61: 	</form>
62: 


Stacktrace:] with root cause

###JSP

<%@ page language="java" contentType="text/html; charset=utf-8"
	import="java.sql.*,javax.naming.*,javax.sql.*, java.text.SimpleDateFormat,java.util.Date"%>
<%@page import="java.util.*"%>
<%@page import="java.text.*"%>

<%@ taglib prefix="c" uri="http://java.sun.com/jsp/jstl/core"%>
<%@ taglib prefix="sql" uri="http://java.sun.com/jsp/jstl/sql"%>

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
    "http://www.w3.org/TR/html4/loose.dtd">

<html>
<sql:setDataSource var="db" driver="com.mysql.jdbc.Driver"
	url="jdbc:mysql://localhost/Library" user="root" password="" />
<sql:query var="rs" dataSource="${db}">
  SELECT book.book_no,book.book_name,genre.genre_name,
  lend.lend_date,book.state FROM book  LEFT JOIN lend ON book.book_no = lend.book_no LEFT JOIN
  user ON user.user_no = lend.user_no LEFT JOIN genre ON book.genre_no = genre.genre_no

  <c:if test="${!empty param['genreNo']}">
  	where book.genre_no = ?
  	<sql:param value="${param['genreNo']}" />
	</c:if>
  ORDER BY book.book_no;
</sql:query>
<sql:query var="rs1" dataSource="${db}">
	select * from genre
</sql:query>
<head>

<link rel="stylesheet" type="text/css" href="css/kanri_mypage.css">

<title>管理者用マイページ</title>
</head>
<body style="text-align: center;">

	<h2>
		ようこそ<%=(String) session.getAttribute("user_id")%>さん
	</h2>
	<form method="POST" action="user_mypage.jsp">

		<%
			Connection con = null;
			PreparedStatement ps = null;
			String sql = null;
			ResultSet rs = null;
			try {

				Context initContext = new InitialContext();
				DataSource ds = (DataSource) initContext
						.lookup("java:comp/env/jdbc/Library");
				con = ds.getConnection();

				sql = /* "select * from book"; */
				 		"select book.book_no,book.book_name,book.genre_no,lend.return_date,book.state"
				 		+" from lend left join book on book.book_no = lend.book_no "
				 		+"where user_no = ?"; //"検索したいuser_id"
				ps = con.prepareStatement(sql);
				rs = ps.executeQuery();
		%>
	</form>


	<%
		GregorianCalendar cal = new GregorianCalendar();
			SimpleDateFormat format = new SimpleDateFormat("yyyy年M月d日 E曜日");
			String datestr = format.format(cal.getTime());
			out.println(datestr);
	%>
	<br>
	<br> あなたが貸出している書籍は下記の通りです。
	<br>
	<br>

	<table border="1" align="center">
		<tr>
			<th>ID</th>
			<th>書籍名</th>
			<th>ジャンル</th>
			<th>返却予定日</th>
			<th>状況</th>
		</tr>
		<tr>
			<%
				while (rs.next()) {
			%>

			<td><%=rs.getString("book_no")%></td>
			<td><%=rs.getString("book_name")%></td>
			<td><%=rs.getString("genre_no")%></td>

			<td>
				<%
					if (rs.getString("state").equals("2")) {
				%>-<%
					} else {
				%> <%
 	if (rs.getString("return_date") == null) {
 %>-<%
 	}

 				if (rs.getString("return_date") != null) {
 %><%=rs.getString("return_date")%> <%
 	}
 			}
 %>
			</td>

			<td>
			???
				<%-- <%
					if (rs.getString("state").equals("2")) {
				%>貸出不可<%
					} else {
				%> <%
 	if (rs.getString("return_date") == null) {
 %>貸出可能<%
 	} else {

 					SimpleDateFormat form = new SimpleDateFormat(
 							"yyyy-MM-dd");
 					Date date = form.parse(rs.getString("return_date"));

 					if ((date.compareTo(new Date())) >= 0) {
 %>貸し出し中<%
 	} else {
 %>返却遅延 <%
 	}
 				}
 			}
 %> --%>
			</td>

		</tr>
		<%
			}
		%>
	</table>
	<%
		} catch (Exception e) {
			throw new ServletException(e);
		} finally {
			try {
				if (rs != null) {
					rs.close();
				}
				if (ps != null) {
					ps.close();
				}
				if (con != null) {
					con.close();
				}
			} catch (Exception e) {
			}
		}
	%>
	<br>



	<form method="POST" action="book_lend.jsp">
		<input type="submit" name="reset" value="書籍貸出へ">
	</form>
	<br>
	<form method="POST" action="Login.jsp">
		<input type="submit" name="reset" value="ログアウト">

	</form>



</body>
</html>

###補足情報(言語/FW/ツール等のバージョンなど)
DBや完成図は前の質問にあるので参考にしてください。

masaya_ohashi

2016/06/15 06:54

見た限りエラーが出ているときのjspと、貼り付けられたjspに相違があります。エラーが出ているjspを貼り付けてください。

l_l_l_l_l_l_l_l

2016/06/15 06:57

失礼致しました。直しました。

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

回答1件

ベストアンサー

解決するかは不明ですが、SQL文に?が含まれているのに、PreparedStatementで一つもパラメータを渡していないのは問題があります。
PreparedStatementとは、SQL文の中に?を置いて代用、後からパラメータを渡すことで初めてデータベースへの要求が完成します。?を使っておきながらパラメータを渡す前にexecuteQueryしてはエラーになります。

今回はセッション内のuser_idを渡したいようですので、以下のようなコードになるかと思います。

Java
1ps = con.prepareStatement(sql);
2ps.setString(1, (String) session.getAttribute("user_id"));//←この行が足りていない
3rs = ps.executeQuery();

投稿2016/06/15 07:01

masaya_ohashi

総合スコア9206

l_l_l_l_l_l_l_l

2016/06/16 00:25

コンパイルも通り実行も出来ましたが、なぜかデータは表示されません。。。

masaya_ohashi

2016/06/16 02:09

https://teratail.com/questions/38153 こちらで回答しておきました。おそらくデータの問題かと思います。

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

あなたの回答

tips

プレビュー

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

質問の解決につながる回答をしましょう。サンプルコードなど、より具体的な説明があると質問者の理解の助けになります。また、読む側のことを考えた、分かりやすい文章を心がけましょう。

15分調べてもわからないことは
teratailで質問しよう！

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう！

Servlet.service() for servlet [servlet.loginservlet] in context with path

関連した質問