try-with-resources文のprepareStatementについて

Question

前提・実現したいこと

フォームから送信されたIDを基に、DBから商品情報を取り出すメソッドを作っています。
現在try文で二重で囲んでいますが、１つだけにしたいです。
この場合、「psmt.setInt(1, id)」はどこにいれるべきなんでしょうか。

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

        try (Connection con = getConnection();
                PreparedStatement psmt = con.prepareStatement(sql.toString());
            ResultSet rs = psmt.executeQuery()) {
              psmt.setInt(1, id);

上記の形でsetIntをtry文の中に入れてしまうと、
java.sql.SQLException: No value specified for parameter 1
というエラーが発生してしまいます。
デバッグではフォームからIDはちゃんと送信されていますので、文法が間違っているのかなと思っています・・・

該当のソースコード

    public ProductBean findOne(int id) throws ClassNotFoundException, SQLException {
        ProductBean productBean = new ProductBean();

        StringBuilder sql = new StringBuilder();
        sql.append("select product_ID, ");
　　　　　・・・省略・・・
 　　　　sql.append("from t_product ");
        sql.append("where product_ID=?");

        try (Connection con = getConnection();
                PreparedStatement psmt = con.prepareStatement(sql.toString())) {
            psmt.setInt(1, id);
            try (ResultSet rs = psmt.executeQuery()) {
                while (rs.next()) {
                    productBean.setId(rs.getInt("product_ID"));
　　　　　　　　　　　・・・省略
                }
            }
        }
        return productBean;
    }

試したこと

    try (Connection con = getConnection();
            PreparedStatement psmt = con.prepareStatement(sql.toString());
        ResultSet rs = psmt.executeQuery();
    	psmt.setInt(1, id)){

トークンに構文エラーがあります。構成の位置が間違っています。

上記のエラーが出てしまいます。

Answer 1

1. 手順エラー

まず最初のエラーは実行時エラーです。javahackさんの解説のとおり、psmt.executeQuery()を実行する前に?変数は全てバインドされていなければならない。

2. try-wth-resourcesの構文エラー

２番目のエラーは、shiketaさんが指摘されていますが、構文エラーです。try () {}の()の中に何が書けるか。リソースの『宣言文』を複数書ける。リソースはjava.lang.AutoCloseableを実装していなければなrない。

The try-with-resources Statement

psmt.setInt(1, id);はリソースの宣言文ですか。いいえ違います。だからコンパイルエラーになります。

結論

あなたの望むようにtry()一つのなかに、宣言文と宣言文以外の文を一緒に書くことはできません。

勧められない回避策

こっそり教えますが、宣言文は書けます。良い子は真似してはいけません。

try (Connection con = getConnection(); 
    PreparedStatement psmt = con.prepareStatement(sql.toString());
    AutoCloseable a = new AutoCloseable() {
      { psmt.setInt(1, id); }
      @Override public void close() throws Exception {}
    };      
    ResultSet rs = psmt.executeQuery()) {
    while (rs.next()) {
      productBean.setId(rs.getInt("product_ID"));
      // ・・・省略
    }
} catch (ClassNotFoundException | SQLException e) {
  throw e;
} catch (Exception e) {
  throw new SQLException(e);
}

推奨しない理由

回避コードpsmt.setInt(1, id);から例外が出ると困る。何のためにtryを使うのか考えてください。
同様にリソース宣言の初期化式からも例外が出ることを考えてください。

Answer 2

try (Connection con = getConnection();
PreparedStatement psmt = con.prepareStatement(sql.toString());
psmt.setInt(1, id);
ResultSet rs = psmt.executeQuery()){

この形にしても下記のエラーが出てしまいます。。
トークンに構文エラーがあります。構成の位置が間違っています

ドキュメントには、こうある。 https://docs.oracle.com/javase/jp/7/technotes/guides/language/try-with-resources.html

try-with-resources 文は、1 つ以上のリソースを宣言する try 文です。リソースは、プログラムでの使用が終わったら閉じられなければいけないオブジェクトです。

PreparedStatement psmt = con.prepareStatement(sql.toString());とResultSet rs = psmt.executeQuery()は、「リソースを宣言する」文だが、psmt.setInt(1, id);は異なる。だから構成の位置が間違っていますとなる。

Comments

[tyai]

なるほど、では発生している問題・エラーメッセージで書いた文はあっている気がするんですが、エラーが出てしまうのは何故でしょうか、、

[shiketa]

> 発生している問題・エラーメッセージで書いた文はあっている気がするんですが...

気のせいです。javahackさんが丁寧に書いてくれているじゃないですか。
年賀状の季節なので、こういう例えだとわかりますかね。ハガキをポストに入れてから宛名を書きますか？おそらく普通のヒトは、ハガキに宛名を書いて(`psmt.setInt(1, id)`)からポストに投函(`psmt.executeQuery()`)します。

Answer 3

2重の時と1つの時で文の順番が違っています。

//2重の時
    psmt.setInt(1, id);
    ResultSet rs = psmt.executeQuery();
//1つの時
    ResultSet rs = psmt.executeQuery();
    psmt.setInt(1, id);

Comments

[tyai]

try (Connection con = getConnection();
PreparedStatement psmt = con.prepareStatement(sql.toString());
psmt.setInt(1, id);
ResultSet rs = psmt.executeQuery()){

この形にしても下記のエラーが出てしまいます。。
トークンに構文エラーがあります。構成の位置が間違っています