array_searchを使用しない処理に置き換えたい

Question

現在array_searchを使用して問題無く動かせている処理があるのですが、
配列の数が増えてくると重くなるのだろうかと思い別の処理を考えています。

実現したいこと

・特定の多次元配列を取得

現在の処理

$data = [
  ['日付', '名前', '番号', 'aaa'],
  ['日付', '名前', '番号', 'bbb']//...
];　

$search_word = 'aaa';
$search_data = array_search($search_word, array_column($data, 3));

$day = $data[$search_data][0];//...

置き換えようとした処理

$data_aaa = ['日付', '名前', '番号'];
$data_bbb = ['日付', '名前', '番号'];//...

$search_word = 'aaa';
$data = ${'data_' . $search_word};

$day = $data[0];//...

下でも同じことはできたのですが
変数が増加するのはあまり効果が無さそうに感じます。

無理にいじろうとせずarray_searchを使用したほうが良いでしょうか？
方法を変えても体感速度に変化が無いのであればそのままにしようと思っています。

何か良い実装方法がありましたらご教授いただければ幸いです。

Answer 1

同じ配列に対して同じ方法で頻繁に検索をかけるのであれば、配列の入れ方を変えるのも1つの手段かと思います。

$data = [
  ['日付', '名前', '番号', 'aaa'],
  ['日付', '名前', '番号', 'bbb']//...
];

$keyed = array_combine(array_column($data, 3), $data);

// 検索方法
$keyed['aaa'];

Comments

[kumo_975]

array_combine、初めて知りました。
これがちょうど欲しかった機能かもしれません。
こちらを元に少し考え直してみようかと思います。
この件以外でも使える良い機能を知ることができました。
ありがとうございます。

[kumo_975]

最終的にarray_combineを実行した結果と同じ形式に配列を変更しました。
$data = [
'aaa' => ['日付', '名前', '番号']
];
$search_word = 'aaa';
$search_data = $data[$search_word];

見た目すっきりして分かりやすくなりました！
多次元配列ばかり使っていて連想配列を使うという発想が全く無かったので大変助かりました。
ありがとうございました。

Answer 2

$data = [
  ['日付', '名前', '番号', 'aaa'],
  ['日付', '名前', '番号', 'bbb'],
  ['日付', '名前', '番号', 'aaa'],
  ['日付', '名前', '番号', 'bbb'],
];

$search_word = 'aaa';
$result=array_filter($data,function($x) use($search_word){
  return in_array($search_word,$x);
});
print_r($result);

Comments

[kumo_975]

ありがとうございます。
$search_wordで調べる配列値に同じ値は存在しませんので1つだけになります。
記載するのを失念しておりました、申し訳ございません。
functionの付くコードのことがまだ理解できていないもので自分には少し難しいですがarray_filter、便利な機能ですね。他の箇所で使ってみようと思います。

Answer 3

パフォーマンスへの影響はほぼないかと思いますが、
変数の増加＝その分の箱の増加
と考えて良いので、それなら箱の中身だけの変化の方が良いかと思いますし、
$dataが実際にどう取得されるかによりますが、外出しも可能なのであれば
メインロジックをいじる必要がなくなる前者のほうがメンテナンス性が良いかと思います。

Comments

[kumo_975]

ありがとうございます。
やはり変数は無駄に作らないほうが良さそうですね。
$dataは配列が初めから完成した状態で別ファイルにまとめてあるもので、include読み込みで取得します。

[m.ts10806]

「外出し」と言った意味では、設定ファイルか、もしくはデータベースにした方が管理はしやすくなると思います。

[kumo_975]

すみません、外出しの言葉の意味が理解できていませんでした、仰る通りです。
本来DBに入れるようなデータを全て配列で管理しているので、
DBの使い方が理解できるようになったときはDBで管理したいと思います。

[m.ts10806]

表現が伝わりづらくて申し訳なかったです。
外に出す＝ハードコーディングしない（ロジック内に直に書かない）と考えてください。