1

Вывести все элементы массива, по три в обертке. Подскажите, как лучше реализовать, пока говнокод и понимаю сам. Дан массив $images с числом изображений (число N). Нужно его обойти

Пример массива 

$images[0], $images[1], $images[n]
$images[0] => $images[0]['popup'],$images[0]['src']
$images[1] => $images[1]['popup'],$images[1]['src']
$images[n] => $images[n]['popup'],$images[n]['src']

$imax=count($images);
for ($i = 0 ; $i < $imax ; $i++) {
    print '<div class="item">';

    $images[$i]['popup']; 
    $i++;if($i==$imax){echo '</div>';break;} 

    $images[$i]['popup']; 
    $i++;if($i==$imax){echo '</div>';break;} 

    $images[$i]['popup']; 
    $i++;if($i==$imax){echo '</div>';break;} 

    print '</div>';
}

Вариант, решения, взятый из комментариев —

$tmpArray = array_chunk($image,3);
foreach($tmpArray as $_image) {
  echo '<div>';
  foreach($_image as $_subImage) {
     echo $_subImage;
  }
  echo '</div>';
}

и

function split_my($arr, $size)
{
    foreach(array_chunk($arr, $size) as $val)
    {
        echo "<div>".
             implode("", array_map(function($value){ 
                 return "<span> ".$value." </span>"; 
             }, $val)).
             "</div>";
    }
}
6
  • покажите пример массива
    – ikerya
    1 июл 2016 в 9:14
  • poprobuy poka eto poka pridumayu luchsh for ($i = 0 ,$imax=count($images),$cnt = 0; $i < $imax ; $i++) { $cnt++; if($cnt == 1){ echo '<div class="item">'; echo '<span>'.$images[$i]['popup'].'<span>'; } else { if($cnt == 3) $cnt = 0; echo '<span>'.$images[$i]['popup'].'<span>'; } } 1 июл 2016 в 9:23
  • array_chunk и implode спасут вас
    – splash58
    1 июл 2016 в 9:28
  • @splash58, эти функции облегчат вывод и получится меньше строк? 1 июл 2016 в 9:29
  • конечно. первой сразу побьете на нужные куски, второй соедините дивами
    – splash58
    1 июл 2016 в 9:30

4 ответа 4

2

Я поправил решение

 $images = [
    ['popup'=>'AAA'],
    ['popup'=>'BBB'],
    ['popup'=>'CCC'],
    ['popup'=>'EEE'],
    ['popup'=>'FFF'],
    ['popup'=>'JJJ'],
];


function showItems($images, $step){
    $cnt = 0;
    for ($i = 0 ,$imgCnt=count($images); $i < $imgCnt ; $i++) {
        $cnt++;
        if($cnt == 1){
            echo '<div class="item">';
        }

        echo  '<span>'.$images[$i]['popup'].'</span>';

        if($cnt == $step or $i == $imgCnt-1){
            $cnt = 0;
            echo '</div>';
        }
    }
}
showItems($images, 3);

Самое на мой взгляд оптимальное и короткое решение. Функция универсальная .Выводит количество $images по заданному $step

13
  • Ваш вариант работает, но 1 div либо лишний, либо недостающий. Скорее лишний 1 июл 2016 в 9:39
  • so hard..........
    – ikerya
    1 июл 2016 в 9:44
  • Там лишнего ничего нет,все корректно открываться и закрывается. Я сейчас постараюсь облегчить его 1 июл 2016 в 9:45
  • а если количество картинок не делится на цело на 3?
    – Grundy
    1 июл 2016 в 11:49
  • У меня было 5 каринок, поэтому что-то с числом дивов было не так 1 июл 2016 в 12:05
1

Итераторы кто-нибудь использует?

reset(), current(), next(), prev()

$i = 0;
$html = '';
reset( $images);
do {
    if( 0 === $i%3)   $html .= '<div class="item">';
    $html .= sprintf('<img src="%s" alt="">', current($images)['popup']);
    if( 0 === ++$i%3) $html .= '</div>' . PHP_EOL;
} while( next( $images));
if( 0 !== $i%3) $html .= '</div>' . PHP_EOL;
echo $html;

Вариант с array_chunk() и array_map():

echo implode("\n", 
    array_map( 'wrapRow',
        array_chunk(
            array_map( 'wrapImage', $images),
            3
        )
    )
);
function wrapRow( $els) {
    return sprintf('<div class="item">%s</div>', implode('', $els));
}
function wrapImage($image) {
    return sprintf('<img src="%s" alt="">', $image['popup']);
}

Сначала из массива $images получаем массив тегов <img> где подставлен url из поля popup. Т.е. такой же длины массив, но там уже теги.

Потом бьём его на группы по три (или меньше в последней) с помощью array_chunk() – получаем массив из массивов по 3, или меньше, <img>.

Потом эти тройки склеиваем implode() и оборачиваем в <div class="item">.

И выводим, склеив <div>'ы символом новой строки.


9
  • сорри но downvote но немогу пройти мимо if в одну строку и т.д. и т.п. память на знаках переноса строки и {} в угоду восприятия экономить не кошерно.
    – Naumov
    1 июл 2016 в 10:06
  • на вкус и цвет минимайзера нет )
    – Sergiks
    1 июл 2016 в 10:06
  • @Naumov, ничего страшного, я компенсирую)) Мне понравился вариант с использованием еще одного array_map и выносом функций за пределы, в отличие от моего ответа с внешним foreach.
    – Alex Krass
    1 июл 2016 в 10:08
  • Да нет не спорю но в первом варианте вы чистый foreach изобрели php.net/manual/ru/control-structures.foreach.php судя по документации.
    – Naumov
    1 июл 2016 в 10:09
  • @Naumov, я несу в народ слово PHP'вое! : ) Здесь это чистый foreach(), но завтра, может, понадобится гулять по большому массиву взад и вперёд!
    – Sergiks
    1 июл 2016 в 10:11
0

Хорошо, я делаю обычно так.

Если мне требуется что-то сделать с массивом, я открываю документацию и смотрю на встроенные функции, могут ли они мне помочь. Опа, смотрите, там есть array_chunk, почему бы и нет.

Но это не все, для прохождения по всем элементам подмассива можно использовать map_array, связать все это через implode и вывести. А давайте теперь все это оформим в универсальную функцию.

function split_my($arr, $size)
{
    foreach(array_chunk($arr, $size) as $val)
    {
        echo "<div>".
             implode("", array_map(function($value){ 
                 return "<span> ".$value." </span>"; 
             }, $val)).
             "</div>";
    }
}

$arr = [1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16];
split_my($arr, 3);

Из минусов данного алгоритма - жрет память на создание копии массива array_chunk. Ну и у Вас это будет не $value, a $value['popup'] насколько я понял.

4
  • Она не очень универсальная - html код перемешан с php. А вот что бы память не "жрало", можно воспользоваться array_slice
    – KoVadim
    1 июл 2016 в 9:43
  • @KoVadim 1. Я сконцентрировался на основной задаче, вроде разделить разметку или подключить шаблонизатор, это уже дело простое. Тут главное обвязка под это. 2. Я попытался избавиться от получения длинны массива и доп.расчетов в явном виде. Если использовать array_slice придется считать offset и проверять вернувшийся массив на пустоту. А если использовать array_splice и избавится им от offset, то остается проблема с изменением исходного массива, что тоже может быть не хорошо и опять же проверка на пустоту. Поправьте, если где-то недодумал.
    – Alex Krass
    1 июл 2016 в 9:50
  • шаблонизатору можно просто вернуть результат array_chunk. я бы разделил на два куска. А вот боятся простых вычислений, но при этом сэкономив память и выиграв в скорости, не нужно. Но это все нужно профайлить, профайлить и ещё раз профайлить.
    – KoVadim
    1 июл 2016 в 9:57
  • Я все же проголосую за Вас, суть ответа была в упрощении кода, для чего и служат те, функции, что Вы и использовали, я бы ради эксперимента использовал вместо implode foreach. Суть Вы донесли первее! Спасибо Вам огромное 1 июл 2016 в 16:27
0

Не понимаю людей удаляющих ответы! вы что минусов боитесь рейтинг потерять?

for ($i = 0; $i < count($image) ; $i++) {
    if($i%3 == 0) {
        echo '<div>';
    }

    echo $image[$i];

    if($i%3 == 0) {
        echo '</div>';
    }
}
if($i%3!=0) {
   echo '</div>'; // что бы если в конце осталось 2-ва элемента закрыть div
}

или

$tmpArray = array_chunk($image,3);
foreach($tmpArray as $_image) {
  echo '<div>';
  foreach($_image as $_subImage) {
     echo $_subImage;
  }
  echo '</div>';
}

второй вариант лучше выглядет

6
  • Смысла нет, чего-то боятся. Без ошибок ничего не выйдет. Вечером буду все варианты пробовать. Определить лучший становится труднее 1 июл 2016 в 9:47
  • Просто там действительно ошибка была, а времени на исправление особо нет, так что решил не засорять.
    – JimmDiGriz
    1 июл 2016 в 9:48
  • @DanielOlivo У вас лишь одна ошибка, в конце вы ` if($i%3 == 0)` эту проверку деления на 3 заменили на что то другое по этому собственно и div недостающий ну и последнее конечно условие. А так ответ же правильный был.
    – Naumov
    1 июл 2016 в 9:51
  • 1
    for ($i = 0; $i < count($image) ; $i++) Tak nikogda nedelayut te kto ponimayut v optimizacii,potomuch to pri kajdoy iteracii budet podschitivatsya count($image),esli itraciy mnogo to ono sojret mnogo pamyati i vremeni. a pravilno ego peremestit v perviy blok vot tak for ($i = 0,$cnt = count($image); $i < $cnt ; $i++),gde inicializaciya $cnt = count($image) budet odin raz i v ostalnix iteraciyax prosto budet postavlyatsya ego znachenie,chto namnogo luchshe,soglasits 1 июл 2016 в 9:57
  • Как по мне, так его лучше вообще вынести за цикл for надо 1 июл 2016 в 12:03

Ваш ответ

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

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