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

Стало гораздо лучше! Теперь мы видим, что функция compactString просто соединяет фрагменты строки. Вероятно, этот факт можно сделать еще более очевидным. Осталось еще много мелких улучшений, которые можно было бы внести в код. Но я не стану мучить вас подробными описаниями остальных изменений и просто приведу окончательный результат в листинге 15.5.

Листинг 15.5. ComparisonCompactor.java (окончательная версия)

package junit.framework;

public class ComparisonCompactor {

  private static final String ELLIPSIS = "...";

  private static final String DELTA_END = "]";

  private static final String DELTA_START = "[";

  private int contextLength;

  private String expected;

  private String actual;

Листинг 15.5 (продолжение)

  private int prefixLength;

  private int suffixLength;

  public ComparisonCompactor(

    int contextLength, String expected, String actual

  ) {

    this.contextLength = contextLength;

    this.expected = expected;

    this.actual = actual;

  }

  public String formatCompactedComparison(String message) {

    String compactExpected = expected;

    String compactActual = actual;

    if (shouldBeCompacted()) {

      findCommonPrefixAndSuffix();

      compactExpected = compact(expected);

      compactActual = compact(actual);

    }

    return Assert.format(message, compactExpected, compactActual);

  }

  private boolean shouldBeCompacted() {

    return !shouldNotBeCompacted();

  }

  private boolean shouldNotBeCompacted() {

    return expected == null ||

           actual == null ||

           expected.equals(actual);

  }

  private void findCommonPrefixAndSuffix() {

    findCommonPrefix();

    suffixLength = 0;

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

      if (charFromEnd(expected, suffixLength) !=

          charFromEnd(actual, suffixLength)

      )

        break;

    }

  }

  private char charFromEnd(String s, int i) {

    return s.charAt(s.length() - i - 1);

  }

  private boolean suffixOverlapsPrefix() {

    return actual.length() - suffixLength <= prefixLength ||

      expected.length() - suffixLength <= prefixLength;

  }

  private void findCommonPrefix() {

    prefixLength = 0;

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

    for (; prefixLength < end; prefixLength++)

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

        break;

  }

  private String compact(String s) {

    return new StringBuilder()

      .append(startingEllipsis())

      .append(startingContext())

      .append(DELTA_START)

      .append(delta(s))

      .append(DELTA_END)

      .append(endingContext())

      .append(endingEllipsis())

      .toString();

  }

  private String startingEllipsis() {

    return prefixLength > contextLength ? ELLIPSIS : "";

  }

  private String startingContext() {

    int contextStart = Math.max(0, prefixLength - contextLength);

    int contextEnd = prefixLength;

    return expected.substring(contextStart, contextEnd);

  }

  private String delta(String s) {

    int deltaStart = prefixLength;

    int deltaEnd = s.length() - suffixLength;

    return s.substring(deltaStart, deltaEnd);

  }