４段階目終了

src/com/rhizome/example/dao/EmployeeDAO.java

@@ -60,15 +60,14 @@ public class EmployeeDAO {
 
 			this.stmt = con.prepareStatement(sql.toString());
 			rs = stmt.executeQuery();
-			//DBのカラムをリストに追加
 			while (rs.next()) {
 				EmployeeDTO dto = new EmployeeDTO();
-				dto.setEmpid(rs.getInt(ID_EMPLOYEE));
+				dto.setID_EMPLOYEE(rs.getInt("ID_EMPLOYEE"));
-				dto.setName(rs.getString(NM_EMPLOYEE));
+				dto.setNM_EMPLOYEE(rs.getString("NM_EMPLOYEE"));
-				dto.setKana(rs.getString(KN_EMPLOYEE));
+				dto.setKN_EMPLOYEE(rs.getString("KN_EMPLOYEE"));
-				dto.setMail(rs.getString(MAIL_ADDRESS));
+				dto.setMAIL_ADDRESS(rs.getString("MAIL_ADDRESS"));
-				dto.setPass(rs.getString(PASSWORD));
+				dto.setPASSWORD(rs.getString("PASSWORD"));
-				dto.setDepid(rs.getInt(ID_DEPARTMENT));
+				dto.setID_DEPARTMENT(rs.getInt("ID_DEPARTMENT"));
 				rtnList.add(dto);
 			}
 		} finally {
@@ -77,65 +76,42 @@ public class EmployeeDAO {
 		return rtnList;
 	}
 
-	public List<EmployeeDTO> selectByld(int id) throws SQLException {
+	//社員テーブルから指定された社員IDに一致するデータを取得し、DTOに設定の上、リスト形式で戻
-		List<EmployeeDTO> rtnList = new ArrayList<>();
+	public List<EmployeeDTO> selectById(int id_employee) throws SQLException {
-		StringBuilder sql = new StringBuilder();
+		List<EmployeeDTO> list = new ArrayList<>();
-		//社員テーブルから指定された社員IDに一致するデータを取得
+		String sql = "SELECT * FROM employee WHERE ID_EMPLOYEE = ?";
-		sql.append(" SELECT ");
-		sql.append("    " + "*");
-		sql.append(" FROM ");
-		sql.append("    " + TBL_NAME);
-		sql.append(" WHERE ");
-		sql.append("    " + ID_EMPLOYEE);
-		sql.append("     =     ");
-		sql.append("    " + id);
-
 		try {
+
 			this.stmt = con.prepareStatement(sql.toString());
-			rs = stmt.executeQuery();
+			this.stmt.setInt(1, id_employee);
-			//DBのカラムをリストに追加
+			this.rs = this.stmt.executeQuery();
 			while (rs.next()) {
 				EmployeeDTO dto = new EmployeeDTO();
-				dto.setEmpid(rs.getInt(ID_EMPLOYEE));
+				dto.setID_EMPLOYEE(rs.getInt("ID_EMPLOYEE"));
-				dto.setName(rs.getString(NM_EMPLOYEE));
+				dto.setNM_EMPLOYEE(rs.getString("NM_EMPLOYEE"));
-				dto.setKana(rs.getString(KN_EMPLOYEE));
+				dto.setKN_EMPLOYEE(rs.getString("KN_EMPLOYEE"));
-				dto.setMail(rs.getString(MAIL_ADDRESS));
+				dto.setMAIL_ADDRESS(rs.getString("MAIL_ADDRESS"));
-				dto.setPass(rs.getString(PASSWORD));
+				dto.setPASSWORD(rs.getString("PASSWORD"));
-				dto.setDepid(rs.getInt(ID_DEPARTMENT));
+				dto.setID_DEPARTMENT(rs.getInt("ID_DEPARTMENT"));
-				rtnList.add(dto);
+				list.add(dto);
 			}
 		} finally {
 			DbUtil.closeStatement(this.stmt);
-		}
-		if (rtnList.size() == 0) {
-			System.out.println("入力された値で情報が見つかりませんでした");
-		}
-		return rtnList;
 
 		}
+		return list;
+	}
 
-	public void empDelete(int id) throws SQLException {
+	// 社員テーブルから指定された社員IDに一致するデータを削除する
-		StringBuilder sql = new StringBuilder();
+	public void empDelete(int id_employee) throws SQLException {
-		//社員テーブルから指定された社員IDに一致するデータを取得
+		String sql = "DELETE FROM employee WHERE ID_EMPLOYEE = ?";
-		sql.append(" DELETE ");
-		sql.append("    " + "*");
-		sql.append(" FROM ");
-		sql.append("    " + TBL_NAME);
-		sql.append(" WHERE ");
-		sql.append("    " + ID_EMPLOYEE);
-		sql.append("     =     ");
-		sql.append("    " + id);
 		try {
-			this.stmt = con.prepareStatement(sql.toString());
+			this.stmt = con.prepareStatement(sql);
+			this.stmt.setInt(1, id_employee);
 			this.stmt.executeUpdate();
-			System.out.println("削除が完了しました");
+
 		} finally {
 			DbUtil.closeStatement(this.stmt);
-
 		}
 	}
-	public void emplnsert (String name, String kana, String mail, String pass, int id) {
-		StringBuilder sql = new StringBuilder();
-		
-	}
 }

src/com/rhizome/example/main/Main.java

@@ -45,8 +45,6 @@ public class Main {
 	 * @throws SQLException
 	 */
 	public void start() throws ClassNotFoundException, SQLException {
-		Connection con = DbUtil.getConnection();
-		EmployeeDAO empDao = new EmployeeDAO(con);
 		System.out.println();
 		System.out.println("現在のメンテナンス対象テーブル：" + targetTableName(tblFlg));
 		System.out.println("1:検索 2:登録 3:更新 4:削除 9:テーブル切替 0:終了");
@@ -73,30 +71,6 @@ public class Main {
 		}
 	}
 
-	private void empDelete() throws SQLException {
-		Connection con = DbUtil.getConnection();
-		EmployeeDAO empDao = new EmployeeDAO(con);
-		try {
-			String strId = inputCheck("ID", true);
-			int id = Integer.parseInt(strId);
-			List<EmployeeDTO> empList = empDao.selectByld(id);
-			if (empList.isEmpty()) {
-			} else {
-				System.out.println("指定されたIDの情報を削除します？　Y/y");
-				System.out.println("Y/y以外の入力であればキャンセル");
-				Scanner sc = new Scanner(System.in);
-				String str = sc.nextLine();
-				if (str.equals("Y") || str.equals("y")) {
-					empDao.empDelete(id);
-				} else {
-					System.out.println("キャンセルしました");
-				}
-			}
-		} finally {
-			DbUtil.closeConnection(empDao.con);
-		}
-	}
-
 	/**
 	 * 社員テーブルから情報を検索する
 	 *
@@ -115,22 +89,58 @@ public class Main {
 				empList = empDao.selectAll();
 				break;
 			case "2":
-				System.out.println("IDを入力してください");
+				String inputId = inputCheck("検索する社員ID", true);
-				int id = sc.nextInt();
+				int input = Integer.parseInt(inputId);
-				empList = empDao.selectByld(id);
+				empList = empDao.selectById(input);
 				break;
 			default:
 				return;
 			}
 			for (EmployeeDTO emp : empList) {
-				System.out.println(emp.getEmpid() + "\t" + emp.getName() + "\t" + emp.getKana() + "\t"
+				System.out.println(emp);
-						+ emp.getMail() + "\t" + emp.getPass() + "\t" + emp.getDepid());
 			}
 		} finally {
 			DbUtil.closeConnection(empDao.con);
 		}
 	}
 
+	public static void empDelete() throws ClassNotFoundException, SQLException {
+
+		Connection con = DbUtil.getConnection();
+		EmployeeDAO empDao = new EmployeeDAO(con);
+		try {
+			System.out.println("IDを入力してください");
+			String input = sc.nextLine();
+			// 空文字が入力された場合
+			if (input.strip().isEmpty()) {
+				System.out.println("入力された値で情報がみつかりませんでした");
+				return;
+			}
+			int inputId = Integer.parseInt(input.strip());
+			List<EmployeeDTO> empList = empDao.selectById(inputId);
+			// 検索結果が存在しないIDの場合
+			if (empList.isEmpty()) {
+				System.out.println("入力された値で情報がみつかりませんでした");
+				return;
+			}
+
+			System.out.println("指定されたIDの情報を削除します？ Y/y");
+			System.out.println("(Y/y以外の入力であればキャンセル)");
+			String confilm = sc.nextLine();
+			// 検索結果が存在した場合
+			if ("Y".equalsIgnoreCase(confilm)) {
+				empDao.empDelete(inputId);
+				System.out.println("社員ID" + input + "のデータを削除しました");
+			} else {
+				System.out.println("削除をキャンセルしました");
+			}
+		} catch (NumberFormatException e) {
+			System.out.println("入力された値で情報がみつかりませんでした");
+		} finally {
+			DbUtil.closeConnection(empDao.con);
+		}
+	}
+
 	/**
 	 * フラグを判定し、テーブル名を取得する
 	 *
@@ -158,7 +168,7 @@ public class Main {
 	 * @return
 	 */
 	public static String inputCheck(String titleName, boolean flg) {
-		String str;
+		String str = null;
 		while (true) {
 			System.out.println(titleName + "を入力してください");
 			str = sc.nextLine();

src/com/rhizome/example/util/DbUtil.java

@@ -44,10 +44,10 @@ public class DbUtil {
 	 * 				  生成されたConnectionを返却する
 	 * 				  ※Connection生成には上記の定数を活用してください
 	 */
-	public static Connection getConnection() throws SQLException {
+	public static Connection getConnection() throws ClassNotFoundException, SQLException {
-			//データベースへの接続
+		Class.forName(DB_DRIVER);
-			con = DriverManager.getConnection(DB_URL, DB_USER ,DB_PASS);
+
-		return con;
+		return con = DriverManager.getConnection(DB_URL, DB_USER, DB_PASS);
 	}
 
 	/*
@@ -59,12 +59,13 @@ public class DbUtil {
 	 * 例外			：SQLException
 	 * 処理			：引数のDBステートメントがnullでない場合、そのDBステートメントをクローズする
 	 */
-	public static void closeStatement(PreparedStatement ps) throws SQLException {
+	public static void closeStatement(PreparedStatement stmt) throws SQLException {
-		//DBステートメントのクローズ
+		if (stmt != null) {
-		if (ps != null) {
+			stmt.close();
-				ps.close(); //DBステートメントをクローズ
 		}
+
 	}
+
 	/*
 	 * TODO 以下の仕様でメソッドを作成してください。
 	 *
@@ -75,10 +76,9 @@ public class DbUtil {
 	 * 処理			：引数のコネクションがnullでない場合、そのコネクションを切断する
 	 */
 	public static void closeConnection(Connection con) throws SQLException {
-		//コネクションの切断
 		if (con != null) {
-				con.close(); //コネクション切断
+			con.close();
-			}
 		}
 	}
 
+}