你能簡化這個 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;
    }
}