Как перебрать массив в массиве и присвоить каждому элементу значения из другого массива?

Question

v1sher47 @v1sher47

Как перебрать массив в массиве и присвоить каждому элементу значения из другого массива?

Всем привет!

Есть участники participants,

const participants = [
    {id: 1, name: 'Anakin Skywalker'},
    {id: 2, name: 'Padmé Amidala'},
    {id: 3, name: 'Luke Skywalker'},
    {id: 4, name: 'Princess Leia'},
]

их нужно расселить по номерам в отеле в любом порядке (без разницы кто с кем).
Отель предоставляет ровно то колличесво мест - сколько участников (5 комнат по одному месту или 2 комнаты по 3 места и тд - не рабочий вариант)

const hotel = [
    {
        room: [
            {id: '', name: ''}
        ]
    },
    {
        room: [
            {id: '', name: ''},
            {id: '', name: ''}
        ]
    },
    {
        room: [
            {id: '', name: ''}
        ]
    }
]

Мое решение

hotel.map(h => {
    h.room.map((r) => {
      r.id = participants[0].id
      r.name = participants[0].name

      participants.shift()

    })
  })

Здесь я перебираю массив hotel и нахожу в нем еще массив room - перебирая его, я присваиваю каждому объекту значения первого участника из массива participants и по завершению установки значений - удаляю его используя метод shift() (аналогичный ему .splice(0, 1))

Когда в room один объект (одно место), то каждый участник находится в своей комнате(отрабатывает на ура), а когда в комнате можно расселить двух участников или больше, происходит перетирание на последнего участника в этой комнате

Результат

const hotel = [
    {
        room: [
            {id: '1', name: 'Anakin Skywalker'}
        ]
    },
    {
        room: [
            {id: '3', name: 'Luke Skywalker'},  // должна быть {id: 2, name: 'Padmé Amidala'},
            {id: '3', name: 'Luke Skywalker'}
        ]
    },
    {
        room: [
            {id: '4', name: 'Princess Leia'}
        ]
    }
]

Так же еще пример результата другого отеля с другими номерами

const hotel = [
    {
        room: [
            {id: '', name: ''},
            {id: '', name: ''}
        ]
    },
    {
        room: [
            {id: '', name: ''},
            {id: '', name: ''}
        ]
    },
]

const hotel = [
    {
        room: [
            {id: 2, name: 'Padmé Amidala'},
            {id: 2, name: 'Padmé Amidala'}
        ]
    },
    {
        room: [
            {id: '4', name: 'Princess Leia'},
            {id: '4', name: 'Princess Leia'}
        ]
    },
]

Помогите пожалуйста понять, что не так я делаю((

Вопрос задан более двух лет назад
285 просмотров

7 комментариев

Подписаться 1 Простой 7 комментариев

Rsa97 @Rsa97
Меня терзают смутные сомненья. А так ли вы создаёте массив hotel? Предполагаю, что в rooms вставляются не отдельные объекты, а один и тот же по ссылке.

const t = [{ id: null }, { id: null }]; t[0].id = 1; t[1].id = 2; console.log(JSON.stringify(t)); // [{"id":1},{"id":2}]

const place = { id: null }; const t = [place, place]; t[0].id = 1; t[1].id = 2; console.log(JSON.stringify(t)); // [{"id":2},{"id":2}]

const place = { id: null }; const t = [Object.assign({}, place), Object.assign({}, place)]; t[0].id = 1; t[1].id = 2; console.log(JSON.stringify(t)); // [{"id":1},{"id":2}]

P.S. И, раз уж вы правите объект in place, то логичнее использовать не map, а forEach.
Написано более двух лет назад
Вадим @MrDecoy Куратор тега JavaScript
Не уверен правда зачем Вам нужно изменять массив участников, но если не задавать Вам лишних вопросов и просто подкорректировать чтоб работало, то как-то так:

hotel.map(h => { h.room.map(() => { return participants.shift() }) return h })
Написано более двух лет назад
v1sher47 @v1sher47 Автор вопроса

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

Написано более двух лет назад
historydev @historydev Куратор тега JavaScript

Вадим, лучше этого не делать)))

Написано более двух лет назад
v1sher47 @v1sher47 Автор вопроса

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

Я бы с удовольствием посмотрел (возможно бы мог использовать) и был бы Вам благодарен, за другой пример - не меняя массив с участниками)

Написано более двух лет назад
Rsa97 @Rsa97

v1sher47, Это не решение. Я вам показываю, что происходит, если в массиве ссылки на один и тот же объект.

Написано более двух лет назад
Вадим @MrDecoy Куратор тега JavaScript
v1sher47, На скорую руку:

let participantIdx = 0 hotel.forEach(h => { h.room.forEach(r => { const p = participants[participantIdx] r.id = p.id r.name = p.name participantIdx += 1 }) })

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

Меня терзают смутные сомненья. А так ли вы создаёте массив hotel? Предполагаю, что в rooms вставляются не отдельные объекты, а один и тот же по ссылке.

const t = [{ id: null }, { id: null }]; t[0].id = 1; t[1].id = 2; console.log(JSON.stringify(t)); // [{"id":1},{"id":2}]

const place = { id: null }; const t = [place, place]; t[0].id = 1; t[1].id = 2; console.log(JSON.stringify(t)); // [{"id":2},{"id":2}]

const place = { id: null }; const t = [Object.assign({}, place), Object.assign({}, place)]; t[0].id = 1; t[1].id = 2; console.log(JSON.stringify(t)); // [{"id":1},{"id":2}]

P.S. И, раз уж вы правите объект in place, то логичнее использовать не map, а forEach.
Не уверен правда зачем Вам нужно изменять массив участников, но если не задавать Вам лишних вопросов и просто подкорректировать чтоб работало, то как-то так:

hotel.map(h => { h.room.map(() => { return participants.shift() }) return h })
Rsa97, Такой массив с логикой уже присутствует и каждый из комнат создается в ручную с добавлением. Не до конца понял ваше решение, но все равно спасибо за помощь!
Вадим, В принципе, не обязательно менять (изменять массив с участниками). Но по той логике, что я описал, я обращаюсь к каждому первому элементу массива и беру его данные, после чего задаю их.

Я бы с удовольствием посмотрел (возможно бы мог использовать) и был бы Вам благодарен, за другой пример - не меняя массив с участниками)
v1sher47, Это не решение. Я вам показываю, что происходит, если в массиве ссылки на один и тот же объект.
v1sher47, На скорую руку:

let participantIdx = 0 hotel.forEach(h => { h.room.forEach(r => { const p = participants[participantIdx] r.id = p.id r.name = p.name participantIdx += 1 }) })

Но это корректно будет работать только если комнат действительно ровно столько же, сколько человек.

Answer 1 · 2022-11-03 10:01:53

Вот эта функция не меняет входные данные, так как она внутри себя создает новые и их возвращает

const checkIntoHotel = (hotel, people) => {
  let index = 0;
  const checkedHotel = hotel.map(({ room }) => room.map(({ id, name }) => {
    const {id: parId, name: parName } = people[index];
    index += 1;
    return { id: parId, name: parName };
  }))
  return checkedHotel;
};

Как перебрать массив в массиве и присвоить каждому элементу значения из другого массива?

Войдите на сайт