用函数简化 if 语句

Question

我正在为学校研讨会做一个日历项目。 它应该打印三个月的时间。 （您指定您希望它打印一月，它打印上一年的十二月、一月和二月）。

我有这个 if 语句，但我想将它简化为三行代码。 我该怎么做？ 是否可以？

if (month == 1) {
    printMonth(12, year-1);
    printMonth(month, year);
    printMonth(month+1, year);
} else if (month == 12) {
    printMonth(month-1, year);
    printMonth(month, year);
    printMonth(1, year+1);
} else {
    printMonth(month-1, year);
    printMonth(month, year);
    printMonth(month+1, year);
}

Answer 1

尝试使用三元运算符：

printMonth(month == 1 ? 12 : month - 1, month == 1 ? year - 1 : year);
printMonth(month, year);
printMonth(month == 12 ? 1 : month + 1, month == 12 ? year + 1 : year);

Answer 2

您可以使用Joda DateTime 类来表示当前的年/月，这可能比将年和月作为独立整数携带更好。

然后你可以写：

printMonth(dt.minusMonths(1));
printMonth(dt);
printMonth(dt.plusMonths(1));

Answer 3

当我自己遇到这样的问题时，我倾向于使用年月结合的方法（也是因为我想节省一些代码行）：

int monthAndYear = year * 12 + month;

printMonth(monthAndYear - 1);
printMonth(monthAndYear);
printMonth(monthAndYear + 1);

如您所见，这当然需要您调整printMonth方法，如果可能的话。

private void printMonth(int monthAndYear) {
    int year = monthAndYear / 12;
    int month = monthAndYear % 12;
    // other code...
}

虽然这些仍然是四行...