Маленькие функции, одно дело

«Одно дело» означает одну причину для изменения, на одном уровне абстракции — а не одну строку. Обработчик, который парсит, валидирует, сохраняет и уведомляет, имеет четыре причины для изменения: формат данных на проводе, бизнес-правила, схему хранения и канал сообщений. Каждая может двигаться независимо. Когда они делят одно тело функции, изменение любой из них заставляет читать сквозь остальные три и рисковать задеть их. Запах — не длина, а то, что строки работают на разных высотах: высокоуровневая оркестрация («затем мы уведомляем пользователя») стоит inline рядом с низкоуровневой деталью (new Date().toISOString(), параметры SMTP). Глазу приходится постоянно переключать передачи.

// одна функция, четыре причины для изменения, смешанные высоты
async function handleSignup(req: Request): Promise<Response> {
  const body = await req.json();
  if (!body.email || !body.email.includes("@")) return bad("email");
  if (!body.password || body.password.length < 8) return bad("password");
  const id = crypto.randomUUID();
  await db.execute("INSERT INTO users(id,email,pw) VALUES(?,?,?)",
    [id, body.email, await hash(body.password)]);
  await mailer.send({ to: body.email, subject: "Welcome", body: tmpl(id) });
  return new Response(JSON.stringify({ id }), { status: 201 });
}

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

async function handleSignup(req: Request): Promise<Response> {
  const input = await parseSignup(req);       // формат на проводе
  const error = validateSignup(input);        // бизнес-правила
  if (error) return bad(error);
  const user = await persistUser(input);      // хранение
  await sendWelcome(user);                     // сообщения
  return created({ id: user.id });
}

Теперь верхняя функция — это четыре названных шага. Чтобы изменить валидацию, ты открываешь validateSignup и больше ничего не читаешь. Чтобы изменить письмо — открываешь sendWelcome. Оркестрация читается как предложение; каждая деталь живёт на уровень ниже, где ей и место.

Выигрыш — это имя, а не меньшее тело. Вот часть, которую люди упускают. validateSignup ценна, даже если в ней всего три строки, потому что имя утверждает намерение: «этот пробег кода — шаг валидации». Выделенная функция — это документация, которую компилятор держит честной: она не может разойтись с кодом так, как может комментарий. Хорошее выделение заменяет комментарий-плюс-блок названным вызовом:

// до: комментарий — единственное, что говорит тебе, что это за блок
// валидируем входные данные регистрации
if (!body.email || !body.email.includes("@")) return bad("email");
if (!body.password || body.password.length < 8) return bad("password");

// после: имя несёт то же намерение и не может устареть
const error = validateSignup(body);
if (error) return bad(error);

Если выделение блока не позволяет дать ему имя информативнее, чем строки внутри него, то выделение ничего не покупает. Тест на «настоящее ли это под-намерение»: могу ли я назвать его по концепции, которая уже заботит читателя? «Валидировать регистрацию», «сохранить пользователя», «отправить приветствие» — да. «Строки с 12 по 17» — нет.

Избыточное выделение — равный и противоположный отказ: крошение связности в погоню за функциями. Как только «маленькое — хорошо» становится рефлексом, люди выделяют однострочники без независимого намерения — function isEmpty(s) { return s.length === 0 }, function addOne(n) { return n + 1 } — и расщепляют единое связное вычисление на шесть функций, по которым приходится прыгать, чтобы понять смысл. Теперь прочитать одну идею значит гоняться за шестью определениями по всему файлу, держа стек вызовов в голове. Это менее связно, а не более: ты увеличил сцепление между фрагментами, разрушив локальность, которая позволяла читать целое сверху вниз.

// избыточно выделено: одна связная формула, разбросанная по файлу
function priceFor(item: Item): number {
  return applyTax(applyDiscount(base(item)));   // надо гнаться за 3 определениями
}
function base(i: Item) { return i.qty * i.unit; }
function applyDiscount(x: number) { return x * 0.9; }   // 0.9? для кого?
function applyTax(x: number) { return x * 1.2; }        // скрытая магия

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