PHP配列同士の比較で、「１つでも一致しなかったら」の判定方法

Question

###実現したいこと
下記$ok_arrをホワイトリストとして、$arrを比較したいです。

具体的には、$ok_arrの中にない値が$arrに１つでもあれば、falseを返したい。というのを実現したいと思っています。（文末に条件について補足あり）

###該当の配列
これが比較したい配列たちです。

$ok_arr = ['apple','melon'];
$arr = ['apple','xxx'];

###試したこと
比較できるものを調べるうちに最も近いと感じたのがこのarray_intersect()ですが、これでは$arr = ['apple'];のような場合しか活用できませんでした。

今回は$arr = ['apple','xxx'];となっていて、xxxは$ok_arrの中にない値なのでfalseを返したいのですが…どう書くのがいいかご教授頂けましたら幸いです。

$ok_arr = ['apple','melon'];
$arr = ['apple','xxx'];

if( array_intersect( $arr, $ok_arr ) ) {
	$bool = true;
}
else{		
	$bool = false;
}

var_dump( $bool );

###補足
上では、「$ok_arrの中にない値が$arrに１つでもあれば、falseを返したい。」だけしか書いていませんが、「$arrに同じ値が複数あるとか、一つも存在しないとか、空だとかの場合もfalseを返したい」とも思っています。失礼致しました。

Answer 1

仕様を満たすだけなら、ごく普通に$arrを foreach で回し、$ok_arrの中にない値があったら即時falseを return で良いかと。

読みやすいし、後で理解もしやすい。

なんかリファクタリングしたぃ～って感じなら、テストいっぱい書いて、検証しながら配列を比較する関数と置き換えてみると良いです。

sample

<?php
$test_arrays = [
  ['apple', 'xxx'],
  ['apple', 'melon'],
  ['melon', 'apple'],
  ['apple', ''],
  ['apple', NULL],
  ['apple', 0],
];
  
function test($test_array){
  $ok_arr = [
    'apple' => true,
    'melon' => true,
  ];
  foreach($test_array as $str){
    if(!isset($ok_arr[$str])){
      return false;
    }
  }
  return true;
}
foreach($test_arrays as $test_array){
  echo (join(",", $test_array)) . ':';
  var_export(test($test_array));
  echo PHP_EOL;
}

apple,xxx:false
apple,melon:true
melon,apple:true
apple,:false
apple,:false
apple,0:false

Comments

[takasima20]

自分が使うならこれですねえ。
どうも、トンチというかクイズ的な考えに流されがちになります。w

[busi]

すみません、普通にfordachで回すとき、空ってどうやって判定するのでしょうか？

下記で$arrを空にすると判定がtrueになってしまいまして、これをfalseにしたいのですが…

$ok_arr = ['apple','melon'];
$arr = [];
$bool = true;
foreach( $arr as $ar ){
if( ! in_array( $ar, $ok_arr ) || empty( $ar ) || is_null( $ar ) || $ar=='' ){
$bool = false;
}
}
var_dump( $bool ); // $arrが空か、または$ok_arrに含まれないものがあったときにfalseを返したい

[takasima20]

$arr の空判定をループ外でやれば?
$arr が空の場合は $ok_arr 以外の適当なものを突っ込んでもいいけど
この場合は素直に処理を分けた方が誤解が少ない気がします。

[busi]

なるほど、どうもありがとうございます。

[busi]

サンプルありがとうございます。ぐぅ、ベストアンサー決めたのに追加でご回答してくださいますか！お優しい…。appleとmelonをキーにしてissetで判定するとはかなりプロいですね。どうもありがとうございました。大変勉強になりました。

Answer 2

仕様が
具体的には、$ok_arrの中にない値が$arrに１つでもあれば、falseを返したい。というのを実現したいと思っています。
だけでいいのであれば（=$arrに同じ値が複数あるとか、一つも存在しないとかでもtrueとするということですよね）

array_diffの第一引数に判定対象、第二引数にホワイトリストを入れて、
戻り値の配列の数が1以上であればfalseとすればいいかと思います。

<?php
$ok_arr = ['apple','melon'];

$arr = ['apple','xxx'];

var_dump(array_diff($arr,$ok_arr));
/*
以下のような配列を取得できるので、配列の要素数を数えて判定
array(1) {
  [1] =>
  string(3) "xxx"
}

*/


//arrに含まれる要素がすべてok_arrに含まれている場合
$ok_arr = ['apple','melon'];
$arr = ['apple'];

var_dump(array_diff($arr,$ok_arr));
/*
array(0) {
}
*/

$ok_arr = ['apple','melon'];
$arr = ['apple','melon'];

var_dump(array_diff($arr,$ok_arr));
/*
array(0) {
}
*/

//値が正しければ複数あってもOKとする
$ok_arr = ['apple','melon'];
$arr = ['apple','apple'];

var_dump(array_diff($arr,$ok_arr));

/*
array(0) {
}
*/

Comments

[busi]

ありがとうございます。ご説明わかりやすくて助かりました。

>$arrに同じ値が複数あるとか、一つも存在しないとかでもtrueとするということですよね
すみません、補足しましたが、もしこのような場合はfalseとしたかったです。

[tanat]

であれば、
重複しているかどうか→array_unique()前後で要素の数を数えて同一であることを確認する
一つも要素が存在しない場合→countで要素数を数えて1以上の時を判定
みたいな感じで条件を追加すればOKです。

多分他にも想定しきれてない仕様があると思われるので、まずは求める仕様を明確にするのが良いですね。

[busi]

なるほど、こちらの不手際でなんどもご説明させてしまってすみません。仕様の明確化を気を付けます。ありがとうございました。

Answer 3

強引ですがこれで…

<?php
$ok_arr = ['apple','melon'];
$arr = ['apple','xxx'];
if (count($ok_arr) == count(array_unique(array_merge($ok_arr, $arr)))) {
    $bool = true;
} else {
    $bool = false;
}
?>

Comments

[busi]

なるほどカウントでアプローチしますか。ありがとうございます。勉強になります。

Answer 4

未検証ですが
$bool = count($arr) === count(array_intersect($arr, $ok_arr)); ではどうでしょう

Comments

[busi]

ありがとうございます。参考にさせて頂きます。