Чистый код. Создание, анализ и рефакторинг

  public static Day make(int index) throws IllegalArgumentException {

    for (Day d : Day.values())

      if (d.index == index)

        return d;

    throw new IllegalArgumentException(

      String.format("Illegal day index: %d.", index));

  }

  public static Day parse(String s) throws IllegalArgumentException {

    String[] shortWeekdayNames =

      dateSymbols.getShortWeekdays();

    String[] weekDayNames =

      dateSymbols.getWeekdays();

    s = s.trim();

    for (Day day : Day.values()) {

      if (s.equalsIgnoreCase(shortWeekdayNames[day.index]) ||

          s.equalsIgnoreCase(weekDayNames[day.index])) {

        return day;

      }

    }

    throw new IllegalArgumentException(

      String.format("%s is not a valid weekday string", s));

  }

  public String toString() {

    return dateSymbols.getWeekdays()[index];

  }

}

В программе две функции getMonths (строки 288–316); первая функция вызывает вторую. Вторая функция не вызывается никем, кроме первой функцией. Я свернул две функции в одну, что привело к значительному упрощению кода [G9],[G12],[F4]. В завершение я переименовал итоговую функцию, присвоив ей более содержательное имя [N1].

public static String[] getMonthNames() {

  return dateFormatSymbols.getMonths();

}

Функция isValidMonthCode (строки 326–346) потеряла актуальность после введения перечисления Month, поэтому я ее удалил [G9].

Функция monthCodeToQuarter (строки 356–375) отдает ФУНКЦИОНАЛЬНОЙ ЗАВИСТЬЮ [Refactoring]; вероятно, ее логичнее включить в перечисление Month в виде метода с именем quarter. Я выполнил замену.

public int quarter() {

  return 1 + (index-1)/3;

}

В результате перечисление Month стало достаточно большим для выделения в отдельный класс. Я убрал его из DayDate по образцу перечисления Day [G11],[G13].

Следующие два метода называются monthCodeToString (строки 377–426). И снова мы видим, как один метод вызывает своего «двойника» с передачей флага. Обычно передавать флаг в аргументе не рекомендуется, особенно если он просто выбирает формат вывода [G15]. Я переименовал, упростил и реструктурировал эти функции и переместил их в перечисление Month [N1],[N3],[C3],[G14].

public String toString() {

  return dateFormatSymbols.getMonths()[index - 1];

}

public String toShortString() {

  return dateFormatSymbols.getShortMonths()[index - 1];

}

Далее в листинге идет метод stringToMonthCode (строки 428–472). Я переименовал его, переместил в перечисление Month и упростил [N1],[N3],[C3],[G14],[G12].

public static Month parse(String s) {

  s = s.trim();

  for (Month m : Month.values())

    if (m.matches(s))

      return m;

  try {

    return make(Integer.parseInt(s));

  }

  catch (NumberFormatException e) {}

  throw new IllegalArgumentException("Invalid month " + s);

}

private boolean matches(String s) {

  return s.equalsIgnoreCase(toString()) ||

         s.equalsIgnoreCase(toShortString());

}

Метод isLeapYear (строки 495–517) можно сделать более выразительным [G16].

public static boolean isLeapYear(int year) {

  boolean fourth = year % 4 == 0;

  boolean hundredth = year % 100 == 0;

  boolean fourHundredth = year % 400 == 0;

  return fourth && (!hundredth || fourHundredth);

}

Следующая функция, leapYearCount (строки 519–536), не принадлежит DayDate. Она не вызывается никем, кроме двух методов SpreadsheetDate. Я переместил ее в производный класс [G6].

Функция lastDayOfMonth (строки 538–560) использует массив LAST_DAY_OF_MONTH. Этот массив принадлежит перечислению Month [G17], поэтому функция была перемещена. Заодно я упростил ее код и сделал его более выразительным [G16].

public static int lastDayOfMonth(Month month, int year) {