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

Приватные методы, действие которых распространяется только на небольшое подмножество класса, — хороший признак для поиска потенциальных усовершенствований. Тем не менее основные усилия следует направить на изменение самой системы. Если бы класс Sql выглядел логически завершенным, то нам не пришлось бы беспокоиться о разделении ответственности. Если бы в обозримом будущем функциональность update не понадобилась, Sql можно было бы оставить в покое. Но как только выясняется, что класс необходимо открыть, нужно рассмотреть возможность усовершенствования его структуры.

Почему бы не воспользоваться решением, представленным в листинге 10.10? Для каждого метода открытого интерфейса, определенного в предыдущей версии Sql из листинга 10.9, создается соответствующий класс, производный от Sql. При этом приватные методы (такие, как valuesList) перемещаются непосредственно туда, где они понадобятся. Общее приватное поведение изолируется в паре вспомогательных классов, Where и ColumnList.

Листинг 10.10. Набор закрытых классов

abstract public class Sql {

   public Sql(String table, Column[] columns)

   abstract public String generate();

}

public class CreateSql extends Sql {

   public CreateSql(String table, Column[] columns)

   @Override public String generate()

}

public class SelectSql extends Sql {

   public SelectSql(String table, Column[] columns)

   @Override public String generate()

}

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

public class InsertSql extends Sql {

   public InsertSql(String table, Column[] columns, Object[] fields)

   @Override public String generate()

   private String valuesList(Object[] fields, final Column[] columns)

}

public class SelectWithCriteriaSql extends Sql {

   public SelectWithCriteriaSql(

      String table, Column[] columns, Criteria criteria)

   @Override public String generate()

}

public class SelectWithMatchSql extends Sql {

   public SelectWithMatchSql(

      String table, Column[] columns, Column column, String pattern)

   @Override public String generate()

}

public class FindByKeySql extends Sql

   public FindByKeySql(

      String table, Column[] columns, String keyColumn, String keyValue)

   @Override public String generate()

}

public class PreparedInsertSql extends Sql {

   public PreparedInsertSql(String table, Column[] columns)

   @Override public String generate() {

   private String placeholderList(Column[] columns)

}

public class Where {

   public Where(String criteria)

   public String generate()

}

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

Что не менее важно, когда придет время добавления update, вам не придется изменять ни один из существующих классов! Логика построения команды update реализуется в новом субклассе Sql с именем UpdateSql. Это изменение не нарушит работу другого кода в системе.

Переработанная логика Sql положительна во всех отношениях. Она поддерживает принцип единой ответственности. Она также поддерживает другой ключевой принцип проектирования классов в ООП, называемый принципом открытости/закрытости [PPP]: классы должны быть открыты для расширений, но закрыты для модификации. Наш переработанный класс Sql открыт для добавления новой функциональности посредством создания производных классов, но при внесении этого изменения все остальные классы остаются закрытыми. Новый класс UpdateSql просто размещается в положенном месте.