Fixed: have chosen to consistently use @player for all instances. 
On Mar 14, 2014, at 12:08 PM, Tim Irwin <notifications@github.com> wrote:

> In app/models/game.rb:
> 
> > +    @books_list = {}
> > +    @game_over = false
> > +    @current_player_index = 0
> > +    @game_is_started = false
> > +  end
> > +
> > +  def start(cards = nil)
> > +    return nil if @game_is_started 
> > +    if cards.nil?
> > +      @pond = Deck.new()
> > +      @pond.shuffle
> > +    else
> > +      @pond = Deck.new(cards)
> > +    end
> > +
> > +    @players.each {  |player| @books_list[player] = [] }
> Why are you accessing @players in some places but just players (from attr_reader) in others?
> 
> —
> Reply to this email directly or view it on GitHub.
>