właściwość not-null odwołuje się do wartości null lub przejściowej


83

W obliczu problemów z zapisaniem obiektu rodzica / dziecka w trybie hibernacji. Każdy pomysł byłby bardzo mile widziany.

org.hibernate.PropertyValueException: not-null property references a null or transient value: example.forms.InvoiceItem.invoice
    at org.hibernate.engine.Nullability.checkNullability(Nullability.java:100)
        .... (truncated)

mapowanie hibernacji:

<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;
    }
}

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;
    }
}

Odpowiedzi:


49

Każdy InvoiceItemmusi mieć Invoicedołączony do niego ze względu na not-null="true"w wielu do jednego mapowania.

Więc podstawową ideą jest to, że musisz ustawić tę jawną relację w kodzie. Można to zrobić na wiele sposobów. Na twojej klasie widzę setItemsmetodę. NIE widzę addInvoiceItemmetody. Kiedy ustawiasz elementy, musisz przejść przez zestaw i wywołać item.setInvoice(this)wszystkie elementy. Jeśli zaimplementujesz addItemmetodę, musisz zrobić to samo. Lub musisz inaczej ustawić fakturę dla każdego InvoiceItemw kolekcji.


82

dla obserwujących ten komunikat o błędzie może również oznaczać „masz odwołanie do obcego obiektu, który nie został jeszcze zapisany w bazie danych” (nawet jeśli tam jest i nie jest pusty).


1
Jak mogę to rozwiązać? Przykład: Mam Entity_A z kolumną Entity_B, aw Entity_B mam kolumnę Entity_A relacja @OneToMany od A do B, a kiedy próbuję zaktualizować Entity_A, daje mi ten wyjątek, jak mogę zaktualizować Entity_A?
FAndrew

1
@FAndrew zapisz najpierw Entity_B?
rogerdpack

3
a jeśli mam na przykład 100 Entity_B to wyszukuję wszystkie edytowane Entity_B i jeśli było edytowane to aktualizuję? Czy nie byłoby lepiej, gdybym po prostu zaktualizował Entity_A?
FAndrew

7

Może to być tak proste, jak:

@Column(name = "Some_Column", nullable = false)

ale podczas ustawiania wartość „Some_Column” jest zerowa, nawet jeśli „Some_Column” nie może być kluczem podstawowym ani obcym.


2

Sprawdź niezapisane wartości klucza podstawowego / identyfikatora obiektu w plikach HBM. Jeśli zautomatyzowałeś tworzenie ID przez hibernację i ustawiasz gdzieś ID, to wyrzuci ten błąd. Domyślnie niezapisana wartość to 0, więc jeśli ustawisz ID na 0, zobaczysz ten błąd.


0

Otrzymywałem ten sam błąd, ale ostatecznie go rozwiązałem, w rzeczywistości nie ustawiałem jednostki obiektu, która jest już zapisana w innej jednostce, a zatem wartość obiektu, którą otrzymywał dla klucza obcego, była zerowa.


0

Rozwiązałem przez usunięcie właściwości @Basic (opcjonalne = false) lub po prostu zaktualizowanie wartości boolean @Basic (opcjonalne = prawda)


-11

Ustaw tę zmienną jako przejściową. Twój problem zostanie rozwiązany.

@Column(name="emp_name", nullable=false, length=30)
    private transient String empName;

3
Mówisz o rozwiązaniu błędu, a nie rozwiązaniu problemu.
Karthik R
Korzystając z naszej strony potwierdzasz, że przeczytałeś(-aś) i rozumiesz nasze zasady używania plików cookie i zasady ochrony prywatności.
Licensed under cc by-sa 3.0 with attribution required.