型の不一致Stringからintへの変換

Question

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

データベースから削除しようとすると、

型の不一致Stringからintには変換できません

と出てくる。
削除したいidの数値は正しく受け渡されていることが、確認できている。

該当のソースコード

public class politicians {

	private String idpoliticians;
	private String name;
	
	
	// DB関連の初期設定
		private Connection conn = null;
		private PreparedStatement pstmt = null;
		private DataSource ds = null;

		// コンストラクタ
		public politicians(HttpServletRequest request) {

			setIdpoliticians(request.getParameter("idpoliticians"));
			setName(request.getParameter("name"));
			
			
		}

		
		
		// データベースへのアクション
		private void doDataBase(String sql) throws Exception {
			
			// コンテキストを取得
			InitialContext ic = new InitialContext();
			// ルックアップしてデータソースを取得
			ds = (DataSource) ic.lookup("java:comp/env/jdbc/politicians");
			conn = ds.getConnection();
			
			// sql文を表示
			System.out.println(sql);
			pstmt = conn.prepareStatement(sql);
			// sql文実行
			pstmt.execute();

			// 使用したオブジェクトを終了させる
			pstmt.close();
			conn.close();
		}
	
	public String getIdpoliticians() {
		return idpoliticians;
	}
	public void setIdpoliticians(String idpoliticians) {
	this.idpoliticians = idpoliticians;
	}
	//autoincrementだからセットすることはない！？
	public String getName() {
		return name;
	}
	public void setName(String name) {
		this.name = name;
	}
	
		public Boolean addData() {
			try {

				// sql文 の作成
				String sql = "insert into politicians( name) values ('" + name + "')";

				// データベース接続＆ｓｑｌの実行
				doDataBase(sql);

				return true;

			} catch (Exception e) {
				e.printStackTrace();
				return false;
			}
		}

		//削除
		public Boolean deleteData() {
			try {

				// sql文 の作成
				String sql = "delete from politicians where idpoliticians=" + idpoliticians;

				// データベース接続＆ｓｑｌの実行
				doDataBase(sql);

				return true;
				
			} catch (Exception e) {
				e.printStackTrace();
				return false;
			}
		}	
	
	
}

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

ここにより詳細な情報を記載してください。

Comments

[firegrape]

where句のコードがないように思いますが。

[panp]

String sql = "delete from politicians where idpoliticians=" + idpoliticians;
ここの部分がwhere句に該当していることになってると認識していたのですがどうでしょうか？

[firegrape]

なるほどです。
getIdpoliticiansはどこで呼ばれていますか？

[m.ts10806]

firegrapeさんのヒアリングも回答も一通り無視してもらってよいと思います。

下記、追記願います。
・DBの種類、バージョン（質問タグにも追加願います）
・politicians テーブルのテーブル定義（CREATE TABLE文）

[Zuishin]

ただのゲッターなので、どこで呼ばれてるとか関係ないんじゃないですか。

Answer 1

確実なのは「削除したいidの数値」と言いながら受けと取っている変数の型が文字列であるということ。

private String idpoliticians;
String sql = "delete from politicians where idpoliticians=" + idpoliticians;

DBによっては暗黙の型変換がないものもありますから、渡すときに数値であれば数値と宣言するのが問題も起きにくいです。

prepareStatement使っているならきちんとパラメータとして値をバインドされては如何でしょうか。
当該deleteに限らずSQLインジェクションも可能な作りになっているように見受けられます。

サーブレット/JSP入門 › データベース接続 / prepareStatementの使用

doDataBase()メソッドはそれなりに改修必要かと思います。
最低限、SQLにセットしたパラメータとその型が引数として必要です。