Should this be private?  It looks like all the relevant fields are set in the paramaterized constructor without getters and setters.