PHP：if文をif-else文にしたいが、波括弧をつけると何も表示されなくなる

Question

お世話になります。
PHP勉強中ですが、独学のため行き詰まってしまいました。
アドバイス、ヒント頂ければ幸いです。

###発生している問題・エラーメッセージ
作成済みのソースコードをif文からif-else文に変更したく思います。
以下のようにしてみましたがエラーなどでうまく動作しません。

###該当のソースコード

<?php
				foreach( $attachments as $id => $attachment ):
					if( get_post_thumbnail_id($post->ID) == $attachment->ID ) 
					continue;
					echo '<div class="img">';
					echo wp_get_attachment_image($id, "thumb_300", false);
					echo '</div>';
				endforeach;
?>

これを↓この様のしましたが「Parse error: syntax error, unexpected 'endif' (T_ENDIF) in～」となってしまいます

<?php
	foreach( $attachments as $id => $attachment ):
		if( get_post_thumbnail_id($post->ID) == $attachment->ID ) {
					continue;
					echo '<div class="img">';
					echo wp_get_attachment_image($id, "thumb_300", false);
					echo '</div>';}
		else {echo "失敗";}
		endif;
	endforeach;
?>

また以下のように試しにelse {echo "失敗";}やendif;を削除すると今度はこの部分（画像が入ります）が何も表示されなくなります。

<?php
	foreach( $attachments as $id => $attachment ):
		if( get_post_thumbnail_id($post->ID) == $attachment->ID ) {
					continue;
					echo '<div class="img">';
					echo wp_get_attachment_image($id, "thumb_300", false);
					echo '</div>';}
	endforeach;
?>

初心者なので何か基本的なことを見落としているのではないかとも思いますが、
お知恵を拝借できれば幸いです。

よろしくお願いいたします。

Answer 1

まず、正しく動いていた処理を分かりやすい様に{}で囲んで纏めてみます。

{は:と同じ意味で、
}はend*****;などの処理と同じ意味です。

foreach( $attachments as $id => $attachment ){
	if( get_post_thumbnail_id($post->ID) == $attachment->ID ){
		continue;
	}
	echo '<div class="img">';
	echo wp_get_attachment_image($id, "thumb_300", false);
	echo '</div>';
}

if文の後に{や:が無い場合、
その次の;までの処理がif文がtrueだった場合に実行されます。
またcontinue;を使うと、それ以降のforeachの処理を省略して、次のループへと進みます。

ですので、上記のコードにエラーはありませんが、
get_post_thumbnail_id($post->ID) == $attachment->IDがfalseだった時のみ、
画像が表示されるという、恐らく想定していない動きになっている様に見えました。

という前提で、改めて問題のコードを見てみます。

    foreach( $attachments as $id => $attachment ):
        if( get_post_thumbnail_id($post->ID) == $attachment->ID ) {
                    continue;
                    echo '<div class="img">';
                    echo wp_get_attachment_image($id, "thumb_300", false);
                    echo '</div>';}
        else {echo "失敗";}
        endif;
    endforeach;

まず、if文などの括りを{}に直します。
また}があるにも関わらず、同じ意味のendif;があるのはおかしいので修正します。

foreach( $attachments as $id => $attachment ){
	if( get_post_thumbnail_id($post->ID) == $attachment->ID ) {
		continue;
		echo '<div class="img">';
		echo wp_get_attachment_image($id, "thumb_300", false);
		echo '</div>';
	}else{
		echo "失敗";
	}
}

しかし、このままだと画像を表示する処理よりも前にcontinue;があります。
ですので、get_post_thumbnail_id($post->ID) == $attachment->IDがfalseの時のみ
失敗と表示される良く分からない内容になってしまっています。

なので順番を見なおして、以下の様にすると上手く行くかと思うのですがいかがでしょうか。

foreach( $attachments as $id => $attachment ){
	if( get_post_thumbnail_id($post->ID) == $attachment->ID ) {
		echo '<div class="img">';
		echo wp_get_attachment_image($id, "thumb_300", false);
		echo '</div>';
		continue;
	}else{
		echo "失敗";
	}
}

もっといえば、continue;の後に省略すべきコードがない為、以下でも大丈夫です。
(ifがtrueであればelseの内容は無視されます)

foreach( $attachments as $id => $attachment ){
	if( get_post_thumbnail_id($post->ID) == $attachment->ID ) {
		echo '<div class="img">';
		echo wp_get_attachment_image($id, "thumb_300", false);
		echo '</div>';
	}else{
		echo "失敗";
	}
}

Comments

[igaiga]

ありがとうございます！いろいろ根本的に間違ってたってことですね。。。
想定通りに動作するようになりました。
丁寧なご指導非常に助かりました。今後とも宜しくお願いします。

[gzrita]

少しでも参考になった様でしたら、よかったです。
頑張ってください！

Answer 2

まず、continue;はループ(foreach)から抜ける、という処理なので、
continue;の後にechoなどの処理をしても表示はされません。

また、普通のif文の場合、endifは必要ありません。
if文の基本の記述は下記のようになります。
波括弧の位置などをしっかり確認して下さい。

foreachも同様、endforeachは不要です。

//ifの場合
<?php
if (条件) {
   //該当の処理
   echo '<div class="img">';
   echo wp_get_attachment_image($id, "thumb_300", false);
   echo '</div>';
} else {
   //該当しなかった場合の処理
   echo "失敗";
}
?>

//foreachの場合
<?php
 foreach ($hoge as $foo) {
   //処理
}

合わせると
<?php
 foreach($hoge as $foo){
    if(条件){
     //処理
   }else{
     //処理
   ｝//ifの終わり
}//foreachの終わり
これでよいです。

もし、HTMLの中にPHPを書いている、と言った場合はendif、endforeachが必要です。
ifとelse、foreachの後ろは:
endifやendforeachの後ろは;
で違うので注意して下さい。
下記の場合、HTMLはそのまま書けばOKです。PHPの処理は<?php ?>で囲みます。

<h1>例えば</h1>

<?php foreach($hoge as $foo): ?>
<?pho if (条件): ?>
   <div class="img">
   <?php echo wp_get_attachment_image($id, "thumb_300", false); ?>
   </div>
<?php else: ?>
   <p>失敗です</p>
   <?php echo '失敗です' ; ?>
<?php endif; >
<?php endforeach; ?>

###まとめますと
記述頂いているコードですが、PHPのファイルに書いているとすれば、

<?php
foreach( $attachments as $id => $attachment ) {
   if ( get_post_thumbnail_id($post->ID) == $attachment->ID ) {
     echo '<div class="img">';
     echo wp_get_attachment_image($id, "thumb_300", false);
     echo '</div>';
   } else {
     echo "失敗";
   } //ifの終わり
}//foreachの終わり

これでコンティニューしなくても、ifの条件に該当すれば画像のecho処理をして次の$idに進みます。
該当しない場合は、失敗と表示されて次の$idに進みます。
?>

HTML内に書いている場合は、下記の様になります。

<?php foreach( $attachments as $id => $attachment ): ?>
   <?php if ( get_post_thumbnail_id($post->ID) == $attachment->ID ): ?>
     <div class="img">
     <?php echo wp_get_attachment_image($id, "thumb_300", false); ?>
     </div>
<?php else: ?>
     <?php echo "失敗"; ?>
<?php endif; ?>
<?php endforeach; ?>

HTMLタグはechoせず普通に書きます。
HTMLファイルにPHPを組み込んでる場合はこれでOKです。

なぜコンティニューしているのかが、ちょっとわからなかったのですが、力になれましたら幸いです。
まずifとforeachの書き方をしっかり確認して下さい。書き方がごちゃごちゃになってしまってます！
長くなってしまってすみません；；；

Comments

[igaiga]

ありがとうございます。
解決いたしました。continue;は意味も理解せずコピペしてたみたいです。。。
すみません。

Answer 3

<?php
    foreach( $attachments as $id => $attachment ):
        if( get_post_thumbnail_id($post->ID) == $attachment->ID ) {
                    continue;
                    echo '<div class="img">';
                    echo wp_get_attachment_image($id, "thumb_300", false);
                    echo '</div>';}
    endforeach;
?>

これの場合ifの条件が成立すると{}内の一番最初がcontinueなので
何もせずに次のループになります
ifが成立しない場合は{}の外に何もないので何もしないので
何もせずに次のループになります

どうしたいのかがわからないのでどう直すべきかはわかりませんが、
とりあえずcontinue;を消せばifが成立した時に画像が表示されるはずです

Comments

[igaiga]

ありがとうございます

Answer 4

<?php
    foreach( $attachments as $id => $attachment ):
        if( get_post_thumbnail_id($post->ID) == $attachment->ID ) {
                    continue;
                    echo '<div class="img">';
                    echo wp_get_attachment_image($id, "thumb_300", false);
                    echo '</div>';}
        else {echo "失敗";}
        endif; // <= これおかしいじゃん
    endforeach;
?>

Comments

[igaiga]

ありがとうございます！
endif; を消してみたところ、失敗と表示されるようになりました。
でも変更前までは画像が表示されていたので( get_post_thumbnail_id($post->ID) == $attachment->ID ) がおかしいんですかね。。。
ちょっと前進しました。ありがとうございました。