Проверка дубликатов не всегда работает

Question

У меня есть функция getNormalList(), которая должна возвращать список с 3 случайными целыми числами 0-5, а не все одинаковые.Это не работает, как я хочу, чтобы это работало.Иногда, несмотря на проверку, он выводит те же 3 числа.

public class SpinResultGenerator {

    public ArrayList<Integer> getNormalList() {
        ArrayList<Integer> integerList = new ArrayList<Integer>();
        Random r = new Random();
        int Low = 0;
        int High = 6;
        for (int i = 0; i < 3; i++) {
            int number = r.nextInt(High - Low) + Low;
            integerList.add(number);
        }
        if (integerList.get(0) == integerList.get(1) && integerList.get(0) == integerList.get(2)
                && integerList.get(1) == integerList.get(2)) {
            integerList.clear();
            for (int i = 0; i < 3; i++) {
                int number = r.nextInt(High - Low) + Low;
                integerList.add(number);
            }
        }
        return integerList;
    }

    public ArrayList<Integer> getJackpotList() {
        ArrayList<Integer> integerList = new ArrayList<Integer>();
        integerList.add(5);
        integerList.add(5);
        integerList.add(5);
        return integerList;
    }
}

Если результатом является, например, [4,4,4], цикл for генерирует новые числа.Тем не менее он все еще может вывести 3 одинаковых целых числа.Почему?

Answer 1

Вы можете сделать это примерно так:

public ArrayList<Integer> getNormalList() {
        Set<Integer> resultSet = new HashSet<>(3);
        Random r = new Random();
        int Low = 0;
        int High = 6;
        while (resultSet.size() < 3) {
            resultSet.add(r.nextInt(High - Low) + Low);
        }
        return new ArrayList<>(resultSet);
    }

Answer 2

Ваш текущий код не работает, потому что, если он генерирует список дубликатов, он просто очищает его и генерирует новый список, без проверки, содержит ли новый список дубликаты .Вот мое предлагаемое альтернативное решение:

do {
    List<Integer> result = ThreadLocalRandom.current().ints(0, 6).limit(3).boxed().collect(toList())
} while (result.get(0) == result.get(1) && result.get(0) == result.get(2))
return result;

Answer 3

Вам нужен вложенный цикл.Ваш цикл for должен повторяться до тех пор, пока не будут найдены 3 приемлемых числа.Я использовал массив, чтобы сделать код более компактным, но то же самое можно сделать с ArrayList.

public List<Integer> getNormalList() {
    Random r = new Random();
    int[] arr = new int[3]; // initialized to 0s by default
    int Low = 0;
    int High = 6;
    while (arr[0] == arr[1] && arr[1] == arr[2]) { // will repeat as long as the 3 number as
                                                   // all equal
        for (int i = 0; i < arr.length; i++) {
            arr[i] = r.nextInt(High - Low) + Low;
        }
    }
    return Arrays.asList(arr);
}