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
– LeChe, Commented Oct 12, 2010 at 12:06

Jigar Joshi · Accepted Answer · 2010-10-12 12:07:47Z

25

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, 2010 at 12:07

answered Oct 12, 2010 at 12:02

Jigar Joshi

241k42 gold badges409 silver badges446 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

willcodejavaforfood Over a year ago

or rebuild the database which might not be an option

Sean Patrick Floyd Over a year ago

minor issue, it should be Set<Item> items = new HashSet<Item>(list);. Apart from that: +1

matt b Over a year ago

"rebuild the database which might not be an option" - This makes no sense, Query.list() always returns a List: docs.jboss.org/hibernate/stable/core/javadoc/org/hibernate/…

Günay Gültekin Over a year ago

@mattb the link is broken.

Community · Accepted Answer · 2017-05-23 11:46:45Z

3

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).

edited May 23, 2017 at 11:46

CommunityBot

11 silver badge

answered Oct 12, 2010 at 12:34

Pascal Thivent

572k140 gold badges1.1k silver badges1.1k bronze badges

1 Comment

willcodejavaforfood Over a year ago

When I need help with JPA/HSQL you are nowhere to be seen, but you appear! outrageous :)

Collectives™ on Stack Overflow

Hibernate -> ArrayList cannot be cast to Set

2 Answers 2

4 Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

4 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related