Did not choose to arrays to concatenate, but refactored to use more explanatory names.
On Mar 14, 2014, at 12:24 PM, Tim Irwin <notifications@github.com> wrote:

> In app/models/result.rb:
> 
> > +      part2 = "He did not make a book."
> > +    else
> > +      if surprise_rank.nil?
> > +        part2 = "He made a book of #{rank}s."
> > +      else
> > +        part2 = "He was surprised to make a book of #{surprise_rank}s."
> > +      end
> > +    end
> > +
> > +    if game_over
> > +      part3 = "\nThe Game is now over"
> > +    else
> > +      part3 = ""
> > +    end
> > +
> > +   part0 + part1 + "\n" + part2 + part3
> Consider: [part1, part2, part3].reject(&:blank?).join("\n")
> 
> —
> Reply to this email directly or view it on GitHub.
>