Supprimer un élément d'un-à-plusieurs collection (Java + HIbernate + Struts)

Je ne peux pas supprimer un objet enfant à partir de la base de données. À partir de la org.apache.struts.action.Action.execute() méthode, je suis de retirer l'enfant de la mère List, et aussi d'appeler session.delete(child). J'ai simplifié le code ci-dessous et ne comprenaient ce que je crois être relavent.

Hibernate Mapping

<class 
    name="xxx.xxx.hibernate.Parent" 
    table="parent">

    ...

    <list
        name="children"
        cascade="all,delete-orphan"
        lazy="true"
        inverse="true">

        <key column="parent_id"/>
        <index column="list_index"/>
        <one-to-many class="xxx.xxx.hibernate.Child"/>
    </list>
</class>

<class 
    name="xxx.xxx.hibernate.Child" 
    table="child">

    ...

    <many-to-one
        name="parent"
        class="xxx.xxx.hibernate.Parent"
        not-null="true"
        column="parent_id" />

</class>

Extrait de la méthode execute ()

Transaction tx = session.beginTransaction();  //session is of type org.hibernate.Session

try {
    Parent parent = (Parent) session.get(Parent.class, getParentId());

    Iterator i = form.getDeleteItems().iterator();  //form is of type org.apache.struts.action.ActionForm
    while(i.hasNext()){
        Child child = (Child) i.next();
        session.delete(child);
        parent.getChildren().remove(child); //getChildren() returns type java.util.List
    }

    session.saveOrUpdate(parent);
    tx.commit();
} ...

J'ai essayé avec seulement session.delete(child); et j'ai essayé avec seulement parent.getChildren().remove(child); et avec les deux lignes, le tout sans succès. Il n'y a pas d'erreurs ou d'exceptions lancées ou quoi que ce soit de la sorte. Je suis sûr que ce code est appelé (je l'ai même utilisé System.out.println(); de tracer ce qui se passe), mais la base de données n'est pas mis à jour. Je peux ajouter des enfants à l'aide de code similaire, modifier la non-perception des propriétés d'enfants existants, modifier les parents propriétés, tout cela fonctionne, tout simplement pas de la suppression!

Selon la Hibernate FAQ je suis en train de faire la cartographie de droit, et selon cette SORTE de question j'ai le droit de la logique. J'ai regardé partout sur internet et qui n'arrive pas à trouver autre chose.

Ce que je fais mal? S'il vous plaît aider! Merci.

Notes sur les versions

Tout est vieux de quelques années:

Java 1.4.2
SQL Server 2005
Hibernate 3.0.5
Struts 1.2.7
Apache Tomcat 5.0.28

Avez-vous remplacer égale/hashcode correctement dans l'Enfant? Avez-vous vérifier que l'Enfant est retiré de Parent après l'appel de supprimer()? Si elle ne l'est pas, vous pourriez en fait d'annuler la suppression de l'action. Il suffit de retirer l'enfant de la liste des parents et de la mise à jour de la mère (ce sera la cause de la list_index être mis à jour).

OriginalL'auteur Peter Di Cecco | 2010-01-07

6

Si vous n'avez pas remplacé la equals() méthode, l'entité n'est probablement pas trouvé dans la liste, car il a été détaché, et est maintenant une instance à l'autre. C'est pourquoi la remove ne fonctionne pas. Alors même si le delete œuvres, les objets sont re-cascacde parce qu'ils existent encore dans la collection. Voici quoi faire:
- supplanter la equals() (et hashCode()) méthode(s), soit en utilisant la id (facile) ou une sorte de busines clé (plus approprié) (recherche stackoverflow pour des conseils pour overrideing ces deux metods), et de ne laisser que getChildren().remove(child)
- Parcourir la collection des enfants dans la première boucle, comme ceci:
```
Iterator<Child> i = form.getDeleteItems().iterator();
while(i.hasNext()){
    Child child = i.next();
    for (Iterator<Child> it = parent.getChildren().iterator();) {
         if (child.getId().equals(it.next().getId()) {
             it.remove(); //this removes the child from the underlying collection
         }
    }
}
```
il ne fonctionne pas, même après primordial equals et hashCode. Retrait de l'objet par l'itérateur n'est pas nessesary, supprimer fait la même chose plus rapidement.
pourquoi pensez-vous ainsi? Je pense que vous avez tort dans les deux déclarations 🙂
L'Enfant Id remplis correctement dans le Formulaire?
me demandez-vous? 🙂
Oui.

OriginalL'auteur Bozho

Je ne suis pas sûr de ce que provoque ce comportement en veille prolongée, vous pouvez l'obtenir en passant par le chargement de la Child premier. Séparément de la suppression de la Child n'est pas nessesary. Mis à jour le code devrait ressembler;

Transaction tx = session.beginTransaction();  //session is of type org.hibernate.Session

try {
    Parent parent = (Parent) session.get(Parent.class, getParentId());

    Iterator i = form.getDeleteItems().iterator();  //form is of type org.apache.struts.action.ActionForm
    while(i.hasNext()){
        Child child = (Child) session.get(Chile.class, ((Child) i.next()).getChildId());
        parent.getChildren().remove(child); //getChildren() returns type java.util.List
    }

    session.saveOrUpdate(parent);
    tx.commit();
} ...

montrer le SQL généré par Hibernate

<property name="show_sql">true</property>
<property name="format_sql">true</property>

Edit:

Découvrez ce Le chapitre 10. Travailler avec des objets

cela fonctionne, mais je pense que le chargement de chaque objet nouveau doit être évitée.
Pourquoi pensez-vous que le chargement de chaque objet nouveau doit être évitée? Ce qui rend votre solution mieux que cela?
si la session est frais, cela signifierait N requêtes à la base de données pour récupérer les éléments.

OriginalL'auteur n002213f

-2

Dans ce cas, l'Enfant de la classe est le propriétaire de la relation inverse, Hibernate va regarder la référence du parent de l'enfant afin de déterminer si la relation est toujours là. Puisque vous ne définissez pas le parent à null, la relation existe et que l'enfant ne peut pas être supprimé. Essayez de faire
```
parent.getChildren().remove(child);
child.parent = null; 
session.delete(child);
```
Également supprimer la non-null="true" à partir de la propriété parent de cartographie.

La meilleure chose à faire lorsque l'on travaille avec des associations inverses, est la mise à jour des deux côtés dans du code Java, de cette façon, vous pouvez continuer à travailler avec les objets en mémoire et que vous n'avez pas à vous soucier de quel côté est propriétaire de la relation.

Une situation similaire est décrit en détail ici: http://simoes.org/docs/hibernate-2.1/155.html

Supprimer orphelin va gérer le retrait de l'enfant, même si vous omettez la 2ème ligne. Et cela ne marchera pas, parce que supprimer(l'enfant) ne pas enlever quoi que ce soit
En effet, j'ai pensé que l'inverse="true" permettrait d'éviter la suppression, mais docs.jboss.org/hibernate/core/3.3/reference/en/html/... les états que les cascades sont toujours traitées lors de l'inverse="true".

OriginalL'auteur Danny Groenewegen

Vous devez vous connecter pour publier un commentaire.