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

Question

[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'}
        ]
    },
]

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

Comments

[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.

[Вадим]

Не уверен правда зачем Вам нужно изменять массив участников, но если не задавать Вам лишних вопросов и просто подкорректировать чтоб работало, то как-то так:

hotel.map(h => {
    h.room.map(() => {
      return participants.shift()
    })
    return h
  })

[v1sher47]

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

[historydev]

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

[v1sher47]

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

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

[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

[GeminiX]

Добавил return в map (через forEach так же работает)

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

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

const result = hotel.map(h => {
    return h.room.map((r) => {
      r.id = participants[0].id
      r.name = participants[0].name
      participants.shift()
      return r
    })
  })

console.log(result)

вот результат

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

Comments

[v1sher47]

Александр, Спасибо Вам большое - решение реально правильное и у меня работает. С Возвращением (return) не до конца разубираюсь(( есть над чем поработать

[v1sher47]

Александр, все бы ничего, но код меняет структуру там в массиве другие массивы, а должно быть массив с объектами внутри которых массивы

ваш код возвращает [Array(1), Array(2), Array(1)]

а нужно [{}, {}, {}]

[v1sher47]

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

массивы с объектами - в которых есть еще массивы с объектами

[v1sher47]

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'}
        ]
    }
]

[GeminiX]

v1sher47, тогда так

const result = hotel.map((h) => {
  return {
    ...h,
    room: h.room.map((r) => {
      r.id = participants[0].id;
      r.name = participants[0].name;
      participants.shift();
      return r;
    }),
  };
});

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

Answer 2

[otdameskapizm]

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

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;
};

Answer 3

[Сергей Соколов]

Удобно использовать Object.assign() — метод копирует все перечисляемые свойства в целевой объект:

const target = { a: '', b: 'V'}; // скопируем сюда
const source = { a: 'AAA', c: 'CCC' }; // отсюда
Object.assign(target, source);
target // { a: "AAA", b: "V", c: "CCC" }

Если не беда, что опустошатся participants, то можно так:

hotel.forEach(({ room }) => room.forEach(slot => Object.assign(slot, participants.pop())));

Ну или так, с нетронутыми participants.

let i = 0;
hotel.forEach(({ room }) =>
  room.forEach((slot) => Object.assign(slot, participants[i++]))
);

Comments

[v1sher47]

Метод безусловно хорошо, но всю структуру нет смысла переносить - когда в принципе нужно задать пара параметров