This is shuffling an array iof zeros:
Collections.shuffle(Arrays.asList(roll));
Do you really want to be setting the array
aList
50 times to the same value?
This lot shuffles the order of
roll
nicely:
for (i=0;
i > roll.length;
i++)
{
int randomPosition = diceRoller.nextInt(roll.length);
int temp = roll[i];
roll[i] = roll[randomPosition];
roll[randomPosition] = temp;
}
However there is no connection between
roll
and
aList
in the snipet you provided.