More of a consistency thing, but you're using the instance variable in the setter and the attributes attr_reader here. I think I would prefer using the instance variable in case the attributes attr_reader is modified.