ifの条件で||を省略して書く方法

Question

このようなif文の条件で

if($value == 'a' || $value == 'b' || $value == 'c' || $value == 'd' || $value == 'e' || $value == 'f')

変数$valueは同じなので何回も||（または）を使うのはおっくうなのですがどうにかして省略できる方法はありますでしょうか。

Comments

[m.ts10806]

条件を仮のものではなく目的に近づけてもっと具体的にされたほうが良いです。
考えられる可能性全てを考慮しようと思うと完全に「提示されたabcdefだけを通せばいい」という条件での大喜利になってしまいます。

[退会済みユーザー]

$value = 0 ってどう考えてるんですか？

[ryo12su29kel]

質問の内容が曖昧ですみませんでした。以後気を付けます。

[m.ts10806]

質問は編集できますので。

Answer 1

if ( in_array( $value, [ 'a', 'b', 'c', 'd', 'e', 'f' ], true ) )

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

【in_arrayを使うときは黙って第三引数を付けること - Qiita】
https://qiita.com/ritukiii/items/3a6add378ae089ab5d70

Answer 2

if(in_array($value, ['a','b','c','d','e','f']))
とか、
if(preg_match('/^[abcdef]$/',$value)){
とか。

後者は1文字の場合限定ですが。

Answer 3

if(!($value != 'a' && $value != 'b' && $value != 'c' && $value != 'd' && $value != 'e' && $value != 'f'))

|| がいやなら && を使えばいいじゃない。

Answer 4

key をチェックするっていうのもよくやる手です。
＊まぁ、面倒くささは変わらんｗ

<?php
$arr = [
    'a' => TRUE,
    'b' => TRUE,
    'c' => TRUE,
    'd' => TRUE,
    'e' => TRUE,
    'f' => TRUE,
    'abc' => TRUE,
];
foreach(['a', 'g', 'abc'] as $value){
    echo $value . ' をテスト';
    if(isset($arr[$value]))echo ' : あたり';
    if(array_key_exists($value, $arr))echo ' : あたり';
    echo PHP_EOL;
}

ちなみに、array_key_exists() は早い！
雑な検証

この処理でパフォーマンス求めることって殆どないと思うけど参考まで。

Answer 5

単一条件ならいっそswitch

switch($value){
    case 'a':
    case 'b':
    case 'c':
    case 'd':
    case 'e':
    case 'f':

       break;
    default:

}

Answer 6

大喜利用

<?php
foreach(['a', 'f', 'g', 'abc'] as $value){
    echo $value . ' をテスト';
    $test = "a";
    while($test !== "g"){
        if($value === $test)echo ' : あたり';
        $test++;
    }
    echo PHP_EOL;
}

入力に一文字限定

foreach(['a', 'f', 'g', 'abc'] as $value){
    echo $value . ' をテスト';
    if(floor((ord($value)-97)/6)===0.0)echo ' : あたり';
    echo PHP_EOL;
}

Answer 7

私は、短くて、見やすくて、修正のしやすいコードが好きです。
なので、私ならこのあたりです。

if (preg_match('/^[a-f]$/', $value)) // 連続した文字限定
if (strpos(' abcdef', $value))

※$valueに入る値は空白以外の1文字であることを前提にしています

Comments

[maisumakun]

strposだと'bc'のようなものにもヒットしてしまいません？

[m.ts10806]

質問要件が具体的でないことが原因ですが、strposは悪手だと思います。
あとpreg_match()の最初のコードは何時間も前に投稿された回答に全く同じものがあるので「大喜利だ」とするのでしたら提示する回答としては如何なものかと。

[hentaiman]

in_arrayは内容変更の際も修正し易いと思いますし他者が読む場合にも意図が伝わりやすいと思います

[miu_ras]

maisumakunさん
そうですね…。$valueに単一の文字しか入らないことを前提に考えていました。そうではない場合も考えるべきだとすると、2文字以上、空文字、null、異なる型、未定義など、どこまで考慮してあげるべきか悩ましいですね。質問者が明示すべきなんでしょうけど。

[miu_ras]

m.ts10806さん
「大喜利だ」とするのだったら、既出の回答と同じものを書いたらいけないのですね。知りませんでした。すみませんでした。(「大喜利だ」と書かなければOKだったということですか？)

[m.ts10806]

>(「大喜利だ」と書かなければOKだったということですか？)

いえ、そういうことではなく、あえて同じ回答を含めるメリットはそもそもありません。

[miu_ras]

m.ts10806さん
そうですよね。『「大喜利だ」とするのでしたら』とあり、あえてそう書いたということは、大喜利だと書いたことが問題だと考えているのだと解釈してしまいました。実質的に無意味な文ということですよね。

[miu_ras]

hentaimanさん
in_arrayは私も場合によっては使います。ただ今回の場合単一文字の判定です。例えば「g」も追加するという場合に、1文字の追加で対応できる書き方もあるが、in_arrayだと「'g', 」の5文字を追加しなければいけません。

それが「私は1文字の修正で済む方が好き」というだけです。好みの問題です。総合的にどちらが優れているかを議論するつもりはありませんでした。すみません。

[退会済みユーザー]

じぶんの案でだしてる strpos だと　

1文字目にヒットした場合、0
ヒットしなかった場合、false

だったきがするけど その条件式で大丈夫?

[miu_ras]

asahina1979さん
ダメだと思います…。先頭に絶対使用しない文字を入れるなどしないと使えませんね。