PHPで配列を比較し、特定のキーに同じ値があれば、別のキーを更新したい。

Question

###実現したいこと
１．$arrと$newを比較し、valが同じ値ならdateを更新し、同じ値でなければ追加したい。
２．$arrは最大4件までとしたい。

宜しくお願い致します。

###該当のソースコード
まず元からある配列として$arrがあります。

$arr = [
    ['val'=>'a1','date'=>'2020-02-25 01:00:00'],
    ['val'=>'a2','date'=>'2020-02-25 02:00:00'],
    ['val'=>'a3','date'=>'2020-02-25 03:00:00']
];

次の$newが与えられた場合、valと同じ値が$arrにあるので、そのdateを更新して$result1を得るのが目的です。

$new = ['val'=>'a2','date'=>'2020-02-25 04:00:00'];

$result1 = [
    ['val'=>'a1','date'=>'2020-02-25 01:00:00'],
    ['val'=>'a2','date'=>'2020-02-25 04:00:00'], // valが同じ値なのでdateを更新
    ['val'=>'a3','date'=>'2020-02-25 03:00:00']
];

もし$newが次の場合、$arrに同じ値がないので、普通に追加して以下$result2を得たいです。

$new = ['val'=>'a4','date'=>'2020-02-25 04:00:00'];

$result2 = [
    ['val'=>'a1','date'=>'2020-02-25 01:00:00'],
    ['val'=>'a2','date'=>'2020-02-25 02:00:00'], 
    ['val'=>'a3','date'=>'2020-02-25 03:00:00'],
    ['val'=>'a4','date'=>'2020-02-25 04:00:00'] // 普通に追加
];

ここまでは１の処理です。もしできれば２の処理として最新の４件までとし、古いものは削除したいのですが、これは文末に「２のロジック」として追記させて頂きました。（あちこち飛んで申し訳ございません。）

###発生している問題（１について）
まず１についてです。

１．$arrと$newを比較し、valが同じ値ならdateを更新したい。

foreachで試したのですが、これでは配列が増えすぎてしまいました。
原因はifの分岐で、同じ値があってもなくても該当して追加されてしまうからでしょうか？

// 同じ値があればその日付を更新し、なければ追加
foreach ( $arr as &$ar ) {
    if ( $ar['val'] == $new['val'] ) {
        $ar['date'] = $new['date'];
    } else {
        $arr[] = $new;
    }
}
var_dump($arr);

###試したこと（１について）
上のifがおかしいなら、更新と追加の処理を次のようにして分けたらどうかと思いましたが、これでもやはり余計な配列が増えてしまいます。

// $newのvalと同じ値があればその日付を更新し、
foreach ( $arr as &$ar ) {
    if ( $ar['val'] == $new['val'] ) {
        $ar['date'] = $new['date']; break;
    }
}
// なければ追加
foreach ( $arr as $ar ) {
    if ( $ar['val'] != $new['val'] ) {
        $arr[] = $new; break;
    }
}
var_dump($arr);

###発生している問題（２について）
そして２についてです

２．$arrは最大4件までとしたい。

これは次のコードしかわからずコメントアウトしてあるようなエラーになります。

「array_keysに渡せるのは配列だけ」というエラーかと思うのですが、でも最大値を見たいのはdateだけなので$arr['date']とする以外にどうしたらいいのかわからず、とりあえずエラーのままのコードですみませんが掲載させて頂きました。

// 4件以上なら古いdateのものを削除
if ( count($arr) >= 4 ) {
    $maxes = array_keys($arr['date'], max($arr['date']));
    $max = $maxes[0];
    unset($arr[$max]);
}
var_dump($arr);

/*
エラーメッセージ
PHP Notice:  Undefined index: date in /aaa/bbb.php on line 0
PHP Notice:  Undefined index: date in /aaa/bbb.php on line 0
PHP Warning:  max(): When only one parameter is given, it must be an array in /aaa/bbb.php on line 0
PHP Warning:  array_keys() expects parameter 1 to be array, null given in /aaa/bbb.php on line 0
*/

###試したこと（２について）
すみません、最大値を得る方法が上の書き方しかわからず、他に試すことが思い浮かびませんでした。

###バージョン
PHP 7.2.13 です。

###２のロジック
２のロジックについてご説明させて頂きます。

以下の場合$newのvalは$arrのどれとも一致しないので、１を介して$arrに普通に追加されることになり$result2がまず得られます。

しかしその結果５件になるので、その５件から古いものを削除し$result3を得たいということを考えています。これが２の処理です。

（自分がわかりやすいので１と２の処理に分けていますが、欲しいのは$result3だけで、その過程にある$result1や$result2の方は不要です。）

$arr = [
    ['val'=>'a1','date'=>'2020-02-25 01:00:00'],
    ['val'=>'a2','date'=>'2020-02-25 02:00:00'],
    ['val'=>'a3','date'=>'2020-02-25 03:00:00'],
    ['val'=>'a4','date'=>'2020-02-25 04:00:00']
];

$new = ['val'=>'a5','date'=>'2020-02-25 05:00:00'];

// ↓ １の処理で$result2が得られる

$result2 = [
    ['val'=>'a1','date'=>'2020-02-25 01:00:00'],
    ['val'=>'a2','date'=>'2020-02-25 02:00:00'],
    ['val'=>'a3','date'=>'2020-02-25 03:00:00'],
    ['val'=>'a4','date'=>'2020-02-25 04:00:00'],
    ['val'=>'a5','date'=>'2020-02-25 05:00:00'] 
];

// ↓ ２の処理で古いものを削除し以下$result3を得たい

$result3 = [
    ['val'=>'a2','date'=>'2020-02-25 02:00:00'],
    ['val'=>'a3','date'=>'2020-02-25 03:00:00'],
    ['val'=>'a4','date'=>'2020-02-25 04:00:00'],
    ['val'=>'a5','date'=>'2020-02-25 05:00:00']
];

Answer 1

１．$arrと$newを比較し、valが同じ値ならdateを更新したい。

冗長な感じはありますが、愚直にやるならこうかなあ。
ポイントとしては$arrを直接いじりたいなら$arr[$key]['date']みたいにする必要がある点です。（これを$row['date'] = $newRow['date']としても$arrは書き換わりません。）

$arr = [
    ['val'=>'a1','date'=>'2020-02-25 01:00:00'],
    ['val'=>'a2','date'=>'2020-02-25 02:00:00'],
    ['val'=>'a3','date'=>'2020-02-25 03:00:00']
];

$new =  ['val'=>'a4','date'=>'2020-02-25 04:00:00'];

$isInArray = false;
foreach($arr as $key => $row ){
  if($row['val'] === $new['val']){
    $arr[$key]['date'] = $new['date'];
    $isInArray = true;
  }
}

if($isInArray === false){
   $arr[] = $new;
}

var_dump($arr);


/*
array(4) {
  [0] =>
  array(2) {
    'val' =>
    string(2) "a1"
    'date' =>
    string(19) "2020-02-25 01:00:00"
  }
  [1] =>
  array(2) {
    'val' =>
    string(2) "a2"
    'date' =>
    string(19) "2020-02-25 02:00:00"
  }
  [2] =>
  array(2) {
    'val' =>
    string(2) "a3"
    'date' =>
    string(19) "2020-02-25 03:00:00"
  }
  [3] =>
  array(2) {
    'val' =>
    string(2) "a4"
    'date' =>
    string(19) "2020-02-25 04:00:00"
  }
}
*/

２．$arrは最大4件までとしたい。

に関しては、
PHPマニュアル array_multisort
PHPの多次元連想配列のソート

を参考にソートして、ソート後の配列を必要な数だけ取り出す という感じで実装できるかと思います。

データの持ち方を変えればもっと楽な例

<?php

//元のデータがこんな感じであるとして
$arr = [
    ['val'=>'a1','date'=>'2020-02-25 01:00:00'],
    ['val'=>'a2','date'=>'2020-02-25 02:00:00'],
    ['val'=>'a3','date'=>'2020-02-25 03:00:00']
];

foreach($arr as $row){
   $newArr[$row['val']] = $row;
}

$new =  ['val'=>'a4','date'=>'2020-02-25 04:00:00'];

//データが無いときは追加であるときは更新
$newArr[$new['val']] = $new;


var_dump($newArr);

Comments

[natsunoomoide]

>$arr[$key]['date']みたいにする必要がある点です
なるほど、＆をつければいいのかと思っていました。ありがとうございます。

[natsunoomoide]

if($row['val'] === $newRow['val'])

の行で、以下のエラーとなりました。

Illegal string offset 'val' in /aaa/bbb.php on line 0

[natsunoomoide]

２に関して参考リンクありがとうございます。見てきます。

[tanat]

> if($row['val'] === $newRow['val'])
> の行で、以下のエラーとなりました。

$newのデータ構造を誤解していたので修正しました。（複数の書き換えをするのかと思いこんでいました）

[tanat]

> なるほど、＆をつければいいのかと思っていました。ありがとうございます。

参照渡しをしても可能です。が、PHPはブロックスコープが無いので後に影響が出ることがある点を注意する必要があります。

[natsunoomoide]

ご修正ありがとうございます。ところで「なければ追加」の方はいかがでしょうか。

「発生している問題（１について）」や「試したこと（１について）」にあるのと同じ流れで以下elseを追加してみたのですが、やはり同じように余分な配列が追加されてしまう問題が発生してしまいました。

以下$newではvalがa4となっていて、これは$arrにないので「なければ追加」をしたいのですが、var_dumpされる配列は$arrの３つに$newの１つが追加された計４つでなく、６つになってしまうという問題です。

$new = ['val'=>'a4','date'=>'2020-02-25 04:00:00'];
foreach($arr as $key => $row ){
if($row['val'] === $new['val']){
$arr[$key]['date'] = $new['date'];
}else{
$arr[] = $new; // ここで「なければ追加」としたつもりです
}
}
var_dump($arr);

[tanat]

あ、ソース中のコメント
// 同じ値があればその日付を更新し、なければ追加
は見ていませんでした。（これは実現したいこと　のところに追記をして頂きたいです。）

[tanat]

「無ければ追記」に対応しました。
これはforeachの中でやっちゃうとループ回数分増えちゃうので、foreachの外で実施するか、最初の一回だけ実施する必要がありますね。

[natsunoomoide]

コメントアウトにしか書いてなくてすみませんでした。実現したいことに追記しました。

$isInArrayなるほどです！質問でわざわざ２回ループしてるの馬鹿すぎましたね。ありがとうございます！

今２の処理はリンク先を参考にしなばら実装中なのでできる限り頑張ってみます。万が一わからないことがございましたらまた宜しくお願い致します。

[natsunoomoide]

こちらで出来たのではないかと！ご助力頂きましてありがとうございました。

何かおかしいところなどないでしょうか。

少し思ったのは最後の「２．最新の4つのみとし、古いのは削除」の処理ですが、下記のようにソートしてから削除という流れではなくて、一発で古いのを見つけて削除のような処理はないものでしょうか。

<?php
/*
前提
-----------------------------------------------*/
// 元からある配列
$arr = [
['val'=>'a1','date'=>'2020-02-25 01:00:00'],
['val'=>'a2','date'=>'2020-02-25 02:00:00'],
['val'=>'a3','date'=>'2020-02-25 03:00:00'],
['val'=>'a4','date'=>'2020-02-25 04:00:00']
];

// 新しい配列
$new = ['val'=>'a2','date'=>'2020-02-25 05:00:00']; // dateを更新する場合はこれ
// $new = ['val'=>'a5','date'=>'2020-02-25 05:00:00']; // 普通に追加する場合はこれ

/*
１．valが同じならdateを更新、なければ追加
-----------------------------------------------*/
$isInArray = false;

// dateを更新
foreach($arr as $key => $row ){
if($row['val'] === $new['val']){
$arr[$key]['date'] = $new['date'];
$isInArray = true;
}
}

// なければ追加
if($isInArray === false){
$arr[] = $new;
}

//var_dump($arr);

/*
２．最新の4つのみとし、古いのは削除
-----------------------------------------------*/
// dateの値でソート
foreach ( $arr as $key => $value) {
$sort[$key] = $value['date'];
}
array_multisort($sort, SORT_ASC, $arr);
//var_dump($arr); // $arrがこの時点でソートされている

// 一番上のものを削除して4件にする
if( count($arr) > 4 ){
unset($arr[0]);
$arr = array_values($arr); // キーを振りなおす
}
var_dump($arr);

[natsunoomoide]

あと、
unset($arr[0]);
は
$arr = unset($arr[0]);
と”すると”ダメなのに、

array_values($arr);
は
$arr = array_values($arr);
と”しないと”ダメ

というの違いってなぜでしょうね。かなりわかりにくくないですか？

[tanat]

参考までにデータの持ち方を変えればそもそももっと楽というケースを追記しました。

[tanat]

> 何かおかしいところなどないでしょうか。
ぱっと見たところ大丈夫だと思いますが、コードレビューはteratailの範囲外だと思いますので、ご自身でテストケースを作って仕様を満たしているか確認してみてください。

[tanat]

> というの違いってなぜでしょうね。かなりわかりにくくないですか？

それぞれの関数の仕様がそうだから。としか言えないです。（PHPの関数の仕様は歴史的な経緯もあって統一性が無くわかりにくいところが多々あります。）

関数を使用する場合は必ずPHPマニュアルの該当ページを確認するように習慣づけて誤解を回避するしかないんじゃないかなと思います。

[natsunoomoide]

>データの持ち方を変えれば
震えましたｗどうもありがとうございました。

[natsunoomoide]

>ご自身でテストケースを作って
ですね。失礼致しました。色々試してみます。

[tanat]

> 下記のようにソートしてから削除という流れではなくて、一発で古いのを見つけて削除のような処理はないものでしょうか。

最小値を自力で調べて、その際の配列のキーを記憶しておく、最後にそのキーの要素をunsetする
とすれば出来るかと思いますが、ソートしてから削除の方があらゆる面で便利だと思います。

[natsunoomoide]

>PHPマニュアルの該当ページを確認するように習慣づけて
統一性がないものって他にもあるんですね、ご忠告感謝です。注意します。

[natsunoomoide]

>ソートしてから削除の方があらゆる面で便利だと思います
なるほど、わかりやすいですしね。細かい質問にまで本当にありがとうございました。

Answer 2

$result=array_map(function($x) use($new){
  return $x["val"]===$new["val"]?$new:$x;
},$arr);
print_r($result);

Comments

[yambejp]

＞ 最大4件までとしたい

5件以上のデータが入ったサンプルと、どういうロジックで
どれを残すのか例示ください

[natsunoomoide]

１の方、ありがとうございます。foreachを使わずに配列が操作できるとは感激でした。ありがとうございます。

実は１には「なければ追加」というのも考えていまして、「２のロジック」の方の$result2をその例とさせて頂きました。

そして２の方失礼致しました。「２のロジック」を追記させて頂きましたのでご査収いただけましたら幸いです。

[yambejp]

最大4つまでとりだすならこう

$max=4;
$result=array_slice($result,count($result)-$max,$max);

[natsunoomoide]

最大４つの処理、ありがとうございます。今回は日付の古いものを削除ということをしたかったので、まずソートが必要みたいでした。

[yambejp]

私の処理は配列の後ろのほうから４件というのやり方です

> 日付の古いもの

となるとソートしてからでしょうね

[natsunoomoide]

なるほど、高レベルな書き方をありがとうございます。今回はわかりやすさ重視でforeachを素直に使いたいと思いますが、大変勉強になりました。また宜しくお願い致します。