PHPのfor文がうまくいきません。

Question

Wordpressにて、複数のクエリをfor文で作成したいです。
次のようにfor文を用意しましたがうまく反映されませんでした。

for($i = 1990; $i <= 2030; $i++){
add_action( "elementor/query/paper_year_{$i}", function( $query ) {
 $query->set( 'post_type', 'paper' ); // 投稿タイプスラッグ
 $query->set( 'meta_key', 'year' ); // ソートで利用するACFのフィールド名（名前）
 $query->set( 'meta_value',"{$i}"); // カスタムフィールドの値
});
}

以下のようなphpのコードを複数用意したい次第です。
当方プログラミング初心者であり、2時間ほど格闘しましたが解決できず、お力添えしていただきたい次第です。🙇‍♂️

// 論文ソート2021
add_action( 'elementor/query/paper_year_2021', function( $query ) {
	$query->set( 'post_type', 'paper' ); // 投稿タイプスラッグ
	$query->set( 'meta_key', 'year' ); // ソートで利用するACFのフィールド名（名前）
	$query->set( 'meta_value', '2021' ); // カスタムフィールドの値
});
// 論文ソート2020
add_action( 'elementor/query/paper_year_2020', function( $query ) {
	$query->set( 'post_type', 'paper' ); // 投稿タイプスラッグ
	$query->set( 'meta_key', 'year' ); // ソートで利用するACFのフィールド名（名前）
	$query->set( 'meta_value', '2020' ); // カスタムフィールドの値
});
// 論文ソート2019
add_action( 'elementor/query/paper_year_2019', function( $query ) {
	$query->set( 'post_type', 'paper' ); // 投稿タイプスラッグ
	$query->set( 'meta_key', 'year' ); // ソートで利用するACFのフィールド名（名前）
	$query->set( 'meta_value', '2019' ); // カスタムフィールドの値
});
// 論文ソート2018
add_action( 'elementor/query/paper_year_2018', function( $query ) {
	$query->set( 'post_type', 'paper' ); // 投稿タイプスラッグ
	$query->set( 'meta_key', 'year' ); // ソートで利用するACFのフィールド名（名前）
	$query->set( 'meta_value', '2018' ); // カスタムフィールドの値
});

Comments

[arcxor]

for 文ではなく下のコードのように列挙した場合は「うまく反映され」たのでしょうか。

[rrr_2020]

arcxorさま、
ご質問ありがとうございます。個別に列挙した場合は反映されました。

なお、個別に列挙せず、for分を使いたい理由は、年度以外のプロパティも個別に設定していく必要があり、
とても時間がかかってしまうため、for文で一括反映したいと考えています、、、

Answer 1

add_action( "elementor/query/paper_year_{$i}", function( $query ) {
↓
add_action( "elementor/query/paper_year_{$i}", function( $query ) use ($i) {

phpでは関数の中は別スコープです
無名関数であればuseを使って変数を渡すことができます

Comments

[rrr_2020]

KazuhiroHatano様
解説まで添えて、ご回答ありがとうございます！
うまくいかなかった理由にも納得してスッキリと問題を解決することができました🙇‍♂️