Here you are using local variables for player1 and player2 rather than the instance variables you initialized. I wouldn't think that would work but I guess it does.