PHPfor内で、引数の値を初期化したい

Question

PHPのforでループする際に、変数を「初期化」できないでしょうか？

「初期化」というのは
「そのループを実行する関数に渡された引数へ初期化」という意味で
「forを実行するrand_insert_postsに渡された$kindへ初期化」という意味です。

つまり以下で、ループのたびに$kind === nullであると判定し、$kinds = ['a','b','c'];をランダムにダンプしたいということになります。

以下はその判定ができず、同じ値になってしまうのです。

<?php

// null を渡すことで、ランダムな $kind を posts テーブルに insert したい
rand_insert_posts( 2, null );

// insert を実行
function rand_insert_posts( $count, $kind = null ){ 
    
	for ( $posted=0; $posted<$count; $posted++ ) {

		// ランダムな $kind を取得
		if ( $kind === null ) {
			$kinds = ['a','b','c'];
			$kind = $kinds[ array_rand( $kinds ) ] ;
		}

		// $kind が同じ値になってしまう
		var_dump($kind); 
		
		// $kind が同じ値になってしまうため、特定の $kind しか insert できないのが問題
		$query = "insert posts (kind) values( $kind )";
	}
	
}

Answer 1

つまり以下で、ループのたびに$kind === nullであると判定し、$kinds = ['a','b','c'];をランダムにダンプしたいということになります。

そうなってますよ。
2回目以降のループでは、$kindがnullではないので if 内が実行されません。

if の直前で$kind = null;とでもしてやれば、当然 if で常にtrueになりますが、if の意味がなくなります。

「初期化」というのは、「そのループを実行する関数、に渡された引数、へ初期化」という意味です。

をもう少し分かりやすい日本語で記述できれば、他の回答が付くかもしれません。

Comments

[wakame2]

「ループのたびに$kind === nullを判定したい」のではありません。
「ループのたびに$kind === nullであると判定したい」です。
「ループのたびに$kind === nullがtrueと判定したい」といえばわかりますか？

[退会済みユーザー]

ループのたびに判定してますよ？

[退会済みユーザー]

なんかコメントが変更されてたせいで
一つ前のコメントが意味不明になっていますね＾＾；

> 「ループのたびに$kind === nullがtrueと判定したい」といえばわかりますか？

回答した通り、2週目の $kind は null ではないので true にはなりません。

[wakame2]

そうですか。ありがとうございます。他の方のご回答を待ちたいと思います。

Answer 2

$arg_kind で判定しました。

// insert を実行
function rand_insert_posts( $count, $kind = null ){ 

    // 引数を $arg_kind として保存
    $arg_kind = $kind;

    for ( $posted=0; $posted<$count; $posted++ ) {

        // $arg_kind で判定し、ランダムな $kind を取得
        if ( $arg_kind === null ) {
            $kinds = ['a','b','c'];
            $kind = $kinds[ array_rand( $kinds ) ] ;
        }

        // ランダムな $kind
        var_dump($kind); 

        // ランダムな $kind で insertする
        $query = "insert posts (kind) values( $kind )";
    }

}

Comments

[退会済みユーザー]

これ、rand_insert_posts の引数として $kind が与えられると、if 内を通らないですけど、意図したものですか？

[wakame2]

はい、大丈夫です。
$kindがnullの場合だけランダムな$kindでinsertする。
$kindがnullでなければその$kindでinsertする。
という処理を意図しています。

[退会済みユーザー]

ようやく要件が理解できました。

$kind の有無によって flag 立てたかったのですね。
回答の
```
$arg_kind = $kind;
```
を
```
$flag = is_null($kind);
```
としてあげると、分かりやすくなると思います。

または
```
$insert _val = $kind??$kinds[ array_rand( $kinds ) ];
```
とか。(array_rand の使い方がちょっと気になるけど。。。)

**全くの余談**
変数の展開による SQLの組み立ては、いかなる時も避けた方が良いです。
今は良くても、リファクタリングにより SQL インジェクションができる場面が発生することがあります。