Welcome to the Treehouse Community

The Treehouse Community is a meeting place for developers, designers, and programmers of all backgrounds and skill levels to get support. Collaborate here on code errors or bugs that you need feedback on, or asking for an extra set of eyes on your latest project. Join thousands of Treehouse students and alumni in the community today. (Note: Only Treehouse students can comment or ask questions, but non-students are welcome to browse our conversations.)

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and a supportive community. Start your free trial today.

Java Hibernate Basics Persisting Data with Hibernate Retrieving Data with Hibernate

M. Brown
M. Brown
29,923 Points

Okay so this is the best I could do for task two after trying for a few days

I thought I would of naturally had an answer by now.

LanguageRepository.java
import java.util.List;
import java.util.ArrayList;

import org.hibernate.Session;
import org.hibernate.SessionFactory;
import org.hibernate.Criteria;

public class LanguageRepository {
  // Assume sessionFactory has been initialized properly by its default constructor
  @SuppressWarnings("unchecked")
  public SessionFactory {
  private static final SessionFactory sessionFactory = new SessionFactory();

  public List<Language> findAll() {
    // Open session
    Session session = sessionFactory.openSession();

    // TODO: Create Criteria
    Criteria criteria = session.createCriteria(Language.class);

    // TODO: Get a list of all persisted Language entities
    List<Language> languages = criteria.list();

    // Close the session
    session.close();

    // Return the list
    return languages;
  }
  }
}

1 Answer

Samuel Ferree
Samuel Ferree
31,721 Points

Your code is mostly correct. You don't need to wrap the code in an inner class (which you've called SessionFactory) So I removed that, and then moved the suppress warnings annotation down to the findAll() method. If you wanted to suppress warnings for the entire class, you could have also put the annotation above the line 'public class LanguageRepository'

import java.util.List;
import java.util.ArrayList;

import org.hibernate.Session;
import org.hibernate.SessionFactory;
import org.hibernate.Criteria;

public class LanguageRepository {
  private static final SessionFactory sessionFactory = new SessionFactory();

  @SuppressWarnings("unchecked")
  public List<Language> findAll() {
    // Open session
    Session session = sessionFactory.openSession();

    // TODO: Create Criteria
    Criteria criteria = session.createCriteria(Language.class);

    // TODO: Get a list of all persisted Language entities
    List<Language> languages = criteria.list();

    // Close the session
    session.close();

    // Return the list
    return languages;
  }
}