Как добавить запятую ко всем элементам в цикле кроме последнего?

Question

[Sergey]

Есть цикл:

<?php if ( have_rows( 'fifth-section' ) ) : ?>
		<?php while ( have_rows( 'fifth-section' ) ) : the_row(); ?>
		<?php if ( get_sub_field( 'display' ) == 1 ) { ?>
			"offers": {
				"@type": "AggregateOffer",
				"offers": [
				<?php if( have_rows('plans') ): ?>
				<?php 
					$loop_counter = 0;
					$count_block=1;
					while( have_rows('plans') ): the_row(); 
					if($loop_counter%4 == 0){
						if($loop_counter > 0){ 
						?>
					<?php } ?>
					<?php }
					$subtitle = get_sub_field('subtitle');
				?>							
				{
					"@type": "Offer",
					"name": "<?php echo $subtitle; ?>",
					"url": "<?php echo get_permalink(); ?>"
				},
				<?php 
					$loop_counter++;
					$count_block++;
					endwhile;
				?>
				<?php endif; ?>
			<?php } else { } ?>
			<?php endwhile; ?>
			<?php endif; ?>
			]
		}
	}

Который выводит кусок для разметки schema

"offers": {
			"@type": "AggregateOffer",
			"offers": [														
				{
					"@type": "Offer",
					"name": "Название",
					"url": "https://site.ru/assets/1"
				},
											
				{
					"@type": "Offer",
					"name": "Название 2",
					"url": "https://site.ru/assets/2"
				},
			]
		}
	}

Проблема в том, что у последнего элемента Offer после закрывающей скобки } не должно быть запятой. Элементов может быть от 1 до 4. Собственно вопрос, как прописать запятую всем элементам, кроме последнего (и, разумеется, вообще ее не добавлять, если элементов 1)?

Спасибо за рекомендации.

Comments

[OVK2015]

Пихать эл-ты в массив, а потом implode,- не вариант?

[Nujabes37]

Господи, что это такое))

Answer 1

[Дмитрий Дерепко]

Добавьте if и сравните в нем "номер текущего элемента не равен номеру последнего"

Comments

[Sergey]

Пробовал разные варианты, только учусь, потому тямы не хватает). Не набросаете пример?

[Mikhail Osher]

Sergey, уж простите, но Вам уже разжевали ответ. Думайте сами дальше.

[Дмитрий Дерепко]

Sergey, вообще для задачи вида "добавить разделитель везде, кроме последнего элемента" обычно решаются через implode()

Можете поступить так же. Сначала формируете массив из этих строк

$elements = [];
foreach(...) {
$elements[] = '{
          "@type": "Offer",
          "name": "Название",
          "url": "https://site.ru/assets/1"
        }';
}

а потом выводите в нужном месте
echo implode($elements, ',');

[Sergey]

Дмитрий Дерепко, спасибо большое за разъяснения, так и сделаю)

Answer 2

[Антон Шаманов]

лучше сформировать массив и вывести его с помощью json_encode, будет короче и читабельнее

Answer 3

[Sergey]

В общем, сам спросил - сам ответил, может кому пригодится. Не сразу сообразил, но гораздо легче добавить запятую перед каждым элементом, кроме первого). Примерно так в моем случае:
<?php if ($loop_counter > 0) echo ',' ?>

Вот так у меня работает корректно:

<?php if ( have_rows( 'fifth-section' ) ) : ?>
    <?php while ( have_rows( 'fifth-section' ) ) : the_row(); ?>
    <?php if ( get_sub_field( 'display' ) == 1 ) { ?>
      "offers": {
        "@type": "AggregateOffer",
        "offers": [
        <?php if( have_rows('plans') ): ?>
        <?php 
          $loop_counter = 0;
          while( have_rows('plans') ): the_row(); 
          if($loop_counter%4 == 0){
            if($loop_counter > 0){ 
            ?>
          <?php } ?>
          <?php }
          $subtitle = get_sub_field('subtitle');
        ?>
	<?php if ($loop_counter > 0) echo ',' ?>
        {
          "@type": "Offer",
          "name": "<?php echo $subtitle; ?>",
          "url": "<?php echo get_permalink(); ?>"
        },
        <?php 
          $loop_counter++;
          endwhile;
        ?>
        <?php endif; ?>
      <?php } else { } ?>
      <?php endwhile; ?>
      <?php endif; ?>
      ]
    }
  }

Comments

[Дмитрий]

Вряд ли ЭТО кому-то пригодится.
Ваш код надо выкинуть и написать новый, нормальный.

[Sergey]

Дмитрий, возможно, я только учусь и не знаю всех тонкостей. Однако знаю одно, ЭТО мне подсказали на stackoverflow и ЭТО работает у меня корректно, и другого пока нет. Вот Ваш снисходительный комментарий точно вряд ли кому-то пригодится. Вы, как куратор тега PHP, кстати, могли бы подсказать новичку что в этом коде не так или просто промолчать, если лень.

[Дмитрий]

Sergey, первое "не так" - куча if(){}elst{}, второе "не так", это пытаться сделать велосипед по расстановке запятых, третье "не так" - нет описания задачи, чтобы понять, что именно Вам надо.
Так что свои ехидные замечания оставьте себе.

[Дмитрий]

Sergey, Вам OVK2015 и Антон Шаманов озвучили лучшие варианты решения, а Дмитрий Дерепко дал практически готовое решение.
Но думать Вы не хотите, довольствуетесь дурнопахнущим кодом.
Так что Вам ещё отвечать?

[Sergey]

Дмитрий, мои ехидные замечания? Если посмотреть выше, то очевидно - это ваша стезя.

нет описания задачи, чтобы понять, что именно Вам надо

т.е. Вы не поняли что требуется, но однозначно решили самоутвердиться, вставив свои «пять копеек» на тему «дурнопахнуещего» кода с переходом на личности. Ок)

if else это к разработчикам ACF. Из их официальной документации взята проверка полей.

Я благодарен OVK2015, Антон Шаманов и Дмитрий Дерепко и принял к сведению их рекомендации, а также изучаю возможности сделать этот код лучше, по их рекомендациям в том числе. Но на данном этапе мне сложно сделать сходу что-то лучшее, повторюсь, я только учусь.

Так что Вам ещё отвечать?

Этого я и не понял. Зачем Вы стали отвечать?

[Дмитрий]

Sergey, если Вы так благодарны ответившим, то Вы обязаны отметить, что их ответы Вам понравились, а лучше выбрать их решением.
Свой ответ с недокодом уберите.

[Sergey]

Дмитрий, спасибо за советы, я прислушаюсь к ним и, возможно, воспользуюсь.

[Дмитрий]

Sergey, не возможно, а так и сделайте.