bad4iz 4 фев 2019 в 15:52

ECMAScript Proposal: Array.prototype.{flat,flatMap}

4 мин

11K

JavaScript*

+27

Комментарии 29

Scf 4 фев 2019 в 16:00

Дождались, аллилуйя. После изучения скалы, мне всегда не хватало flatMap в яваскрипте.

sshikov 4 фев 2019 в 16:09

А вам что-то мешало его допилить? Он же сравнительно несложный внутри.

Scf 4 фев 2019 в 16:18

менять прототайп массива — не круто, чревато проблемами в будущем.

Zibx 4 фев 2019 в 18:49

Какими? Если не втаскивать кривых библиотек которые добавляют в прототип функции с аналогичными именами, то от расширения прототипов Function, Array, String, Number ничего страшного не произойдёт. Исключением является расширения прототипа Object — тут, действительно, не так может пойти очень многое.

-1

Scf 4 фев 2019 в 18:54

Ну вот написал я свое расширение flatMap, а тут разработчики браузеров втащили кривую библиотеку с другой реализацией flatMap.

Сделать их идентичными не так просто из-за заранее неизвестных соглашений об обработке "необычных" типов.

justboris 4 фев 2019 в 19:15

Какими?

Например тем, что вы напишите функцию flat для бесконечной вложенности, а в стандарт войдет реализация только для первого уровня вложенности, и весь ваш код сломается после обновления браузеров.

https://developers.google.com/web/updates/2018/03/smooshgate

Zibx 4 фев 2019 в 23:27

Так надо понимать что происходит. Если написать:

Array.prototype.flat = MyFlatFunction;

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

justboris 4 фев 2019 в 23:45

отлетит сторонний код который положится на реализацию вошедшую в стандарт.

Вот именно это и плохо. Если у пользователя перестанет открываться сайт, ему неважно, ваш это был код, или сторонний.

Zibx 5 фев 2019 в 02:57

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

justboris 5 фев 2019 в 03:52

Если у пользователя перестанет открываться сайт, то такой код должен отлететь уже на этапе тестов

Так не получится, потому что не новый код ломается, а уже существующий в проде по причине обновления браузера.

vanxant 5 фев 2019 в 00:09

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

sshikov 4 фев 2019 в 20:17

>После изучения скалы, мне всегда не хватало flatMap в яваскрипте.
Вообще-то, в вашем оригинальном пожелании нет ни слова о прототипе :)

-1

greabock 4 фев 2019 в 16:05

лодаш рип

biziwalker 4 фев 2019 в 16:17

Что-то не единого слова, что flat и flatMap включены в ES2019 github.com/tc39/proposals/blob/master/finished-proposals.md

Подправьте статью, данное предложение больше не является кандидатом (стадия 3)

bad4iz 4 фев 2019 в 16:20

да он перешел в stage 4 и в самом низу вашей ссылки
github.com/tc39/proposal-flatMap

bad4iz 4 фев 2019 в 16:46

`Array.prototype.{flat,flatMap}` to stage 4, per 2019.01.29 TC39

rd_nino 4 фев 2019 в 18:27

Можно ли добавить в статью пример практического применения?
На ряде чисел.- не понятна суть этого улучшения. В чем польза?

Scf 4 фев 2019 в 18:58

const persons = [...];
const maleAges = persons.flatMap(p => {
  if (p.male) return [p.age]; else return [];
})

rd_nino 4 фев 2019 в 19:03

Спасибо! Теперь все ясно. :)

symbix 5 фев 2019 в 02:33

Ну вот, один из моих любимых FizzBuzz-подобных вопросов испортили. :-)

С другой стороны, если кандидат знает такие новшества, это хороший признак, и всегда можно попросить написать без flat().

Chamie 5 фев 2019 в 13:16

А в чём похожесть на FizzBuzz? В FizzBuzz проблема в том, что хочется написать «красиво», а тут как раз легко написать «красиво».

symbix 5 фев 2019 в 14:49

В том, что цель одинаковая: отсеять за 3 минуты тех, кто вообще не умеет программировать.

Конечно же, принимается любое корректное решение. Рекурсия или стек, reduce или цикл ручками — не суть важно.

Chamie 5 фев 2019 в 13:12

Самого короткого варианта замены не написали:

const ar = [[1,2],[3,4],[5,6]];
ar.reduce((a,b)=>[...a,...b]);
//[1,2,3,4,5,6]

Это если вложенные элементы — гарантированно массивы, конечно. Если нет, то

ar.reduce((a, b) => [...a, ...(Array.isArray(b) ? b : [b])]);

justboris 5 фев 2019 в 13:28

Еще короче

[].concat(...arr)

Chamie 5 фев 2019 в 14:39

Вообще шикарно, спасибо. Жаль только, не чейнится — в середину цепочки преобразований не вставить, только в начало.

FanAs 5 фев 2019 в 15:51

Не советую использовать spread синтаксис. Spread (...) добавит все значения в стек и передаст как список параметров, что при больших размерах массивов (10^6) сгенерирует stack overflow исключение.