Как красиво/правильно скомбинировать значения из двух массивов, один из которых короче?

Question

[Тамара Ламова]

Есть два массива:

$arr1 = ['a',  'b',  'c',  'd',  'e',  'f',  'g',  'h',  'i',  'j',  'k',  'l',  'm',  'n']
$arr2 = [1, 2, 3, 4];

Есть необходимость «слить» эти два массива в один,
причём, когда во втором заканчиваются значения, возвращать его к началу,
чтобы получилось так:

$result = ['a1',  'b2',  'c3',  'd4',  'e1',  'f2',  'g3',  'h4',  'i1',  'j2',  'k3',  'l4',  'm1',  'n2'];

Пока что сделала незамысловато:

$index = 0; $result = [];
foreach ($arr1 as $item) {
    if (!isset($arr2[$index])) $index = 0;
    $result[] = $item.$arr2[$index];
    $index++;
}

Но выглядит это неочень…

Как сделать лаконичнее / правильнее ?

Comments

[FanatPHP]

Ну тут если только по мелочи.
Постфиксный инкремент не используется по назначению, то есть алгоритм вполне можно сократить на одну строчку.
Вместо условия можно написать тернарный оператор.
И оформить код по PSR, то есть $result = []; написать отдельной строкой.

А в остальном код уже лаконичный и правильный.
Иногда не нужно искать более лаконичное решение - оно может оказаться менее эффективным и/или более сложным для понимания.

[FanatPHP]

Интересно, откуда идет такой трафик на этот вопрос

[FanatPHP]

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

[FanatPHP]

Ну и если делать совсем по феншую, то надо проверить, какой массив больше, и не пустые ли они.

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

FanatPHP, читаем условия внимательно:

когда во втором заканчиваются значения

[FanatPHP]

Сергей Соколов, в том-то и вопрос - какой считать вторым.
С опытом приходит понимание что входные данные не всегда бывают такими, как ожидается

[Тамара Ламова]

FanatPHP, в данном случае ничего не нужно )) :
задача поставлена точно,
номера массивов обозначены цифрами,
важно обойти полностью именно первый массив ))

[FanatPHP]

Да я не про задачу, а про реальную жизнь.
Которая начинается после учебных заданий

Answer 1

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

У вас вполне хороший код. Можно чуть изменить, полагаясь на то, что именно второй массив короче первого:

$result = [];
$limit = count($arr2);
foreach ($arr1 as $index => $item) {
    $result[] = $item . $arr2[$index % $limit];
}

Или короче, хуже, менее читаемо:

Ацкий однострочник

$result = array_map(function($item, $i) use ($arr2) {return $item . $arr2[$i % count($arr2)];}, $arr1, array_keys($arr1));

Comments

[FanatPHP]

Ооо, полетело "усрусь но не сдамся" на вентилятор

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

FanatPHP, ))) давненько не брал я в руки шашки. Насколько это всё удобнее в JS.

[Тамара Ламова]

ОМГ!
Я конечно предполагала через array_map, но это слишком головоломно.
Тот кто будет разбирать/править этот код после — проклянёт меня )))
Дайти попроще )

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

Тамара Ламова, ну у вас и правда вполне хороший код. Можно чуть изменить:

$result = [];
foreach ($arr1 as $index => $item) {
    $result[] = $item . $arr2[$index % count($arr2)];
}

[Тамара Ламова]

Сергей Соколов, вычислять count($arr2) в каждой итерации цикла — такое себе %-)
А так, то что надо, спасибо )))
Выделите в отдельный ответ и отметим решением.

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

Тамара Ламова, обновил ответ

[FanatPHP]

на самом деле нет никакой проблемы получать count в каждой итерации, никаких "вычислений " там нет

[Тамара Ламова]

FanatPHP, в тех учебниках по которым я училась за count внутри цикла делали атата )))
даже за вот такой простой как тут

[FanatPHP]

Это могли быть либо учебники по другим языкам, либо их писали некомпетентные авторы, что в случае с РНР, увы, скорее правило, чем исключение.