0

Здравствуйте. Есть одномерный массив объектов в которых хранится информация о баннерных изображениях на сайте. Например 4 элемента:

images = [
[0] => Object {src : path, size : '256x256'},
[1] => Object {src : path, size : '128x128'},
[2] => Object {src : path, size : '256x256'},
[3] => Object {src : path, size : '128x128'},
]

Мне нужно создать многомерный массив сгруппировав по ключу (size).

Что бы получилось:

result = [
[256x256] => [
   [0] => 'src_img1',
   [1] => 'src_img3'
]
[128x238] => [
   [0] => 'src_img2',
   [1] => 'src_img4'
]
]

Я делаю примерно вот так:

for (var i in images){
    result[images[i].size] = [];
    result[images[i].size][i] = images[i].src;
}

В таком случае в каждом из массивов ключи выглядят вот так:

result = [
    [256x256] => [
       [0] => 'src_img1',
       [1] => 'src_img3'
    ]
    [128x238] => [
       [2] => 'src_img2',  <-- неверная индексация
       [3] => 'src_img4'
    ]
    ]

Подскажите пожалуйста, каким образом можно построить этот массив, что бы индексация в каждом одиночном массиве начиналась с нуля?

3
  • не указывайте индекс [i] в конце своей реализации и индексация будет по порядку
    – Lexx918
    14 ноя 2017 в 12:10
  • @Lexx918 если не указать индекс - массив перетрётся строкой. Здесь вам тут не php)
    – Darth
    14 ноя 2017 в 12:11
  • @Darth не буквально же) .push() конечно, если надо в конец списка добавлять.
    – Lexx918
    14 ноя 2017 в 12:15

2 ответа 2

1
result = {}
for (var i in images){
    // чтоб не затирать массив, создаем только если его нет
    if (! result[images[i].size]) result[images[i].size] = []
    result[images[i].size].push(images[i].src);
}
// { '256x256': [ '1', '3' ], '128x128': [ '2', '4' ] }
9
  • тут лучше сделать result объектом
    – Grundy
    14 ноя 2017 в 12:24
  • согласен, затупил - дурное влияние php :)
    – splash58
    14 ноя 2017 в 12:25
  • @Grundy воообще этот подход не будет работать, потому что это же будет не тот результат, что ожидается в вопросе.
    – Darth
    14 ноя 2017 в 12:26
  • @Darth - eval.in/899697
    – splash58
    14 ноя 2017 в 12:27
  • @splash58 1) ТС хотел всё же массив, а не объект в результате 2) мой комментарий не в ваш огород)
    – Darth
    14 ноя 2017 в 12:29
1

Элементарно, Ватсон. Воспользуйтесь методом push

for (var i in images){
    result[images[i].size] = [];
    result[images[i].size].push(images[i].src);
}
4
  • вообще этот подход не должен работать, потому что в этом случае для каждого размера будет только один элемент, даже если их было больше
    – Grundy
    14 ноя 2017 в 12:12
  • @Grundy это к автору вопроса вопрос)
    – Darth
    14 ноя 2017 в 12:12
  • 1
    Это к автору ответа, потому что результат в ответе не тот, что ожидается в вопросе, хотя там то же самое - только номер ключа может быть другим
    – Grundy
    14 ноя 2017 в 12:14
  • Да, все работает как нужно. Пропустил очевидный вариант. Спасибо!
    – user137
    14 ноя 2017 в 12:19

Ваш ответ

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

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