サンプルプログラムでよくわからない箇所

Question

下記は本に掲載されていたサンプルプログラムの抜粋です

//IDが指定されていない場合や、指定されたIDの記事が見つからない場合は一覧にリダイレクト
$id and $data['article'] = Model_Article::find($id);
if (!$data['article']) {
    Response::redirect('articles');
}

引数で受け取った$idを使ってデータを取得する処理で、$idは初期値に0が指定されており、$idは1から始まるので$idの指定がなければデータは取得できずリダイレクト処理となります。

最初の$id and はあっても無くても関係なさそうですが、何の意味があるのでしょう？

Answer 1

基本的には最初に回答しているyng13さんの通りです。

補足として、
このソースの場合、idが入らなかった時(0)に$data['article']は定義されていないのでNotice: Undefined indexエラーが発生します。
それを解決させるには下記のようにissetやemptyを挟む必要があります。

if(!isset($data['article']) || !$data['article']) {
    Response::redirect('articles');
}

個人的にはソースの明瞭性に欠けていたり、判定の内容が異なると感じるので以下のように区切ったほうが良いと思います。
あくまでも個人的な意見です。

if(!empty($id)) Response::redirect('articles');
$data['article'] = Model_Article::find($id);

if(!$data['article']) Response::redirect('articles');

Comments

[msx2]

$data['article']が定義されていないというのが不思議でPHPマニュアルを見ていたら「短絡評価」という言葉を始めて知りました。
まさか$idが入らなかった時点で右側の$data['article']=Model_Article::find($id)は評価されないとは…。それで補足のissetやemptyが必要になるってことなんですね。
また一つPHPの理解が進みました、ありがとうございます。

Answer 2

このコードだけで全貌がわかりませんが、コード内のコメントにあるとおり、

//IDが指定されていない場合や、指定されたIDの記事が見つからない場合は一覧にリダイレクト

idが指定されていない場合を評価しているのが $id andの記述です。

$idは初期値に0が指定されており

ということなので、$idが初期値のままで0の場合、$id andの条件？は真(true)になりません。
つまり、$idが指定されているかどうかを検査する、という意味を持っています。

$idが指定されているかどうか、と指定された$idのModel_Articleが存在するかどうか、は別の問題ですよね？
例えば、ユーザーが$idに相当する数字を入力しなければいけない画面で、

「IDを指定してください」というエラーを表示することと、「指定されたIDのArticleは存在しません」という表示では意味が違いますよね？

Comments

[msx2]

簡易的なプログラムだったので深く考えていませんでしたが、ご指摘の通り$idが指定されていないのと$idのArticleが見つからないのは意味が違いますね。
そういうところも考えながらプログラムを作っていかないといけないのですね。
ご回答ありがとうございます！