0

В хтмл должно выглядеть примерно так:

<div class="slide">                                 <div class="ph1"><img src="img/hotel/ph1.png" alt="" /></div>
<div class="ph2"><img src="img/hotel/ph2.png" alt="" /></div>
<div class="ph3"><img src="img/hotel/ph3.png" alt="" /></div>
<div class="ph4"><img src="img/hotel/ph4.png" alt="" /></div>
</div>

<div class="slide">
<div class="ph1"><img src="img/hotel/ph1.png" alt="" /></div>
<div class="ph2"><img src="img/hotel/ph2.png" alt="" /></div>   
<div class="ph3"><img src="img/hotel/ph3.png" alt="" /></div>
<div class="ph4"><img src="img/hotel/ph4.png" alt="" /></div>
</div>

В пхп я сделал так:

<?$countmorephoto = 1;?>
<?foreach ($arrayimg as $keyimg => $imgmore):?>
<?$keyimg++;?>
    <?if ($countmorephoto == 1){?>
        <div class="slide">
    <?}?>
    <div class="ph<?=$countmorephoto?>"><img src="<?=$imgmore?>" alt="" /></div>
    <?if ($countmorephoto == 4 || $countmorephoto == 8 || $countmorephoto == 12 || $countmorephoto == 16){?>
        <?$countmorephoto = 0;?>
        </div>
        <div class="slide">
    <?}?>
<?if (!next($arrayimg)):?>  
        </div>
<?endif;?>
    <?$countmorephoto++;?>
<?endforeach;?>

Но работает честно говоря не очень :)

Как лучше разделять по 4 изображения, зная что их не может быть более 16.

2 ответа 2

2

Вы таки не поверите:

<?
$n = 0;
foreach ($arrayimg as $keyimg => $imgmore)
{
  $n++;
  if ($n % 4 == 1) { ?><div class="slide"><? }
  ?><div class="ph<?=$n?>"><img src="<?=$imgmore?>" alt="" /></div><?
  if ($n % 4 == 0) { ?></div><? }
}
if ($n % 4 > 0) { ?></div><? }
?>
0

Вы всегда пишете новую строку заключая её в <??> ?

Вам на говнокод, простите конечно.

<?$countmorephoto = 1;?>
<?foreach ($arrayimg as $keyimg => $imgmore):?>
<?$keyimg++;?>
    <?if ($countmorephoto == 1){?>
        <div class="slide">
    <?}?>
    <div class="ph<?=$countmorephoto?>"><img src="<?=$imgmore?>" alt="" /></div>
    <?if ($countmorephoto == 4 || $countmorephoto == 8 || $countmorephoto == 12 || $countmorephoto == 16){?>
        <?$countmorephoto = 0;?>
        </div>
        <div class="slide">
    <?}?>
<?if (!next($arrayimg)):?>  
        </div>
<?endif;?>
    <?$countmorephoto++;?>
<?endforeach;?>
5
  • 1
    реально жесть)
    – Palmervan
    12 янв 2012 в 14:06
  • А я о чем.
    – Artem
    12 янв 2012 в 14:10
  • 2
    Не используйте в php || // не делайте этого !11!АДИН // используйте or
    – user3545
    12 янв 2012 в 14:12
  • а разница ? Ну приоритеты разные, но это должен знать каждый! 12 янв 2012 в 20:35
  • 1
    @shurik, так это самое же: if ($countmorephoto == 4 || $countmorephoto == 8 || $countmorephoto == 12 || $countmorephoto == 16) if ($countmorephoto == (4 || $countmorephoto) == (8 || $countmorephoto) == (12 || $countmorephoto) == 16) У == приоритет 7, у || - 2. Поэтому либо в скобки, либо OR. Кстати, сам только недавно постиг эту штуку, писал все с огромным кол-вом скобок)
    – Sh4dow
    12 янв 2012 в 20:40

Ваш ответ

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

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