SQLで取得したデータを一覧表としてweb上に表示したい

Question

前提・実現したいこと

フォームで入力した情報をもとにSQLでデータを取得し、一覧表としてweb上に表示する検索機能を作成しています。以前の質問にいただいた回答をもとに改良を加えてみました。デバッグで確認したところ、フォームで入力した値を受け取り、SQL文で実行するところまではできているみたいなのですが、取得した情報がページで表示されずに困っています。

__作成したBeans__
public User(String loginid,String name,String birthdate) {
		this.loginid=loginid;
		this.name=name;
		this.birthdate=birthdate;
	}
__以下、アクセサのため省略__

__作成したDAO__
public User searchUser(String loginid,String name,String birthdatestart,String birthdateend) {
    	Connection conn=null;
    	try {
    		conn = DBManager.getConnection();


			String sql = "SELECT * FROM user WHERE login_id<>'admin'";
                        __以下、各フォームが空欄でない場合、SOL文に条件を付与していく__
			if(!loginid.equals("")) {
				sql += " AND login_id = '" + loginid + "'";
			}

			if(!name.equals("")) {
				sql += " AND name LIKE '%" + name + "%'";
			}

			if(!birthdatestart.equals("")) {
				sql += " AND birth_date<'" + birthdatestart + "'";
			}

			if(!birthdateend.equals("")) {
				sql += " AND birth_date>'" + birthdateend + "'";
			}

			
                       
                        
			Statement stmt = conn.createStatement();
            ResultSet rs = stmt.executeQuery(sql);

　　　　　　　　　　　　　　    		if (!rs.next()) {
                return null;
            }
    		String loginIdData = rs.getString("login_id");
            String nameData = rs.getString("name");
            String birthDate = rs.getString("birth_date");


            return new User(loginIdData,nameData,birthDate);
    	}catch (SQLException e) {
    		e.printStackTrace();
    		return null;
    	}finally {
			if (conn != null) {
	          	try {
	                		conn.close();
	            	} catch (SQLException e) {
	                		e.printStackTrace();
	                		return null;
	            	}
		         }
	          }

    }

__サーブレットのdoPostメソッド__
protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		request.setCharacterEncoding("UTF-8");

		String loginid=request.getParameter("loginid");
		String name=request.getParameter("user-name");
		String birthdateStart=request.getParameter("date-start");
		String birthdateEnd=request.getParameter("date-end");

        __パラメータの入力項目を引数に渡し、DAOのメソッドを実行__
		UserDao userDao=new UserDao();
		User user=userDao.searchUser(loginid, name, birthdateStart,birthdateEnd);

　　　　　__スコープに情報をセット__
        request.setAttribute("user", user);
                                    **⬆️デバッグするとここがnullになってしまいます**
		RequestDispatcher dispatcher = request.getRequestDispatcher("/WEB-INF/jsp/userList.jsp");
		dispatcher.forward(request, response);




	}

__JSPファイル__
  <div class=table-responsive>
     <table class="table table-striped">
     <table class="table table-bordered">
       <thead class="thead-light">
        <tr>
          <th>ログインID</th>
          <th>ユーザ名</th>
          <th>生年月日</th>
          <th></th>
        </tr>
       </thead>

      <tbody>
       __データベースに登録している全てのユーザーの情報を表示__
      　<c:forEach var="userlist" items="${requestScope.userList}" >
       __検索して取得したユーザーの情報を表示__
　　　　 <c:forEach var="userlist" items="${requestScope.user}" >



        <tr>

          <td>${userlist.loginid}</td>
          <td>${userlist.name}</td>
          <td>${userlist.birthdate}</td>
  
          <td>${user.loginid}</td>
          <td>${user.name}</td>
          <td>${user.birthdate}</td>






         
        </tr>
        </c:forEach>
        </c:forEach>



      </tbody>
      </table>
      </table>

   </div>

デバッグした結果nullになってしまう部分とjspファイル内のEL式の書き方あたりがうまくできていない気がします。
お力をお貸しいただきたいです。よろしくお願いします。

Comments

[m.ts10806]

実際のコードにはない記述はコードブロックから出していただけると確認しやすくなります

[m.ts10806]

その前の質問で私の回答の中にある
「アプリケーションから実行する前に想定通りの結果が得られるSQLを直にDBに対して実行確認しましょう。」
は確認されたのでしょうか。
日付範囲のためのbetweenも利用された形跡がありませんし、
前の質問のコードにはあったプリペアドステートメントが消えてます。
あまり望ましくない実装を地でいっているので、あまりアドバイスした意味がなかったのではないかと感じていますが。。

[m.ts10806]

とりあえず前の質問に回答した者としてのアドバイスは「前の回答をきちんと見直して」です。

Answer 1

nullになっているのは、SQLを実行した結果エラーになっているからではないでしょうか。

Answer 2

return new User(loginIdData,nameData,birthDate);

の処理が通っているのであれば

 User user=userDao.searchUser(loginid, name, birthdateStart,birthdateEnd);

の中身がnullになることはありません

select文の結果が0件だった場合とSQLExceptionをcatchした場合にnullを返しているので
このどちらかを通っているものと思われます

・生成されるSQL文が正しいものか確認する
・データベースに正しく中身が入っているか確認する

ことが必要でしょう。

また、処理を見る限り業務ではなく研修や課題で作成しているものと勝手に推測しますが

preparedStatementを扱うこと（すでに指摘されていますが）
select * ～～ですべての検索結果を取得するのは避けることは徹底したほうが良いかと。