Гибернация: две или более транзакции одновременно: транзакция уже активна

Попробуйте переписать код как

public class HibernateSession {

    public Session session;
    int id;
    Transaction t = null;

    public void createTransaction() {
        session = HibernateUtil.getSessionFactory().getCurrentSession();
        t = session.beginTransaction();
    }

    public void commitclose() {
        t.commit();
        session.close();
    }

    public void rollbackclose() {

        try {
            t.rollback();
            session.close();
        } catch (Exception hibernateexception) {
            hibernateexception.printStackTrace();
        }

    }

}

Конечно, для каждой ссылки t требуются нулевые проверки.

Согласно вашей логике createTransaction, вы получаете текущий сеанс от SessionFactory и запускаете из него транзакцию.

Вот в чем проблема.

Предположим, вы создали объект HibernateSession и начали транзакцию, но она все еще выполняется. Значит, вы еще не закрыли транзакцию.

Теперь вы создали еще один HibernateSession и попытаетесь запустить транзакцию, при этом будет сгенерировано исключение.

Итак, ваш этот код

public void createTransaction() {
    session = HibernateUtil.getSessionFactory().getCurrentSession();
    session.beginTransaction();
}

должно быть это

public void createTransaction() {
    session = HibernateUtil.getSessionFactory().openSession();
    session.beginTransaction();
}

getSessionFactory (). openSession () всегда открывает новый сеанс, который необходимо закрыть после завершения операций.

getSessionFactory (). getCurrentSession () возвращает сеанс, привязанный к контексту - вам не нужно его закрывать.

На самом деле в вашем реальном коде много недостатков:

• Вы создаете новый экземпляр своего класса HibernateSession каждый раз, когда вызываете свой код, поэтому у вас будет много его экземпляров, пытающихся получить доступ к одному и тому же сеансу.
• Каждый из этих экземпляров HibernateSession будет пытаться создать новую транзакцию, когда вы вызываете hs.createTransaction();, и поэтому вы получаете исключение в этой строке, потому что уже есть открытые транзакции, и вы пытаетесь открыть новую, потому что вы звонить session.beginTransaction(); каждый раз, не звоня в transaction.close();.

Здесь вы можете сделать свой HibernateSessionclass синглтоном, чтобы был доступен только один экземпляр, вы можете проверить этот Реализация одноэлементного шаблона для получения дополнительных сведений о том, как его реализовать.

И в методе createTransaction, который лучше называть getTransaction, а не просто вызывать session.beginTransaction();, вам нужно получить текущую транзакцию, если она существует, вы можете проверить ее с помощью метода session.getTransaction() и убедиться, что вы заключили свой код в блок try ...catch, вы можете проверить это Сеанс гибернации учебник для получения дополнительной информации.

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

1. Вы не должны использовать getCurrentSession()

Как указано в jboss документация, вы должны использовать следующую идиому:

 Session sess = factory.openSession();
 Transaction tx;
 try {
     tx = sess.beginTransaction();
     //do some work
     ...
     tx.commit();
 }
 catch (Exception e) {
     if (tx!=null) tx.rollback();
     throw e;
 }
 finally {
     sess.close();
 }

2. Вы должны следовать шаблону CRUD

Для каждой операции (создание, чтение, обновление, удаление) создайте метод в своем классе. Каждый метод должен создать свой сеанс собственный и транзакцию для работы, потому что, как указано в этот ответ:

session is not a thread safe object - cannot be shared by multiple threads. You should always use "one session per request" or "one session per transaction"

Ваши методы несколько верны, но также посмотрите на объекты DAO, созданные в этот учебник

Решение:

Причина сбоя вашего кода заключается просто в том, что транзакция все еще выполняется после вашего предыдущего взаимодействия с базой данных. Вот почему вы делаете finally{ session.close() } - чтобы сделать Конечно, чтобы сессия закрывалась при выходе из метода. Я предполагаю, что в какой-то момент ваш commit() не был успешным и оставил вашу гибернацию в транзакции / сеансе, которая с тех пор не закрывалась.

Чтобы исправить это, вы должны один раз вставить session.close () в свой код, выполнить его, а затем реализовать предложенный мной блок try-catch-finally, чтобы убедиться, что он закрыт в будущем.

Я только что изменился

session = HibernateUtil.getSessionFactory().getCurrentSession();

с участием

session = HibernateUtil.getSessionFactory().openSession();

Я уже открывал сеанс Фабрики.