Выделение класса

Группа данных и кластер методов сами говорят, где шов, — ты его не выдумываешь, ты его читаешь. «Выделение класса» — это принцип единственной ответственности, применённый задним числом, и сигнал того, что две обязанности слиплись, — структурный, а не эстетический. Ищи группу данных: набор полей, которые всегда появляются вместе (areaCode + number или street + city + zip). Потом смотри, какие методы касаются только этого подмножества полей и игнорируют остальной класс. Это перекрытие — поля, которые кластеризуются, и методы, которые обслуживают только кластер, — и есть шов.

class Person {
  name: string;
  // ── the data clump: these three always move together ──
  areaCode: string;
  number: string;
  // ── the method cluster: it only ever touches the clump above ──
  phoneAsString(): string { return `(${this.areaCode}) ${this.number}`; }
}

Если у кандидата в «обязанности» есть данные, но нет методов, обслуживающих их исключительно, — или есть методы, но нет общих данных, — у тебя нет класса, готового выйти наружу: у тебя есть что-то меньшее (перемещаемый метод, переименованное поле). Шов должен проявиться в обоих измерениях, прежде чем выделение себя окупит.

Создай новый класс пустым и свяжи его — первое перемещение не меняет ничего наблюдаемого. Начни с шага без эффекта: введи TelephoneNumber и дай Person его экземпляр. На этом этапе старые поля ещё существуют; ничего не перемещено. Прогони тесты — они должны остаться зелёными, потому что ты добавил обратную ссылку и не изменил поведения. Эта пустая оболочка — твои страховочные леса: каждое последующее перемещение теперь становится делегированием, заменяемым по одному полю и по одному методу за раз.

class TelephoneNumber {} // empty on purpose

class Person {
  name: string;
  areaCode: string;
  number: string;
  private telephoneNumber = new TelephoneNumber(); // linked, unused — tests still green
  phoneAsString(): string { return `(${this.areaCode}) ${this.number}`; }
}

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

Перемещай поля по одному, заменяя каждое делегирующим аксессором, тесты зелёные после каждого перемещения. Возьми areaCode. Добавь его в TelephoneNumber с геттером/сеттером, затем сделай Person.areaCode делегирующим к this.telephoneNumber. Хранилище поля переехало; его публичная поверхность на Person не изменилась, поэтому вызывающие и тесты ничего не замечают. Повтори для number.

class TelephoneNumber {
  private _areaCode = "";
  get areaCode() { return this._areaCode; }
  set areaCode(v: string) { this._areaCode = v; }
}

class Person {
  // storage now lives in TelephoneNumber; Person just forwards
  get areaCode() { return this.telephoneNumber.areaCode; }
  set areaCode(v: string) { this.telephoneNumber.areaCode = v; }
  phoneAsString(): string { return `(${this.areaCode}) ${this.number}`; }
}

Причина делегировать, а не выдирать поле, — безопасность ссылок. Ты можешь не знать каждого, кто вызывает person.areaCode. Делегирующий аксессор оставляет их всех работающими, пока данные переезжают, так что ссылки можно переместить в более позднем, отдельном проходе, а не все сразу под давлением компилятора.

Перемести методы к данным, затем уведи вызывающих с пробросчиков Person и удали их. Теперь phoneAsString() читает только данные, живущие в TelephoneNumber, поэтому оно принадлежит ему. Перемести его. Person.phoneAsString() становится однострочным делегатом, а затем — как только вызывающие обновлены, чтобы спрашивать телефонный номер напрямую, — ты удаляешь делегат. Это и есть результирующее перемещение: поведение переехало к своим данным, в чём и весь смысл. Класс связен, когда его методы и данные, которые они используют, сидят вместе.

class TelephoneNumber {
  areaCode = "";
  number = "";
  toString(): string { return `(${this.areaCode}) ${this.number}`; }
}

class Person {
  name = "";
  telephoneNumber = new TelephoneNumber();
  // delegate kept only until callers move; then deleted
  phoneAsString(): string { return this.telephoneNumber.toString(); }
}

Когда последний вызывающий пробросчика исчезает, исчезает и сам пробросчик. Конечное состояние — два класса, у каждого из которых одна причина меняться: меняешь формат телефона — трогаешь TelephoneNumber; меняешь правило про человека — трогаешь Person. Что критично, TelephoneNumber несёт поведение (форматирование, и валидация теперь тоже может жить там) — это не структура данных.