php if 分岐＋foreach処理でforeachの2回目以降からifが効かない

Question

WP_Queryを用いてAdvanced Custom Fieldsプラグインのデータを抽出してリストした際、
画像カスタムフィールド「flyimg」が有る場合と無い場合で以下内容で分岐をしたいです。

有る場合は画像を配置。
無い場合の処理はカスタムフィールド 「artists」の画像を上限4枚並べるという内容で以下の構文を書きました。

4枚の画像を並べる方法はforeachを用いておりますが、
foreachが一度行われた後のループが、「flyimg」がある記事もforeachの内容が適応されてしまい、
解決策を見つける事ができず困っております。

前提として、「flyimg」の有無かかわらず、
検索対象記事に全てに「artists」の項目が入力されています。

1度目のforeachが行われる前までは、「flyimg」 trueの処理を行ってくれます。
foreachの記事が一番目に有る場合は、全ての結果がforeach内の処理となってしまいます。

<?php

  $currnet_date = date_i18n( 'y/m/d' );
  $args = array(
    'posts_per_page' => 10,
    'order' => 'DESC',
    'meta_key' => 'pickup',
    'meta_value' => true,
    'meta_query' => array(
            'relation' => 'AND',
            array(
                'key' => 'e-date',
                'value' => $currnet_date,
                'compare' => '>=',
                'type' => 'DATE'
            )
        )
  );

  $my_query = new WP_Query($args);
  if ($my_query->have_posts()) :
    while ($my_query->have_posts()) :
      $my_query->the_post();

　$FlyImg = get_field('flyimg',$value->ID);

?>

<?php if($FlyImg):?>
  <img class="img-fluid" src="<?php echo $Aimg; ?>" />
  <?php else: ?>
  <?php $artists = get_field('artist'); ?>
  <?php if($artists): ?>
  <div class="row no-gutters">
    <?php $count = 0 ;?>
    <?php foreach((array)$artists as $value):?>
    <?php if($count >= 4){ break; } ?>
        <div class="col-6">
          <img class="img-fluid" src="<?php echo $Bimg; ?>" />
        </div>
    <?php $count++; ?>
    <?php endforeach; ?>
  </div>
  <?php wp_reset_postdata();?>
  <?php endif; ?>
<?php endif;?>

助言いただければ幸いです。何卒宜しくお願い申し上げます。

Comments

[m.ts10806]

while がどこで閉じられているか分かりづらいのでインデントを整理していただきたいのですが・・・ ifもendif使ってたり}で閉じてたりで統一性がないので、そこも整理していただきたいです。

Answer 1

$FlyImg = get_field('flyimg',$value->ID);

そりゃまあ、毎回同じ投稿のflyimgの
空白判定してるわけですから
そうなるでしょうね

$value->ID←これ、要らないんじゃないですか？

Comments

[nats]

ご回答ありがとうございます！こちら削除したら問題が解決しました。
まだまだphpの初歩も理解できておらず、色々なページを参考にソースを組んでおりますので、無駄な処理を行ってたみたいです。本当にありがとうございました！