Hibernate -> ArrayList cannot be cast to Set

Question

I have a Java EE application and I use Hibernate. The domain objects, I changed the List / ArrayList to Set / HashSet, because it is better to use Sets.

But in my Dao implementation I run into a problem:

public Set<Person> getAllPersons() {
    SessionFactory sessionFactory = HibernateUtil.getSessionFactory();
    Session sess = sessionFactory.getCurrentSession();

    Transaction tx = sess.beginTransaction();
    @SuppressWarnings("unchecked")
    Set<Item> items = (Set<Item>) sess.createQuery("from Item").list();
    tx.commit();

    return items;
}

Here I get an error:

java.lang.ClassCastException: java.util.ArrayList cannot be cast to java.util.Set

What can I do to avoid this error?

Thank you in advance & Best Regards.

Are you sure that you know what the difference between a set and a list is? Because in no way is it 'better' to use a Set. You use Sets whenever you want a single instance of an object in your collection, whereas you can have multiple instances of the same object in a list. — LeChe, Oct 12 '10 at 12:06

jmj · Accepted Answer · 2010-10-12T12:07:47.427

23

List<Item> list = sess.createQuery("from Item").list();
Set<Item> items = new HashSet<Item>(list);

sess.createQuery("from Item").list(); will return a array list of resulting items, if you need it in Set you can make it as shown in code

edited Oct 12 '10 at 12:07

answered Oct 12 '10 at 12:02

jmj

225,392
41
383
426

or rebuild the database which might not be an option – willcodejavaforfood Oct 12 '10 at 12:03
minor issue, it should be `Set items = new HashSet(list);`. Apart from that: +1 – Sean Patrick Floyd Oct 12 '10 at 12:07
"rebuild the database which might not be an option" - This makes no sense, `Query.list()` always returns a `List`: http://docs.jboss.org/hibernate/stable/core/javadoc/org/hibernate/Query.html#list() – matt b Oct 12 '10 at 12:37
@mattb the link is broken. – Günay Gültekin Nov 12 '13 at 12:32

score 3 · Answer 2 · edited May 23 '17 at 11:46

The domain objects, I changed the List / ArrayList to Set / HashSet, because it is better to use Sets.

It's not "better" to use Set, it depends entirely on what you need. But anyway, the collection type you use in your domain objects and the return type of Query#list() are two unrelated things.

Personally, I can't say that I see much value in converting the result of a query into a Set (apart from eating more memory), unless you explicitly want to remove duplicates from query results of course (but in that case, I would challenge the mappings).

Now, if you insist, the various Set implementations have a constructor that takes a Collection (and the question is essentially a duplicate of Easiest way to convert a List to a Set? - Java).

When I need help with JPA/HSQL you are nowhere to be seen, but you appear! outrageous :) — willcodejavaforfood, Oct 12 '10 at 12:39

Hibernate -> ArrayList cannot be cast to Set

2 Answers2

Linked