Oh and your attr_accessors. But should those be attr_reader accessors only?