Hibernate Hibernate liest Daten zu oft aus!

[moessi91]

Hallo,
ich verwende eine Postgresql Datenbank und greife darauf mit Hibernate zu. Das hat alles in letzter Zeit immer gut funktioniert. Allerdings habe ich seit einiger Zeit einen Fehler. Und zwar jedes mal wenn ich meine Liste an Benutzern auslesen möchte dann erhalte ich einen User 5mal obwohl er nur einmal in der Datenbank vorhanden ist. Ich habe die Funktion schon bis direkt vor das auslesen hineindebugged und erhalte gleich nach dem crit.list() Befehl 5 einträge in meine Liste. Es sind aber nur 2 User in der Tabelle.
Weiß irgendjemand hilfe?

Session session = HibernateUtil_Postgres.getSessionFactory().openSession();
        Transaction tx = session.beginTransaction();

        Criteria crit = session.createCriteria(Kunde.class);
            crit.setMaxResults(10);
            crit.setFirstResult(iFrom);
            
            
        List<Kunde> lKunden = crit.list();
        
        tx.commit();
       
        return lKunden;

 

[brauner1990]

Referenzierst du bereits irgendwo auf die User innerhalb der User?

Hast du die Criteria mal selber geschrieben und via Hibernate gepostet?

 

[Ebenius]

Ich hab keine Idee. Aber lass Dir mal das SQL ausgeben. Sieht das ordentlich aus? Welchen Wert hat iFrom? Wie sieht die Klasse Kunde aus?

Ebenius

 

[nillehammer]

Das Criterium ist ja nun sehr einfach. Da sollte jeder Kunde nur einmal in die Liste kommen. Anhand der Infos kann man nur raten. Wie Ebenius schon geschrieben hat, sollte man sich mal die SQL-Query anschauen, die auf die Datenbank abesetzt wird (in der hbm.xml output SQL auf true setzen). Ich würde mal auf ein verkorkstes Mapping tippen, das zu einem Kreuzprodukt (OUTER JOIN von Kunde-Tabelle auf sich selbst) in der SQL-Query führt.

 

[moessi91]

okay werde ich mal probieren den sql code auszugeben. und mir das ganze dann genauer ansehen.
Das lustige dabei ist einfach dass der andere Benutzer nur einmal ausgegeben wird nur der eine wird mehrmals ausgegeben?!?!

 

[JimPanse]

wie ist deine Klasse Kunde aufgebaut? Ist in der Klasse zufälliger weise eine ManyToMany Relation enthalten? (nur so eine Ahnung)

 

[Ebenius]

Kannst Du mal die Klasse Kunde inkl. Mapping hier einstellen?

Beim dritten mal ignoriert der TO die Frage bestimmt nicht. :-D

Ebenius

 

[moessi91]

Also hier mal meine ganze Kunden Klasse:

/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package persistence;

import controling.Functions;
import hibernate.HibernateUtil_Postgres;
import java.io.Serializable;
import javax.persistence.*;
import org.hibernate.HibernateException;
import org.hibernate.Session;
import org.hibernate.Transaction;
import org.hibernate.annotations.LazyToOne;
import org.hibernate.annotations.LazyToOneOption;
import org.hibernate.criterion.Restrictions;

/**
 *
 * @author Administrator
 */
@Entity 
@Table (name="kunde")
public class Kunde implements Serializable, Persis {
    @Id @GeneratedValue
    @Column (name="pk_kunde")
    private long ID;

    @Column (name="vorname")
    private String sVorname;
    
    @Column (name="nachname")
    private String sNachname;
    
    @Column (name="titel")
    private String sTitel;

    @Column (name="username")
    private String sUsername;
    
    @Column (name="passwort")
    private String sPasswort;
    
    @Column (name="geschlaecht")
    private boolean bGeschlaecht;
    
    @Column (name="level")
    private int iLevel;
    
    @JoinColumn(name="fk_adresse", referencedColumnName="pk_adresse")
    @OneToOne(cascade= CascadeType.ALL)
    @LazyToOne(value= LazyToOneOption.FALSE)
    private Adresse adresse;
    
    @JoinColumn(name="fk_kundenstatistik", referencedColumnName="pk_kundenstatistik")
    @OneToOne(cascade= CascadeType.ALL)
    private Kundenstatistik kundenstatistik;
    
    @JoinColumn(name="fk_rabatt", referencedColumnName="pk_rabatt")
    @OneToOne(cascade= CascadeType.ALL)
    private Rabatt rabatt;

    public Kunde() {
        this.ID = -1;
        this.rabatt = new Rabatt(0, "A");
        this.kundenstatistik = new Kundenstatistik(0, 0, null, null);
    }

    public Kunde(String sVorname, String sNachname, String sTitel, String sUsername, String sPasswort, boolean bGeschlaecht, Adresse adresse, Kundenstatistik kundenstatistik, Rabatt rabatt, int iLevel) {
        this.sVorname = sVorname;
        this.sNachname = sNachname;
        this.sTitel = sTitel;
        this.sUsername = sUsername;
        this.sPasswort = sPasswort;
        this.bGeschlaecht = bGeschlaecht;
        this.adresse = adresse;
        this.kundenstatistik = kundenstatistik;
        this.rabatt = rabatt;
        this.iLevel = iLevel;
    }

    public long getID() {
        return ID;
    }

    public void setID(long ID) {
        this.ID = ID;
    }

    public Adresse getAdresse() {
        return adresse;
    }

    public void setAdresse(Adresse adresse) {
        this.adresse = adresse;
    }

    public boolean isbGeschlaecht() {
        return bGeschlaecht;
    }

    public void setbGeschlaecht(boolean bGeschlaecht) {
        this.bGeschlaecht = bGeschlaecht;
    }

    public Kundenstatistik getKundenstatistik() {
        return kundenstatistik;
    }

    public void setKundenstatistik(Kundenstatistik kundenstatistik) {
        this.kundenstatistik = kundenstatistik;
    }

    public Rabatt getRabatt() {
        return rabatt;
    }

    public void setRabatt(Rabatt rabatt) {
        this.rabatt = rabatt;
    }

    public String getsNachname() {
        return sNachname;
    }

    public void setsNachname(String sNachname) {
        this.sNachname = sNachname;
    }

    public String getsPasswort() {
        return sPasswort;
    }

    public void setsPasswort(String sPasswort) {
        this.sPasswort = sPasswort;
    }

    public String getsTitel() {
        return sTitel;
    }

    public void setsTitel(String sTitel) {
        this.sTitel = sTitel;
    }

    public String getsUsername() {
        return sUsername;
    }

    public void setsUsername(String sUsername) {
        this.sUsername = sUsername;
    }

    public String getsVorname() {
        return sVorname;
    }

    public void setsVorname(String sVorname) {
        this.sVorname = sVorname;
    }

    public int getiLevel() {
        return iLevel;
    }

    public void setiLevel(int iLevel) {
        this.iLevel = iLevel;
    }   

    @Override
    public void save() throws HibernateException {
        Session session = HibernateUtil_Postgres.getSessionFactory().openSession();
        Transaction tx = session.beginTransaction();
        session.save(this);
        tx.commit();
        session.close();
        
    }

    @Override
    public void load() throws HibernateException, Exception {
        if(ID == -1)
            throw new Exception("Keine ID angegeben");
        
        Session session = HibernateUtil_Postgres.getSessionFactory().openSession();
        Transaction tx = session.beginTransaction();
        session.load(this, ID);
        
        tx.commit();
        session.close();
    }

    @Override
    public void delete() throws HibernateException {
        throw new UnsupportedOperationException("Not supported yet.");
    }
/**
     * Diese funktion ließt anhand der Übergebenen Parameter einen User aus. Ist keine übereinstimmung vorhanden, wird null zurückgegeben
     * @param sUsername
     * @param sPasswort
     * @return Kunde or null if not valid
     * @throws HibernateException 
     */
    public Kunde loadLogin(String sUsername, String sPasswort) throws HibernateException{
       Session session = HibernateUtil_Postgres.getSessionFactory().openSession();
       Transaction tx = session.beginTransaction();
       Kunde k = (Kunde) session.createCriteria(Kunde.class)
             .add(Restrictions.and(Restrictions.eq("sUsername", sUsername),Restrictions.eq("sPasswort",sPasswort)))
             .uniqueResult();
       tx.commit();
       session.close();
       return k;
    }
    
    public String getFormattedAddress()
    {
        return Functions.utfDecode(this.sNachname+" "+this.sVorname+"<br/>"+
               this.adresse.getsStrasse()+" "+this.adresse.getsHausnummer()+"<br/>"+
               this.adresse.getPlz().getsPostleitzahl()+" "+this.adresse.getPlz().getsOrt()+"<br/>"+
               this.adresse.getPlz().getsLand());
                
    }
    
    
}

Ich verwende bei meinen einstellungen die eine Java Datei wie kann ich hier einstellen, dass mir der Code ausgegeben wird?
Hier ist die Datei:

/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package hibernate;

import org.hibernate.cfg.AnnotationConfiguration;
import org.hibernate.SessionFactory;
import org.hibernate.tool.hbm2ddl.SchemaExport;
/**
 * Hibernate Utility class with a convenient method to get Session Factory object.
 *
 * @author Administrator
 */
public class HibernateUtil_Postgres {

    private static final SessionFactory sessionFactory;
    
    static {
        try {
            AnnotationConfiguration cfg = new AnnotationConfiguration();
            cfg.setProperty("hibernate.dialect", "org.hibernate.dialect.PostgreSQLDialect");
            cfg.setProperty("hibernate.connection.driver_class", "org.postgresql.Driver");
            cfg.setProperty("hibernate.connection.url", "jdbc:postgresql://localhost:5432/webshop");
            cfg.setProperty("hibernate.connection.username", "postgres");
            cfg.setProperty("hibernate.connection.password", "cisco");

            cfg.addAnnotatedClass(persistence.Adresse.class);
            cfg.addAnnotatedClass(persistence.Bankverbindung.class);
            cfg.addAnnotatedClass(persistence.Bestellposition.class);
            cfg.addAnnotatedClass(persistence.Bestellung.class);
            cfg.addAnnotatedClass(persistence.Bonitaet.class);
            cfg.addAnnotatedClass(persistence.KreditkartenInfo.class);
            cfg.addAnnotatedClass(persistence.Kunde.class);
            cfg.addAnnotatedClass(persistence.Kundenstatistik.class);
            cfg.addAnnotatedClass(persistence.Lieferant.class);
            cfg.addAnnotatedClass(persistence.Mengenrabatt.class);
            cfg.addAnnotatedClass(persistence.Plz.class);
            cfg.addAnnotatedClass(persistence.Rabatt.class);
            cfg.addAnnotatedClass(persistence.RechnungsAdresse.class);
            cfg.addAnnotatedClass(persistence.Ware.class);
            cfg.addAnnotatedClass(persistence.Zahlungsform.class);
            cfg.addPackage("or_mapping");     
            SchemaExport se = new SchemaExport(cfg);
            se.create(false, false);
            sessionFactory=cfg.buildSessionFactory();
        } catch (Throwable ex) {
            // Log the exception. 
            System.err.println("Initial SessionFactory creation failed." + ex);
            throw new ExceptionInInitializerError(ex);
        }
    }
    
    public static SessionFactory getSessionFactory() {
        return sessionFactory;
    }
    public static void shutdown() {
//Caches und Verbindungspools schließen
        getSessionFactory().close();
    }
}

 

[SlaterB]

cfg.setProperty("hibernate.show_sql", "true");

wie stellst du überhaupt die Anzahl 5 fest, ist das list.size() oder irgendeine komplizierte Weiterverarbeitung, die andere Fehler enthalten kann?

was kannst du über die Kunden in der Liste sagen, da es 2 statt 5 gibt wird es spannend,
schau dir die Ids und Namen an, ist es 5x einer der zwei in der DB oder welche Aufteilung?

falls irgendwo gleiche Werte, ergeben die Objekte untereinander mit == verglichen Übereinstimmung
oder sind es verschiedene Objekte gleichen Inhalts?

offensichtliche Tests weiterhin:
- die Kunde-Klasse um alle Attribute bis auf Id und Name temporär kürzen
- HQL-Query 'from Kunde' statt Criteria, falls nicht zu schwer zu lernen
- HQL 'select count(*) from Kunde'
- SQL 'select count(*) from Kunde',
- SQL 'select * from Kunde',

 

[DanZ]

Und guck mal wieviele KundenStatistiken/Rabatt Einträge mit dem entsprechenden Kunden verbunden sind. Bei deinem Defaultkonstruktor hab ich eine blöde Vorahnung

 

[moessi91]

Die Anzahl 6 muss mich entschuldigen habe mich vertan: Stelle ich beim Debuggen fest.
Nachdem crit.list(); aufgerufen wurde steht im Debugger das die Liste 6 Objekte hat obwohl eben nur 2 in der Datenbank sind. Eines ist dabei 5 mal vorhanden das andere 1x.
Anbei ein Bild der Tabelle von pkadmin wo man deutlich sieht dass die Objekte nur 2x vorhanden sind und dann noch ein Auszug der Variablen im Debug Modus wo man erneut genau erkennt, dass der eine User 5x ausgelesen wurde mit exakt dem gleichen Hashwert.
Hier noch der SQL Code der dabei ausgeführt wurde. Werde aus diesem nicht ganz schlüssig:

Information: Hibernate: select this_.pk_kunde as pk1_6_12_, this_.fk_adresse as fk9_6_12_, this_.geschlaecht as geschlae2_6_12_, this_.level as level6_12_, this_.fk_kundenstatistik as fk10_6_12_, this_.fk_rabatt as fk11_6_12_, this_.nachname as nachname6_12_, this_.passwort as passwort6_12_, this_.titel as titel6_12_, this_.username as username6_12_, this_.vorname as vorname6_12_, adresse2_.pk_adresse as pk1_0_0_, adresse2_.fk_plz as fk4_0_0_, adresse2_.hausnummer as hausnummer0_0_, adresse2_.strasse as strasse0_0_, plz3_.pk_plz as pk1_10_1_, plz3_.land as land10_1_, plz3_.ort as ort10_1_, plz3_.postleitzahl as postleit4_10_1_, kundenstat4_.pk_kundenstatistik as pk1_7_2_, kundenstat4_.fk_bonitaet as fk4_7_2_, kundenstat4_.jahresumsatz as jahresum2_7_2_, kundenstat4_.monatsumsatz as monatsum3_7_2_, albestellu5_.kundenstatistik_pk_kundenstatistik as kundenst4_14_, albestellu5_.pk_bestellung as pk1_14_, albestellu5_.pk_bestellung as pk1_3_3_, albestellu5_.bestelldatum as bestelld2_3_3_, albestellu5_.fk_kunde as fk3_3_3_, albestellu5_.kundenstatistik_pk_kundenstatistik as kundenst4_3_3_, albestellu5_.fk_zahlungsform as fk5_3_3_, kunde6_.pk_kunde as pk1_6_4_, kunde6_.fk_adresse as fk9_6_4_, kunde6_.geschlaecht as geschlae2_6_4_, kunde6_.level as level6_4_, kunde6_.fk_kundenstatistik as fk10_6_4_, kunde6_.fk_rabatt as fk11_6_4_, kunde6_.nachname as nachname6_4_, kunde6_.passwort as passwort6_4_, kunde6_.titel as titel6_4_, kunde6_.username as username6_4_, kunde6_.vorname as vorname6_4_, rabatt7_.pk_rabatt as pk1_11_5_, rabatt7_.prozent as prozent11_5_, rabatt7_.bezeichner as bezeichner11_5_, zahlungsfo8_.pk_zahlungsform as pk1_14_6_, zahlungsfo8_.fk_bankverbindung as fk2_14_6_, zahlungsfo8_.fk_kreditkarteninfo as fk3_14_6_, zahlungsfo8_.fk_rechnungsadresse as fk4_14_6_, bankverbin9_.pk_bankverbindung as pk1_1_7_, bankverbin9_.blz as blz1_7_, bankverbin9_.bic as bic1_7_, bankverbin9_.iban as iban1_7_, bankverbin9_.inhaber as inhaber1_7_, bankverbin9_.kontonummer as kontonum6_1_7_, kreditkart10_.pk_kreditkarteninfo as pk1_5_8_, kreditkart10_.ablaufdatum as ablaufda2_5_8_, kreditkart10_.inhaber as inhaber5_8_, kreditkart10_.kartennummer as kartennu4_5_8_, kreditkart10_.sicherheitscode as sicherhe5_5_8_, rechnungsa11_.pk_rechnungsadresse as pk1_12_9_, rechnungsa11_.fk_adresse as fk4_12_9_, rechnungsa11_.nachname as nachname12_9_, rechnungsa11_.vorname as vorname12_9_, adresse12_.pk_adresse as pk1_0_10_, adresse12_.fk_plz as fk4_0_10_, adresse12_.hausnummer as hausnummer0_10_, adresse12_.strasse as strasse0_10_, bonitaet13_.pk_bonitaet as pk1_4_11_, bonitaet13_.verzuege as verzuege4_11_, bonitaet13_.zahlungsaufforderungen as zahlungs3_4_11_ from kunde this_ left outer join adresse adresse2_ on this_.fk_adresse=adresse2_.pk_adresse left outer join plz plz3_ on adresse2_.fk_plz=plz3_.pk_plz left outer join kundenstatistik kundenstat4_ on this_.fk_kundenstatistik=kundenstat4_.pk_kundenstatistik left outer join bestellung albestellu5_ on kundenstat4_.pk_kundenstatistik=albestellu5_.kundenstatistik_pk_kundenstatistik left outer join kunde kunde6_ on albestellu5_.fk_kunde=kunde6_.pk_kunde left outer join rabatt rabatt7_ on kunde6_.fk_rabatt=rabatt7_.pk_rabatt left outer join zahlungsform zahlungsfo8_ on albestellu5_.fk_zahlungsform=zahlungsfo8_.pk_zahlungsform left outer join bankverbindung bankverbin9_ on zahlungsfo8_.fk_bankverbindung=bankverbin9_.pk_bankverbindung left outer join kreditkarteninfo kreditkart10_ on zahlungsfo8_.fk_kreditkarteninfo=kreditkart10_.pk_kreditkarteninfo left outer join rechnungsadresse rechnungsa11_ on zahlungsfo8_.fk_rechnungsadresse=rechnungsa11_.pk_rechnungsadresse left outer join adresse adresse12_ on rechnungsa11_.fk_adresse=adresse12_.pk_adresse left outer join bonitaet bonitaet13_ on kundenstat4_.fk_bonitaet=bonitaet13_.pk_bonitaet limit ?
Information: Hibernate: select rabatt0_.pk_rabatt as pk1_11_0_, rabatt0_.prozent as prozent11_0_, rabatt0_.bezeichner as bezeichner11_0_ from rabatt rabatt0_ where rabatt0_.pk_rabatt=?

Vl weiß ja jemand von euch was hier der fehler ist.
schon mal vielen Dank für die HIlfe bis jetzt.

[EDIT]Wenn ich den SQL Code herauskopiere und direkt auf der Datenbank eingebe erhalte ich auch 5 mal den einen User.!!![/EDIT]

 

[moessi91]

Und guck mal wieviele KundenStatistiken/Rabatt Einträge mit dem entsprechenden Kunden verbunden sind. Bei deinem Defaultkonstruktor hab ich eine blöde Vorahnung

Also Kundenstatistiken und Rabatte haben auch alle die selben. Wie gesagt es ist 5x das exakt selbe Objekt mit dem Selben hashwert.
Hier noch die Rabatt klasse:

@Entity
@Table (name="rabatt")
public class Rabatt implements Serializable {
    @Id @GeneratedValue
    @Column (name="pk_rabatt")
    private long ID;
    
    @Column (name="prozent")
    private double lfProzent;
    
    @Column (name="bezeichner")
    private String sBezeichner;

    public Rabatt() {
    }
    
    public Rabatt(double lfProzent, String sBezeichner) {
        this.lfProzent = lfProzent;
        this.sBezeichner = sBezeichner;
    }

    public double getLfProzent() {
        return lfProzent;
    }

    public String getsBezeichner() {
        return sBezeichner;
    }
        
    public long getID() {
        return ID;
    }

    public void setID(long ID) {
        this.ID = ID;
    }

    @Override
    public String toString() {
        return "Rabatt{" + "ID=" + ID + ", lfProzent=" + lfProzent + ", sBezeichner=" + sBezeichner + '}';
    }

 

[SlaterB]

'hibernate criteria multiple results' ist ein Suchbegriff
->
https://forum.hibernate.org/viewtopic.php?t=960677
->
criteria.setResultTransformer(Criteria.DISTINCT_ROOT_ENTITY)

---

auf temporäre Ausblendung der Verknüpfungen verzichtest du?

------

> Also Kundenstatistiken und Rabatte haben auch alle die selben. Wie gesagt es ist 5x das exakt selbe Objekt mit dem Selben hashwert.

wieviele? hat der 5er-Kunde zufällig genau 5 Verknüpfungen

 

[DanZ]

Also Kundenstatistiken und Rabatte haben auch alle die selben. Wie gesagt es ist 5x das exakt selbe Objekt mit dem Selben hashwert.

Ich meinte in der Datenbank, nicht in der geladenen Liste

 

[moessi91]

Ich meinte in der Datenbank, nicht in der geladenen Liste

Kunenstatistik und Rabatte haben jeweils 2 einträge welche auf die jeweiligen klassen verweisen.

 

[moessi91]

Habe das Problem jetzt gelöst durch SlaterB.
Dank mit dem Befehl funktioniert es. Bzw. verstehe ich jetzt auch warum es so aussieht.
Aber das Gelbe vom Ei ist diese Lösung ja eigentlich auch nicht, da man ja zu viele Daten ausliest und dann wieder welche verwirft. Eigentlich sollten ja nur so viele wie benötigt werden ausgelesen werden, oder?
Soll ich das Thema jetzt als erledigt zeichnen oder noch warten ob jemand eine andere Lösung hat?

Mit freundlichen Grüßen
Vincenz

 

[SlaterB]

ein Join ist nunmal ein Join, entweder mehrere Queries oder mehrere Daten parallel,
was Criteria dann zurückgibt ist ne andere Frage