将 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);
    }
}

请注意，代码假定 max/min 就位，我省略了根据给定输入确定 max/min 整数的逻辑。您需要添加此内容。

Answer 2

您忘记将 'result' 重置为 1。

此外，如果它只是检查第一个的否定，则不需要另一个 if 语句，只需使用 else。

我还修改了代码样式指南以遵循标准 Java 指南：

您使用的大括号样式是C/C++中常用的样式。
即使 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!");
    }

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

Program that takes 2 numbers as an input and finds the factorials of those numbers and every number in between?

java

computer-science

loops

factorial

java.util.scanner