2009-02-26 4 views
1

に条件付きでPHPスクリプトを実行するための最良の方法、それは、これを行う方が良いです:私はつべこべされるかもしれないユーザのログイン

if ($loggedin) { 
    // normal process 
} 
else { 
    header('Location: login.php'); 
} 

またはこの:

if (!$loggedin) { 
    header('Location: login.php'); 
    exit(); 
} 
// normal process 

それともそれだけで問題ありません。 ?

答えて

5

「通常のプロセス」は単純なチェックのためにインデントレベルが深くなっていないので、私は2番目を好む。私はPHPがおそらくこれを最適化してパフォーマンスが無関係になると思うので、その時点では読みやすさの問題であり、2番目のものはあなたのすべてのロジックを1つの巨大なものにラッピングするよりも( "ログインしていない、リダイレクトする" IF。

5

「//通常のプロセス」は長いコードである可能性が高いので、私は2番目のスタイルを好むので、最後の}(elseブランチから)はちょっと混乱するかもしれません。

1

本当に違いはありません。私の個人的な意見では、ユーザーがログインしていないときに何が起こっているかを見るために最後までスクロールする必要がないので、2番目のブランチはわかりやすいです。

0

リダイレクトしないように常にexitまたはdieを使用してください。さらなるコードが実行される。

しかし、それ以外にも、私は$loggedinがあなたの「通常のプロセス」の前提条件なので、第2のものも好む。

0

間違いなく2番目の。それはあなたのコードの残りの部分でより良く読み込まれ、よりよく動作します。

関連する問題