質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.79%

finallyの処理する中身をメソッドでまとめて出力したい

解決済

回答 2

投稿

  • 評価
  • クリップ 1
  • VIEW 289

score 2

登録以外の処理をする際に利用できるようにcloseメソッドでまとめようと考えています。
closeメソッドのConnectionとPreparedStatementをnullではなく実際に利用しているinsertメソッドからConnectionとPreparedStatementを持ってきてクローズしたいのですがどのように持ってくるのかがわかりません。
ご教授頂ければ幸いです。

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.SQLException;

public class Dao {
    private static final String URL = "jdbc:mysql://localhost:3306/aaa?characterEncoding=UTF-8&serverTimezone=JST";
    private static final String USER = "bbb";
    private static final String PASS = "ccc";
    private static final String DRIVE = "com.mysql.cj.jdbc.Driver";

    private Connection getConnection() throws SQLException, ClassNotFoundException {
        Connection con = null;
        Class.forName(DRIVE);
        con = DriverManager.getConnection(URL, USER, PASS);
        con.setAutoCommit(false);
        return con;
    }

    void close() throws SQLException, ClassNotFoundException {
//ここのConnectionとPreparedStatementの部分です。
        Connection con = null;
        PreparedStatement ps = null;
            try {
                if (ps != null) {
                    ps.close();
                }
            } catch (SQLException e) {
         e.printStackTrace();
            }
            try {
                if (con != null) {
                    con.close();
                }
            } catch (SQLException e) {
         e.printStackTrace();
            }
        }
    public void insert(Dto dto) throws SQLException {
             PreparedStatement ps = null;
        Connection con = null;
        try {
          con = getConnection();

StringBuilder INSERT_SQL = new StringBuilder();
            INSERT_SQL.append("INSERT INTO aaa (");
            INSERT_SQL.append("ID,");
            INSERT_SQL.append("name,");
            INSERT_SQL.append("remarks)");
            INSERT_SQL.append("VALUES (?,?,?)");
            String sql = INSERT_SQL.toString();
            ps = con.prepareStatement(sql);
            ps.setInt(1, dto.getID());
            ps.setString(2, dto.getName());
            ps.setString(3, dto.getRemarks());
            ps.executeUpdate();
            con.commit();
        } catch (ClassNotFoundException | SQLException e) {
            con.rollback();
        } finally {
           close();
        }
    }
}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • dodox86

    2021/01/28 17:57

    > finallyの処理する中身をメソッドでまとめて出力したい

    言いまわしが少し謎(?)なので確認です。close()メソッドにConnectionとPreparedStatementの引数を増やせばよいと思うのですが、そういう話ではなくて? でしょうか。

    キャンセル

  • ai_atoz

    2021/01/28 22:35

    説明がわかりにくく申し訳ありません。
    引数を増やすというのはclose()メソッドにどのような処理を記述すればよろしいのでしょうか?

    キャンセル

回答 2

checkベストアンサー

0

closeメソッドのConnectionとPreparedStatementをnullではなく実際に利用しているinsertメソッドからConnectionとPreparedStatementを持ってきてクローズしたいのですがどのように持ってくるのかがわかりません。

最低限、質問者さんの現状のコードとやりたいことに沿うかたちにするならば、insert()メソッド内で生成したConnectionPreparedStatementclose()メソッドの引数にすることが考えられます。以下はそのようにしたコード例です。closeAnyway()メソッドをご覧ください。

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.SQLException;

public class Dao {
    private static final String URL = "jdbc:mysql://localhost:3306/aaa?characterEncoding=UTF-8&serverTimezone=JST";
    private static final String USER = "bbb";
    private static final String PASS = "ccc";
    private static final String DRIVE = "com.mysql.cj.jdbc.Driver";

    private Connection getConnection() throws SQLException, ClassNotFoundException {
        Connection con = null;
        Class.forName(DRIVE);
        con = DriverManager.getConnection(URL, USER, PASS);
        con.setAutoCommit(false);
        return con;
    }

    void close() throws SQLException, ClassNotFoundException {
//ここのConnectionとPreparedStatementの部分です。
        Connection con = null;
        PreparedStatement ps = null;
            try {
                if (ps != null) {
                    ps.close();
                }
            } catch (SQLException e) {
                e.printStackTrace();
            }
            try {
                if (con != null) {
                    con.close();
                }
            } catch (SQLException e) {
                e.printStackTrace();
            }
        }

    // SQLExceptionもClassNotFoundExceptionもthrowしないので、throws...は不要。
    // インスタンスメソッドである必要もないので、staticなクラスメソッドで十分。
    private static void closeAnyway(Connection con, PreparedStatement ps) {
        try {
            if (ps != null) {
                ps.close();
            }
        } catch (SQLException e) {
           e.printStackTrace();
        }

        try {
            if (con != null) {
                con.close();
            }
        } catch (SQLException e) {
            e.printStackTrace();
        }
    }

    public void insert(Dto dto) throws SQLException {
        PreparedStatement ps = null;
        Connection con = null;
        try {
            con = getConnection();

            StringBuilder INSERT_SQL = new StringBuilder();
            INSERT_SQL.append("INSERT INTO aaa (");
            INSERT_SQL.append("ID,");
            INSERT_SQL.append("name,");
            INSERT_SQL.append("remarks)");
            INSERT_SQL.append("VALUES (?,?,?)");
            String sql = INSERT_SQL.toString();
            ps = con.prepareStatement(sql);
            ps.setInt(1, dto.getID());
            ps.setString(2, dto.getName());
            ps.setString(3, dto.getRemarks());
            ps.executeUpdate();
            con.commit();
        } catch (ClassNotFoundException | SQLException e) {
            if (con != null) {
                con.rollback();
            }
        } finally {
            //close();
            // とにかく可能な限りcloseする、と言う意味を込めて:
            closeAnyway(con, ps);
        }
    }
}

SQL実行中の例外(Exception)関連のハンドリングは意外と難しく、様々なやり方が提案されています。上記例はtrycatchif文だらけで仕方が無いとは言え、コードとして見た目が不格好ですよね。上記の例にこだわらず、他の良質なサンプルコードなどを見、理解した上で必要に応じて改善していくのも良いことだと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2021/01/29 19:27

    質問にお答えくださりありがとうございました。
    コードを参考にさせていただきました。
    理解できていないところも多々ありましたので学習しながら理解を深めていきたいと思います。

    キャンセル

0

try {
con = getConnection();

これは入れ替えておかないと、catchのときに、ヌルポになりますよ

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2021/01/28 22:38

    そうなのですね。
    ご指摘ありがとうございます。

    キャンセル

  • 2021/01/28 22:40

    getConnection() の実行で例外でたら、ってことですね

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.79%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る