Почему я больше не цепляю всё подряд в 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