Нужно вставить div через каждые 4 блок, но почему вставляет по своему?

Question

[Гарольд]

Есть 24 блока, по количеству они не измены.

$i = 0;
$all = count($result["rows"]);
$k = round($all / 6);

foreach ($result['rows'] as $row)
{	
	
	if($i == 0)
		{
			$all--;
			$k = (round($all / 6) > 0 ? round($all / 6) : 1);			
			echo '<div class="column large-12 list__delimiter"></div>';
		}

		$i++;

	if($i == $k)
	{			
		$i = 0;
	}	

echo '<div class="list"><img></div>';
}

Все нормально выводит только для трех первых строчек (по 4 блока и далее разделитель), а дальше начинается по 3 блока в строчке, заканчивая 2 блоками.

Почему такое происходит?

Answer 1

[Алексей]

Я если честно не понял манипуляцю с $all, нужен ведь только счетчик

$i = 0;
foreach ($result['rows'] as $row) {	
	if ($i === 4) {
		echo '<div class="column large-12 list__delimiter"></div>';
		$i = 0;
	} else {
		$i++;
	};
  echo '<div class="list"><img></div>';
};

Comments

[Гарольд]

Спасибо

echo '<div class="column large-12 list__delimiter"></div>';
$i = 1;

Теперь работает

[Сергей delphinpro]

Зачем счетчик-то обнулять

$i = 0;
foreach ($result['rows'] as $row) {	
  if ($i++ % 4 === 0) {
    echo '<div class="column large-12 list__delimiter"></div>';
  };
  echo '<div class="list"><img></div>';
};

[Алексей]

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

Answer 2

[Артем]

Для разбиения массива на части воспользуйтесь функцией array_chunk, и выводите поблочно.

Comments

[Гарольд]

Спасибо