[PHP]配列の中から特定の条件に当てはまるレコードを抽出して処理したい

Question

前提・実現したいこと

いつも大変お世話になっております。

DBから引っ張ってきた下記の二次元配列の中から

①statusが4で
②かつkindが重複しているものがあれば、
③updateが古いほう（より過去のもの）のstatusおよびflagだけを0にする

ということをやりたいのですが、
途中から行き詰ってしまって…皆さまのお力添えをいただきたいです。

<?php
$list = Array ( 
	[0] => Array ( [no] => 8 [kind] => 5 [flag] => 1 [status] => 4 [update] => 2014-08-20 ),
	[1] => Array ( [no] => 15 [kind] => 3 [flag] => 0 [status] => 3 [update] => 2016-12-25 ),
	[2] => Array ( [no] => 22 [kind] => 6 [flag] => 1 [status] => 4 [update] => 2017-03-01 ),
	[3] => Array ( [no] => 48 [kind] => 9 [flag] => 1 [status] => 4 [update] => 2018-05-22 ),
	[4] => Array ( [no] => 52 [kind] => 5 [flag] => 1 [status] => 4 [update] => 2014-03-04 ),
	[5] => Array ( [no] => 55 [kind] => 23 [flag] => 1 [status] => 4 [update] => 2015-09-28 ),
	[6] => Array ( [no] => 104 [kind] => 2 [flag] => 0 [status] => 3 [update] => 2018-11-11 ),
	[7] => Array ( [no] => 107 [kind] => 3 [flag] => 1 [status] => 4 [update] => 2015-04-28 ),
	[8] => Array ( [no] => 109 [kind] => 1 [flag] => 0 [status] => 3 [update] => 2016-10-15 ),
	[9] => Array ( [no] => 116 [kind] => 23 [flag] => 1 [status] => 4 [update] => 2018-07-10 ),
	[10] => Array ( [no] => 131 [kind] => 11 [flag] => 0 [status] => 3 [update] => 2013-01-02 ),
);
?>

テーブル定義は以下のようになってます。

CREATE TABLE TableName(
no int(11) unsigned auto_increment,
kind tinyint(3) unsigned,
flag tinyint(4),
status tinyint(4) DEFAULT 0,
update date,
primary key (no)
);

試したこと

まずループでstatusが4のものを抽出して、
そのうえでkindが重複しているものを抽出し、
updateが古いほうのstatusとactiveFlagを書き換える。

という手順がいいかと考えて、下記までは書いたのですが、
【kindが重複しているものをさらに抽出してそのうちupdateが古いほうを】
というのをどうすればいいか、どうにも思い浮かばない状態です。

<?php

$sql = "SELECT * from TableName where status='3' OR status='4'";
$rs = mysqli_query($link,$sql);
$count=0;
while($row = mysqli_fetch_assoc($rs)){
	$Array[$count]["no"] = $row["no"];
	$Array[$count]["kind"] = $row["kind"];
	$Array[$count]["flag"] = $row["flag"];
	$Array[$count]["status"] = $row["status"];
	$Array[$count]["update"] = $row["update"];
	$count++;
}

foreach ($Array as $val) {
	if ($val['status'] == '4') {
		//ここでkindの重複および日付の判定をしたい
		$val['status'] = 0;
		$val['activeFlag'] = 0;
	}
}
?>

DBからstatus3と4を取得しているのは他の処理でも3のレコードが必要なためです。
そもそもこの取得を一度にやろうとせず、もうちょっと考えるべきでしょうか。

間違っていたり、もっと効率的な方法がありましたら
ご指摘頂戴できれば大変助かります。

補足情報（FW/ツールのバージョンなど）

所謂レガシーシステムを扱っておりまして、
PHP環境は5.3になります。
array_columnなどが使えない状態です。

Comments

[m.ts10806]

>DBからstatus3と4を取得しているのは他の処理でも3のレコードが必要なためです。
そもそもこの取得を一度にやろうとせず、もうちょっと考えるべきでしょうか。

status='3'でやろうとしている他の処理次第です。
要件次第でもしかしたらSQL一発投げるだけで終わる可能性もあります。

[uthio]

コメントありがとうございます。
status='3'でやる処理は、statusが3の場合でupdateが現在の日時を過ぎた場合、そのレコードのstatusを4にしてflagを1にする、といったことになります。（updateは公開日、という位置づけで未来の日付になっているのがあります）

つまりまずその処理を行った上で、今回質問に上げた処理を行いたい状態です。

[yambejp]

SQL側でやってもよいのでしょうか？

[uthio]

>yambejp様
コメントありがとうございます。
SQL側でやる、というのがすみません、理解しきれてないですが、むしろ最終的にはSQLを更新するのが目的になります。
特に表示させる必要はなく、実行用のPHPを作ろうと思っています。
これで答えとしては正しいでしょうか…？

[yambejp]

そういうことならSQLでstatusが4でkindがダブっている古い日付のstatusを0にするという
処理を書けばいいような・・・

[uthio]

statusが4の時にそういう処理を書いたSQLを書く、という認識で大丈夫ですか？

なるほど、そもそもSQLでそういう処理ができる方法を知りませんでした。。
知識不足ですみませんでした。
もうちょっと探してみます。

Answer 1

SQLで処理

CREATE TABLE tbl(
no int(11) unsigned auto_increment,
kind tinyint(3) unsigned,
flag tinyint(4),
status tinyint(4) DEFAULT 0,
`update` date,
primary key (no)
);

insert into tbl values
(  8, 5,1,4,"2014-08-20"),
( 15, 3,0,3,"2016-12-25"),
( 22, 6,1,4,"2017-03-01"),
( 48, 9,1,4,"2018-05-22"),
( 52, 5,1,4,"2014-03-04"),
( 55,23,1,4,"2015-09-28"),
(104, 2,0,3,"2018-11-11"),
(107, 3,1,4,"2015-04-28"),
(109, 1,0,3,"2016-10-15"),
(116,23,1,4,"2018-07-10"),
(131,11,0,3,"2013-01-02");

だとして
statusが4で、kindがダブっていて、最新ではないデータはこう

select * from tbl as t1
where exists(
select 1 from tbl
where status=t1.status
and kind=t1.kind
and `update`>t1.`update`
)
and status=4

上記を応用してupdate

update tbl set status=0
where no in(select no from(
select no from tbl as t1
where exists(
select 1 from tbl
where status=t1.status
and kind=t1.kind
and `update`>t1.`update`
)
and status=4
) as sub)

Comments

[uthio]

何から何までありがとうございます。
SQLの知識不足が根本的に問題ですね。。
PHPで無理やりやろうと思ったところから改めねばなりません。

頂いたものでどうにか対応できる気がします。
まず中身をしっかり把握して進めます。

また終わりましたらこの場を借りてご報告いたします。
取り急ぎお礼まで。

[uthio]

こちら回答が大変遅くなり失礼しました。
お陰様で解決しましたし、そもそもSQLの知識が足りないということも気付かせていただき、誠にありがとうございました。

Answer 2

PHPでやるなら

$list = [
    [ "no" => 8   ,"kind" => 5     ,"flag" => 1, "status" => 4, "update" => "2014-08-20" ],
    [ "no" => 15  ,"kind" => 3     ,"flag" => 0, "status" => 3, "update" => "2016-12-25" ],
    [ "no" => 22  ,"kind" => 6     ,"flag" => 1, "status" => 4, "update" => "2017-03-01" ],
    [ "no" => 48  ,"kind" => 9     ,"flag" => 1, "status" => 4, "update" => "2018-05-22" ],
    [ "no" => 52  ,"kind" => 5     ,"flag" => 1, "status" => 4, "update" => "2014-03-04" ],
    [ "no" => 55  ,"kind" => 23    ,"flag" => 1, "status" => 4, "update" => "2015-09-28" ],
    [ "no" => 104 ,"kind" => 2     ,"flag" => 0, "status" => 3, "update" => "2018-11-11" ],
    [ "no" => 107 ,"kind" => 3     ,"flag" => 1, "status" => 4, "update" => "2015-04-28" ],
    [ "no" => 109 ,"kind" => 1     ,"flag" => 0, "status" => 3, "update" => "2016-10-15" ],
    [ "no" => 116 ,"kind" => 23    ,"flag" => 1, "status" => 4, "update" => "2018-07-10" ],
    [ "no" => 131 ,"kind" => 11    ,"flag" => 0, "status" => 3, "update" => "2013-01-02" ],
];
$list_a = []; //status4の配列

//status4 のデータ配列を作る　SQLから取れるならとったほうがいい。
for ($i = 0; $i < count($list); $i++) {
     if($list[$i]["status"] == 4){
         array_push($list_a,$list[$i]);
     }
}

//メイン処理
for ($i = 0; $i < count($list_a); $i++) {
    for ($j = $i+1; $j < count($list_a); $j++) {
        if($list_a[$i]["kind"] == $list_a[$j]["kind"]){
            if($list_a[$i]["update"] > $list_a[$j]["kind"]){
                $list_a[$j]["flag"] = 0;
                $list_a[$j]["status"] = 0;
                print_r($list_a[$j]); //printの代わりにSQLをアップデート
            }else {
                $list_a[$i]["flag"] = 0;
                $list_a[$i]["status"] = 0;
                print_r($list_a[$i]); //printの代わりにSQLをアップデート
            }
        }
    }
}

Comments

[uthio]

ご丁寧にありがとうございます。
なるほどです、私が当初やりたかったことはこういうやり方で可能なのですね。

こちらもきちんと内容確認して対応してみたいと思います。
yambejp様からいただいたものも含めて、終わりましたら改めてご報告いたします。

まずはお礼を伝えさせてください。ありがとうございました。

[uthio]

こちら返答が大変遅くなってしまってすみません。
頂いたもので動作を試したところ、私がやりたい通りのことができました。

ただそもそも私の知識不足のせいなのですが、最終的には別のやり方で対応することになってしまいました。
お時間頂いたにも関わらずすみませんでした。
せめてお礼だけこの場でお伝えさせてください。
ありがとうございました。