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

      for (ITestNGMethod m : tr.getAfterSuiteMethods()) {

        afterSuiteMethods.put(m.getMethod(), m);

      }

    }

...

Переменные экземпляров, напротив, должны объявляться в начале класса. Это не увеличивает вертикальное расстояние между переменными, потому что в хорошо спроектированном классе они используются многими, если не всеми, методами класса.

Размещение переменных экземпляров становилось причиной ожесточенных споров. В C++ обычно применялось так называемое правило ножниц, при котором все переменные экземпляров размещаются внизу. С другой стороны, в Java они обычно размещаются в начале класса. Я не вижу причин для использования каких-либо других конвенций. Здесь важно то, что переменные экземпляров должны объявляться в одном хорошо известном месте. Все должны знать, где следует искать объявления.

Для примера рассмотрим странный класс TestSuite из JUnit 4.3.1. Я основательно сократил этот класс, чтобы лучше выразить свою мысль. Где-то в середине листинга вдруг обнаруживаются объявления двух переменных экземпляров. Если бы автор сознательно хотел спрятать их, трудно найти более подходящее место. Читатель кода может наткнуться на эти объявления только случайно (как я).

public class TestSuite implements Test {

  static public Test createTest(Class theClass,

                                String name) {

     ...

  }

  public static Constructor

    getTestConstructor(Class theClass)

    throws NoSuchMethodException {

  ...

  }

  public static Test warning(final String message) {

  ...

  }

  private static String exceptionToString(Throwable t) {

  ...

  }

  private String fName;

  private Vector fTests= new Vector(10);

  public TestSuite() {

  }

  public TestSuite(final Class theClass) {

   ...

  }

  public TestSuite(Class  theClass, String name) {

  ...

  }

  ... ... ... ... ...

}

Зависимые функции. Если одна функция вызывает другую, то эти функции должны располагаться вблизи друг от друга по вертикали, а вызывающая функция должна находиться над вызываемой (если это возможно). Тем самым формируется естественная структура программного кода. Если это правило будет последовательно соблюдаться, читатели кода будут уверены в том, что определения функций следуют неподалеку от их вызовов. Для примера возьмем фрагмент FitNesse из листинга 5.5. Обратите внимание на то, как верхняя функция вызывает нижние, и как они, в свою очередь, вызывают функции более низкого уровня. Такая структура позволяет легко найти вызываемые функции и значительно улучшает удобочитаемость всего модуля.

Листинг 5.5. WikiPageResponder.java

public class WikiPageResponder implements SecureResponder {

  protected WikiPage page;

  protected PageData pageData;

  protected String pageTitle;

  protected Request request;

  protected PageCrawler crawler;

  public Response makeResponse(FitNesseContext context, Request request)

    throws Exception {

    String pageName = getPageNameOrDefault(request, "FrontPage");

    loadPage(pageName, context);

    if (page == null)

      return notFoundResponse(context, request);

    else

      return makePageResponse(context);

  }

  private String getPageNameOrDefault(Request request, String defaultPageName)

  {

    String pageName = request.getResource();

    if (StringUtil.isBlank(pageName))

      pageName = defaultPageName;

    return pageName;

  }

  protected void loadPage(String resource, FitNesseContext context)

    throws Exception {

    WikiPagePath path = PathParser.parse(resource);

    crawler = context.root.getPageCrawler();

    crawler.setDeadEndStrategy(new VirtualEnabledPageCrawler());

    page = crawler.getPage(context.root, path);

    if (page != null)