try с ресурсами - будет ли выполнен close();

Обсуждается

Сonnection connection = Util.getMySQLConnection();
try (Statement statement = connection.createStatement()) {
    //code
    connection.commit();
} catch (SQLException e) {
    connection.rollback();
}

Если мне не изменяет память, вроде я где -то читал, что если в try c ресурсами помещен хоть один объект реализующий инерфейс AutoCloseable, то автоматически close будет вызван у всей цепочки. Т.е. в том числе и у

Сonnection connection = Util.getMySQLConnection();

так ли это?

Комментарии (4)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

hidden #598481

Уровень 27

7 декабря 2022, 08:37

Сдаётся мне, что это зависит не от try с ресурсами, а от того, как реализован close() в твоей имплементации Statement. Если стейтмент обращается к коннекшену и вызывает у него close(), то закроется, а если нет, то нет.

seni

Уровень 34

7 декабря 2022, 09:04

он выглядит так 😅

public void close() throws SQLException {
    try {
        this.realClose(true, true);
    } catch (CJException var2) {
        throw SQLExceptionsMapping.translateException(var2, this.getExceptionInterceptor());
    }
}

класса StatementImpl а this.realClose так

protected void realClose(boolean calledExplicitly, boolean closeOpenResults) throws SQLException {
    JdbcConnection locallyScopedConn = this.connection;
    if (locallyScopedConn != null && !this.isClosed) {
        if (!(Boolean)this.dontTrackOpenResources.getValue()) {
            locallyScopedConn.unregisterStatement(this);
        }

        if (this.useUsageAdvisor && !calledExplicitly) {
            this.session.getProfilerEventHandler().processEvent((byte)0, this.session, this, (Resultset)null, 0L, new Throwable(), Messages.getString("Statement.63"));
        }

        if (closeOpenResults) {
            closeOpenResults = !this.holdResultsOpenOverClose && !(Boolean)this.dontTrackOpenResources.getValue();
        }

        if (closeOpenResults) {
            if (this.results != null) {
                try {
                    this.results.close();
                } catch (Exception var6) {
                }
            }

            if (this.generatedKeysResults != null) {
                try {
                    this.generatedKeysResults.close();
                } catch (Exception var5) {
                }
            }

            this.closeAllOpenResults();
        }

        this.clearAttributes();
        this.isClosed = true;
        this.closeQuery();
        this.results = null;
        this.generatedKeysResults = null;
        this.connection = null;
       // more..

seni

Уровень 34

7 декабря 2022, 09:04

мне пока сложно ориентироваться в таком 😕

hidden #598481

Уровень 27

7 декабря 2022, 09:39

Ну по логике вещей в твоём коннекшене может быть много стейтментов, и закрытие одного не должно вызывать закрытие всего соединения. Ведь у тебя в этот момент могут выполняться и другие запросы к базе. И в описании закрытия стейтмента, как и в коде, я вижу, что по цепочку будут закрываться нижележащие ResultSet-ы, а не вышестоящий стейтмент. Поэтому скорее всего нет, коннекшен закрываться не должен.

this.connection = null;

Тут просто разрывается связь между твоим стейтментом и существующим коннекшеном, и всё. Я не претендую на истину в последней инстанции и не разбирался глубоко в этом вопросе, но по крайней мере это рассуждение мне кажется логичным. Может, кто-то меня исправит или дополнит.