My general opinion is to use the instance variables in the construction and then user the attr_reader elsewhere.