ｐｈｐで配列を連結すると、消えてしまう

Question

PHPの配列連結についての質問になります。

'name'の値が同じとき'post_arr'を連結し、違うときは外側に連結したいです。

###具体例
まず下記（例１）は同じ'name'なので、'post_arr'を連結して$resultとしたいです。

（例１）

$new_wrapper = [
    [
        'name' => 'nameA',
        'post_arr' => [ ['id'=>3,'type'=>'c'] ],
    ],
];

$leave_wrapper = [
    [
        'name' => 'nameA',
        'post_arr' => [ ['id'=>1,'type'=>'a'] ],
    ],
];

$result = [
    [
        'name' => 'nameA',
        'post_arr' => [ ['id'=>1,'type'=>'a'], ['id'=>3,'type'=>'c'] ],
    ],
];

そして下記（例２）は違う'name'なので、外側に連結して$resultとしたいです。

（例２）

$new_wrapper = [
    [
        'name' => 'nameC',
        'post_arr' => [ ['id'=>3,'type'=>'c'] ],
    ],
];

$leave_wrapper = [
    [
        'name' => 'nameA',
        'post_arr' => [ ['id'=>1,'type'=>'a'] ],
    ],
];

$result = [
    [
        'name' => 'nameA',
        'post_arr' => [ ['id'=>1,'type'=>'a'] ],
    ],
    [
        'name' => 'nameC',
        'post_arr' => [ ['id'=>3,'type'=>'c'] ],
    ],
];

###ソースコード
上のような連結を実現したいと思っていて、それが下記コードのmy_marge()という関数ですが、何も取得できなくなってしまいました。

foreachでのループのあたりが怪しいと思うのですが…、どうすれば上のような連結ができるようになりますでしょうか。

<?php
// 新しく追加する配列
$new_wrapper = [
    [
        'name' => 'nameC',
        'post_arr' => [ ['id'=>3,'type'=>'c'] ],
    ],
    [
        'name' => 'nameD',
        'post_arr' => [ ['id'=>4,'type'=>'d'] ],
    ],
];

// 最終的な配列
$result = my_marge( $new_wrapper );
var_dump( $result );

// 最終的な配列を取得
function my_marge( $new_wrapper ){
    $result = [];
    
    // 現在の配列
    $leave_wrapper = [
        [
            'name' => 'nameA',
            'post_arr' => [ ['id'=>1,'type'=>'a'] ],
        ],
        [
            'name' => 'nameB',
            'post_arr' => [ ['id'=>2,'type'=>'b'] ],
        ],
    ];

	foreach( $leave_wrapper as $leave ){
		$leave_name = $leave['name'];
		$leave_post_arr = $leave['post_arr'];
 
		foreach( $new_wrapper as $new ){
			$new_name = $new['name'];
			$new_post_arr = $new['post_arr'];    
			
			// nameが同じならpost_arrを連結
			if( $new_name == $leave_name ){
			    foreach( $new_post_arr as $new_post ){
				    $leave['post_arr'][] = $new_post;
			    }
			}		
			
			// nameが違うなら外側に連結
			else{
			    array_merge( $leave_wrapper, $new_wrapper ); 
			    exit();
			}
		}
    }
	
	return $leave_wrapper;
}

Comments

[退会済みユーザー]

例が乏しすぎてよくわからない。$new_wrapperや$leave_wrapperの中に、'name'と'post_arr'の組が複数個存在するケースが有るなら、そこにも言及してほしい。

[anpanmama]

はい、ソースコードにあるように、'name'と'post_arr'の組は複数存在します。

[m.ts10806]

>ｐｈｐで
細かいのですが、名称などの英数は全角を使われない方が良いです。
なるべく公式通りに書きましょう。プログラムは書いた通りにしか動かないので、
こういうところを注意できないと凡ミスで時間を浪費することになります。

Answer 1

おそらくですが、途中でexitしているので、何も取得できないのではないでしょうか？

Comments

[papinianus]

exit したら取得できなくないですか？なんでこの回答がマイナスなの。

[anpanmama]

nameが違うときはforeachから抜けたいのでecitとしたのですが、そうですよね。ありがとうございます。

Answer 2

頭が痛くなってきた。
こういう構造にできないものかねぇ。

$new_wrapper = [
    'nameA' => [
        'name' => 'nameA',
        'post_arr' => [
            3 => ['id'=>3,'type'=>'c'] 
        ],
    ],
];

とか、いっそ

$new_wrapper = [
    'nameA' => [
        'post_arr' => [
            3 => ['type'=>'c'] 
        ],
    ],
];

とか。

ユニークキーがつかない、配列の入れ子って、ユニークさを扱うのがちょー面倒なんですけど。

ほんの、手慰み。

<?php


$new_wrapper = [
    'nameA' => [
        'name' => 'nameA',
        'post_arr' => [
            3 => ['id'=>3,'type'=>'c'] 
        ],
    ],
];

$leave_wrapper = [
    'nameA' => [
        'name' => 'nameA',
        'post_arr' => [ 
            1 => ['id'=>1,'type'=>'a'] 
        ],
    ],
];

/*
$new_wrapper = [
    'nameC' => [
        'name' => 'nameC',
        'post_arr' => [ 
            3 => ['id'=>3,'type'=>'c'] 
        ],
    ],
];

$leave_wrapper = [
    'nameA' => [
        'name' => 'nameA',
        'post_arr' => [ 
            1 => ['id'=>1,'type'=>'a'] 
        ],
    ],
];
*/

$result = $new_wrapper;
foreach ($leave_wrapper as $key => $arr) {
    if (isset($result[$key])) {
        foreach ($arr['post_arr'] as $k => $v) {
            $result[$key]['post_arr'][$k] = $v;
        }
    }
    else {
        $result[$key] = $arr;
    }
}
var_export($result);

ついでに、質問文中にあった構造をちゃんと入力としておきたい場合は、
こういう関数を書いておくと私のサンプルコードのような形式になるよ、と。

function apply_unique_key($arr) {
    $result = [];
    foreach ($arr as $item) {
        if (isset($item['name'])) {
            $result2 = [];
            foreach ($item['post_arr'] as $item2) {
                if (isset($item2['id'])) {
                    $result2[$item2['id']] = $item2;
                }
            }
            $result[$item['name']] = [
                'name' => $item['name'],
                'post_arr' => $result2
            ];
                
        }
    }
    return $result;
}

Comments

[anpanmama]

関数化までどうもありがとうございます。助かりました！構造もよく考えておきたいと思います。

[anpanmama]

後学のためにお伺いしたいのですが、連番キーでなくユニークキーを（つけられるなら）つけるべきという考えでいた方がよろしいでしょうか？

3 => ['id'=>3,'type'=>'c']

とか

'nameA' => [
'name' => 'nameA',
],

とかですと、重複した情報がある点をどうかと思っていたのですが、今回のケースでその重複より得られるメリットが大きいなら、ユニークキーをつけるべきかも知れないなと思いました。

[退会済みユーザー]

PHPマニュアルの配列： https://www.php.net/manual/ja/language.types.array.php 他適切な情報に触れていただきたいです。
keyとvalueのペアの方が、いちいちvalueの中身をチェックしてから採否判定しなくてもkeyの存在確認だけで済むので楽っていう話です。
それも理由として、他の言語で言ういわゆる連想配列（keyとvalueを組み合わせた配列データの持ち方）が実装されているわけで。

[anpanmama]

なるほどです。度重なるアドバイスに感謝申し上げます。

Answer 3

コレと同じですかね？

PHPによる配列の変換で、ループの処理がうまくいかないです

個人的には、元のデータの持ち方を再検討したほうが良いと思います。

Comments

[anpanmama]

はい、元はソレを考えていて、かなり簡易なかたちで再挑戦したのが今回の質問になっています。元のデータの持ち方を再検討とは？

[退会済みユーザー]

コードを試しに書いてみて、データの持ち方がおかしいという点に気づいたことは、同じ意見かも。

[退会済みユーザー]

> 元のデータの持ち方を再検討とは？

リンク先で提案しています。

[anpanmama]

すみませんリンク先を今拝見しましたが関係なかったです。先程した質問（https://teratail.com/questions/236555）かと勝手に思ってしまいした。m6uさんの方向で進めてみたいと思います。ありがとうございました。