Shouldn't the dice be attached to the board, not the player? It's an action performed for the game by a player, not performed for the player... eh, whatever.
In any case, the problem about the looping is that he's not actually doing looping in the right place - he needs to put curlies after the for and after the last System.out, if he wants both of those.
_________________________
UBB.classic: Love it or hate it, it was mine.