Чисто, а не хитро

Хитрый код оптимизирован под автора; чистый код оптимизирован под следующего читателя и следующее изменение. Хитрую конструкцию оценивают по тому, насколько её мало и сколько она делает на символ. Чистую конструкцию оценивают по тому, как быстро её понимает посторонний и как безопасно он может её изменить. Эти цели реально конфликтуют: самое плотное выражение вычисления почти никогда не самое ясное. Возьмём оценку списка баллов:

// clever: one expression, three behaviours folded together
const grade = (s: number[]) =>
  s.reduce((a, x) => a + (x >= 50 ? x : 0), 0) / s.filter((x) => x >= 50).length;

Оно работает, и оно компактно. Но чтобы это прочитать, нужно дважды удержать в голове предикат фильтра, вывести, что усредняются только проходные баллы, и заметить, что массив обходится дважды. Намерение — «среднее проходных баллов» — ты реконструируешь сам, а не код заявляет. Автор сэкономил нажатия клавиш; каждый читатель платит за их расшифровку.

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

// clean: longer, but the intent is on the surface
function averagePassingScore(scores: number[]): number {
  const passing = scores.filter((score) => score >= PASS_MARK);
  if (passing.length === 0) return 0; // clever version divided by zero → NaN
  const total = passing.reduce((sum, score) => sum + score, 0);
  return total / passing.length;
}

Больше строк, меньше сюрпризов. Имя заявляет намерение, так что его никто не реконструирует. Защита от пустого списка видна — и именно её выписывание обнажило деление на ноль, которое хитрый однострочник прятал за своей плотностью. Стоимостью никогда не была длина; ей была непрозрачность. Чистый код тратит пару лишних строк, чтобы купить более дешёвое следующее изменение, — та же ставка, которой учил урок про стоимость изменения: заплати немного сейчас там, где это удешевит будущее.

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

// HOT PATH: called per-pixel in the render loop; profiled 3.1ms → 0.4ms.
// Bit trick: (x & (x - 1)) === 0 tests power-of-two without a loop or log2.
const isPowerOfTwo = (x: number) => x > 0 && (x & (x - 1)) === 0;

Комментарий — это квитанция об уплате налога. Он называет измеренную причину и переводит трюк, так что следующий читатель платит один раз (прочитать комментарий) вместо каждого раза (реверс-инжиниринг битов). Плотность без такой квитанции — это налог, который ты наложил на других, чтобы польстить себе.

Режим отказа: «чисто», прочитанное как «максимально абстрагировано», — это собственная хитрость. Противоположность хитрому однострочнику — это не пятислойная абстракция. Читатель, который услышал «чистый код» и потянулся за обобщённым Pipeline<T>, реестром стратегий и шиной событий, чтобы отформатировать одну дату, не убрал хитрость — он переместил её из лаконичности в архитектуру. И то, и другое заставляет читателя реконструировать намерение; и то, и другое повышает стоимость следующего изменения. Чисто значит очевидно, а не вычурно и не гибко-ради-самой-гибкости.

// "clever-as-abstract": indirection no requirement asked for
const formatter = FormatterFactory.create(LocaleStrategy.from(config)).build();
return formatter.apply(new DateValue(d));

// clean: obvious, names the intent, no ceremony
return d.toLocaleDateString("en-GB"); // e.g. 23/06/2026

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