if文の書き方

質問

プログラム初心者です

if文の処理速度が速いのは一般的に考えてどちらでしょうか?

別々にif文で処理する方法

try {

if(!isset($_REQUEST['id'])&& !is_string($_REQUEST['id'])){
throw new Exception('error');
}

if(!isset($_COOKIE['date'])&& !is_string($_COOKIE['date'])){
throw new Exception('error');
}

} catch (Exception $e) {
$msg = $e->getMessage();
}

一緒に処理する方法

try {

if(!isset($_REQUEST['id']) && !is_string($_REQUEST['id']) && !isset($_COOKIE['date']) && !is_string($_COOKIE['date'])){
throw new Exception('error');
}

} catch (Exception $e) {
$msg = $e->getMessage();
}

よろしくお願いします

回答No.1

agunuz · ベストアンサー

>if(!isset($_REQUEST['id'])&& !is_string($_REQUEST['id'])){

これって、(isset($_REQUEST['id']) && is_string($_REQUEST['id'])) の否定でしょうか？もしそうなら(!isset($_REQUEST['id']) || !is_string($_REQUEST['id'])) と書かないといけませんが・・・

なので合わせて書くなら

if(!isset($_REQUEST['id']) || !is_string($_REQUEST['id']) || !isset($_COOKIE['date']) || !is_string($_COOKIE['date'])) {

ではないんですかね。

なお、実行速度は大差ないです。基本的には（該当しないのがデフォなので）全部チェックするのですから）。

個人的には、見易さ・パッと見たときの意味の掴みやすさから

if (!isset($_REQUEST['id']) || !is_string($_REQUEST['id'])) { throw new Exception('error'); }
if (!isset($_COOKIE['date']) || !is_string($_COOKIE['date'])) { throw new Exception('error'); }

が好みです。

お礼

回答ありがとうございます
すいません||でした
見易さ優先で行こうと思います

回答No.2

agunuz

追記です。

基本的に「処理速度」よりも、ソースの読みやすさ（内容を把握しやすい記述）を心がけてください。明らかに無駄な処理とかボトルネックになる処理でもない限り、処理速度は誤差の範囲でしかありません。逆に読みにくくてバグを生みやすいソースは決定的にアウトです。

お礼

見易さのほうが大事でした