wp_reset_postdata(); は、どこに書けばいいですか？

Question

wp_reset_postdata();は、下記の【ここ】に書いても問題ありませんでしょうか？

<?php if ( $the_query->have_posts() ) : ?>
	
	<?php while ( $the_query->have_posts() ) : $the_query->the_post(); ?>		

		echo '<p>'.the_title().'</p>';

	<?php
	endwhile;
	wp_reset_postdata(); //【ここ】
	?>
	
<?php else : ?>
	
	<p>記事がありません</p>
	
<?php endif; ?>

なぜこんな質問をしたのかっていいますと、<p>記事がありません</p>のところで別のサブループを書きたいからです。

しかしどのサイトを見ても、wp_reset_postdata();は、一番最後のendif;の下に書かれているので、endwhile;の下に書くのはどうなのかと思いまして。

問題ないにせよ、あるにせよ、できればそのソースも含めて教えてもらえますとありがく思います。

もし知ってたら教えてください。宜しくお願い致します！

Comments

[退会済みユーザー]

matsuzakaqやろ

[maxbet]

すいません、matsuzakaqがわからなかったんですけど、何をしたらいいんでしょうか？

Answer 1

$the_query->the_post()に到達したときのみwp_reset_postdata()が必要になるので提示コードで問題ありません。

$the_query->the_post()を呼び出すと$the_queryクエリループの投稿が、wp_reset_postdata()を呼び出すとメインクエリループの投稿がグローバル変数$postにセットされます。

Comments

[maxbet]

ありがとうございます。ということは、別に「wp_reset_postdata」を【ここ】に書いてリセットしないままで、質問の「記事がありません」の部分を下記のように変更して２回目のループを書いても、特に問題ないということでしょうか？？

---------------------------------------------
<?php else : ?>

<p>記事がありません</p>

<?php endif; ?>
---------------------------------------------

↓変更

---------------------------------------------
<?php else : ?>

<?php
$args = array( 'post_type' => 'monday' );
$the_query = new WP_Query( $args );

if ( $the_query->have_posts() ) :
while ( $the_query->have_posts() ) :
$the_query->the_post();
echo '<p>'.the_title().'</p>';
endwhile;
else :
echo '<p>2回目もありません'</p>';
endif;
?>

<?php endif; ?>
---------------------------------------------

上の変更後の「$the_query->the_post()」の時点で、新しい「monday」の「$the_query」がセットされるということであれば、別に質問の【ここ】でリセットする必要はない。と解釈したのですが、あってますか？

[yhg]

【ここ】の部分のwp_reset_postdata()でリセットしなかった場合、最初のクエリーに投稿が存在すると最終的にリセットされないままになってしまうので必須です。
入れ子にしたかったら、無難に両方のwhileループの後ろにwp_reset_postdata()を配置するのが良いのではないでしょうか。

[maxbet]

最初のクエリーはリセットしないままですが、そのままでも、

$the_query->the_post()を呼び出したときに、$the_queryの投稿（mondayの条件）が、新しく$postとしてセットされる。というわけですよね？

これはつまり、最初のクエリーに投稿が存在しリセットしなくても、新しくセットされる（つまり意図してリセットしなくても、自動的にリセットされる）という意味ではないのでしょうか？

すみません。無難につけとけよという話ですがｗ、良い機会なので詳しく知りたくて…

[yhg]

リセットせずにおくと、その処理よりも後ろでメインクエリーを表示する場合に不具合が出てきますので、メインクエリーの表示有無に限らず無難につけとくのが良いですね。

またプログラミング全般に言えることですが、グローバル領域への影響が複雑に絡み合うと保守がとてつもなく大変になってきますので、グローバル領域への影響は必要最小限になるよう努めるべきです。

[maxbet]

なるほどいずれもおっしゃる通りですね。とことん納得できました。夜おそくまでありがとうございました！