Почему я больше не цепляю всё подряд в JavaScript — Мэтт Смит

Я раньше писал много JavaScript вот так:

const result = users
  .filter(user => user.active)
  .map(user => user.name)
  .sort()
  .slice(0, 5);

Здесь нет ничего плохого. Я писал такой код постоянно. Но это как раз тот самый случай: сначала всё выглядит нормально, а потом постепенно становится сложнее с этим работать.

Цепочки — это здорово… пока не становится иначе

Проблема не в .map() или .filter(). Проблема в том, что происходит, когда вы складываете их друг за другом. Вы перестаёте писать шаги и начинаете писать «конвейеры».

Конвейеры выглядят аккуратно, но всё равно вам приходится мысленно проходить по ним: filter → map → sort → slice.

Это нормально один-два раза. Но если делать так по всему файлу, это начинает изматывать.

Сравните с этим:

const activeUsers = users.filter(user => user.active);
const names = activeUsers.map(user => user.name);

names.sort();

return names.slice(0, 5);

Да, строк больше. Но каждый шаг просто стоит на своём месте. Никакой «расшифровки» не требуется.

Та же проблема, три способа написать

Вот тот же замысел, выраженный тремя разными способами.

Если я делаю цепочку:

users.filter(u => u.active).map(u => u.name)[0]

Это выглядит опрятно. Раньше я очень часто тянулся именно к этому. Но оно обрабатывает всё, хотя мне нужно только одно значение.

Если я пишу по шагам:

const user = users.find(u => u.active);
const name = user?.name;

Обычно я прихожу именно сюда. Это останавливается раньше, и если что-то кажется странным, я могу проверить каждую часть.

Если мне нужен полный контроль:

for (const u of users) {
  if (u.active) return u.name;
}

Это самое явное, и честно говоря, иногда — самое понятное, когда мне действительно важно, что именно происходит.

Где становится немного беспорядочно

Это быстрее всего проявляется, когда вы пытаетесь отлаживать.

Допустим, что-то кажется неправильным, и вы хотите проверить результаты после фильтрации. В цепочке вы в итоге делаете вот так:

const result = users
  .filter(user => {
    console.log(user);
    return user.active;
  })
  .map(user => user.name);

Теперь ваша логика смешана с отладочным кодом. Или вы сдаётесь и в любом случае разрываете цепочку на части.

Вы можете делать больше работы, чем нужно

Цепочки подталкивают к «обработке всего», даже когда вы этого не имели в виду.

const firstActiveUser = users
  .filter(user => user.active)
  .map(user => user.name)[0];

Сначала фильтруется весь массив, затем результат маппится, а потом берётся один элемент.

А то, что вы на самом деле хотели бы, выглядит так:

const user = users.find(user => user.active);
const name = user?.name;

Или так:

for (const user of users) {
  if (user.active) {
    return user.name;
  }
}

Где это начинает по-настоящему болеть

Это не только про читаемость. Эта дополнительная работа накапливается на больших массивах или в «горячих» участках кода. А длинные цепочки могут неожиданно раздражать при отладке в продакшене.

Я раньше писал довольно жуткие цепочки. Возвращаться к ним позже —… унизительно.

Плавность не всегда означает ясность

Есть причина, почему цепочки популярны: сначала они читаются приятно.

data
  .transform()
  .normalize()
  .validate()
  .save();

Но теперь вы начинаете гадать, что возвращает каждый шаг, куда вообще ставить брейкпоинт и можно ли что-то из этого переиспользовать.

Разбиение на шаги сразу отвечает на эти вопросы.

Асинхронные цепочки имеют ту же проблему

Цепочки промисов могут выглядеть элегантно:

const data = await fetchUsers()
  .then(res => res.json())
  .then(users => users.filter(u => u.active))
  .then(users => users.map(u => u.name));

Но теперь вы смешиваете асинхронное управление потоком (загрузка, парсинг) и преобразование данных в одной цепочке.

Разбить на части обычно проще для понимания:

const res = await fetchUsers();
const users = await res.json();
const activeNames = users.filter(u => u.active).map(u => u.name);

Жёсткое правило, которому я следую

Я не говорю «никогда не цепляйте»

Короткие цепочки — нормально. Я всё ещё пишу их. Но как только доходит до трёх или четырёх шагов, я делаю паузу.

Как я думаю об этом сейчас

Цепочки — это здорово, когда вы быстро пишете код. Разбиение на шаги лучше, когда этот код нужно будет читать позже.

Это не одно и то же.

Как я обычно распутываю это

Это спасло меня от множества головных болей.

JavaScript даёт вам кучу инструментов, но вам не нужно использовать их все сразу.

JavaScript