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

  for (; actualSuffix >= prefixIndex && expectedSuffix >= prefixIndex;

    actualSuffix--, expectedSuffix--) {

    if (expected.charAt(expectedSuffix) != actual.charAt(actualSuffix))

      break;

  }

  return expected.length() - expectedSuffix;

}

Также следует изменить имена переменных класса так, чтобы они стали чуть более точными [N1]; в конце концов, обе переменные представляют собой индексы.

Тщательное изучение findCommonSuffix выявляет скрытую временную привязку [G31]; работа функции зависит от того, что значение prefixIndex вычисляется функцией findCommonPrefix. Если вызвать эти две функции в неверном порядке, вам предстоит непростой сеанс отладки. Чтобы эта временная привязка стала очевидной, значение prefixIndex будет передаваться при вызове findCommonSuffix в аргументе.

private void compactExpectedAndActual() {

  prefixIndex = findCommonPrefix();

  suffixIndex = findCommonSuffix(prefixIndex);

  compactExpected = compactString(expected);

  compactActual = compactString(actual);

}

private int findCommonSuffix(int prefixIndex) {

  int expectedSuffix = expected.length() - 1;

  int actualSuffix = actual.length() - 1;

  for (; actualSuffix >= prefixIndex && expectedSuffix >= prefixIndex;

       actualSuffix--, expectedSuffix--) {

    if (expected.charAt(expectedSuffix) != actual.charAt(actualSuffix))

      break;

  }

  return expected.length() - expectedSuffix;

}

Но и такое решение оставляет желать лучшего. Передача аргумента prefixIndex выглядит нелогично [G32]. Она устанавливает порядок вызова, но никоим образом не объясняет необходимость именно такого порядка. Другой программист может отменить внесенное изменение, так как ничто не указывает на то, что этот параметр действительно необходим.

private void compactExpectedAndActual() {

  findCommonPrefixAndSuffix();

  compactExpected = compactString(expected);

  compactActual = compactString(actual);

}

private void findCommonPrefixAndSuffix() {

  findCommonPrefix();

  int expectedSuffix = expected.length() - 1;

  int actualSuffix = actual.length() - 1;

  for (;

       actualSuffix >= prefixIndex && expectedSuffix >= prefixIndex;

       actualSuffix--, expectedSuffix--

    ) {

    if (expected.charAt(expectedSuffix) != actual.charAt(actualSuffix))

      break;

  }

  suffixIndex = expected.length() - expectedSuffix;

}

private void findCommonPrefix() {

  prefixIndex = 0;

  int end = Math.min(expected.length(), actual.length());

  for (; prefixIndex < end; prefixIndex++)

    if (expected.charAt(prefixIndex) != actual.charAt(prefixIndex))

      break;

}

Функции findCommonPrefix и findCommonSuffix возвращаются к прежнему виду, функция findCommonSuffix переименовывается в findCommonPrefixAndSuffix, и в нее включается вызов findCommonPrefix до выполнения каких-либо других действий. Тем самым временная связь двух функций устанавливается гораздо более радикально, чем в предыдущем решении. Кроме того, новое решение со всей очевидностью демонстрирует, насколько уродлива функция findCommonPrefixAndSuffix. Давайте немного почистим ее.

private void findCommonPrefixAndSuffix() {

  findCommonPrefix();

  int suffixLength = 1;

  for (; !suffixOverlapsPrefix(suffixLength); suffixLength++) {

    if (charFromEnd(expected, suffixLength) !=

         charFromEnd(actual, suffixLength))

      break;

  }

  suffixIndex = suffixLength;

}

private char charFromEnd(String s, int i) {