This should probably be attr_reader (you don't want other things setting this, do you?)