Hibernate: Objekt wird nicht gelöscht.

[Dandro]

Hallo Leute

Ich hab mal wieder ein Problem, diesmal mit Hibernate.
Ich versuche das mal so genau zu schildern wie möglich um allen möglichen Frage vorzubeugen.

Erstmal der Code abschnitt in dem das Problem auftritt:

Client lClient = event.getClient();
		Account lAccount = lClient.getAccount();
		lAccount.acquireWriteLock();
		try {
			for (CharacterData lData : lAccount.getCharacters()) {
				if (lData.getName().equals(event.getSource().getCharacterName())) {
					Transaction lTransaction = _dbSession.beginTransaction();				
					
					/*  re-attach objects */
					_dbSession.update(lAccount.getData());
					_dbSession.update(lData);
					
					lAccount.removeCharacter(lData);
					_dbSession.delete(lData);
					
					lTransaction.commit();
					break;
				}
			}
		} finally {
			lAccount.releaseLock();
		}

Das Problem liegt beim Delete, das Objekt wird einfach nicht gelöscht.
Aufgrund der Programmarchitektur ist es mir nicht möglich Sessions lange aufzuhalten, so werden die Objekte detached. Sprich in einer früheren Session wird das Objekt "AccountData" geladen (Bestandteil von der Klasse "Account", siehe lAccount.getData()). AccountData sowie CharacterData sind Annotierte Klassen und der Configuration bekannt. AccountData hat eine OneToMany Beziehung zu CharacterData und andersrum eine ManyToOne Beziehung, also Bidirektional. Die OneToMany Beziehung spiegelt dabei die ManyToOne Beziehung (inverse). Der FetchType beider beziehungen ist EAGER.

Meiner Meinung nach muss ich in der neuen Session die alten Datenobjekte wieder attachen, was ich laut hibernate.org und diversen Büchern mit "update(), saveOrUpdate() oder merge()" machen kann. Allerdings funktioniert das löschen bei allen 3 alternativen nicht.
In der Log taucht keine Warn oder Error meldung auf, leider.

Ich hab scheinbar irgendwo den Blick dafür momentan verloren oder weiß einfach noch nicht woran es liegen kann.
Vielleicht könnt ihr mir dabei helfen? ^.^

 

[byte]

Hast Du Dir mal das erzeugte SQL angeguckt? Wird ein DELETE abgesetzt?
Wird das zu löschende Objekt evtl. noch von anderen Objekten referenziert?
Welche Kaskadierung ist zwischen den Objekten konfiguriert?

 

[ARadauer]

hilft dir das?
http://www.java-forum.org/de/topic73287_hibernate-probleme.html

@OneToMany(mappedBy = "order", cascade = CascadeType.ALL) 
@org.hibernate.annotations.Cascade( 
   value = org.hibernate.annotations.CascadeType.DELETE_ORPHAN 
) 
public List<OrderedProduct> getOrderedProducts() { 
   return orderedProducts; 
}

DELETE_ORPHAN ist unser held...

 

[Dandro]

Hast Du Dir mal das erzeugte SQL angeguckt? Wird ein DELETE abgesetzt?
Wird das zu löschende Objekt evtl. noch von anderen Objekten referenziert?
Welche Kaskadierung ist zwischen den Objekten konfiguriert?

Ja, das SQL habe ich mir angeguckt, es wird kein DELETE abgesetzt. Auch das "_dbSession.update()" bleibt ohne ein SQL Statement (was meines Wissens an dieser Stelle aber auch nicht zwingend ist).
Es gibt bisher nur diese beiden Datenobjekte, und die Referenz auf CharacterData lösche ich ja eigentlich explizit mit "removeCharacter" ... Was auch gut klappt, denn die Referenz darauf geht "verloren" danach.
Bisher habe ich noch keine Kaskadierung zwischen den Objekten konfiguriert. Inwiefern ist die hier zu nutzen?

@ARadauer
Das werde ich gleich mal ausprobieren

<edit>
Ich hab deinen Vorschlag nun einmal ausprobiert @ARadauer. Leider funktioniert dies nicht, und das löschen bleibt weiterhin ergebnislos
</edit>

Und noch eine Ergänzung:
Ich habe nun den mal ein wenig rumgetestet und wenn ich

@OneToMany(mappedBy = "account", fetch = FetchType.EAGER, cascade = CascadeType.ALL)
	private Set<CharacterData> characters = new HashSet<CharacterData>();

Und dann das Update nur für das AccountData Objekt aufrufe, dann re-attacht er scheinbar die Objekte an die Session, denn es werde SQL Statements abgesetzt die das ganze updaten. Allerdings funktioniert das eigentliche löschen immernoch nicht Ich hab hier nochmal ein paar Codestellen die vielleicht helfen mir zu helfen

Nach der Umstellung sieht der Block zur Löschung nun so aus:

Client lClient = event.getClient();
		Account lAccount = lClient.getAccount();
		lAccount.acquireWriteLock();
		try {
			Transaction lTransaction = _dbSession.beginTransaction();
			_dbSession.update(lAccount.getData()); /* re-attach */
			for (CharacterData lData : lAccount.getCharacters()) {
				if (lData.getName().equals(event.getSource().getCharacterName())) {			
					lAccount.removeCharacter(lData);
					_dbSession.delete(lData);
					
					break;
				}
			}
			lTransaction.commit();
		} finally {
			lAccount.releaseLock();
		}

Die entsprechende Funktion in der Account.class

/**
	 * Removes a character.
	 * 
	 * @param aCharacter Character to remove.
	 */
	public void removeCharacter(CharacterData aCharacter) {
		removeCharacter(aCharacter.getRealm(), aCharacter.getSlot());
	}
	
	/**
	 * Removes a character in the specified slot.
	 * 
	 * @param aSlot
	 */
	public void removeCharacter(Realm aRealm, int aSlot) {
		Set<CharacterData> lCharacters = _data.getCharacters();
		for (Iterator<CharacterData> lIterator = lCharacters.iterator(); lIterator.hasNext();) {
			CharacterData lData = lIterator.next();
			if (lData.getSlot() == aSlot && lData.getRealm() == aRealm) {
				lIterator.remove();
			}
		}
		_data.setCharacters(lCharacters);
	}

Das Datenobjekt in Account.class ist so deklariert:

private AccountData _data;

 

[ARadauer]

mhn ich schreib meine annotations immer beim getter, vielleicht mach das für die @org.hibernate.annotations.Cascade( einen unterschied...

ich habe in meiner anwendung eine bestellung mit artikel, ich entferne einfach den artikel über remove aus der arraylist der bestellung und mach ein merge auf die bestellung, dann wird bei mir der artikel gelöscht... versuch das mal..

Client lClient = event.getClient(); 
      Account lAccount = lClient.getAccount(); 
      lAccount.acquireWriteLock(); 
      try { 
         Transaction lTransaction = _dbSession.beginTransaction(); 
         _dbSession.update(lAccount.getData()); /* re-attach */ 
         for (CharacterData lData : lAccount.getCharacters()) { 
            if (lData.getName().equals(event.getSource().getCharacterName())) {          
               lAccount.removeCharacter(lData); 

               _dbSession.merge(lAccount); // <---
                
               break; 
            } 
         } 
         lTransaction.commit(); 
      } finally { 
         lAccount.releaseLock(); 
      }

ps: warum machst du eigentlich immer _ und l vor die variablen (parameter und lokale variable oder?) ich find das eher störend..

 

[Dandro]

Danke Es klappt nun und ich bin überglücklich

Die Benennung der Variablen hab ich so mehr oder weniger aus dem Beruf übernohmen, dort gibt es auch solche Richtlinien um den Code teilweise übersichtlicher zu gestalten, damit es fremdentwickler leichter haben sich zurechtzufinden.

_ steht für ein Field, also Klassenvariable
l für eine lokale
a für einen Parameter