This is a small thing, but I think I would move this and line 16 into a private method. But since this is state of the object I'm not sure it's a big deal. It's mostly because I try not to use instance variables directly too much.