java - 哪种实现更好(DRY 和 KISS)

Question

关闭。这个问题是 opinion-based 。目前不接受答案。

---

想要改进这个问题吗？更新问题，以便 editing this post 可以用事实和引文来回答它。 .

已关闭 4 年前。

Improve this question

下面我有返回月份名称的方法。在第一个实现中，我使用 switch/case，此方法较长，验证位于最后一行。在第二个中，我在第一行进行验证，而不是使用 switch/case 来声明带有月份名称的表。

当我考虑 KISS 和 DRY 原则时，哪一个更好？

public String getMonthName(int month) {
    switch (month) {
        case 1:
            return "January";
        case 2:
            return "February";
        case 3:
            return "March";
        case 4:
            return "April";
        case 5:
            return "May";
        case 6:
            return "June";
        case 7:
            return "July";
        case 8:
            return "August";
        case 9:
            return "September";
        case 10:
            return "October";
        case 11:
            return "November";
        case 12:
            return "December";
        default:
            throw new IllegalArgumentException("month must be in range 1 to 12");
    }
}

或者也许是这个？

public String getMonthNameNew(int month) {
    if ((month < 1) || (month > 12)) throw new IllegalArgumentException("month must be in range 1 to 12");
    String[] months = {
            "January",
            "February",
            "March",
            "April",
            "May",
            "June",
            "July",
            "August",
            "September",
            "October",
            "November",
            "December"
    };
    return months[month - 1];
}

Answer 1

我发现第二个更容易阅读。它更短，并且带有前提条件检查，可以立即告诉您允许哪些值。在第一个示例中，您必须浏览整个方法主体才能理解这一点。

综上所述，该方法应该使用 java.time.Month 编写如:

public String getMonthNameNew(int month) {
  return Month.of(month).getDisplayName(TextStyle.FULL, Locale.ENGLISH);
}