change attr_accessor to attr_reader