Hibernate und Fehlermeldung

[NatGro]

// neue Transaction
Transaction newtran = newSession.beginTransaction();

List contacts =  newSession.createQuery("from Contact m order by m.id asc").list();

System.out.println( contacts.size() + " message(s) found:" );

// Ausgabe
for ( Iterator iter = contacts.iterator();
iter.hasNext(); ) {
Contact loadedCon = (Contact) iter.next();
System.out.println( loadedCon.getFirstName() + "  ID:" + loadedCon.getId() );
}

Oben seht Ihr den Teil meines Quellcodes. Ich habe ein Hibernate Tutorial angefangen und es klappt auch soweit etwas in die DB zu schreiben und auch wieder auszulesen.

Mein Problem ist, das mir Eclipse meldet:
Iterator is a raw type. References to generic type Iterator<E> should be parameterized
bzw.
List is a raw type. References to generic type Iterator<E> should be parameterized

Ist das ein Problem?

 

[dunhillone]

Das hat nix mit Hibernate zu tun. Vielmehr geht es um Generics, welche du raw verwendest, also unparametrisiert.

Schau mal hier: http://java.sun.com/j2se/1.5/pdf/generics-tutorial.pdf

 

[NatGro]

So etwas hatte ich mir schon gedacht.

Es müsste also ähnlich wie hier aussehen z.B. oder?

List<Integer> myIntList = new LinkedList<Integer>();

Wenn ich das also beispielsweise auf die Zeile

List contacts =  newSession.createQuery("from Contact m order by m.id asc").list();

beziehe, sollte diese dann ja "links" so aussehen, oder?

List<XYZ> contacts =  newSession.createQuery("from Contact m order by m.id asc").list();

Nur weiß ich ja nicht, was mir das Hibernate Framework mit "newSession.createQuery("from Contact m order by m.id asc").list();" zurückgibt!?

 

[SlaterB]

doch, du weißt es, jedenfalls soweit es relevant ist,
Generics ist nur für den Compiler interessant, zur Laufzeit quasi nicht mehr vorhanden,
die Methodensignatur sagt dir, dass eine beliebige Liste zurückkommt, nicht mehr und nicht weniger,

genauso wie du bei get() auf der Liste nach Integer castest, weil du das aus dem Quellcode, aus dem HQL mehr oder weniger erkennst,
genauso müsstest du die Liste manuell zu einer List<wasauchimmer> machen, das kann nicht automatisch funktionieren
weil Hibernate eine beliebige Liste zurückgibt,

ich empfehle, niemals .list() einzeln aufzurufen, sondern über eine statische Methode:

public static <T>List<T> getList(Query query)
{
    return query.list();
}

hier kann man
List<Integer> myIntList = getList(query);
schreiben, der generische Parameter wird im normalen Code passend ohne Fehlermeldung/ Warnung angenommen,
nur in der statischen Methoden wird genau einmal gemeckert

in der Methode kann man dann auch gut loggen, Queries zählen usw.

 

[NatGro]

Also wie gesagt, ich habe kürzlich erst mit Hibernate angefangen.
Wenn ich Dich richtig verstehe, müsste ich also den Befehl "query.list();" in eine separate Methode auslagern, richtig?

Die Bedeutung von "<T>List<T>" verstehe ich ehrlich gesagt nicht ganz.
???:L

 

[SlaterB]

du musst sie nicht auslagern und <T>List<T> auch nicht verstehen, wenn du aber beides verwendest kannst du
List<Integer> myIntList = getList(query);
schreiben, wie gesagt

anderenfalls nicht, dann hast du erstmal nur eine List,
wenn du sie wie in deinem Code ganz oben verarbeitest bekommst du Warnings,
ein echtes Problem sind sie nicht unbedingt wenn du Generics nicht verwenden willst, kann man auch abschalten

> Also wie gesagt, ich habe kürzlich erst mit Hibernate angefangen.

hat absolut nichts mit Hibernate zu tun, außer das Hibernate dir eine einfache List liefert,
das kann jedes 10-Zeilen-Programm auch

 

[dunhillone]

Die Bedeutung von "<T>List<T>" verstehe ich ehrlich gesagt nicht ganz.
???:L

Es geht ihm darum, die Warnung, welche vom Java Compiler ausgegeben wird, in eine statische Methode auszulagern. Manche stören sich auch in der IDE an der Warnung. Es gibt jedoch auch noch die Möglichkeit, die Warnung für eine Methode auszuschalten.

@SuppressWarnings("unchecked")
public void meineMethode() {

dann stört sie auch nicht mehr..

 

[NatGro]

Es geht ihm darum, die Warnung, welche vom Java Compiler ausgegeben wird, in eine statische Methode auszulagern. Manche stören sich auch in der IDE an der Warnung. Es gibt jedoch auch noch die Möglichkeit, die Warnung für eine Methode auszuschalten.

@SuppressWarnings("unchecked")
public void meineMethode() {

dann stört sie auch nicht mehr..

Also das bedeutet in meinem Falle, das ich diese Warnung theoretisch einfach ignorieren kann,
weil ich ja weiß das ich eine "Irgendwas"-Liste zurückbekomme, oder?

Die Warnung ist also eher für den Programmierer gedacht, damit es nicht so schnell Probleme mit unkompatiblen Typen gibt?

 

[NatGro]

du musst sie nicht auslagern und <T>List<T> auch nicht verstehen, wenn du aber beides verwendest kannst du
List<Integer> myIntList = getList(query);
schreiben, wie gesagt

anderenfalls nicht, dann hast du erstmal nur eine List,
wenn du sie wie in deinem Code ganz oben verarbeitest bekommst du Warnings,
ein echtes Problem sind sie nicht unbedingt wenn du Generics nicht verwenden willst, kann man auch abschalten

> Also wie gesagt, ich habe kürzlich erst mit Hibernate angefangen.

hat absolut nichts mit Hibernate zu tun, außer das Hibernate dir eine einfache List liefert,
das kann jedes 10-Zeilen-Programm auch

Könntest Du mir ein kurzes Beispiel coden und hier posten bitte?
Bin wie gesagt ziemlicher Anfänger.

 

[SlaterB]

wofür denn ein Beispiel?

für ein 10 Zeilen-Programm dass nicht Hibernate heißt und dich vor dasselbe Problem stellt?
gerne

public class Test {

public List test() {
  return new ArrayList();
}

}

 

[NatGro]

wofür denn ein Beispiel?

für ein 10 Zeilen-Programm dass nicht Hibernate heißt und dich vor dasselbe Problem stellt?
gerne

public class Test {

public List test() {
  return new ArrayList();
}

}

Naja, ich dachte da mehr an das "Gegenstück" dazu.
Also das, was die "List" dann entgegen nimmt sozusagen.

 

[SlaterB]

List l = test();
wie du es in deinem Programm doch auch hast

 

[NatGro]

doch, du weißt es, jedenfalls soweit es relevant ist,
...
ich empfehle, niemals .list() einzeln aufzurufen, sondern über eine statische Methode:

public static <T>List<T> getList(Query query)
{
    return query.list();
}

hier kann man
List<Integer> myIntList = getList(query);
schreiben, der generische Parameter wird im normalen Code passend ohne Fehlermeldung/ Warnung ...

Oben hattest Du es etwas anders vorgeschlagen, so meinte ich das.
Wir würde das jetzt hier konkret aussehen müssen!?

 

[SlaterB]

so?

public class Test {
    public static void main(String[] args)  {
        Test t = new Test();

        // Warning: using raw type, Casten nötig
        List l1 = t.test();

        // nötige Umwandlung, durch SuppressWarnings gerechtfertigt
        @SuppressWarnings("unchecked")
        List<Integer> l2 = t.test();

        @SuppressWarnings("unchecked")
        List<Integer> l3 = t.test();

        @SuppressWarnings("unchecked")
        List<Integer> l4 = t.test();

        // muss man jedesmal schreiben, tausende Male in einem großen Programm
        @SuppressWarnings("unchecked")
        List<Integer> l5 = t.test();


        // kein SuppressWarnings mehr nötig, nur einmal in der statischen Methode
        List<Integer> l6 = getTestList(t);
        List<Integer> l7 = getTestList(t);
        List<Integer> l8 = getTestList(t);
        List<Integer> l9 = getTestList(t);
    }


    @SuppressWarnings("unchecked")
    public static <T>List<T> getTestList(Test t) {
        return t.test();
    }

    public List test() {
        return new ArrayList();
    }
}

 

[fastjack]

Parametrisiere doch einfach den Iterator und die Listen mit den Objekten vom Typ Contact die da vorhanden sind.

Iterator<Contact>, List<Contact>

Dann sparst Du Dir die casterei auf Contact.

Von den ganzen @SuppressWarnings halte ich persönlich nichts.

 

[NatGro]

Parametrisiere doch einfach den Iterator und die Listen mit den Objekten vom Typ Contact die da vorhanden sind.

Iterator<Contact>, List<Contact>

Dann sparst Du Dir die casterei auf Contact.

Von den ganzen @SuppressWarnings halte ich persönlich nichts.

Du meinst sicherlich so in etwa, oder?
(Hinweis: Contact ist eine eigene Klasse für meine gespeicherten Personen)

List <Contact> contacts =  newSession.createQuery("from Contact m order by m.id asc").list();

Nun, das habe ich versucht, das Problem ist allerdings, das ich ja "rechts" auch etwas angeben müsste!?
So wie hier z.B.

List<Integer> myIntList = new LinkedList<Integer>();

Ich wüsste jedoch nicht wie ich das bei

List contacts =  newSession.createQuery("from Contact m order by m.id asc").list();

mit der richtigen Syntax schreiben soll und außerdem kommt ja von "newSession.createQuery("from Contact m order by m.id asc").list();" soweit ich weiß nicht nur etwas vom Typ Contact zurück!?

 

[SlaterB]

wie anfangs schon geschrieben:
welcher Art die Objekte in der Liste sind hängt vom HQL-String ab,
das ist pures Raten im Quellcode, welches zur ClassCastException führen kann, egal wie man es programmiert,
dieser unsichere Schritt ist unumgänglich


ansonsten ist das genau die Situation wie in meinem Beispiel:
> List <Contact> contacts = newSession.createQuery("...").list();
entspricht
> List<Integer> l2 = t.test();
mit nötigen SuppressWarnings, falls die Warnmeldung eingeschaltet ist (man kann sie natürlich auch einfach immer anzeigen lassen)

und wie vorgeschlagen ginge
> List<Contact> contacts = getList(newSession.createQuery("..."));
ohne jede Warning, wenn nur eine entsprechende Methode existiert,
was darin <T>List<T> bedeutet ist eher müßig zu erklären, es sorgt einfach dafür dass es geht

 

[fastjack]

dazu schreibst Du die Methode wie SlaterB sie erwähnte, dann paßt der Lack!

 

[NatGro]

Ich danke Euch für Eure ganzen Tips und werde es dann mal so probieren.
Wirklich ein super Forum hier! Großes Lob an dieser Stelle!