递归函数中的错误

克尔蒂瓦桑

我正在尝试获取仅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

Tunaki

问题是您没有返回递归调用的结果。将您的代码更改为：

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()多次。调用一次并存储结果。

本文收集自互联网，转载请注明来源。

如有侵权，请联系 [email protected] 删除。