change to attr_accessor