org.hibernate.PersistentObjectException: отдельная сущность передана, чтобы сохранить - PullRequest
73 голосов
/ 17 июня 2011

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

org.hibernate.PersistentObjectException: detached entity passed to persist: example.forms.InvoiceItem
    at org.hibernate.event.def.DefaultPersistEventListener.onPersist(DefaultPersistEventListener.java:127)
    at org.hibernate.impl.SessionImpl.firePersist(SessionImpl.java:799)
    at org.hibernate.impl.SessionImpl.persist(SessionImpl.java:791)
    .... (truncated)

отображение спящего режима:

<hibernate-mapping package="example.forms">
    <class name="Invoice" table="Invoices">
        <id name="id" type="long">
            <generator class="native" />
        </id>
        <property name="invDate" type="timestamp" />
        <property name="customerId" type="int" />
        <set cascade="all" inverse="true" lazy="true" name="items" order-by="id">
            <key column="invoiceId" />
            <one-to-many class="InvoiceItem" />
        </set>
    </class>
    <class name="InvoiceItem" table="InvoiceItems">
        <id column="id" name="itemId" type="long">
            <generator class="native" />
        </id>
        <property name="productId" type="long" />
        <property name="packname" type="string" />
        <property name="quantity" type="int" />
        <property name="price" type="double" />
        <many-to-one class="example.forms.Invoice" column="invoiceId" name="invoice" not-null="true" />
    </class>
</hibernate-mapping>

РЕДАКТИРОВАТЬ: InvoiceManager.java

class InvoiceManager {

    public Long save(Invoice theInvoice) throws RemoteException {
        Session session = HbmUtils.getSessionFactory().getCurrentSession();
        Transaction tx = null;
        Long id = null;
        try {
            tx = session.beginTransaction();
            session.persist(theInvoice);
            tx.commit();
            id = theInvoice.getId();
        } catch (RuntimeException e) {
            if (tx != null)
                tx.rollback();
            e.printStackTrace();
            throw new RemoteException("Invoice could not be saved");
        } finally {
            if (session.isOpen())
                session.close();
        }
        return id;
    }

    public Invoice getInvoice(Long cid) throws RemoteException {
        Session session = HbmUtils.getSessionFactory().getCurrentSession();
        Transaction tx = null;
        Invoice theInvoice = null;
        try {
            tx = session.beginTransaction();
            Query q = session
                    .createQuery(
                            "from Invoice as invoice " +
                            "left join fetch invoice.items as invoiceItems " +
                            "where invoice.id = :id ")
                    .setReadOnly(true);
            q.setParameter("id", cid);
            theInvoice = (Invoice) q.uniqueResult();
            tx.commit();
        } catch (RuntimeException e) {
            tx.rollback();
        } finally {
            if (session.isOpen())
                session.close();
        }
        return theInvoice;
    }
}

Invoice.java

public class Invoice implements java.io.Serializable {

    private Long id;
    private Date invDate;
    private int customerId;
    private Set<InvoiceItem> items;

    public Long getId() {
        return id;
    }

    public Date getInvDate() {
        return invDate;
    }

    public int getCustomerId() {
        return customerId;
    }

    public Set<InvoiceItem> getItems() {
        return items;
    }

    void setId(Long id) {
        this.id = id;
    }

    void setInvDate(Date invDate) {
        this.invDate = invDate;
    }

    void setCustomerId(int customerId) {
        this.customerId = customerId;
    }

    void setItems(Set<InvoiceItem> items) {
        this.items = items;
    }
}

InvoiceItem.java

public class InvoiceItem implements java.io.Serializable {

    private Long itemId;
    private long productId;
    private String packname;
    private int quantity;
    private double price;
    private Invoice invoice;

    public Long getItemId() {
        return itemId;
    }

    public long getProductId() {
        return productId;
    }

    public String getPackname() {
        return packname;
    }

    public int getQuantity() {
        return quantity;
    }

    public double getPrice() {
        return price;
    }

    public Invoice getInvoice() {
        return invoice;
    }

    void setItemId(Long itemId) {
        this.itemId = itemId;
    }

    void setProductId(long productId) {
        this.productId = productId;
    }

    void setPackname(String packname) {
        this.packname = packname;
    }

    void setQuantity(int quantity) {
        this.quantity = quantity;
    }

    void setPrice(double price) {
        this.price = price;
    }

    void setInvoice(Invoice invoice) {
        this.invoice = invoice;
    }
}

РЕДАКТИРОВАТЬ: JSON-объект отправлен от клиента:

{"id":null,"customerId":3,"invDate":"2005-06-07T04:00:00.000Z","items":[
{"itemId":1,"productId":1,"quantity":10,"price":100},
{"itemId":2,"productId":2,"quantity":20,"price":200},
{"itemId":3,"productId":3,"quantity":30,"price":300}]}

РЕДАКТИРОВАТЬ: Некоторые детали:
Я попытался сохранить счет двумя способами:

  1. Изготовлено вручную вышеупомянутым JSON объект и передал его свежему сессия сервера. В этом случае абсолютно никакая деятельность не была сделана до звонка метод сохранения, поэтому не должно быть открытых сессий кроме открытого в методе сохранения

  2. Загружены существующие данные с помощью метод getInvoice, и они прошли то же самое данные после удаления значения ключа. Это тоже я считаю следует закрыть сессию перед сохранением как транзакция фиксируется в методе getInvoice.

В обоих случаях я получаю одно и то же сообщение об ошибке, которое заставляет меня поверить, что что-то не так с файлом конфигурации hibernate или классами сущностей или методом save.

Пожалуйста, дайте мне знать, если я должен предоставить более подробную информацию

Ответы [ 5 ]

101 голосов
/ 17 июня 2011

Вы не предоставили много соответствующих подробностей, поэтому я предполагаю, что вы вызвали getInvoice, а затем использовали объект результата для установки некоторых значений и вызвали save с предположением, что изменения вашего объекта будут сохранены.

Однако операция persist предназначена для совершенно новых временных объектов и завершается ошибкой, если идентификатор уже назначен. В вашем случае вы, вероятно, хотите позвонить saveOrUpdate вместо persist.

Вы можете найти некоторые обсуждения и ссылки здесь "отдельная сущность передана для сохранения ошибки" с кодом JPA / EJB

10 голосов
/ 26 февраля 2014

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

Следовательно, проблема наступает.

3 голосов
/ 17 июня 2011

Скорее всего, проблема лежит за пределами кода, который вы показываете нам здесь.Вы пытаетесь обновить объект, который не связан с текущим сеансом.Если это не Invoice, то, возможно, это InvoiceItem, который уже был сохранен, получен из базы данных, сохранен в каком-либо сеансе, а затем вы пытаетесь сохранить его в новом сеансе.Это невозможно.Как правило, никогда не сохраняйте ваши постоянные объекты живыми между сеансами.

Решение будет, например, получать весь граф объектов из того же сеанса, с которым вы пытаетесь его сохранить.В веб-среде это будет означать:

  • Получение сеанса
  • Выбор объектов, которые необходимо обновить или добавить ассоциации.Предпочтительно по первичному ключу
  • Изменить то, что нужно
  • Сохранить / обновить / удалить / удалить то, что вы хотите
  • Закрыть / зафиксировать сеанс / транзакцию

Если у вас возникли проблемы, опубликуйте код, который вызывает вашу службу.

2 голосов
/ 10 мая 2019

Это существует в отношении @ManyToOne. Я решил эту проблему, просто используя CascadeType.MERGE вместо CascadeType.PERSIST или CascadeType.ALL. Надеюсь, это поможет вам.

@ManyToOne(cascade = CascadeType.ALL)
@JoinColumn(name="updated_by", referencedColumnName = "id")
private Admin admin;

Решение:

@ManyToOne(cascade = CascadeType.MERGE)
@JoinColumn(name="updated_by", referencedColumnName = "id")
private Admin admin;
0 голосов
/ 18 июня 2019

Для исправления JPA с использованием EntityManager merge () вместо persist ()

EntityManager em = getEntityManager();
    try {
        em.getTransaction().begin();
        em.merge(fieldValue);
        em.getTransaction().commit();
    } catch (Exception e) {
        //do smthng
    } finally {
        em.close();
    }
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...