Введение объекта-параметра

Триггер — это группа данных или длинный список параметров: одни и те же поля, путешествующие вместе. Группа — это два и более значения, которые постоянно появляются как набор: (startDate, endDate), (x, y), (amount, currency), (host, port, protocol). Признак механический: если удалить одно из них, остальные перестают иметь смысл. Длинный список параметров — та же болезнь в одном месте вызова: пять, шесть, восемь позиционных аргументов, где читатель уже не может сказать, который из них что.

// before: the (start, end) clump rides through every function, plus boolean noise
function priceStay(
  start: Date,
  end: Date,
  ratePerNight: number,
  taxRate: number,
  includeCleaning: boolean,
  isWeekendPremium: boolean,
): number { /* ... */ }

book(start, end, 120, 0.2, true, false); // which boolean is which?

Позиционные булевы нечитаемы, а пара (start, end) уже появилась в четырёх функциях. И то и другое — сигналы, а не шум: они говорят тебе, что не хватает концепции.

Делай это безопасными шагами, оставаясь зелёным всю дорогу. Introduce Parameter Object — это механический рефакторинг с заданной последовательностью, и тесты остаются зелёными после каждого шага, а не переписываются одним прыжком:

1. Создай новый тип с полями группы. Пока не меняй ни одной сигнатуры.
2. Добавь объект как новый параметр целевой функции, сохранив старые; тело пусть читает из нового объекта. Тесты по-прежнему зелёные.
3. Обнови одного вызывающего, чтобы он передавал объект. Зелёный. Затем следующего вызывающего. Зелёный.
4. Как только каждый вызывающий передаёт объект, удали ставшие неиспользуемыми отдельные параметры.
5. Перенеси поведение, оперировавшее значениями группы, на новый тип (следующий шаг).

type DateRange = { readonly start: Date; readonly end: Date };

function priceStay(range: DateRange, rate: RateCard): number { /* ... */ }

Каждый шаг обратим и проверен, поэтому ошибка ловится в одном ходу от того места, где ты её совершил, — в этом весь смысл рефакторинга под тестами, а не через big-bang-переписывание.

Настоящая выгода: поведение и валидация притягиваются к новому объекту. Как только DateRange появился, у каждой проверки «валиден ли этот диапазон?» / «сколько ночей?» / «пересекаются ли они?», дублировавшейся по вызывающим, есть очевидный дом — на типе. Мартин Фаулер называет это более глубокой причиной рефакторинга: новый объект становится магнитом, притягивающим разбросанное вокруг него поведение.

class DateRange {
  constructor(readonly start: Date, readonly end: Date) {
    if (end < start) throw new RangeError("end before start");
  }
  get nights(): number {
    return Math.round((+this.end - +this.start) / 86_400_000);
  }
  overlaps(other: DateRange): boolean {
    return this.start < other.end && other.start < this.end;
  }
}

Валидация end < start теперь происходит один раз, при конструировании, — так что ни одна нижестоящая функция уже не может получить перевёрнутый диапазон. nights вычисляется в одном месте, а не выводится заново (с off-by-one-багами) в прайсинге, календаре и отчётности. В этом senior-мысль: объект-параметр не просто укорачивает сигнатуры, он называет недостающую концепцию и даёт связанной логике место, где жить, что убивает целый класс дублирования.

Целый объект (whole-object): передай объект, который у тебя уже есть, вместо его распаковки. Близкий родственник: когда вызывающий вытаскивает несколько полей из существующего объекта лишь для того, чтобы передать их как отдельные аргументы, передай сам объект. Это сокращает список параметров, развязывает вызываемого от раскладки полей вызывающего и позволяет вызываемому позже спросить объект о большем без изменения сигнатуры.

// before: caller unpacks, callee re-couples to the field names
const within = isWithinBudget(quote.low, quote.high, order.total);

// after: pass the whole object; the rule moves where it belongs
const within = quote.contains(order.total);

Когда НЕ стоит: если вызываемому действительно нужно лишь одно поле, передача целого объекта переусиливает связь с типом, который он едва использует — предпочти единственное значение (это напряжение между Законом Деметры и сегрегацией интерфейсов). Whole-object уместен, когда вызываемый использует несколько полей или когда «правило живёт на объекте» читается естественнее, чем «вызывающий вычисляет правило».