Ошибка в рекурсивной функции

Question

Я пытаюсь получить номер булавки всего 4 цифры. Но цифры, которые меньше 1000, также печатаются. Что происходит в этом коде?

import java.util.Random; 

public class Util { 

    public static int getRandomPINNumber(){ 
     Random randomGenerator = new Random(); 
     int randomNo = randomGenerator.nextInt(10000); 
     if(randomNo < 1000) 
     getRandomPINNumber(); //repeat if it is less than 1000,some bug here 
     return randomNo; 
    } 

    public static void main(String[] args){ 
     for(int i = 0;i<100; i++) 
      System.out.println(getRandomPINNumber()); 
    } 

} 

Выход

6413 
1692 
5734 
105 <--- why 105 is getting printed, it should reevaluate the pin right? 
4857 
6348 
1355 

Answer 1

Проблема заключается в том, что вы не возвращает результат рекурсивного вызова. Изменение вам код:

public static int getRandomPINNumber(){ 
    Random randomGenerator = new Random(); 
    int randomNo = randomGenerator.nextInt(10000); 
    if(randomNo < 1000) 
     return getRandomPINNumber(); //repeat if it is less than 1000 
    return randomNo; 
} 

При вызове функции в первый раз и число меньше 1000 генерируется, вы рекурсивный вызов getRandomPINNumber но игнорировать возвращаемое значение.

Кроме того, вы не должны называть new Random() несколько раз. Вызовите его один раз и сохраните результат.

Answer 2

Три вещи, в порядке возрастания педантичность

1. Вы должны вернуть значение рекурсивной функции: return getRandomPINNumber(); еще вы отбрасывая результат.

2. Не звоните Random randomGenerator = new Random(); несколько раз, иначе вы разрушаете статистические свойства генератора. У вас есть randomGenerator как поле вашего класса или передайте его вашей функции.

3. Никогда не отбрасывайте результаты генератора случайных чисел, иначе вы введете некоторый статистический уклон. (В течение многих лет очень много исследований были развенчаны из-за неправильного использования генератора случайных чисел). В вашем случае вы можете использовать int randomNo = randomGenerator.nextInt(9000) + 1000; и вообще отказаться от отбрасывания.

Answer 3

Только намек, на самом деле вам не нужно использовать рекурсию здесь:

return randomGenerator.nextInt(9000) + 1000; 

является гораздо более простым решением.

Answer 4

Вы должны приложить код, который хотите обработать после инструкции if в фигурных скобках, и назначить результат метода случайному.

int randomNo = randomGenerator.nextInt(10000); 
if (randomNo < 1000) { 
    randomNo = getRandomPINNumber(); //repeat if it is less than 1000,some bug here 
} 
return randomNo; 

Можно также избежать значения, < 1000 с помощью

int randomNo = randomGenerator.nextInt(9000) + 1000; 

Это возвратит число от 1000 до 9999 и является гораздо чище, чем решение рекурсии.

Answer 5

Я сделал код этим путем,

public class Util { 

    static Random randomGenerator = new Random(); 

    public static int getRandomPINNumber(){ 
     int randomNo = randomGenerator.nextInt(9000) + 1000; 
     return randomNo; 
    } 


    public static void main(String[] args){ 
     for(int i = 0;i<100; i++) 
      System.out.println(getRandomPINNumber()); 
    } 

} 

Проводка полностью Буг кода следующего всей информации/советы от других ответов