diff --git a/forge-game/src/main/java/forge/game/player/Player.java b/forge-game/src/main/java/forge/game/player/Player.java index df9a75eab5e..4813494144d 100644 --- a/forge-game/src/main/java/forge/game/player/Player.java +++ b/forge-game/src/main/java/forge/game/player/Player.java @@ -1608,26 +1608,8 @@ public class Player extends GameEntity implements Comparable { return; } - // overdone but wanted to make sure it was really random - final Random random = MyRandom.getRandom(); - Collections.shuffle(list, random); - Collections.shuffle(list, random); - Collections.shuffle(list, random); - Collections.shuffle(list, random); - Collections.shuffle(list, random); - Collections.shuffle(list, random); - - int s = list.size(); - for (int i = 0; i < s; i++) { - list.add(random.nextInt(s - 1), list.remove(random.nextInt(s))); - } - - Collections.shuffle(list, random); - Collections.shuffle(list, random); - Collections.shuffle(list, random); - Collections.shuffle(list, random); - Collections.shuffle(list, random); - Collections.shuffle(list, random); + // Note: Shuffling once is sufficient. + Collections.shuffle(list, MyRandom.getRandom()); getZone(ZoneType.Library).setCards(getController().cheatShuffle(list));