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

          ORD = ORD + 1;

          SQUARE = P[ORD] * P[ORD];

          MULT[ORD - 1] = J;

        }

        N = 2;

        JPRIME = true;

        while (N < ORD && JPRIME) {

          while (MULT[N] < J)

            MULT[N] = MULT[N] + P[N] + P[N];

          if (MULT[N] == J)

            JPRIME = false;

          N = N + 1;

        }

      } while (!JPRIME);

      K = K + 1;

      P[K] = J;

    }

    {

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

      PAGENUMBER = 1;

      PAGEOFFSET = 1;

      while (PAGEOFFSET <= M) {

        System.out.println("The First " + M +

                             " Prime Numbers --- Page " + PAGENUMBER);

        System.out.println("");

        for (ROWOFFSET = PAGEOFFSET; ROWOFFSET < PAGEOFFSET + RR; ROWOFFSET++){

          for (C = 0; C < CC;C++)

            if (ROWOFFSET + C * RR <= M)

              System.out.format("%10d", P[ROWOFFSET + C * RR]);

          System.out.println("");

        }

        System.out.println("\f");

        PAGENUMBER = PAGENUMBER + 1;

        PAGEOFFSET = PAGEOFFSET + RR * CC;

      }

    }

  }

}

Записанная в виде одной функции, эта программа представляет собой полную неразбериху. Многоуровневая вложенность, множество странных переменных, структура с жесткой привязкой… По крайней мере, одну большую функцию следует разбить на несколько меньших функций.

В листингах 10.6–10.8 показано, что получается после разбиения кода из листинга 10.5 на меньшие классы и функции, с выбором осмысленных имен для классов, функций и переменных.

Листинг 10.6. PrimePrinter.java (переработанная версия)

package literatePrimes;

public class PrimePrinter {

  public static void main(String[] args) {

    final int NUMBER_OF_PRIMES = 1000;

    int[] primes = PrimeGenerator.generate(NUMBER_OF_PRIMES);

    final int ROWS_PER_PAGE = 50;

    final int COLUMNS_PER_PAGE = 4;

    RowColumnPagePrinter tablePrinter =

      new RowColumnPagePrinter(ROWS_PER_PAGE,

                               COLUMNS_PER_PAGE,

                               "The First " + NUMBER_OF_PRIMES +

                               " Prime Numbers");

    tablePrinter.print(primes);

  }

}

Листинг 10.7. RowColumnPagePrinter.java

package literatePrimes;

import java.io.PrintStream;

public class RowColumnPagePrinter {

  private int rowsPerPage;

  private int columnsPerPage;

  private int numbersPerPage;

  private String pageHeader;

  private PrintStream printStream;

  public RowColumnPagePrinter(int rowsPerPage,

                              int columnsPerPage,

                              String pageHeader) {

    this.rowsPerPage = rowsPerPage;

    this.columnsPerPage = columnsPerPage;

    this.pageHeader = pageHeader;

    numbersPerPage = rowsPerPage * columnsPerPage;

    printStream = System.out;

  }

  public void print(int data[]) {

    int pageNumber = 1;

    for (int firstIndexOnPage = 0;

         firstIndexOnPage < data.length;

         firstIndexOnPage += numbersPerPage) {

      int lastIndexOnPage =

        Math.min(firstIndexOnPage + numbersPerPage - 1,

                 data.length - 1);

      printPageHeader(pageHeader, pageNumber);

      printPage(firstIndexOnPage, lastIndexOnPage, data);

      printStream.println("\f");

      pageNumber++;

    }

  }

  private void printPage(int firstIndexOnPage,

                         int lastIndexOnPage,

                         int[] data) {

    int firstIndexOfLastRowOnPage =

      firstIndexOnPage + rowsPerPage - 1;

    for (int firstIndexInRow = firstIndexOnPage;

         firstIndexInRow <= firstIndexOfLastRowOnPage;

         firstIndexInRow++) {