1

Здравствуйте, помогите пожалуйста вызвать метод закрытия, я понимаю что возможно это просто и для человека с опытом не составит никакого труда, необходимо в main после вызова метода updateImages закрыть соединение, для этого есть метод close(), так как getImagesDAO() уже создаёт экземпляр класса, необходимо именно его закрыть после того как метод отработал

public class DAOFactory implements DAOFactotyInterface {

Connection connection = null;

public Connection getConnection() throws DAOException {
    //получение соединения
    } return connection;
}

@Override
public ImagesDAO getImagesDAO() throws DAOException {
    return new MySQLImagesDAO(getConnection());




public class MySQLImagesDAO implements ImagesDAO {

private Connection connection = null;
private PreparedStatement preparedStatement = null;

public MySQLImagesDAO(Connection connection) {
    this.connection = connection;
}

private void getPreparedStatement(String sql) throws DAOException {
    if (preparedStatement == null) {
        try {
            preparedStatement = connection.prepareStatement(sql);
        } catch (Exception e) {
            throw new DAOException("Get preparedStatment failed.", e);
        }
    }
}
public void updateImages(Images images) throws DAOException {
    //метод update
}
}public void close() throws DAOException {
    //закрытие ps и соединения
}




public class Main {
public static void main (String[] args){
    DAOFactory test = new DAOFactory();
    Images images = new Images();
    images.setIdProfile(1);
    images.setAvatar("testLink");
    images.setId(1);

    try {
        test.getImagesDAO().updateImages(images);
        //close?
    } catch (DAOException e) {
        e.printStackTrace();
    }
}
1
  • нет, static методов тут нет, необходимо просто вызвать метод close так, чтобы он закрыл уже созданное DAO для выполнения метода update
    – PolkovnikJ
    15 июн 2016 в 16:44

1 ответ 1

3

Получите ссылку на экземпляр так же как и когда вызываете другие методы.

test.getImagesDAO().close();

Плюс измените геттер так, чтобы он не создавал каждый раз новый объект

public class DAOFactory implements DAOFactotyInterface {

Connection connection = null;

public Connection getConnection() throws DAOException {
    //получение соединения
    } return connection;
}

MySQLImagesDAO instance;

@Override
public ImagesDAO getImagesDAO() throws DAOException {
    if(instance==null)
       { instance = new MySQLImagesDAO(getConnection()) }
    return instance;
} 
}

UPD_0:

Если менять исходный код нельзя, то сделайте так:

ImagesDAO instance = test.getImagesDAO();
instance.updateImages(images);
instance.close();
11
  • А разве при этом я не создаю опять новый объект и закрываю его сразу же, по сути это второй объект, первый не будет закрыт, поправьте если это не так.
    – PolkovnikJ
    15 июн 2016 в 21:13
  • 1
    @PolkovnikJ, а, ну да. Попробуйте вынести ImagesDAO в переменную класса и в getImagesDAO() не создавать каждый раз новый экземпляр, а возвращать один и тот же.
    – ЮрийСПб
    15 июн 2016 в 21:16
  • 1
    @PolkovnikJ, обновил ответ. В приведённом коде getImagesDAO() будет возвращать всегда один и тот же объект
    – ЮрийСПб
    15 июн 2016 в 21:40
  • 1
    @PolkovnikJ, обновил ответ
    – ЮрийСПб
    16 июн 2016 в 9:29
  • 1
    отлично, жаль что не догадался до такого простого решения, спасибо за помощь!
    – PolkovnikJ
    16 июн 2016 в 9:56

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.