同じクラスのメソッドの値を入れたい

Question

public static void getConnection(String URL, String USER, String PASS)のURL,USER,PASSの値をpublic void insertDto(Dto dto)のcon = DriverManager.getConnection(URL, USER, PASS);に値を入れたいです。
直接記入やクラス内での記入だとうまくはいくのですが、別メソッドからも呼び出して使えるようにしたいです。

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

public class Dao {
	private static final String SQL = "INSERT INTO product (ID,name,remarks) VALUES (?,?,?);";
	private static final String DRIVE = "com.mysql.cj.jdbc.Driver";

	public static void getConnection(String URL, String USER, String PASS) throws SQLException {
		URL = "jdbc:mysql://localhost:3306/aaa?characterEncoding=UTF-8&serverTimezone=JST";
		USER = "bbb";
		PASS = "ccc";

		try {
			con = DriverManager.getConnection(URL, USER, PASS);
		} catch (SQLException e) {
		}
	}

	void close() throws SQLException {
		con = DriverManager.getConnection(URL);
		con.close();
	}

	static Connection con = null;
	PreparedStatement ps = null;

	public void insertDto(Dto dto) throws SQLException {
		try {
			Class.forName(DRIVE);
			con = DriverManager.getConnection(URL, USER, PASS);
			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 {
			try {
				if (ps != null) {
					ps.close();
				}
			} catch (SQLException e) {
			}
			try {
				if (con != null) {
					con.close();
				}
			} catch (SQLException e) {
			}
		}
	}
}

Comments

[momon-ga]

static Connection con = null;
っていう定義が、すでに不具合を起こす可能性がありますが・・・

getConnectionが呼ばれた時点で、insertDtoのconも生成済みなので、
> con = DriverManager.getConnection(URL, USER, PASS);
自体不要ですよ。

[dodox86]

現状のコードですと
public static void getConnection(String URL, String USER, String PASS)
の引数URL, URSER, PASSにメソッド内部で上書きしていますが、これは。。。？

> void close() throws SQLException {
> con = DriverManager.getConnection(URL);
> con.close();
> }

も、(JDBCの実装にもよるでしょうが）新しいコネクションを取得して、すぐ閉じているような。
全般的に、まだ試行錯誤中のコードでしょうか。

質問自体の解決策としては、URL, USER, PASSをDaoクラスのインスタンスフィールドにすればよいのではないかと思うのですが、そういう話ではないのですか？

Answer 1

ありがとうございました。
public static void getConnection(String URL, String USER, String PASS)で上書きしていたり、他にもミスがあったので修正して値を出力することができました。