> @my? Could you use a more descriptive instance variable name?
> 
Not sure what you are getting at here.  “my” seems like a nice, short choice for “current_player".  I am thinking that all views are from the current_player’s perspective, so personifying that as “my” seems descriptive.  e.g., in a view I have “my” hand (my.hand) and the other hands (@game.player(X).hand), "my" status and the other statuses.


Does that make sense, or am I not seeing your point?

Greg


On Mar 14, 2014, at 12:00 PM, Tim Irwin <notifications@github.com> wrote:

> In app/controllers/games_controller.rb:
> 
> > @@ -0,0 +1,20 @@
> > +class GamesController < ApplicationController
> > +
> > +  def index
> > +    
> > +  end
> > +
> > +
> > +  def show
> > +    # future: if there is a saved game, then restore it
> > +    if @game.nil?
> > +      @game = Game.new()
> > +      @game.add_player(1, "Greg")
> > +      @game.start()
> > +      @my = @game.current_player
> @my? Could you use a more descriptive instance variable name?
> 
> —
> Reply to this email directly or view it on GitHub.
>