phpでどちらでもなかった場合の判定の省略した書き方

Question

ifの条件式で変数の値がONでもOFFでもない値が来た場合の処理を書きたいのですが、下記のようにオーソドックスに&&で区切る以外の方法はありませんでしょうか。書き方がくどいので一瞥して何がしたいかわかりにくいため複合条件の省略した書き方を探しています。

if ($test !== 'ON' && $test !== 'OFF') {
    throw new Exception('どちらでもない');
}

Answer 1

FILTER_VALIDATE_BOOLEAN フィルタおよび FILTER_NULL_ON_FAILURE フラグを使うのがおすすめです。

PHP: 検証フィルタ - Manual

"1"、"true"、"on" および "yes" の場合に TRUE、 それ以外の場合に FALSE を返します。

FILTER_NULL_ON_FAILURE が設定されている場合は、 FALSE が返されるのは "0"、"false"、"off"、"no" および "" の場合のみとなります。 boolean 以外の値については NULL を返します。

厳密には "on" "off" だけではありませんが、"1" や "0" も同等に扱ったほうが便利な場合が多いです。論理値を文字列に変換したときはこうなるので。また、大文字と小文字は区別されません。

$bool = filter_var($test, FILTER_VALIDATE_BOOLEAN, FILTER_NULL_ON_FAILURE);
if ($bool === null) {
    throw new \Exception('どちらでもない');
}
var_dump($bool); // bool(true) か bool(false) のいずれか

Comments

[mpyw]

この方法のきれいなところは、あらゆる型のあらゆる値を TRUE / FALSE / NULL のいずれか3通りまで落とし込んでくれるところにありますね。フィルタを名乗るだけあって有能です。

[motuo]

恥ずかしながら、この命令知りませんでした…
確かに有能な機能ですね！

[enigumalu]

ありがとうございます。こういった関数があることを期待していたのですがズバリです

Answer 2

こういう書き方もあります。

if ( !in_array( $test, [ 'ON', 'OFF' ], true ) ) {}

【PHP: in_array - Manual】
http://php.net/manual/ja/function.in-array.php

【PHPのin_arrayは罠が多いので注意喚起が必要 - Qiita】
http://qiita.com/tadsan/items/2a4c3e6b0b74a408c038

Comments

[enigumalu]

最初に思いついたのがin_arrayで結局これが一番わかりやすい気がしているのですが
in_arrayって配列に値があるかチェックするという仕様記載なので、
文字列に配列の値があるかという利用だと後から見た人がどう思うのかなとちょっと悩んでます

Answer 3

正規表現でフィルタする場合はこうです

if(filter_var($test,FILTER_VALIDATE_REGEXP,["options"=>["regexp"=>"/^(ON|OFF)$/"]])!==false){

}

Answer 4

頻繁に使うなら関数にするのもいいかも?

function is_normal($x) {
    if (!isset($x)) return false;
    if (!is_string($x)) return false;
    if ($x == 'ON') return true;
    if ($x == 'OFF') return true;
    return false;
}


if (!is_normal($test)) {
    throw new Exception('どちらでもない');
}

Answer 5

じゃ、こっちでｗ

<?php
$switch = [
    'ON' => true,
    'OFF' => true
    ];
$test='';
if (!isset($switch[$test])) {
    throw new Exception('どちらでもない');
}

Answer 6

分かり易く伝える、という意味では下記ではどうでしょうか？

if ($test === 'ON') {
    //ONの処理
} elseif ($test === 'OFF') {
    //OFFの処理
}else{
    throw new Exception('どちらでもない');
}

Answer 7

if (!preg_match("/^(ON|OFF)$/", $test )) {
     throw new Exception('どちらでもない');
}

比較候補が３つ４つになってきたら正規表現で書いたほうが !== と && をたくさん並べるよりも見やすくなるかなと思いますが、ON/OFFの２つなら対して変わらないですね。

Answer 8

phpでどちらでもなかった場合の判定の省略した書き方

IFではないですが、こういう書き方もできますね

switch ($test) {
	case 'OK':
	case 'NG':
		# 何もしない場合は何も書かない
		break;
	
	default:
		# どちらでもない時の処理
		break;
}

Comments

[KiyoshiMotoki]

横から失礼します。

$test = 0 のとき、このコードは case 句が実行されてしまいますよ。

なぜなら、PHP の switch構文は緩やかな比較（`==`による比較のこと）を行うからです。

http://php.net/manual/ja/control-structures.switch.php
> 注意:
> switch/case が行うのは、 緩やかな比較 であることに注意しましょう。

http://php.net/manual/ja/language.operators.comparison.php
> 整数値を文字列と比較したり、比較に数値形式の文字が含まれる場合は、文字列が 数値に変換され、 数値としての比較を行います。

http://php.net/manual/ja/language.types.string.php#language.types.string.conversion
> 文字列の最初の部分により値が決まります。文字列が、 有効な数値データから始まる場合、この値が使用されます。その他の場合、 値は 0 (ゼロ) となります。

[raichi]

おっと、コメント気が付きませんでした、そういやそうでしたね。
厳密にやりたけりゃ
switch (true) {
case $test === 'OK':
case $test === 'NG':
# 何もしない場合は何も書かない
break;

default:
# どちらでもない時の処理
break;
}
とかですかね。
よっぽど&&で同じ条件つかない限り、普通にif文書いたほうがよさそうですが。