change attr_reader to attr_accessor