-1
const orderIds = [4, 2, 1, 3]

const people = [
    { id: 3, name: 'Tom'},
    { id: 2, name: 'Bill'},
    { id: 1, name: 'Jack'},
    { id: 4, name: 'Bob'}
]
// Bob Bill Jack Tom

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

4
  • цикл по массиву и потом цикл по объекту. в чём трабл? 27 авг 2021 в 13:22
  • Задание неясное. (rest, spread)? Где-то сказано, что нужно использовать в решении эти операторы? Тогда почему нет этого в самом задании? Нужно вывести данные из объекта? Из какого объекта? Есть целый массив из объектов. В порядке, указанном в массиве? Этот порядок связан с id каждого объекта или с чем? Поменять ключи в объекте??? На что поменять, в каком объекте? Единственное осмысленное здесь ожидаемый вывод: Bob Bill Jack Tom - он действительно соответствует заданию?
    – Leonid
    27 авг 2021 в 15:37
  • Да, такие люди всегда найдутся. Всем всё всегда понятно, но будет такой буквоед, что спасения от него не будет. Это ведь не техническое задание в компании, никто за это денег не даст, никто не понесёт ответственности, это вообще по факту для всех присутствующих - ничто! Скажите, вы писали это в конце рабочего дня? После ссоры с менеджером проекта или собственником проекта? Вас отругали за неточное выполнение задания? Или вас регулярно забрасывают плохо поставленными ТЗ? К чему это было?! Более того, ДО вашего сообщения уже было дано несколько правильных ответов.
    – Spawnet
    28 авг 2021 в 17:12
  • @Spawnet лично я поставил вопросу + и поставил ответам плюсы, именно для этого я и хотел, чтобы вопрос был корректно сформулирован. А так не совсем ясно: правильные ответы или нет. Я считаю, что у вас должна быть обратная связь. Поставил + вопросу потому, что есть очень интересные ответы и только поэтому. Кто-то поставил минус (а их 4) и просто пошел дальше. Отметил закрытие (тоже 4) и пошел дальше. А я пишу, и вы знаете что мне не нравится и можете сделать выводы. Что до вопроса, то можете для эксперимента убрать комментарий с ожидаемым выводом и узнать, сможет ли кто-то понять что надо.
    – Leonid
    29 авг 2021 в 13:34

3 ответа 3

4

Без возможных потерь данных (если помимо name в объектах будут еще свойства):

const orderIds = [4, 2, 1, 3];
const people = [
  { id: 3, name: 'Tom'},
  { id: 2, name: 'Bill'},
  { id: 1, name: 'Jack'},
  { id: 4, name: 'Bob'},
];

(
  ppl => console.log(...orderIds.map(id => ppl[id].name))  // 2. вывод данных, два прохода
)(
  Object.fromEntries(people.map(     // 1. преобразование из массива в объект, два прохода
    ({ id, ...rest }) => [id, { ...rest }]
  ))
);

Конструкции вида Object.fromEntries(arr) и ...arr.map() содержат "скрытые" циклы - вместо них можно использовать reduce или процедурный цикл, в каждом таком случае получив один проход по данным вместо двух.
Таким образом, это примерно среднее по эффективности решение. Более скоростная комбинация редьюсов/форов и IIFE выглядела бы уж чересчур эзотерично :)

Кстати об IIFE, оно здесь только чтобы веселее было (это по сути делает хэштаблицу автоосвобождаемой, готовой к GC сразу после выполнения выражения) - так делать совсем не обязательно, под хранение ХТ можно и переменную ввести. Показываю только потому что на собеседованиях такие трюки иногда добавляют "звездочек".

А вложенные циклы (перепоиск/перефильтровка элементов на каждой итерации, как в принятом ответе) == зло: такие решения наименее производительны, плюс иногда приводят к гейзенбагам 😱. Минздрав предупреждает.

11
  • «на собеседованиях такие трюки иногда добавляют "звездочек"» - тут ключевое слово "иногда" :) Более беспроигрышный вариант зазвездиться на собеседовании, имхо, это ООП-решение через паттерны... сложно, зато ынтерпразно (сам не понимаю чем так важны JS-разработчику паттерны, но меня почти на каждом техн. собеседовании о них спрашивали 🤷).
    – yar85
    27 авг 2021 в 15:16
  • console.log(...orderIds.map(id => people.find(item => item.id === id).name)); - а почему нельзя так? И почему вообще все выводят только имена объектов, когда вроде надо вывести объекты в порядке, указанном в orderIds?
    – Leonid
    27 авг 2021 в 15:24
  • 1
    @Leonid, так можно, но тогда поиск (вызов find проходящий по массиву) будет выполняться на каждой итерации map - это неэффективная лишняя возня с данными, плюс она добавляет вероятность ошибки. Поэтому каст массива в хэштабличку предпочтительнее: один раз пробежались, и далее используем моментальный доступ по ключу. «И почему вообще все выводят только имена объектов, когда вроде надо вывести объекты в порядке, указанном в orderIds?» - потому что несмотря на формулировку ТСа, коммент в коде вoпроса показывает желаемый результат... лично я больше доверяю коду, чем словесному описанию)))
    – yar85
    27 авг 2021 в 15:33
  • 1
    Я заменил Object.values(people).map() на people.map() и ничего не изменилось.
    – Leonid
    27 авг 2021 в 15:53
  • 1
    Потратил минут 5-10, чтобы понять код, после уже кажется не таким уж и сложным) Однозначно плюсик. 28 авг 2021 в 5:41
2

Коллега, подобный мелкий саботаж учебных заданий дорого обойдется при трудоустройстве. Сокращенный синтаксис управления объектами как множеством полей недвусмысленно сокращает время написания кода, его нужно знать!

const orderIds = [4, 2, 1, 3]

const people = [
    { id: 3, name: 'Tom'},
    { id: 2, name: 'Bill'},
    { id: 1, name: 'Jack'},
    { id: 4, name: 'Bob'}
]

const peopleMap = people.reduce((acm, cur) => ({...acm, [cur.id]: cur.name}), {});
console.log(...orderIds.map((id) => peopleMap[id]));
3
  • Спасибо за ответ. Но не очень понял на счёт саботажа. Мне задали такую задачу на собеседовании, я не справился. Попросил помощи здесь. В чём саботаж? Не думаю, что кому-то создаст проблем этот вопрос, если он наткнётся на него перед своим собеседованием. Это должно быть просто мировое совпадение, если именно тому человеку попадётся этот вопрос именно на его следующем собеседовании)
    – Spawnet
    27 авг 2021 в 13:47
  • 2
    @Spawnet В таком случае, обознался) Вопрос выглядит как домашняя работа абстрактного студента абстрактного института 27 авг 2021 в 13:53
  • Может так это и выглядит) Но по факту, это долбанный провал и потеря очень хорошей вакансии, чтоб меня( Никак не идёт углублённый ES6.
    – Spawnet
    27 авг 2021 в 13:56
2

const orderIds = [4, 2, 1, 3]

const people = [
    { id: 3, name: 'Tom'},
    { id: 2, name: 'Bill'},
    { id: 1, name: 'Jack'},
    { id: 4, name: 'Bob'}
]

let result = orderIds.map(el => people.filter(item => item.id == el)[0].name)
console.log(...result)

2
  • Благодарю! Компактно и понятно (станет со временем, когда это всё уляжется в голове)
    – Spawnet
    27 авг 2021 в 14:00
  • Тогда логичнее использовать find() вместо filter()[0]. Выглядит очевиднее.
    – Leonid
    27 авг 2021 в 15:28

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.