ServletでSQLの結果をロジックからコントローラに返却する仕様に変更したいです。

Question

Servlet/MySQLを使ってログイン画面を作成しています。

LoginServlet.java(コントローラ側)とLoginServeltLogic.java(ロジック側)に分けているのですが、
まだコントローラ側にロジックの処理が残ってしまっています。
具体的には、
1: 入力されたパラメータでDBを検索する(userid , password でdbを検索)
2: 検索結果が1件なら成功、0件なら失敗

現時点ですと、2をコントローラ側で行っている形になっており、
可能であれば2もロジック側で行いたいです。

もう少し具体的にいうと、ログイン可否をロジックからコントローラ側に返却する形にしたいのですが、
ヒントなどがあれば、ご回答頂けますと幸いです。
よろしくお願いいたします。

LoginServlet.java
```ここに言語を入力
package budget_tracker.login;

import java.io.IOException;
import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;

import javax.naming.InitialContext;
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.sql.DataSource;

/**

• Servlet implementation class LoginServlet

*/
@WebServlet("/LoginServlet")
public class LoginServlet extends HttpServlet {
private static final long serialVersionUID = 1L;
LoginServletLogic loginServletLogic;
Connection conn = null;
PreparedStatement pstmt = null;
ResultSet rset = null;

// データソースの作成
DataSource ds;

// 初期化処理
public void init() throws ServletException {
	try {
		// 初期コンテキストを取得
		InitialContext ic = new InitialContext();
		// ルックアップしてデータソースを取得
		ds = (DataSource) ic.lookup("java:comp/env/jdbc/searchman");
				
	} catch (Exception e) {

	} finally {
		try {
			// just in case, terminate the DB connection with finally statement
			conn.close();
		} catch (Exception e) {
		}
	}
}

/**
 * @see HttpServlet#HttpServlet()
 */
public LoginServlet() {
	super();
	// TODO Auto-generated constructor stub
}

/**
 * @see HttpServlet#doGet(HttpServletRequest request, HttpServletResponse
 *      response)
 */
protected void doGet(HttpServletRequest request, HttpServletResponse response)
		throws ServletException, IOException {
	// TODO Auto-generated method stub
	response.getWriter().append("Served at: ").append(request.getContextPath());
}

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

	// setting text code
	request.setCharacterEncoding("Windows-31J");

	// get the id in index.jsp
	String inputId = request.getParameter("id");

	// get the password in index.jsp
	String inputPassword = request.getParameter("password");

	System.out.println(inputId);
	System.out.println(inputPassword);

	try {
		
		loginServletLogic = new LoginServletLogic();
		
		rset = (ResultSet) loginServletLogic.login(inputId, inputPassword);
		

// transfer the data to the transition page(put it by Attribute)
request.setAttribute("SqlResult", rset);

		// move on to loginResult.jsp or loginResultFailed.jsp
		if (rset.next()) {
			request.getRequestDispatcher("mainSearch.jsp").forward(request, response);
		} else {
			request.getRequestDispatcher("loginFailed.jsp").forward(request, response);
		}

		// terminate the used objects
		rset.close();
		pstmt.close();
		conn.close();

	} catch (Exception e) {
		e.printStackTrace();

		doGet(request, response);
	} finally {
		try {
			// just in case, terminate the DB connection with finally statement
			conn.close();
		} catch (Exception e) {
		}
	}

}

}

LoginServletLogic.java
```ここに言語を入力
package budget_tracker.login;

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

import javax.naming.InitialContext;
import javax.sql.DataSource;

public class LoginServletLogic {

	ResultSet rset = null;
	DataSource ds;
	Connection conn = null;
	PreparedStatement pstmt = null;

	public ResultSet login(String inputId, String inputPassword) throws SQLException {
		try {

			// 初期コンテキストを取得
			InitialContext ic = new InitialContext();
			// ルックアップしてデータソースを取得
			ds = (DataSource) ic.lookup("java:comp/env/jdbc/searchman");

			// データソースからConnectionを取得
			conn = ds.getConnection();

			StringBuilder sql = new StringBuilder();

			sql.append("select * from login_table where id = ? && password = ? ");

			pstmt = conn.prepareStatement(new String(sql));

			pstmt.setString(1, inputId);

			pstmt.setString(2, inputPassword);
			
			rset = pstmt.executeQuery();
			
		} catch (Exception e) {
			e.printStackTrace();

		} 
		
		return rset;
		

	}

}

Comments

[m.ts10806]

まず思ったように書いてみては。
赤の他人には何が問題なのか分かりません

[dodox86]

動くと言えば動くのかもしれませんけど、取得したResultSetをmainSearch.jspにforward()したり、DBコネクションやトランザクションの取り回しが心配になる作りに見えますが、今はそういうのもアリなのでしょうか。

Answer 1

public ResultSet login(String inputId, String inputPassword) throws SQLException {

を

public boolean login(String inputId, String inputPassword) throws SQLException {

に変えて、コントローラでやっている処理

rset.next()

の結果（と、ResultSetのクローズ処理）をLogic側で書けばよいのでは？

コントローラ側は

if (loginServletLogic.login(inputId, inputPassword)) {
    request.getRequestDispatcher("mainSearch.jsp").forward(request, response);
} else {
    request.getRequestDispatcher("loginFailed.jsp").forward(request, response);
}

みたいになるんじゃないでしょうか。

request.setAttribute("SqlResult", rset);

の部分は、どうなるかは気になりますが・・・