I'm not familiar with this pattern of using an attr_reader in private. Where did you see this before? Could it just be instance variables instead?