DTOでsetした値をgetしたい

Question

前提・実現したいこと

DTOを使ってsetした値をgetしたい
//でコメントしているところを見てください。

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

java.sql.SQLIntegrityConstraintViolationException: Column 'user_id' cannot be null
	at com.mysql.cj.jdbc.exceptions.SQLError.createSQLException(SQLError.java:117)
	at com.mysql.cj.jdbc.exceptions.SQLError.createSQLException(SQLError.java:97)
	at com.mysql.cj.jdbc.exceptions.SQLExceptionsMapping.translateException(SQLExceptionsMapping.java:122)
	at com.mysql.cj.jdbc.ClientPreparedStatement.executeInternal(ClientPreparedStatement.java:955)
	at com.mysql.cj.jdbc.ClientPreparedStatement.executeUpdateInternal(ClientPreparedStatement.java:1094)
	at com.mysql.cj.jdbc.ClientPreparedStatement.executeUpdateInternal(ClientPreparedStatement.java:1042)
	at com.mysql.cj.jdbc.ClientPreparedStatement.executeLargeUpdate(ClientPreparedStatement.java:1345)
	at com.mysql.cj.jdbc.ClientPreparedStatement.executeUpdate(ClientPreparedStatement.java:1027)
	at DB.DAO.regAttendance(DAO.java:180)
	at login.AttendanceInfomation.doPost(AttendanceInfomation.java:60)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:648)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:729)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:292)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:207)
	at org.apache.tomcat.websocket.server.WsFilter.doFilter(WsFilter.java:52)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:240)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:207)
	at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:212)
	at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:94)
	at org.apache.catalina.authenticator.AuthenticatorBase.invoke(AuthenticatorBase.java:504)
	at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:141)
	at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:79)
	at org.apache.catalina.valves.AbstractAccessLogValve.invoke(AbstractAccessLogValve.java:620)
	at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:88)
	at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:502)
	at org.apache.coyote.http11.AbstractHttp11Processor.process(AbstractHttp11Processor.java:1132)
	at org.apache.coyote.AbstractProtocol$AbstractConnectionHandler.process(AbstractProtocol.java:684)

該当のソースコード

/**
 * Servlet implementation class AttendanceInfomation
 */
@WebServlet("/AttendanceInfo")
public class AttendanceInfomation extends HttpServlet {
	private static final long serialVersionUID = 1L;

	/**
	 * @see HttpServlet#HttpServlet()
	 */
	public AttendanceInfomation() {
		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
		DAO dao = new DAO();

		DTO dto = new DTO();

		dao.regAttendance(dto.getUserid());
	}

}

public class DAO {
　　　　private static Connection getConnection() {
　　　　final String DSN = "*";//
　　　　final String USER = "*";
　　　　final String PASSWORD = "*";

　　　　try {
　　　　　　　　Class.forName("com.mysql.cj.jdbc.Driver");
	         return DriverManager.getConnection(DSN,USER,PASSWORD);
　　　　} catch (Exception e) {
	         throw new IllegalArgumentException(e);
　　　　}
　　　　}

　　　　private static void allClose(PreparedStatement statement, Connection connection) {
　　　　　　　　　if (statement != null) {
　　　　　　　　　　　　　try {
		                statement.close();
　　　　　　　　　　　　　} catch (SQLException e) {
		                e.printStackTrace();
　　　　　　　　　　　　　}
　　　　　　　　　}
		        if (connection != null) {
		            try {
		                connection.close();
		            } catch (SQLException e) {
		                e.printStackTrace();
		            }
		        }

		    }
		 static Connection connection = null;
		    static PreparedStatement statement = null;


public DTO loginUser(String id,String password) {
    DTO dto = new DTO();

    try {
        connection = getConnection();

　　　　statement = connection.prepareStatement("SELECT * FROM login WHERE id = ? and password = ?");
　　　　statement.setString(1, id);
　　　　statement.setString(2, password);
　　　　ResultSet resultSet = statement.executeQuery();

	if (!resultSet.next()) {
         	return null;
　　　　}
　　　　else {
              //このsetした値をregAttendanceでgetしたい
　　　　　　　　dto.setUserid(id);
　　　　}

　　　　} catch (SQLException e) {
	　　　　e.printStackTrace();
　　　　} finally {
　　　　　　　　allClose(statement, connection);
　　　　}
	return dto;
}

public int regAttendance(String id) {
　　　　int result = 0;
　　　　try {
　　　　connection = getConnection();

　　　　statement = connection.prepareStatement("INSERT INTO time_card(user_id,date,attendance_time) VALUES (?, now(), now())");
　　　　statement.setString(1, id);
　　　　result = statement.executeUpdate();

　　　　} catch (SQLException e) {
　　　　　　　　e.printStackTrace();
　　　　} finally {
　　　　　　　　allClose(statement, connection);
　　　　}
        return result;

public class DTO {
    private String userid;
    private String password;

    public String getUserid() {
        return userid;
    }
    public String getPassword() {
        return password;
    }


    public void setUserid(String userid) {
        this.userid = userid;
    }
    public void setPassword(String password) {
        this.password = password;
    }

}

試したこと

DAOのグローバル変数にDTOのインスタンスを作りそこにsetした。

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

Eclipse

Comments

[ironya]

NullPointerException になった時点のStackTraceを追加してください。

[ironya]

//dao.regAttendance(id);

ここで渡してるidがどこで定義され、どのような値がセットされているかソースから読み取れません。本当にこのソースを実行して出たエラーでしょうか？

[takoyaking]

申し訳ございません。記入漏れがありました。
修正します。

Answer 1

public DTO loginUser(String id,String password) {
    DTO dto = new DTO();
～略～
　　dto.setUserid(id);
～略～
    return dto;
}

    protected void doPost(HttpServletRequest request, HttpServletResponse response)
            throws ServletException, IOException {
        // TODO Auto-generated method stub
        DAO dao = new DAO();

        DTO dto = new DTO();

        dao.regAttendance(dto.getUserid());
    }

loginUser()でdtoをnewして、setUseridで値をセットしていますが、提示されているコードの
doPost()内で別なdtoをnewしている（loginUser()で生成したdtoとは別なものを使っている）
ことが原因です。
loginUser()をどのタイミングで呼び出すかが不明なため、コードの修正自体は致しかねますが
loginUser()でdtoをreturnしているので、それをsessionで保持する形が自然ではないかと思います。

コメントに対して

loginUser()を使うところのプログラムも追記した方がわかりやすいでしょうか

今発生している問題を解決するためだけであれば、必須ではないです。
ただ　回答者によってはコードを読み解く手間が増える　等の指摘を受けるかもしれません。
（例えば今回の例で言うと、この問題が発生する以前に別な箇所でloginUser()を呼んでいることを推測する必要があります。）
追記依頼を受けた場合に載せればいいかな、と個人的には思ってます。

このプログラム以外のところでもloginUser()のdtoを取得したいので…

でしたら尚更sessionの利用が適切かなと思われます。

Comments

[takoyaking]

loginUser()を使うところのプログラムも追記した方がわかりやすいでしょうか？
できれば、このプログラム以外のところでもloginUser()のdtoを取得したいのでloginUser()のdtoを保持しておきたいです。

[takoyaking]

dto自体をsessionで保持するということでよろしいでしょうか？

[tetutetu]

そうですね。
ネーミング的にログインする際にユーザのＩＤ，パスワードを格納するDTOなのかなと思うのですが
（パスワードを保持するリスクとかは本題から逸れるので無視）
loginUser()を呼び出した時点で戻り値のdtoをsessionで保持、必要な時に呼び出す形でいいんじゃないでしょうか。

[takoyaking]

なるほど！わかりました。
参考にしながらプログラムを進めてみます。
ありがとうございます。