Question

这是我的代码。

问题：我得到的测试用例输出不正确：a = 100 b = 54。

发现问题：为什么当方法if中的第一个computeGcd条件被调用时（即当a==b或a可以被b整除时），它不是从这个if块返回到main方法中的那一行，从那里调用它？

相反，它将转到方法中的最后一个return语句，并从那里返回旧值'b'。我错过了什么？

public static void main(String[] args) {
    Scanner sc = new Scanner(System.in);
    int a = sc.nextInt();
    int b = sc.nextInt();
    if (a >= b) {
        System.out.println("\n\nfinal values are: " + computeGcd(a, b) 
                + " for a is=" + a + " and b=" + b);}
    else
        System.out.println(computeGcd(b, a));
    sc.close();
}

public static int computeGcd(int a, int b) {
    System.out.println("out side a is=" + a + " and b=" + b);
    if (a == b || a % b == 0) {
        System.out.println("Inside final : a is=" + a + " and b=" + b);
        return b;
    } else {
        a = (a - b * (a / b));
        if (a > b) {
            System.out.println("Inside test a>b : a is=" + a + " and b=" + b);
            computeGcd(a, b);
        }
        if (b > a) {
            System.out.println("Inside test a<b : a is=" + a + " and b=" + b);
            computeGcd(b, a);
        } 
    }
    System.out.println("exiting else");
    System.out.println("i m here :P ");
    return b;
}

调试测试用例：100 54

Answer 1

您的递归电话不是return。

if (a > b) {
    System.out.println("Inside test a>b : a is=" + a + " and b=" + b);
    return computeGcd(a, b); // <-- add return
} else { // if (b > a) {
    System.out.println("Inside test a<b : a is=" + a + " and b=" + b);
    return computeGcd(b, a); // <-- add return
}

<强>替代地

最大可能的gcd是两个术语中最小值的平方根。您可以从该值开始并向后迭代。像，

public static int computeGcd(int a, int b) {
    if (a == b) {
        return a;
    }
    for (int i = (int) Math.sqrt(Math.min(a, b)); i >= 2; i--) {
        if (a % i == 0 && b % i == 0) {
            return i;
        }
    }
    return 1;
}

返回2（对于100, 54），因为54的一半是27，其中3 ³只剩下唯一的共同点2和1。

对于某些测试用例，函数没有返回到正确的return语句

1 个答案: