Just change the i to something else (everywhere you wanted that i), another variable is already named 'i' 
Also, the for loop itself is fubbered. It's aim is to cycle through each player, so there's no point defining it as a player

Also, the for loop itself is fubbered. It's aim is to cycle through each player, so there's no point defining it as a player
