My experiences and background were similar, and I also had a hard time finding a tutorial that addressed my development preferences (which seem similar to yours):
It would have made so much more sense to me if the available documentation would take a stand and actively push me in this direction instead of giving me a bunch of options and wondering where to go. Lacking that, I learned the following lessons by converting the standard Hibernate Tutorial over to annotations.
- Prefer JPA annotations over XML, only adding Hibernate annotations when the JPA counterparts do not suffice
- Use Hibernate's SessionFactory / Session to access persistent data rather than JPA EntityManager
- Use immutable classes wherever possible
It would have made so much more sense to me if the available documentation would take a stand and actively push me in this direction instead of giving me a bunch of options and wondering where to go. Lacking that, I learned the following lessons by converting the standard Hibernate Tutorial over to annotations.
Configuration
- Keep hibernate.cfg.xml to the bare minimum (see below).
- Define database column / table names explicitly with the JPA annotations so you get exactly the schema you want. You can double-check it by using SchemaExport.create(true, false). Note that you may not get the exact schema you expect if Hibernate tries to update the schema on an existing database (for example, you can't add a NOT NULL constraint to a column that already contains null values).
- Create the SessionFactory's Configuration using AnnotationConfiguration.addAnnotatedClass().addAnnotatedClass()...
org.hsqldb.jdbcDriver jdbc:hsqldb:hsql://localhost sa 1 org.hibernate.dialect.HSQLDialect update thread org.hibernate.cache.NoCacheProvider false
Scope
- Begin/commit/rollback transactions outside each Manager / DAO class. I have no idea why the tutorial's EventManager starts/commits its own transactions, as the book identifies this as being an anti-pattern. For the servlet in the tutorial, this can be done by making a Filter that starts the transaction, invokes the rest of the FilterChain, and then commits/rolls back the transaction.
- Similarly, make the SessionFactory outside and pass it in to each Manager / DAO class, which then calls SessionFactory.getCurrentSession() any time it needs to access data. When it comes time to unit test your DAO classes, make your own SessionFactory that connects to an in-memory database (such as "jdbc:hsqldb:mem:test") and pass it into the DAO being tested.
public final class EventManager { private final SessionFactory sessionFactory; /** Default constructor for use with JSPs */ public EventManager() { this.sessionFactory = HibernateUtil.getSessionFactory(); } /** @param Nonnull access to sessions with the data store */ public EventManager(SessionFactory sessionFactory) { this.sessionFactory = sessionFactory; } /** @return Nonnull events; empty if none exist */ public ListgetEvents() { final Session db = this.sessionFactory.getCurrentSession(); return db.createCriteria(Event.class).list(); } /** * Creates and stores an Event for which no people are yet registered. * @param title Nonnull; see {@link Event} * @param date Nonnull; see {@link Event} * @return Nonnull event that was created */ public Event createEvent(String title, Date date) { final Event event = new Event(title, date, new HashSet ()); this.sessionFactory.getCurrentSession().save(event); return event; } /** * Registers the specified person for the specified event. * @param personId ID of an existing person * @param eventId ID of an existing event */ public void register(long personId, long eventId) { final Session db = this.sessionFactory.getCurrentSession(); final Person person = (Person) db.load(Person.class, personId); final Event event = (Event) db.load(Event.class, eventId); person.addEvent(event); event.register(person); } ...other query / update methods... }
Data classes
- Fields are private
- Getter methods return defensive copies since Hibernate can access the fields directly
- Don't add setter methods unless you really have to.
- When it is necessary to update some field in the class, do it in a way that preserves encapsulation. It's much safer to call something like event.addPerson(person) instead of event.getPeople().add(person)
@Entity(name="EVENT") public final class Event { private @Id @GeneratedValue @Column(name="EVENT_ID") Long id; //Nullable /* Business key properties (assumed to always be present) */ private @Column(name="TITLE", nullable=false) String title; @Column(name="DATE", nullable=false) @Temporal(TemporalType.TIMESTAMP) private Date date; /* Relationships to other objects */ @ManyToMany @JoinTable( name = "EVENT_PERSON", joinColumns = {@JoinColumn(name="EVENT_ID_FK", nullable=false)}, inverseJoinColumns = {@JoinColumn(name="PERSON_ID_FK", nullable=false)}) private Set<Person> registrants; //Nonnull public Event() { /* Required for framework use */ } /** * @param title Non-empty name of the event * @param date Nonnull date at which the event takes place * @param participants Nonnull people participating in this event */ public Event(String title, Date date, Set<Person> participants) { this.title = title; this.date = new Date(date.getTime()); this.registrants = new HashSet<Person> (participants); } /* Query methods */ /** @return Nullable ID used for persistence */ public Long getId() { return this.id; } public String getTitle() { return this.title; } public Date getDate() { return new Date(this.date.getTime()); } /** @return Nonnull people registered for this event, if any. */ public Set<Person> getRegistrants() { return new HashSet<Person> (this.registrants); } /* Update methods */ public void register(Person person) { this.registrants.add(person); } }Hope that helps!
No comments:
Post a Comment