未定義の変数に対して条件分岐する際の対処法

質問

PHP 5.3 で次のような関数があったとして、配列$optionの内容によって条件を分岐するとします。
$optionに渡される値は不定で、空の場合もあれば$option['param3']などが渡されることもあります。

function sample($option=array()){
if($option['param1'] == 'test'){
echo "A";
} elseif($option['param2'] == 'test'){
echo "B";
}
}

もし引数に「$option['param3']='test'」だけが渡されたら、条件分岐の際に未定義の「param1」と「param2」はNoticeエラーを返すことになります。
「@」でとりあえず抑制したり、予想される全てのキーに対して初期化するなどがあると思いますが、
エラー表示レベルを変えずにこのエラーを消すにはどうするのが一般的ですか？

回答No.2

LancerVII · ベストアンサー

こんにちは。

No.1ですカッコ一つ忘れてました。訂正です。すみません。

if ( isset($option['param1']) && $option['param1'] === 'test' ) {
}

お礼

狙い通りの動作が確認できました。
@でごまかすよりすっきりしているので今後この方法でやっていこうと思います。
ありがとうございます。

回答No.3

yambejp

すでに質の良い回答はでていますが、配列の要素を前提とするなら

array_key_exists('param1',$option)

のような書き方の方が、理解が深まるかもしれません。
既出の回答と結果は同じです

またおそらく#1さんが常識の範囲としてあえて触れていませんが
$option['param1'] == 'test'
はありえません。最低でも
$option['param1'] === 'test'
としてください

変数（配列をふくむ）と文字列との比較の場合、
仮に$option['param1']が0でも成り立ってしまいますよ

お礼

そういう書き方もあるのですね。
比較演算子のほうもご指摘ありがとうございます。
大きな勘違いをしていたようです。

回答No.1

LancerVII

こんにちは。

issetでチェックすればよいのではないでしょうか。

if ( isset($option['param1'] && $option['param1'] === 'test' ) {
}