JDBCプログラミングでデータベースが更新できない

Question

前提・実現したいこと

ここに質問の内容を詳しく書いてください。
JDBCプログラミングおよびサーブレットとJSPを用いて、タスク管理プログラムを作っています。
その中で、JSPのフォームにIDを入力して送信するとそのIDをメンバー表から探し出し、当該ID存在した場合はタスク管理表にIDを追加する機能を実装しております。

発生している問題

単体でのエラーは発生しないのですが、結合して実際にフォームに入力して送信してもデータベースへの反映がうまくいきません。
エラーメッセージや例外も発生していないため、ここからどうすればいいのか手詰まりの状況です。

該当のソースコード1/5（業務ロジック）

package mod4;

import java.io.IOException;
import java.sql.SQLException;
import java.util.Date;

import javax.servlet.RequestDispatcher;
import javax.servlet.ServletException;
import javax.servlet.annotation.WebServlet;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;

@WebServlet("/ChooseMember")
public class ChooseMember extends HttpServlet{
	
	protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
		  
		  //エンコーディング方式の指定
		  request.setCharacterEncoding("UTF-8");
		  //オブジェクトの定義
		    MemberDAO mmbDAO=new MemberDAO("localhost","mysql","mysql");
		    ManHoursDAO mmhDAO = new ManHoursDAO("localhost","mysql","mysql");
		    Member mmb = new Member();
		    Manhours mmh = new Manhours();
		    //memberテーブルからIDを取得
		  try {
				//memberDBへの接続
				mmbDAO.connect();
				//memberテーブルへIDをキーに検索し、mmbに格納
				mmb = mmbDAO.getMember("memberId");
				
				if(mmb != null) {
					mmb.print();
				}else {
					System.out.println("該当者が見つかりません。id:" + "memberId");
				}
			}catch(SQLException e) {
				e.printStackTrace();
			}finally {
				try {
					//接続のクローズ
					mmbDAO.close();
				}catch(SQLException e) {
					e.printStackTrace();
				}
			}
		  //mmbをもとにmanhoursテーブルを更新
		  try {
				//DBへの接続
				mmhDAO.connect();
				//MemberテーブルにIDをキーに格納
				if(mmb != null) {
					mmh.setMemrid(mmb.getMemberId());
				}else {
					System.out.println("該当者が見つかりません。id:" + "memberId");
				}
			}catch(SQLException e) {
				e.printStackTrace();
			}finally {
				try {
					//接続のクローズ
					mmhDAO.close();
				}catch(SQLException e) {
					e.printStackTrace();
				}
			}
	  }
}

該当のソースコード2/5（MemberDAO）

package mod4;

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.util.ArrayList;

public class MemberDAO {

	   private Connection con;  // 接続オブジェクト
	   private String server;   // DBサーバ名
	   private String user;     // ユーザー名
	   private String password; // パスワード

	   // コンストラクタ
	   public MemberDAO(String server, String user, String password) {
	       this.server = server;
	       this.user = user;
	       this.password = password;
	   }

	   // 接続
	   public void connect() throws SQLException {
	       String url = "jdbc:mysql://" + server + ":3306/ourDB";
	       con = DriverManager.getConnection(url, user, password);
	   }

	   // 接続のクローズ
	   public void close() throws SQLException {
	       if (con != null) {
	           con.close();
	       }
	   }

	   // 1件検索
	   public Member getMember(String memberId) throws SQLException {
	   	ResultSet res = null;
	       PreparedStatement stmt = null;
	       Member mmb = null;
	       String sql = "select * from member where memberID=?";

	       try {	
	           stmt = con.prepareStatement(sql);
	           stmt.setString(1, memberId);
	           res = stmt.executeQuery();

	           if (res.next()) {
	               mmb = new Member();
	               mmb.setMemberId(res.getString(1));
	               mmb.setName(res.getString(2));
	           }
	       } finally {
	           if (res != null) {
	               res.close();
	           }
	           if (stmt != null) {
	               stmt.close();
	           }
	       }
	       return mmb;
	   }
	}

該当のソースコード3/5（ManHoursDAO）

package mod4;

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.util.ArrayList;

public class ManHoursDAO {

    private Connection con;  // 接続オブジェクト
    private String server;   // DBサーバ名
    private String user;     // ユーザー名
    private String password; // パスワード

    // コンストラクタ
    public ManHoursDAO(String server, String user, String password) {
        this.server = server;
        this.user = user;
        this.password = password;
    }

    // 接続
    public void connect() throws SQLException {
        String url ="jdbc:mysql://" + server + ":3306/ourDB";
        con = DriverManager.getConnection(url, user, password);
    }

    // 接続のクローズ
    public void close() throws SQLException {
        if (con != null) {
            con.close();
        }
    }

    // レコードの更新
    public void updateManhours(Manhours man) throws SQLException {
        PreparedStatement stmt = null;
        String sql = "UPDATE manhours" +
        		"SET (mh01 = ?, mh02 = ?, mh03 = ?, mh04 = ?, mh05 = ?, mh06 = ?, mh07 = ?, mh08 = ?, mh09 = ?, mh10 = ?, mh11 = ?, mh12 = ?, mh13 = ?, mh14 = ?, mh15 = ?" +
        		", mh16 = ?, mh17 = ?, mh18 = ?, mh19 = ?, mh20 = ?, mh21 = ?, mh22 = ?, mh23 = ?, mh24 = ?, mh25 = ?, mh26 = ?, mh27 = ?, mh28 = ?, mh29 = ?, mh30 = ?, mh31 = ? )" +
        		"(WHERE 'MemberID =?', 'TaskID =?')";

        try {
            stmt = con.prepareStatement(sql);
            stmt.setString(1, man.getMemberId());
            stmt.setInt(2, man.getTaskId());
            stmt.setDate(3, man.getYm());
            stmt.setDate(4, man.getSchesta());
            stmt.setDate(5, man.getScheend());
            stmt.setDate(6, man.getSta());
            stmt.setDate(7, man.getEnd());
            stmt.setInt(8, man.getSchemh());
            stmt.setInt(9, man.getMh1());
            stmt.setInt(10, man.getMh2());
            stmt.setInt(11, man.getMh3());
            stmt.setInt(12, man.getMh4());
            stmt.setInt(13, man.getMh5());
            stmt.setInt(14, man.getMh6());
            stmt.setInt(15, man.getMh7());
            stmt.setInt(16, man.getMh8());
            stmt.setInt(17, man.getMh9());
            stmt.setInt(18, man.getMh10());
            stmt.setInt(19, man.getMh11());
            stmt.setInt(20, man.getMh12());
            stmt.setInt(21, man.getMh13());
            stmt.setInt(22, man.getMh14());
            stmt.setInt(23, man.getMh15());
            stmt.setInt(24, man.getMh16());
            stmt.setInt(25, man.getMh17());
            stmt.setInt(26, man.getMh18());
            stmt.setInt(27, man.getMh19());
            stmt.setInt(28, man.getMh20());
            stmt.setInt(29, man.getMh21());
            stmt.setInt(30, man.getMh22());
            stmt.setInt(31, man.getMh23());
            stmt.setInt(32, man.getMh24());
            stmt.setInt(33, man.getMh25());
            stmt.setInt(34, man.getMh26());
            stmt.setInt(35, man.getMh27());
            stmt.setInt(36, man.getMh28());
            stmt.setInt(37, man.getMh29());
            stmt.setInt(38, man.getMh30());
            stmt.setInt(39, man.getMh31());
            stmt.executeUpdate();
        } finally {
            if (stmt != null) {
                stmt.close();
            }
        }
    }

    // レコードの削除
    public void deleteManhours(String memberId) throws SQLException {
        PreparedStatement stmt = null;
        String sql = "DELETE FROM manhours" +
        		"(WHERE 'MemberID =?', 'TaskID =?')";

        try {
            stmt = con.prepareStatement(sql);
            stmt.setString(1, memberId);
            stmt.executeUpdate();
        } finally {
            if (stmt != null) {
                stmt.close();
            }
        }
    }
}

該当のソースコード4/5（Member）

package mod4;

import java.io.Serializable;

public class Member implements Serializable {
    private String memberId;          // ID
    private String name;     // 名前

    public Member() { // コンストラクタ
    }
    public void print() { // データの表示
        System.out.print(memberId + ",");
        System.out.print(name + ",");
    }
    public void setMemberId(String memberId) { // IDを設定
        this.memberId = memberId;
    }
    public void setName(String name) { // 名前を設定
        this.name = name;
    }
    public String getMemberId() { // IDを取得
        return memberId;
    }
    public String getName() { // 名前を取得
        return name;
    }
}

該当のソースコード5/5（MemberIdInputForm）

<%@ page language="java" contentType="text/html; charset=UTF-8"
    pageEncoding="UTF-8"%>
<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>Insert title here</title>
</head>
<body>
<table border="1">
<tr style="background-color:#99cccc;">
<th >担当者ID</th>
<th >担当者</th>
</tr>
<tr>
<td>
<form method="get" action="../jdbc/mod4/ChooseMember.java"> 
<input type="text" name="memberId"><br>
</form>
</td>
</tr>
<tr>
<td>
<input type="submit">
<input type="reset">
</td>
</tr>
</table>
</body>
</html>

memberデータベース

manhoursデータベース

試したこと

ChooseMember.java（業務ロジック）単体で動かしてみたところ、当然idの受け取りをしていないので「該当者が見つかりません」が2回表示されました。
また、フォワード先URLに対し不備がある可能性も探ってみましたが問題ありませんでした。

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

Java SE 11.0.8
Tomcat 9
Java開発環境 Eclipse 4.13

データベース MySQL
データベース開発環境A5:SQL Mk-2

Comments

[shinami]

ChooseMemberクラス(1/5業務ロジック)にmmhDAO.updateManhours()が無いんですが書き忘れですか？
更新できるようなそれらしきソースはそこしか無いのでそれが原因とは思いますがupdateやinsertを実行しないとデータベースへ反映できないかと。

Answer 1

ManHoursDAO.javaのUPDATE文ですが、テーブル名の後ろにスペースが必要なのでは？

String sql = "UPDATE manhours" +
                "SET (mh01 = ?, mh02 = ?, mh03 = ?, mh04 = ?, mh05 = ?, mh06 = ?, mh07 = ?, mh08 = ?, mh09 = ?, mh10 = ?, mh11 = ?, mh12 = ?, mh13 = ?, mh14 = ?, mh15 = ?" +
                ", mh16 = ?, mh17 = ?, mh18 = ?, mh19 = ?, mh20 = ?, mh21 = ?, mh22 = ?, mh23 = ?, mh24 = ?, mh25 = ?, mh26 = ?, mh27 = ?, mh28 = ?, mh29 = ?, mh30 = ?, mh31 = ? )" +
                "(WHERE 'MemberID =?', 'TaskID =?')";

↓

String sql = "UPDATE manhours " +
                "SET (mh01 = ?, mh02 = ?, mh03 = ?, mh04 = ?, mh05 = ?, mh06 = ?, mh07 = ?, mh08 = ?, mh09 = ?, mh10 = ?, mh11 = ?, mh12 = ?, mh13 = ?, mh14 = ?, mh15 = ?" +
                ", mh16 = ?, mh17 = ?, mh18 = ?, mh19 = ?, mh20 = ?, mh21 = ?, mh22 = ?, mh23 = ?, mh24 = ?, mh25 = ?, mh26 = ?, mh27 = ?, mh28 = ?, mh29 = ?, mh30 = ?, mh31 = ? )" +
                "(WHERE 'MemberID =?', 'TaskID =?')";