posterSQLをJAVAで使用しINSERT文を入れる場合についての質問です。

Question

前提・実現したいこと

JDBCプログラミングにおいて、INSERT文を用いてデータベースのテーブルに、指定したデータを入れたいのですが、エラーが発生してしまう。エラー文を見てもわからなかったので、ネットで調べても出てこなかったので、こちらで質問させていただきます。

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

org.postgresql.util.PSQLException: PreparedStatement でクエリ文字列を取るクエリメソッドは使えません。
	at org.postgresql.jdbc.PgPreparedStatement.executeUpdate(PgPreparedStatement.java:115)
	at model.dao.UserDAO.accountMake(UserDAO.java:64)
	at servlet.CreateAccount.doPost(CreateAccount.java:43)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:660)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:741)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:231)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:166)
	at org.apache.tomcat.websocket.server.WsFilter.doFilter(WsFilter.java:53)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:193)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:166)
	at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:199)
	at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:96)
	at org.apache.catalina.authenticator.AuthenticatorBase.invoke(AuthenticatorBase.java:491)
	at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:139)
	at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:92)
	at org.apache.catalina.valves.AbstractAccessLogValve.invoke(AbstractAccessLogValve.java:668)
	at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:87)
	at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:343)
	at org.apache.coyote.http11.Http11Processor.service(Http11Processor.java:408)
	at org.apache.coyote.AbstractProcessorLight.process(AbstractProcessorLight.java:66)
	at org.apache.coyote.AbstractProtocol$ConnectionHandler.process(AbstractProtocol.java:764)
	at org.apache.tomcat.util.net.NioEndpoint$SocketProcessor.doRun(NioEndpoint.java:1388)
	at org.apache.tomcat.util.net.SocketProcessorBase.run(SocketProcessorBase.java:49)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1135)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
	at org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:61)
	at java.base/java.lang.Thread.run(Thread.java:844)

該当のソースコード

package servlet;

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

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;

import model.dao.UserDAO;

@WebServlet("/CreateAccount")

public class CreateAccount extends HttpServlet {
	private static final long serialVersionUID = 1L;


    public CreateAccount() {
        super();

    }


	protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {




	}
	protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {


		//リクエストパラメータの取得
		request.setCharacterEncoding("UTF-8");
		String id = request.getParameter("id");
		String password = request.getParameter("password");
		String name = request.getParameter("name");
UserDAO dao=new UserDAO();
try {
dao.accountMake(id, password, name);
boolean isLogin=dao.loginCheck(id, password);
if(isLogin) {


		//アカウント登録成功画面をフォワード
		RequestDispatcher dispatcher = request.getRequestDispatcher("/WEB-INF/jsp/CreateAccountResult.jsp");
		dispatcher.forward(request,response);

		}else {
		//アカウント登録失敗画面をフォワードする
		RequestDispatcher dispatcher = request.getRequestDispatcher("/WEB-INF/jsp/CreateAccountFailure.jsp");
		dispatcher.forward(request, response);
		}

	}catch(SQLException e) {
		e.printStackTrace();
	}catch(ClassNotFoundException e) {
		e.printStackTrace();
	}

	}
	}

package model.dao;

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

import model.UserBean;

public class UserDAO {

	private UserBean user=new UserBean();
	//テーブル内の情報をリストに取得されるメソッド
	public List<UserBean> userList() throws SQLException,ClassNotFoundException{
		try(Connection con=ConnectionManager.getConnection()){
			List<UserBean> userList=new ArrayList<>();
			String sql="SELECT * FROM accounttable";
			PreparedStatement pStmt=con.prepareStatement(sql);
			List<UserBean> userlist=new ArrayList<>();
			ResultSet re=pStmt.executeQuery();
			while(re.next()) {
				String id=re.getString("ID");
				String pass=re.getString("PASSWORD");
				String name=re.getString("NAME");
				int lavel=re.getInt("LEVEL");
				UserBean user=new UserBean(id,pass,name,lavel);
				userList.add(user);
				}
			return userList;
		}
	}


		//ログイン処理メソッド
		public boolean loginCheck(String id,String pass) throws SQLException,ClassNotFoundException{
			try (Connection con=ConnectionManager.getConnection()){
				List<UserBean> list=new ArrayList<>();
				list=userList();

				for(UserBean user1:list) {
					if(user1.getId().equals(id)&&user1.getPassword().equals(pass)) {
						this.user=user1;
						return true;

					}


				}


			}
			return false;
			}
public void accountMake(String id,String password,String name) throws SQLException,ClassNotFoundException {
	try(Connection con=ConnectionManager.getConnection()){
		con.setAutoCommit(false);
		String sql="INSERT INTO accounttable(id,password,name) VALUES(?,?,?)";
		PreparedStatement pStmt=con.prepareStatement(sql);
		pStmt.setString(1, id);
		pStmt.setString(2, password);
		pStmt.setString(3, name);
		int result =pStmt.executeUpdate(sql);
		  System.out.println(result + "行が追加されました。");
		con.commit();
		
	}

}
public UserBean sendUser() {
	return this.user;
}
}

以上がソースコードです。

試したこと

特には行っておらず、ネットで調べるのみです。
調べたことでしたら、PreparedStatement でクエリ文字列を取るクエリメソッドは使えません。って単語を調べたぐらいです。特に自分が求めていたデータは得られなかったので、見たページはないです。

Comments

[m.ts10806]

「試したこと」は「調べた上で試したこと」と言う風にもなると思います。
調べたことがあるのでしたらキーワードや参考にした記事など、例示してください。

Answer 1

PreparedStatement　ではsql文字列を引数とするようなexecuteUpdateは呼び出せなかったと思います
（おそらくPreparedStatementの親インタフェースのStatementではメソッド定義されているので書けてはしまうのですが）

PreparedStatement pStmt=con.prepareStatement(sql);

ですでにsqlは取り込んでいるのであとは引数無しでexecuteUpdateを呼べば動くと思うのですが試していただけますでしょうか

Comments

[kenken2525]

ありがとうございます。
いわれたとおりに打ち直したら、動きました。
誠にありがとうございました。