你能简化这个 if 语句吗？

Question

我希望以某种方式简化以下内容： monthNbr==11||monthNbr==4||monthNbr==6||monthNbr==9这里：

public int daysPerMonth (int monthNbr){
    if(monthNbr==11||monthNbr==4||monthNbr==6||monthNbr==9){
        return 30;
    } else if (monthNbr==2) {
        return 28;
    } else {
        return 31;
    }
}

Answer 1

也许使用新的 switch 表达式会更好看

public int daysPerMonth(int monthNbr) {
    return switch (monthNbr) {
        case 11, 4, 6, 9 -> 30;
        case 2 -> 28;
        default -> 31;
    };
}

Answer 2

不要重新发明轮子：

Month.of(monthNbr).length(isLeapYear)

Answer 3

由于 Java 8，您还可以这样做：

public int daysPerMonth(int monthNbr) {
    return YearMonth.of(Year.now().getValue(),monthNbr).lengthOfMonth();
}

这将为您提供当年的月数。

另外，请注意， Switch Expressions只能在 Java 12 之后使用。

Answer 4

这是一个较小的版本，它也应该很容易扩展以支持闰年。 如果您想要闰年，请发送年份或发送 2000 年。

import java.util.*; 
import java.time.*;

public class HelloDays
{
     public static int daysPerMonth(int monthNbr)
     {
        YearMonth yearMonthObject = YearMonth.of(1999, monthNbr);
        return yearMonthObject.lengthOfMonth();  
     }
     
     public static void main(String []args)
     {
        System.out.println(daysPerMonth(1));  // 31
        System.out.println(daysPerMonth(2));  // 28
        System.out.println(daysPerMonth(4));  // 30
        
     }
}

Answer 5

您可以像这样使用 stream 。

public int daysPerMonth(int monthNbr) {
    if (IntStream.of(11, 4, 6, 9).anyMatch(i -> monthNbr == i)) {
        return 30;
    } else if (monthNbr == 2) {
        return 28;
    } else {
        return 31;
    }
}