Триада сопровождаемости

Сопровождаемость — не одно свойство, а три измеримых, разделяемых. «Дёшево менять» связывает в пучок три разные стоимости, и ты должен уметь оценивать код по каждой независимо:

• Читаемость — стоимость понять, что этот код делает и зачем, с холодного старта. Измеряется тем, сколько компетентному читателю нужно времени, чтобы быть уверенным.
• Изменяемость — стоимость безопасно изменить: сколько мест придётся тронуть, насколько велик радиус поражения, насколько ты можешь быть уверен, что не сломал вызывающего.
• Тестируемость — стоимость проверить единицу в изоляции: можешь ли ты прогнать эту логику, не поднимая базу, не подменяя часы и не запуская всё приложение?

Это оси, по которым senior реально выставляет оценки. «Хороший код» — это высокие баллы по всем трём; «плохой код» обычно проваливает одну конкретную ось, и назвать, какую именно, — в этом весь смысл.

Они коррелируют, но разделяемы — и зазоры между ними и есть места, где прячутся баги. Свойства склонны двигаться вместе: хорошо названный, связный код обычно легко читать, менять и тестировать. Эта корреляция реальна, поэтому новички и относятся к «хорошему коду» как к чему-то одному. Но они расходятся достаточно часто, чтобы их смешение было senior-уровневой ошибкой:

// Читаемо. Очевидное намерение. Нетестируемо.
function isTrialExpired(user: User): boolean {
  const elapsedDays = (Date.now() - user.signupAt) / 86_400_000;
  return elapsedDays > 14;
}

Ты прочитаешь это за пять секунд — читаемость высока. Но написать на это детерминированный юнит-тест нельзя: Date.now() делает результат зависящим от того, когда запускается тест. Чтобы проверить «истёк» против «не истёк», пришлось бы мокать глобальное время, подделывать системные часы или спать. Функция читаема и нетестируема одновременно. Два свойства просто разошлись.

Обычный виновник низкой тестируемости — скрытая зависимость: время, IO, случайность или глобальное состояние, к которым тянутся напрямую. Date.now(), Math.random(), process.env, fetch, fs.readFileSync, синглтон уровня модуля — каждое из них зависимость, которую функция хватает из окружающей среды вместо того, чтобы получить. Читается нормально, потому что зависимость невидима в сигнатуре. Именно эта невидимость и убивает тестируемость: тест может управлять только тем, что он способен передать, а ты не можешь передать то, к чему функция тянется у тебя за спиной.

// Читаемо и так же нетестируемо: тянет конфиг из окружающей среды.
function buildApiUrl(path: string): string {
  const base = process.env.API_BASE ?? "https://prod.example.com";
  return `${base}${path}`;
}

Чтобы протестировать ветку staging, пришлось бы менять process.env (глобальное, протекает между тестами) и не забыть его восстановить. Сигнатура (path: string) => string врёт: функция на деле зависит от path и от окружения. Нетестируемость — это обычно сигнатура, которая не говорит правду о своих входах.

Внедри скрытую зависимость — и тестируемость растёт, а читаемость не тронута. Сделай зависимость параметром — значением now, часами, объектом конфига. Логика не меняется; сигнатура перестаёт врать; тест получает шов, через который можно передавать значения.

// Та же логика, та же ясность — теперь детерминированно тестируемо.
function isTrialExpired(user: User, now: number): boolean {
  const elapsedDays = (now - user.signupAt) / 86_400_000;
  return elapsedDays > 14;
}

// Тест управляет временем. Без моков, без сна, без глобального состояния.
isTrialExpired({ signupAt: 0 }, 13 * 86_400_000); // false
isTrialExpired({ signupAt: 0 }, 20 * 86_400_000); // true

Прочитай ещё раз: понимать это ровно так же легко, как раньше. Мы не потратили ни капли читаемости и купили полную тестируемость. Это идеальный ход — и он показывает, что свойства разделяемы и в хорошую сторону: можно поднять одно, не платя другим. Стоимость вытолкнута наверх, к вызывающему (он обязан подать now: Date.now() один раз на краю), а это ровно то место, где реальным, нетестируемым часам и положено быть.

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

// «Тестируемо» — но читаемость и изменяемость теперь угроблены.
class TrialChecker {
  constructor(
    private clock: IClock,
    private config: IConfig,
    private logger: ILogger,
    private featureFlags: IFeatureFlags,
  ) {}
  isExpired(user: User): boolean {
    this.logger.debug("checking");
    const limit = this.config.get("trialDays");
    if (!this.featureFlags.isOn("trials")) return false;
    return (this.clock.now() - user.signupAt) / 86_400_000 > limit;
  }
}

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