变量可能尚未初始化[重复]

Question

对于我的 Comp Sci 课程，我必须编写一个程序来查找给定月份 (1-12) 中的天数，你知道为什么我会收到错误“可变天数可能尚未初始化" 当试图从交换机返回 int "days" 时？代码如下：

public static int getNumberofDays(int month,int year)
   {
      // Imports the required Scanner
      Scanner kbd = new Scanner(System.in);

      final String month;
      final int days;
      switch (month) {
         case 1:  days = 31;
            break;
         case 2:  if ((year % 4 == 0) && year % 100 != 0)
                  {
                     days = 29;
                  }
                  else if ((year % 4 == 0) && (year % 100 == 0) && (year % 400 == 0))
                  {
                     days = 29;
                  }
                  else
                  {
                     days = 28;
                  }
            break;
         case 3:  days = 31;
            break;
         case 4:  days = 30;
            break;
         case 5:  days = 30;
            break;
         case 6:  days = 31;
            break;
         case 7:  days = 31;
            break;
         case 8:  days = 31;
            break;
         case 9:  days = 30;
            break;
         case 10: days = 31;
            break;
         case 11: days = 30;
            break;
         case 12: days = 31;
            break;
         default: month = "invalid";
            break;
      }
      return days;
   }

Answer 1

为什么我收到错误“可能尚未初始化可变天数”

因为：

final int days;

应该是：

int days = 0;

为什么？

• 方法中的变量应该被初始化
• 最终变量无法修改。

---

其他错误：

• 您没有涵盖switch 中month 的所有可能值
• 你定义一个String month，隐藏参数int month
• Scanner 未使用
• 还有更多... XD

---

我的解决方案用你的方式

另外，如果你不破坏一个 case 块，下一个将被执行，这意味着你可以这样做：

public static int getNumberofDays(int month,int year) {
    switch (month) {
       case 1:  
       case 3:  
       case 6:  
       case 7:  
       case 8:
       case 10: 
       case 12: 
             days = 31;
          break;
       case 4:
       case 5:
       case 9:
       case 11: 
             days = 30;
          break;
       case 2:  
          if ((year % 4 == 0) && year % 100 != 0)
          {
             days = 29;
          }
          else if ((year % 4 == 0) && (year % 100 == 0) && (year % 400 == 0))
          {
             days = 29;
          }
          else
          {
             days = 28;
          }
          break;
       default: days = -1;
          break;
    }
    return days;
}

---

我的解决方案：Calendar!:

public static int getNumberofDays(int month,int year)
{
    Calendar cal = Calendar.getInstance();
    cal.set(Calendar.MONTH, month);
    cal.set(Calendar.YEAR, year);
    return cal.getActualMaximum(Calendar.DAY_OF_MONTH); 
}

---

Neil 解决方案：Java-8

public static int getNumberofDays(int month,int year)
{
    YearMonth yearMonthObject = YearMonth.of(year, month); 
    return yearMonthObject.lengthOfMonth();
}

Answer 2

并非所有可能通过您的代码的路径都为 days 变量分配一个值。正是默认块没有。

default: month = "invalid";
           break;

如果你改成

default: month = "invalid";
         days=0
           break;

警告消失。

我的其他问题是：

• 为什么要定义字符串月份，从而遮蔽参数月份？
• 扫描仪有什么用？
• 为什么你在默认块中将invalid分配给month，你永远不会返回它。

Answer 3

1. final String month; 无效，因为 month 作为参数提供给函数。

2. 您不会在所有控制路径上初始化 days：switch 中的 default 块不会这样做。这是 Java 的一个很好的特性，因为它可以帮助您实现程序稳定性。没有什么比 编译器 告诉你该做什么更好的了。 不要通过写final int days = 0; 来强制解决问题；而是在 default 块中为其分配一个值。

Answer 4

将days = 0; 添加到您的默认情况下，如下所示：

default: 
    month = "invalid";
    days = 0;
        break;

在尝试返回之前，您必须在代码路径中的某处给 days 一个值。您还应该将final int days; 替换为int days;。 此外，您不能拥有final String month，因为month 作为参数传递给函数.. 只需删除month 的重新声明即可。