# Почему я больше не цепляю всё в JavaScript — Matt Smith
Перевод статьи: Why I don't chain everything in JavaScript anymore - Matt Smith
Я раньше писал много 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;
}
}
Где это начинает болеть
Это не только про читаемость. Дополнительная работа накапливается на больших массивах или в горячих участках. А длинные цепочки могут неожиданно раздражать при отладке в production.
Я раньше писал довольно жуткие цепочки. Возвращаться к ним позже — это… унизительно.
Fluent не всегда значит clear
Есть причина, почему цепочки популярны: сначала они читаются приятно.
data
.transform()
.normalize()
.validate()
.save();
Но теперь вы начинаете думать о том, что возвращает каждый шаг, куда ставить breakpoint и можно ли это как-то переиспользовать.
Разбиение на шаги отвечает на эти вопросы сразу.
Асинхронные цепочки имеют ту же проблему
Цепочки промисов могут выглядеть изящно:
const data = await fetchUsers()
.then(res => res.json())
.then(users => users.filter(u => u.active))
.then(users => users.map(u => u.name));
Но теперь вы смешиваете асинхронное управление потоком (fetching, parsing) с преобразованием данных в одной цепочке.
Разбивать на части обычно проще для понимания:
const res = await fetchUsers();
const users = await res.json();
const activeNames = users.filter(u => u.active).map(u => u.name);
Грубое правило, которого я придерживаюсь
| Длина цепочки | Рекомендация | Пример |
|---|---|---|
| 1 шаг | Вполне нормально | users.map(u => u.name) |
| 2 шага | Обычно нормально | users.filter(u => u.active).map(u => u.name) |
| 3–4 шага | Пауза, подумайте о разбиении | users.filter(...).map(...).sort(...).slice(...) |
| 5+ шагов | Определённо разбейте на шаги | Сложные преобразования или асинхронные цепочки |
Я не говорю «никогда не цепляйте»
Короткие цепочки — это нормально. Я всё ещё пишу их. Как только я добираюсь до трёх или четырёх шагов, я делаю паузу.
Как я думаю об этом сейчас
Цепочки — это здорово, когда вы быстро пишете код. Разбиение на шаги лучше, когда этот код нужно будет читать позже.
Это не одно и то же.
Как я обычно распутываю это
| Шаг | Что делать | Пример |
|---|---|---|
| 1 | Дайте промежуточным значениям имена | const activeUsers = users.filter(u => u.active) |
| 2 | Логически разделите преобразования | const names = activeUsers.map(u => u.name) |
| 3 | Цепляйте только то, что понятно | names.sort() |
Это спасло меня от множества головных болей.
JavaScript даёт вам много инструментов, но вам не нужно использовать их все одновременно.