質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.92%

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

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 241

score 244

実現したいこと

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);
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+1

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

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

↓↓↓

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/11/13 04:49

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

    キャンセル

+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 段構えにして、それらを結合して返す、といった作りにすると思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/11/13 04:48

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.92%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る