Расходящееся изменение и стрельба дробью

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

// OrderService.ts — задеваемый тремя несвязанными силами
class OrderService {
  charge(order: Order) { /* специфика Stripe API — меняется, когда меняются платежи */ }
  computeTax(order: Order) { /* правила НДС — меняются, когда меняется налоговый закон */ }
  sendReceipt(order: Order) { /* тело HTML-письма — меняется, когда меняется маркетинг */ }
}

Улика — в истории: один и тот же файл появляется в коммитах, чьи сообщения не имеют друг к другу никакого отношения — «перейти на Adyen», «добавить НДС ЕС», «новый дизайн чека». Лечение — разбить по осям изменения, чтобы у каждого нового требования был один очевидный дом.

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

// ОДНО И ТО ЖЕ правило (как рисовать деньги) переписано в пяти местах
cart.ts:      const label = currency === 'USD' ? '$' + amt : '€' + amt;
checkout.ts:  const label = currency === 'USD' ? '$' + amt : '€' + amt;
invoice.ts:   const label = currency === 'USD' ? '$' + amt : '€' + amt;
// ...и ещё два

Улика снова в истории: один коммит (или, хуже, несколько коммитов, гоняющихся за одним багом) задевает разброс файлов, у которых не должно быть ничего общего. Лечение — собрать разбросанные осколки за одной границей, чтобы следующая валюта была одной правкой.

Это эмпирические запахи SRP — их читают по совместным изменениям, а не спорят о них. Принцип единственной ответственности обычно формулируют как «у класса должна быть одна причина для изменения». Это определение операциональное, а не философское: причина для изменения наблюдаема. Построй карту совместных изменений — для каждой пары файлов (или для множества причин коммитов каждого файла): как часто они меняются вместе и почему?

// грубый сигнал совместного изменения из истории (концептуально)
// git log --name-only --pretty=format:%s  →  группируй файлы по коммиту
// расходящееся изменение:  один файл, много несвязанных тем коммитов
// стрельба дробью:         одна тема коммита, много несвязанных файлов

• Расходящееся изменение = один файл, высокое разнообразие причин изменения. Связность низкая; файл делает несколько работ. Разбей его.
• Стрельба дробью = одна причина изменения, высокое разнообразие задетых файлов. Связанность высокая; одна работа размазана. Собери её.

Вот почему запахи бьют абстрактный спор: два сеньора могут вечно спорить, есть ли у OrderService «одна обязанность», но не могут спорить, что его история показывает три независимые оси изменения. Репозиторий — судья.

Направление рефакторинга для каждого противоположное — и выбрать неверно значит сделать хуже. Расходящемуся изменению нужно разделение: Extract Class / Move Method, пока каждый модуль не станет варьироваться по одной причине. Стрельбе дробью нужно собирание: Move Method / Inline / Combine, пока каждая причина не заживёт в одном модуле. Они тянут в противоположные стороны, поэтому диагноз должен идти первым.

// расходящееся изменение → РАЗБИТЬ (поднять связность)
class PaymentGateway { charge(order: Order) {/* … */} }
class TaxCalculator  { computeTax(order: Order) {/* … */} }
class ReceiptMailer  { send(order: Order) {/* … */} }

// стрельба дробью → СОБРАТЬ (снизить связанность)
// money.ts — единственное место, где живёт правило «рисовать деньги»
export function formatMoney(amt: number, currency: Currency): string { /* … */ }
// cart/checkout/invoice теперь все вызывают formatMoney(amt, currency)

Если «починить» расходящееся изменение собиранием, построишь god-класс побольше. Если «починить» стрельбу дробью разбиением, разбросаешь обязанность ещё сильнее. Запах называет лекарство; в этом и весь смысл того, чтобы называть их по отдельности.