PHPで条件に適合する配列を連結したい

Question

###実現したいこと
PHP初心者で、配列の合体がどうしてもできません。

下記の「現在の配列$current」と「新しい配列$new」を合体して、「目的の配列$update」にしたいと考えています。

###そのための条件
合体の条件は３つありまして、

➀thread_idが同じなら次の➁➂で合体する。違えば「新しい配列$new_array」を先頭にして普通に連結する。

合体させるときは、
➁time、sender_id、readの３つについて、新しい配列の値だけを使う。
➂新しい配列を先頭に持っていく。

です。

これを実現するために文末のソースコードを書いて奮闘しているのですが、どうしてもできず、質問させて頂きました。

###「現在の配列（$current_array）」と「新しい配列（$current_array）」
まずこの２つの配列のうちthread_idが同じものを合体させたいです。

<?php
/* 
* 現在の配列（$current_array）
* いくつあるか不明（例として次の2つ）
*/
$current_array = array(
    array(
        'action'=>'message',
        'time'=>'16',
        'sender_id'=>'9',
        'recipient_id'=>'1',
        'read'=>'ok',
        'thread_id'=>'8'
    ),  
    array(
        'action'=>'message',
        'time'=>'13',
        'sender_id'=>'4',
        'recipient_id'=>'1',
        'read'=>'ok',
        'thread_id'=>'5'
    ),    
);

/* 
* 新しい配列（$new_array）
* １つだけ
*/
$new_array = array(
    array(
        'action'=>'message',
        'time'=>'18',
        'sender_id'=>'9',
        'recipient_id'=>'1',
        'read'=>'yet',
        'thread_id'=>'5'
    ),  
);

###「目的の配列（$current）」
上の２つの合体で作りたいのが下記になります。

<?php
/* 
* 目的の配列（$update_array）
*/
$current_array = array(
    array( // thread_idが同じなので合体した
        'action'=>'message',
        'time'=>'18',
        'sender_id'=>'9',
        'recipient_id'=>'1',
        'read'=>'yet',
        'thread_id'=>'5'
    ),  
    array( // thread_idが違うので変化なし
        'action'=>'message',
        'time'=>'16',
        'sender_id'=>'9',
        'recipient_id'=>'1',
        'read'=>'ok',
        'thread_id'=>'8'
    ),    
);

###考えているソースコード
あくまでイメージですが、このような流れになるのではと書いてみました。
素人の付け焼刃ゆえ、お目汚しかとは思いますが…

<?php
/* 
* $current_arrayと$new_arrayの合体の関数
* usage: $update_array = mymerge($new_array, $current_array); 
*/
function mymerge($arg1, $arg2) {
    foreach ($arg1 as $new => $newVal) {
        foreach ($arg2 as $current => $currentVal) {
            // thread_id が同じ場合は、新しい配列の値を採用して、新しい配列を先頭に持っていく
            if ( $new[5]['thread_id'] == $current[5]['thread_id'] ) {
                $currentVal['time'] = $newVal['time'];
                $currentVal['sender_id'] = $newVal['sender_id'];
                $currentVal['read'] = $newVal['read'];
                return $arg2;
            }
            // thread_id が異なる場合は現在の配列と新しい配列を結合して、新しい配列を先頭に持っていく
            else{
                $arg1_arg2 = array_merge($arg1, $arg2);
                return $arg1_arg2;
            }
        }
    }
}


/* 
* 上の関数を使って、目的の配列 ($update_array)を作る
*/
$update_array = mymerge( $new_array, $current_array );
var_export($update_array);
?>

どのようにすればmymergeは「目的の配列（$update_array）」を作れるような関数にできがるでしょうか。
質問で分かりにくい点などございましたらご指摘ください。

Comments

[m.ts10806]

せっかくコードを作られたのでしたら実行されてみては？デバッグも行えば見えてくるものもあると思います。何のために置かれたかわからないbreakとか

[love_kinniky]

ありがとうございます。breakはforeachの最後に必ず必要なものだと思っていますが、そのようなことなはないのでしょうか？

[m.ts10806]

ないです。foreachは配列を全てループするので勝手に抜けます。
意図的にループを抜けたいときにbreakを使うことはありますが、それならwhileでループ抜ける条件をつけるべきで、switch以外でbreakを使う機会はあまりありません。

[love_kinniky]

そうでしたか…breakを消して、いくつか手直してしてみました。ありがとうございます。

Answer 1

前回の回答がお目汚しと認知されてしまったようですので。

前提とすること

• $new_arrayは1つしかない
• $new_array['thread_id']とマッチするthread_idは$current_arrayにはたかだか1つしかない
• 順序性を重んじる

前回のように、先頭に追加してそのうち20個だけを取る、それも一部要素だけ、とかっていうphp的にトリッキーな制約がなければループしなくてもarray_mergeでできるんで、そこまで酷いコードにはなりません。

<?php
/* 
* 現在の配列（$current_array）
* いくつあるか不明（例として次の2つ）
*/
$current_array = array(
    array(
        'action'=>'message',
        'time'=>'16',
        'sender_id'=>'9',
        'recipient_id'=>'1',
        'read'=>'ok',
        'thread_id'=>'8'
    ),  
    array(
        'action'=>'message',
        'time'=>'13',
        'sender_id'=>'4',
        'recipient_id'=>'1',
        'read'=>'ok',
        'thread_id'=>'5'
    ),    
);

/* 
* 新しい配列（$new_array）
* １つだけ
*/
$new_array = array(
    array(
        'action'=>'message1', //この値を利用していないことの確認のため1を付与
        'time'=>'18',
        'sender_id'=>'9',
        'recipient_id'=>'11', //この値を利用していないことの確認のため1を付与
        'read'=>'yet',
        'thread_id'=>'5'
    ),  
);

$res = insertMerge($new_array, $current_array);
var_dump($res);

function insertMerge($ins, $data) {
    $ins = array_pop($ins);
    $needle = $ins['thread_id'];
    $thread_ids = array_column($data, 'thread_id');
    if($idx = array_search($needle, $thread_ids,true) === false) { return $data; } //thread_idがなかったら元のままにする
    unset($ins['recipient_id']);
    unset($ins['action']);
    $ins = array_merge($data[$idx], $ins);
    unset($data[$idx]);
    return array_merge([$ins], $data);
}

Comments

[love_kinniky]

こんばんは。遅くなってしまい申し訳ございません。ループでなくてもいいんですね。thread_idと一致するかどうか検知するのにループ必須かと思っていたので、array_columnなど大変勉強になりました。ありがとうございます。

Answer 2

条件を簡潔にすると良いです。

$new_arrayからthread_idだけを取り出し、$current_array[n]['thread_id']と比較すると楽です。

補足
関数の使い方が理解できていないようです。
提示されたコードで言うと
・関数の引数の関数内での使用方法
・$item のスコープ
・return の使用箇所
がおかしいです。

追記
そのままじゃ使えないけどサンプルコードｗ

$result_array = $new_array;
$target_thread_id = $new_array[0]['thread_id'];

foreach($current_arrays as $current_array){
    if($current_array['thread_id'] === $target_thread_id){
        $result_array[0] = array(
            'action'=>$current_array['action'],
            'time'=>$new_array[0]['time'],
            'sender_id'=>$new_array[0]['sender_id'],
            'recipient_id'=>$current_array['recipient_id'],
            'read'=>$new_array[0]['read'],
            'thread_id'=>$current_array['thread_id']
        );
    } else {
        $result_array[] = $current_array;
    }
}
var_dump($result_array);

概要：
$result_array という結果を入れる配列を作り、$new_array を突っ込んでおきます。
つまり $result_array[0]。これが結構コードをシンプルにしてます。
で、$current_arrays をぐるぐるまわし、thread_id が合致した場合、$result_array[0] を置き換え、合致しなければ、$result_array の後ろへ。

注意：
提示されたコードの $current_array は $current_arrays として使用しています。

Comments

[love_kinniky]

その楽な方法が難しいですｗ
自分としては
foreach ($arg1 as $item) {
で新しい配列を１件ずつ処理して
foreach ($arg2 as $key=>$val) {
で現在の配列を１件ずつ処理して
そして
if ( $item['thread_id'] == $val['thread_id'] ) {
でそれら１件ずつの中の「thread_id」が一致したら、
というまさに仰る条件にしているつもりでして…

[退会済みユーザー]

概要をコードにしてみました。
$new_array が1件しかないのであれば、foreach を使用する必要ないです。
先頭に持ってくる件は、最初から先頭においておけば良い。

プログラミングでは時々順番を入れ替えてコードをシンプルにすることがあります。
どの錠処理が「絶対に順番を守らなければならないのか？」を考えるのも面白いですよ。

[love_kinniky]

回答のご修正、どうもありがとうございます。もうやめるしかないと思っていたのでとてもうれしいです。

どちらかを操作して出力するのではなくて、出力用のresult_arrayという配列を作ってその値を順にコピーしていくという方法ですか！すごいや！ありがとうございます。

でもどうして $current_arrays にされたのでしょうか？ $current_array にすればそのまま使えますよね？

[退会済みユーザー]

foreach で回すのは複数のものなので、たいてい複数形で命名します。
で、foreach の中では単数形で使うってのが、ワリとわかりやすい記述なので変更してます。
つまり、コード見た時にわかりやすくなるルールにしたかったんで変えました。

[love_kinniky]

今自分でアレンジしていたらできなくなって、まさにそこが原因でしたｗ名前、大事ですね。どうもありがとうございました！