程序需要2个数字作为输入，并找到这些数字以及之间的每个数字的阶乘？

Question

这段代码应该采用2个数字，并找出介于这些数字之间的每个数字的阶乘。我没有得到正确的输出，但是无法弄清楚我在做什么错。

Scanner scan = new Scanner(System.in);
long result = 1;
int m = scan.nextInt();
int n = scan.nextInt();
scan.close();

if (n > 0 && m > 0)             //want factorial greater than zero
for(int j = n; j <= m; j++)
{
    for(int i = 1; i <= j; i++)
    {
        result = result * i;    //find factorial
    }
    System.out.println(result);
}
    if(n <= 0 || m <= 0)        //if value is les than zero
    {
        System.out.println("Not Valid!");
    }

Answer 1

类似的东西应该可以工作：

public class RangeFactorial {
    public static void main(String[] args) throws Exception {
        Scanner scan = new Scanner(System.in);
        int max = scan.nextInt();
        int min = scan.nextInt();

        if (max < 0 || min < 0) {
            System.out.println("Invalid Params");
        }

        for (int i = min; i <= max; i++) {
            System.out.println("Factorial for " + i + " is: " + factorial(i));
        }

        scan.close();
    }

    private static int factorial(int i) {
        if (i <= 1) {
            return 1;
        }

        return i * factorial(i-1);
    }
}

请注意，代码假定最大/最小值落在适当的位置，我省略了根据给定输入确定最大/最小值整数的逻辑。您需要添加它。

Answer 2

您忘了将“结果”重置为1。

[此外，如果仅检查第一个语句的取反，则无需使用另一个if语句，只需使用else即可。

我还固定了代码样式指南，以遵循标准的Java指南：

1. 您使用的大括号样式是C / C ++中通常使用的样式。
2. 即使if语句或循环在它们之后仅包含一行，在Java中，无论如何也要使用大括号。

如果想了解更多，请看Google Java Style Guide。

    Scanner scan = new Scanner(System.in);
    long result = 1;
    int m = scan.nextInt();
    int n = scan.nextInt();
    scan.close();

    if (n > 0 && m > 0){
        for(int j = n; j <= m; j++){
            result = 1; //You forgot to reset 'result'
            for(int i = 1; i <= j; i++){
                result *= i;   
            }
        System.out.println(result);
    } else { // No need for another if statement
        System.out.println("Not Valid!");
    }