PHP多次元連想配列で、あるキーの値が等しいとき、別キーの値を統合したい。

Question

前提・実現したいこと

$sourceのidが同じとき、preを統合させて、$targetを作りたいです。

$source = array(
    array('id'=>1, 'pre'=>'東京', 'rank'=>'A'),
    array('id'=>1, 'pre'=>'大阪', 'rank'=>'A'),
    array('id'=>4, 'pre'=>'沖縄', 'rank'=>'B'),
);

↓統合させたい

$target = array(
    array('id'=>1, 'pre'=>['東京','大阪'], 'rank'=>'A'),
    array('id'=>4, 'pre'=>['沖縄'], 'rank'=>'B'),
);

発生している問題・エラーメッセージ

Uncaught Error: [] operator not supported for strings

該当のソースコード

実現することと近い以下のソースコードをネットから教えて頂きました。

以下のソースコードは
"東京,大阪"

となるのですが、その部分が
["東京","大阪"]

となれば実現したいことができます。

$target = array();
foreach ($source as $item) {
    $id = $item['id'];
    foreach ($target as $skey=>$out) {
        if ($out['id'] === $id) {
            $outkey = $skey;
        }
    }
    if (!isset($outkey)) {
        $target[] = $item;
    } else {
        $target[$outkey]['pre'] .= ',' . $item['pre'];
        unset($outkey);
    }
}
print_r($target);

試したこと

実現したいことのために、こちらの部分を

        $target[$outkey]['pre'] .= ',' . $item['pre'];

次のように変更しますと、先のエラーとなってしまうのです。

        $target[$outkey]['pre'][] = $item['pre'];

エラーに対して試したこと

そこでエラーを見てますとstringsが悪いようで、こちらでstringsがそのまま追加されているのが悪いのかとあたりをつけまして、

    if (!isset($outkey)) {
        $target[] = $item;
    }

次のように変更し、preなら配列として扱うようにしてみたのですがやはりできず、どうしてもわからないので質問させて頂きました。

    if (!isset($outkey)) {
        foreach ( $item as $k=>$v ) {
            if( $k === 'pre' ){
                $target['pre'][] = $v;
            }else{
                $target[$k] = $v;
            }
        }
    }

Answer 1

preが文字列だったり配列だったり型が混在するのは避けるべきです
またrankが異なるときどちらが優先されるかも不明です

Comments

[jonsmith]

申し訳ございません。型は配列でrankが異なることはないのでその2点を修正致しました。

[yambejp]

idとrankは常に連動しているということですね。

$a = [
['id'=>1, 'pre'=>'東京', 'rank'=>'A'],
['id'=>1, 'pre'=>'大阪', 'rank'=>'A'],
['id'=>4, 'pre'=>'沖縄', 'rank'=>'C'],
];
$b=[];
foreach($a as $val){
$c=&$b[$val["id"]];
if(!is_array($c["pre"])) $c["pre"]=[];
$c["id"]=$val["id"];
array_push($c["pre"],$val["pre"]);
$c["rank"]=$val["rank"];
}
$b=array_values($b);

print_r($b);

[jonsmith]

だいぶスッキリしました。ありがとうございます。
一つ補足で、$sourceが質問のように固定でない場合があります。たとえば以下の札幌のように「age」だったりということです。なので以下としてみました。「age」にも対応できるようにforeachで変数とした流れです。

<?php
$a = [
['id'=>1, 'pre'=>'東京', 'rank'=>'A'],
['id'=>1, 'pre'=>'大阪', 'rank'=>'A'],
['id'=>4, 'pre'=>'沖縄', 'rank'=>'C'],
['id'=>5, 'pre'=>'札幌', 'age'=>'20']
];
$b=[];
foreach($a as $val){
$c=&$b[$val["id"]];
if(!is_array($c["pre"])) $c["pre"]=[];
foreach( $val as $k => $v ){
if ( $k == 'pre' ){
array_push($c["pre"],$val["pre"]);
}else{
$c[$k]=$val[$k];
}
}
}
$b=array_values($b);

[jonsmith]

そして、質問よろしいでしょうか。

上記をvar_dump($b)しますと、札幌がある配列（$b[2]の配列）に「&」という記号が付きます。下記です。

array(3) {
[0]=>
array(3) {
["pre"]=>
array(2) {
[0]=>
string(6) "東京"
[1]=>
string(6) "大阪"
}
["id"]=>
int(1)
["rank"]=>
string(1) "A"
}
[1]=>
array(3) {
["pre"]=>
array(1) {
[0]=>
string(6) "沖縄"
}
["id"]=>
int(4)
["rank"]=>
string(1) "C"
}
[2]=>
&array(3) {
["pre"]=>
array(1) {
[0]=>
string(6) "札幌"
}
["id"]=>
int(5)
["age"]=>
string(2) "20"
}
}

この「&」が何か…？もしご存じでしたら教えて頂けませんでしょうか？

見た感じ「$c=&$b[$val["id"]];」に関係していそうですが、この行の意味が解読できないために、札幌の配列に「&」が付く理由もわからないでおります。

[yambejp]

なるほどキーが一律ではないのですね

$a = [
['id'=>1, 'pre'=>'東京', 'rank'=>'A'],
['id'=>1, 'pre'=>'大阪', 'rank'=>'A'],
['id'=>4, 'pre'=>'沖縄', 'rank'=>'C'],
['id'=>5, 'pre'=>'札幌', 'age'=>'20']
];
$b=[];
foreach($a as $val){
$c=&$b[$val["id"]];
if(!is_array($c["pre"])){
$c=array_filter($val,function($x){return $x!=="pre";},ARRAY_FILTER_USE_KEY);
$c["pre"]=[];
}
array_push($c["pre"],$val["pre"]);
}
unset($c);
$b=array_values($b);
var_dump($b);

「＆」マークが付くのは参照渡ししているからです
unsetしてやってください。

[jonsmith]

なんとご丁寧に…細やかにご対応頂き感謝致します。参照渡しは聞いたことしかありあませんでした。array_filterも使ったことがなく、大変噛み応えのあるコードで勉強になります。どうもありがとうございました。

Answer 2

こんな感じにするしかないのでは？

    if (!isset($outkey)) {
        $target[] = ['id' => $item['id'], 'pre' => [$item['pre']], 'rank' => $item['rank']];
    } else {
        $target[$outkey]['pre'][] = $item['pre'];
        unset($outkey);
    }

Comments

[jonsmith]

なるほど１つずつ改めて入れなおすと。ありがとうございます。

[Y.H.]

まとめて入れても結局どこかで$target[$outkey]['pre']をarray()にすげ替える為のifブロックが必要になるだけですし。