foreach( $array as $key => $value ) において、ループの最中に $array を変更したい

Question

###実現したいこと
PHPのforeachで、一回目のループで取得した値を、二回目のループにねじ込みたいです。

###ソースコード・問題点
次の関数print_thread_datas()をご覧ください。
引数$arg_wrapperをループし、一回目のループで掲示板のレコードを取得し、二回目のループでコメントのレコードを取得するという関数です。

ここでコメントの取得に際して問題があります。
コメントの取得では、掲示板のレコードだけが持つ値top_comment_idが必要となっており、二回目のループにそれをねじ込みたいのですが、なぜかできません。

$arg_wrapper = [
    // thread エリアに thread_id=1 の掲示板を出力するための引数
    'thread' => [
        'process' => 'pjax',
        'arg' => ['type'=>'thread', 'thread_id'=>1],
    ],
    // comments エリアにthread_id=1の掲示板へのコメントを出力するための引数
    'comments' => [
        'process' => 'pjax',
        'arg' => ['type'=>'comment', 'thread_id'=>1, 'limit'=>2, 'last_comment_id'=>20, 'order_by'=>'popular_order'],
    ]
];
$results = print_thread_datas( $arg_wrapper );
var_dump($results);

// $arg_wrapperに応じてスレッドのタイトルとコメントリストを返す
function print_thread_datas( $arg_wrapper ) {
    $results = [];
    foreach ( $arg_wrapper as $print_area => $info ) {
        
        // レコードを取得
        $arg = $info['arg'];
        $rows = get_rows( $arg );
        if ( $print_area === 'thread' ) {
            $results[$print_area] = $rows[0];  // thrread は一件だけ
        } else {
            $results[$print_area] = $rows; // comments は複数
        }
        
        // thread の場合
        if ( $print_area === 'thread' ) {
            // コメントリストの先頭に固定されるコメントを取得するための引数をねじ込む
            $top_comment_id = $rows[0]['top_comment_id'] ?? '';
            if ( $top_comment_id ) {
                $arg_wrapper['comments']['arg']['top_comment_id'] = $top_comment_id;
            }
            
            // ねじ込めたことを確認
            var_dump($arg_wrapper);
        }
    }
    return $results;
}


// $argに応じてデータベースからレコード($rows)を取得する例
function get_rows( $arg ){
    $rows;
    $type = $arg['type'];
    if ( $type === 'thread' ) {
        $rows = [ ['type'=>'thread', 'thread_id'=>1, 'title'=>'1の掲示板', 'author_id'=>100, 'author_name'=>'鈴木', 'top_comment_id'=>99] ];
    } elseif ( $type === 'comment' ) {
        $rows = [
            ['type'=>'comment', 'thread_id'=>1, 'comment_id'=>30, 'content'=>'こんにちは', 'author_id'=>200, 'author_name'=>'佐藤'],
            ['type'=>'comment', 'thread_id'=>1, 'comment_id'=>40, 'content'=>'さようなら', 'author_id'=>300, 'author_name'=>'山田']
        ];
        // コメントリストの先頭に固定されるコメントを取得
        if ( isset( $arg['top_comment_id'] )  ) {
            $top_comment = ['type'=>'comment', 'thread_id'=>1, 'comment_id'=>99, 'content'=>'先頭に固定されるコメントです', 'author_id'=>400, 'author_name'=>'高橋'];
            array_unshift( $rows, $top_comment );
        }
    }
    return $rows;
}

###現状と目的
上記コードで得られる現状には、先頭に固定するコメント（'comment_id'=>99）が追加されず以下となります。

$results = [
    'thread' => ['type'=>'thread', 'thread_id'=>1, 'title'=>'1の掲示板', 'author_id'=>100, 'author_name'=>'鈴木', 'top_comment_id'=>99],
    'comments' => [
        ['type'=>'comment', 'thread_id'=>1, 'comment_id'=>30, 'content'=>'こんにちは', 'author_id'=>200, 'author_name'=>'佐藤'],
        ['type'=>'comment', 'thread_id'=>1, 'comment_id'=>40, 'content'=>'さようなら', 'author_id'=>300, 'author_name'=>'山田']    
    ]
];

目的はこうで、commentsの配列に先頭に固定するコメント（'comment_id'=>99）を追加したものになります。

$results = [
    'thread' => ['type'=>'thread', 'thread_id'=>1, 'title'=>'1の掲示板', 'author_id'=>100, 'author_name'=>'鈴木', 'top_comment_id'=>99],
    'comments' => [
        ['type'=>'comment', 'thread_id'=>1, 'comment_id'=>99, 'content'=>'先頭に固定されるコメントです', 'author_id'=>400, 'author_name'=>'高橋'],
        ['type'=>'comment', 'thread_id'=>1, 'comment_id'=>30, 'content'=>'こんにちは', 'author_id'=>200, 'author_name'=>'佐藤'],
        ['type'=>'comment', 'thread_id'=>1, 'comment_id'=>40, 'content'=>'さようなら', 'author_id'=>300, 'author_name'=>'山田']    
    ]
];

###試したこと
上記ソースコードにございますが// ねじ込めたことを確認のダンプによって確認はとれていて、確かに先頭に固定するコメント（'comment_id'=>99）が、$arg_wrapperにねじこめていると思うのですが、なぜ目的のような結果にできないのか…わかりません。

また以下ミニマムなarray_unshiftは問題なく追加できましたし、原因が特定できずにおります。

$rows = [ ['comment_id'=>30], ['comment_id'=>40] ];
$top_comment = [ 'comment_id'=>99 ];
array_unshift($rows,$top_comment);
var_dump($rows);

Answer 1

解決策ではないのですが、 php では foreach のときに配列がコピーされるので、ループ中に取り出される $print_area や $info と var_dump している $arg_wrapper は別物です。

PHP 'foreach' の内部挙動について - スタック・オーバーフロー

ご提示のパターンだと、他の $print_area の $info を更新しないといけないのでやりかたが分からないです。

やるとしたら、foreach ( $arg_wrapper as $print_area => &$info ) のところを thread と comment に分離して(キーが他にもあったら厄介ですが)、thread を get_row, comment と 固定id の配列を連結した配列を作ってから get_row の 2 段構えにして、それらを結合して返す、といった作りにすると思います。

Comments

[ikatako]

別物になるのですね、どうりで…
ループの外側の$arg_wrapperを変更はできないようですが、yambejp様が内側の$infoを変更する方法を伝授してくださいましたので今回はなんとかなりそうです。
ご回答ありがとうございます。

Answer 2

質問の意図がわかりかねますが、以下でどうでしょう？

foreach ( $arg_wrapper as $print_area => $info ) {

↓↓↓

foreach ( $arg_wrapper as $print_area => &$info ) {

Comments

[ikatako]

おお、「&」で変えられるのですか。いつもありがとうございます。