or even use attr_accessor?