java - 由于 return 语句，我无法理解哪些代码无法编译

Question

import java.util.Scanner;
public class Test{
   public static int[] calculateFactors(int n){
      int[] array = new int[n];
      for(int i = 0; i<=n;i++){
         if(n%i==0){
            return array[i];
         }
      }
   }

   public static void main(String[] args){
      System.out.println("Please enter the number you want to find the factors for:");
      Scanner input = new Scanner(System.in);
      int n = input.nextInt();
      System.out.println(calculateFactors(n));
   }
}

我不明白为什么会出现错误。我正在尝试编写一个代码，用户输入一个数字，程序返回这个数字的所有因素。谁能告诉我为什么我无法返回具有相应 i 因子的数组。

谢谢

Answer 1

查看您的代码(为清楚起见添加了空格):

public static int[] calculateFactors(int n) {
    int[] array = new int[n];
    for(int i = 0; i <= n; i++) {
        if(n % i == 0) {
            return array[i];
        }
    }
}

有两个问题:

• 您的返回语句与声明的返回类型不匹配。您正在尝试返回 int ，但您的方法声明它返回一个 int[]
• 由于您的方法是非空的，因此方法的结尾必须是不可到达的...您必须通过任何可能的路径返回一个值。您的代码不是这种情况。如果 n 会发生什么是-1？它永远不会经历循环，所以它永远不会返回。更一般地说，编译器将假定(几乎)任何 if条件可能为假。

现在，至于如何修复它 - 给定方法名称，听起来您确实确实想要返回所有因素...所以您不应该一返回就返回我找到了第一个。我会在 List<Integer> 中收集这些因素而不是在数组中。转换 return声明 add调用，然后在循环结束时返回。哦，你可能想用 i 开始循环作为 1 而不是 0...

不过，所有这些都是针对这个特定问题的。更一般地说，您应该查看编译器给您的错误。他们应该为您提供您自己解决此问题所需的所有信息。既然您确实理解了问题，请再次阅读消息并尝试弄清楚是什么阻止了您之前理解问题。您能否自己解决这个问题，或者您是否遗漏了一些重要的概念？