Рефакторинг к паттернам

Паттерн — это пункт назначения, а не стартовая точка. Каталог «Банды четырёх» описывает повторяющиеся решения; он не говорит тебе, когда проблема присутствует. Переосмысление Кериевски — «рефакторинг к паттернам» — трактует каждый паттерн как цель серии маленьких рефакторингов, сохраняющих поведение, применяемых только тогда, когда код демонстрирует запах, который этот паттерн растворяет. Ты не решаешь «здесь нужна Стратегия» на дизайн-митинге; ты замечаешь условный оператор, который продолжает расти вдоль одной оси, и именно это наблюдение делает Стратегию правильным ходом. Паттерн заслуживает своё место, убирая конкретную боль, а не появляясь в учебнике.

Практическое следствие: ты никогда не пишешь interface PaymentStrategy в первый день. Ты пишешь самое простое, что работает, и позволяешь второму и третьему способу оплаты показать тебе дублирование, прежде чем извлекать абстракцию.

Дублированная логика создания тянет тебя к Фабрике. Когда одно и то же многошаговое создание объекта появляется в нескольких местах вызова — и особенно когда создаваемый тип зависит от значения времени выполнения — запах состоит в дублированном, ветвящемся создании. Рефакторинг, который его убирает, приземляет тебя на фабрику.

// SMELL: the same "which class?" decision is duplicated at every call site
const a = type === "pdf" ? new PdfReport(data) : new CsvReport(data);
// ...elsewhere, again:
const b = kind === "pdf" ? new PdfReport(other) : new CsvReport(other);

Решение «какой подкласс Report» скопировано через копипасту. Централизуй его:

function createReport(type: string, data: Data): Report {
  if (type === "pdf") return new PdfReport(data);
  return new CsvReport(data);
}
const a = createReport(type, data);
const b = createReport(kind, other);

Теперь новый формат — это одна правка внутри createReport; вызывающие зависят от концепции «сделать отчёт», а не от конкретных классов. Ты не планировал Фабрику — ты убрал дублирование, и Фабрика — это то, что произвело устранение дублирования.

Условный оператор, растущий вдоль одной оси, тянет тебя к Стратегии. switch или цепочка if, которая продолжает набирать ветви по одной и той же причине — новый способ доставки, новое правило ценообразования, новый формат экспорта, — это запах «операторов switch». Каждая ветвь — взаимозаменяемый алгоритм, втиснутый в одну функцию. Стратегия — это то, куда тебя приводит устранение этого запаха: вытащи каждую ветвь в её собственный объект за общим интерфейсом, и условный оператор схлопывается в поиск по таблице.

// SMELL: one function that grows a branch every time a method is added
function shippingCost(method: string, weight: number): number {
  if (method === "standard") return weight * 1.5;
  if (method === "express")  return weight * 1.5 + 10;
  if (method === "overnight") return weight * 3 + 25;
  throw new Error("unknown method");
}

Рефакторь к Стратегии — но только когда третья или четвёртая ветвь доказали, что ось реальна:

type ShippingStrategy = (weight: number) => number;
const strategies: Record<string, ShippingStrategy> = {
  standard:  (w) => w * 1.5,
  express:   (w) => w * 1.5 + 10,
  overnight: (w) => w * 3 + 25,
};
function shippingCost(method: string, weight: number): number {
  const strategy = strategies[method];
  if (!strategy) throw new Error("unknown method");
  return strategy(weight);
}

Добавить способ теперь — это запись в таблице, а не ветвь в растущей функции, и не связанные между собой способы перестают делить общий путь кода, который приходится перетестировать.

Условный оператор, ветвящийся по фазе жизненного цикла, тянет тебя к Состоянию. Двоюродный брат Стратегии: когда условный оператор не выбирает алгоритм, а проверяет «в каком режиме я сейчас?» — и переходы между режимами разбросаны как присваивания this.status = "..." — тяга направлена к паттерну Состояние. Признак — поле статуса, по которому ветвятся сразу несколько методов, с одной и той же повторяющейся лестницей if (status === ...).

// SMELL: every method re-derives behaviour from a status string
class Order {
  status = "draft";
  submit() { if (this.status !== "draft") throw Error("can't submit"); this.status = "submitted"; }
  pay()    { if (this.status !== "submitted") throw Error("can't pay"); this.status = "paid"; }
  cancel() { if (this.status === "paid") throw Error("can't cancel"); this.status = "cancelled"; }
}

Состояние извлекает каждую фазу в собственный объект, который владеет своими легальными переходами, так что нелегальный переход невозможен по построению, а не охраняется цепочкой строковых проверок. Но обрати внимание на цену: Состояние тяжелее Стратегии. Для трёх состояний ты можешь оставить явные проверки; паттерн заслуживает себя только тогда, когда число состояний и переходов делает разбросанную лестницу по-настоящему трудной для понимания. Запах — продублированный if (status === ...) по многим методам — это ворота.