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

    String failure= new ComparisonCompactor(10, "S&P500", "0").compact(null);

    assertEquals("expected:<[S&P50]0> but was:<[]0>", failure);

  }

}

Я провел для ComparisonCompactor анализ покрытия кода на основе этих тестов. В ходе тестирования обеспечивалось 100%-ное покрытие: была выполнена каждая строка кода, каждая команда if и цикл for. Я удостоверился в том, что код работает правильно, а также преисполнился уважения к мастерству его авторов.

Код ComparisonCompactor приведен в листинге 15.2. Не жалейте времени и как следует разберитесь в нем. Вероятно, вы согласитесь с тем, что код достаточно выразителен, обладает логичным разбиением и простой структурой. А когда вы закончите, мы вместе начнем придираться к мелочам.

Листинг 15.2. 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 fContextLength;

  private String fExpected;

  private String fActual;

  private int fPrefix;

  private int fSuffix;

  public ComparisonCompactor(int contextLength,

                             String expected,

                               String actual) {

    fContextLength = contextLength;

    fExpected = expected;

    fActual = actual;

  }

  public String compact(String message) {

    if (fExpected == null || fActual == null || areStringsEqual())

      return Assert.format(message, fExpected, fActual);

    findCommonPrefix();

    findCommonSuffix();

    String expected = compactString(fExpected);

    String actual = compactString(fActual);

    return Assert.format(message, expected, actual);

  }

  private String compactString(String source) {

    String result = DELTA_START +

                      source.substring(fPrefix, source.length() -

                        fSuffix + 1) + DELTA_END;

    if (fPrefix > 0)

      result = computeCommonPrefix() + result;

    if (fSuffix > 0)

      result = result + computeCommonSuffix();

    return result;

  }

  private void findCommonPrefix() {

    fPrefix = 0;

    int end = Math.min(fExpected.length(), fActual.length());

    for (; fPrefix < end; fPrefix++) {

      if (fExpected.charAt(fPrefix) != fActual.charAt(fPrefix))

        break;

    }

  }

  private void findCommonSuffix() {

    int expectedSuffix = fExpected.length() - 1;

    int actualSuffix = fActual.length() - 1;

    for (;

         actualSuffix >= fPrefix && expectedSuffix >= fPrefix;

          actualSuffix--, expectedSuffix--) {

      if (fExpected.charAt(expectedSuffix) != fActual.charAt(actualSuffix))

        break;

    }

    fSuffix = fExpected.length() - expectedSuffix;

  }

  private String computeCommonPrefix() {

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

    return (fPrefix > fContextLength ? ELLIPSIS : "") +

             fExpected.substring(Math.max(0, fPrefix - fContextLength),

                                    fPrefix);

  }

  private String computeCommonSuffix() {

    int end = Math.min(fExpected.length() - fSuffix + 1 + fContextLength,

                         fExpected.length());

    return fExpected.substring(fExpected.length() - fSuffix + 1, end) +

           (fExpected.length() - fSuffix + 1 < fExpected.length() -

            fContextLength ? ELLIPSIS : "");

  }