Would it make more sense for `@current_player` to be a User object?  It reads odd to compare a username with a player.