ps.executeUpdate();を利用して会員削除

Question

ps.executeUpdate();を利用してDBを削除

javaで会員サイトの削除機能を作ろうとしています。
DBに接続して会員idから探して削除したいです。
検索してもps.executeUpdate();の使い方などわからなかったので書き込みました。

DBに接続して会員idから探すクラス

package service;

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

import data.Customer;

public class DeleteService {
	public List<Customer> deleteCustomer (String id) {
		String dbName = "system_user";
		String dbPropaties = "?cvharacterEncoding=UTF-8&serverTimezone=JST";
		String dbUrl ="jdbc:mySQL://localhost:3306/" + dbName +dbPropaties;
		String dbUser =  "root";
		String dbPassword = "admin";
		String sql = "delete from customer where id=?;";
		List<Customer>customerList = new ArrayList<Customer>();
		try {
			Class.forName("com.mysql.jdbc.Driver");
			Connection coon = DriverManager.getConnection(dbUrl, dbUser, dbPassword);
			PreparedStatement ps = coon.prepareStatement(sql);
			ps.setString(1, id);
			ResultSet rs = ps.executeQuery();
			ps.executeUpdate();
			while (rs.next()) {
				Customer customer = new Customer();
				customer.setEmail(rs.getString("email"));
				customer.setPassword(rs.getString("password"));
				customer.setName(rs.getString("name"));
				customer.setGender(rs.getString("gender"));
				customer.setBlood(rs.getString("blood"));
				customer.setOld(rs.getInt("old"));
				customerList.add(customer);
			}
		}catch (SQLException | ClassNotFoundException e) {
			e.printStackTrace();
		}
		return customerList ;

	}
}

上記を呼び出すクラス

package controller;

import java.io.IOException;
import java.util.List;

import javax.servlet.ServletContext;
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 data.Customer;
import service.UpdateService;

@WebServlet("/delete")

public class DeleteController  extends HttpServlet {
	protected void doPost (HttpServletRequest request, HttpServletResponse response)
			throws ServletException, IOException{
		
	}
}

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

Eclipse
tomcat

Answer 1

削除機能、であれば、以下で十分でしょう。
まず、削除機能が何かのリストを返そうとすること自体がまちがいなのではないでしょうか。
また、Tomcat云々以前の問題なので、mainメソッドを定義するなり、JUnitでテストを書くなりして、deleteCustmerメソッドを呼び出すテストをしてみてはいかがでしょうか。

    public void deleteCustomer (String id) {
        String dbName = "system_user";
        String dbPropaties = "?cvharacterEncoding=UTF-8&serverTimezone=JST";
        String dbUrl ="jdbc:mySQL://localhost:3306/" + dbName +dbPropaties;
        String dbUser =  "root";
        String dbPassword = "admin";
        try {
            Class.forName("com.mysql.jdbc.Driver");
            Connection coon = DriverManager.getConnection(dbUrl, dbUser, dbPassword);
            String sql = "delete from customer where id=?;";
            PreparedStatement ps = coon.prepareStatement(sql);
            ps.setString(1, id);
            ps.executeUpdate();
            ps.close();
            coon.close();
        } catch (SQLException | ClassNotFoundException e) {
            e.printStackTrace();
        }
    }

Comments

[arin]

回答ありがとうございます。
JUnit初めて聞きましたが調べて取り組んでみます！
void型にしたら呼び出して処理できなくないですか？

package controller;

import java.io.IOException;

import javax.servlet.ServletContext;
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 service.DeleteService;

@WebServlet("/delete")

public class DeleteController extends HttpServlet {
protected void doPost (HttpServletRequest request, HttpServletResponse response)
throws ServletException, IOException{
String id = request.getParameter("id");
DeleteService deleteservice = new DeleteService();
deleteservice.deleteCustomer(id);
ServletContext sc = getServletContext();
sc.getRequestDispatcher("/" + "delete.jsp").forward(request, response);
}
}

[shiketa]

> void型にしたら呼び出して処理できなくないですか？

これは「戻り地がvoidのメソッドは呼び出すことができないのではないか」という意味でしょうか？
https://docs.oracle.com/javase/jp/11/docs/api/index.html は参照したことありますよね。戻り値がvoidのメソッドなんて山ほどあります。メソッドなのでもちろん呼び出して処理できます。そしてそれらのメソッドは何も返さない。それだけです。わたしは、削除処理に戻り値は不要とかんがえましたので回答「案」ではvoidとしました。案です。
たとえば、ps.executeUpdate()メソッドは、処理した件数を返してくれますので、deleteCustomer()メソッドの戻り値をintにして、削除した件数を返すようにしてもいいかもしれません。必要に応じて、お好きにどうぞ。

[arin]

なるほど、ありがとうございます。