где мы должны использовать commit (session), в try или finally?

Question

где мы должны использовать commit (session), в try или finally?

1

Если я хочу использовать фиксацию (сеанс) после успешного выполнения операции с базой данных, где ее лучше положить в try или в finally block? Здесь я использовал его, в конце концов, должен ли он попробовать?

public void delete( --- ) {
    Session session = init();
    try {
        ----
    } catch (HibernateException e) {
        rollback(session);
        logger.error("delete failed", e);
        throw e;
    } finally {
        commit(session);
        close(session);
    }
}

knowledge flow 03 март 2015, в 08:12

Источник

Теги:

java

hibernate

exception

try-catch

4 ответа

0

Это должно быть в try. finally будет также вызван после catch.

Jens 03 март 2015, в 06:04

0

Он должен определенно быть помещен в блок try. Окончательный блок используется только для закрытия любого соединения.

René Winkler 03 март 2015, в 05:21

0

Что ж, блок finally также следует использовать для отката в этом случае, IMO, если сеанс не был зафиксирован. В противном случае любые необработанные исключения приведут к утечке этих сеансов.
Jon Skeet 03 март 2015, в 08:20

0

вам нужно поместить его в блок try, иначе вы не узнаете никаких ошибок, и вы не сможете откатить

kism3t 03 март 2015, в 04:38

Ещё вопросы

Что ж, блок finally также следует использовать для отката в этом случае, IMO, если сеанс не был зафиксирован. В противном случае любые необработанные исключения приведут к утечке этих сеансов.

Jon Skeet · Accepted Answer · 2015-03-03T05-55-00.000Z

Это должно быть в try, по двум причинам:

Вы commit сеанс, если какое-либо исключение или ошибка, кроме исключения HibernateException, и вы почти наверняка не хотите этого делать
Вы вызовете commit после вызова rollback. Я не могу вспомнить, позволяет ли Hibernate сделать это (молча игнорируя откат), но, по крайней мере, это уродливо. Каждая сессия должна быть зафиксирована или отменена.

Нормальное решение здесь, чтобы сохранить отдельную boolean переменную, которая устанавливается, когда вы успешно совершить, и убедитесь, что в finally, откат в случае необходимости:

boolean committed = false;
try {
    // Do stuff
    commit(session);
    committed = true;
} catch (HibernateException e) {
    logger.error("delete failed", e);
    throw e;
} finally {
    if (!committed) {
        rollback(session);
    }
    // TODO: This won't execute if rollback fails.
    // Check whether that a problem.
    close(session);
}

Спасибо за подробный ответ, Джон.
Я думаю, что вы должны переместить откат внутри блока catch (без флага фиксации) вместо использования причины флага фиксации, если в результате фиксации произошла исключительная ситуация.
@ AmjadAbdul-Ghani Нет, потому что тогда не будет выполнено откат, если возникнет какое-либо другое исключение. И если фиксация не удалась, я бы по крайней мере ожидал, что она будет действительной, чтобы затем вызвать откат
это правда, если вы не правильно обрабатывали исключения, если вы когда-либо видели реализацию весенней транзакции - catch, throwable >> rethrow ошибка или исключение, если необходимо, оба могут работать, но я не люблю по скромному мнению ставить Откат заявления в блоке finally, это мои предпочтения, и спасибо за ваш отзыв
@ Amjad: В то время как я предпочитаю использовать блок finally, если вы не ловите Throwable. В противном случае слишком легко пропускать транзакции, хотя и временно. Наконец, блоки предназначены именно для той очистки, которую мы хотим здесь.