返された関数の結果が条件文に混乱しています

Question

私はPHPでデータベースで動作するクラスを持っています。このクラスのAdd関数は次のとおりです。

function Add($post_id) 
    { 
     if(!is_numeric($post_id)) 
     { 
      return 1181; 
     } 

     $query = "..."; 
     $result = mysql_query($query, $this->link); 
     return $result; 
    } 

また、フォームデータを取得してこのクラスに渡すページもあります。ページのコードは次のとおりです。

$result = $obj->Add($post_id); 

if($result == 1181) 
{ 
    echo 'invalid'; 
} 
else if($result) 
{ 
    echo 'success'; 
} 
else 
{ 
    echo 'error'; 
} 

返される値は1で、出力が「成功」でなければなりませんが、私は「無効」メッセージが表示されます。私が「無効」と「成功」条件文を入れ替えれば、すべてうまくいくが、私はこの問題が何であるか知りたいのですか？

Answer 1

var_dump($result);は良い出発点です。ブール値コンテキストでは1181はtrueに変換されるため、successが印刷されただけで、それが成功するとは思われません。

おそらく間違ってpost_idを渡します。警告と通知の表示を有効にします。クレイジーマジック定数を使用しないでください。falseを使用するか、例外をスローしてください。戻り値は常にmysql_queryであることを確認してください。

私が推測する必要はないので、進歩して有意義な質問をすることができます。

Answer 2

他の人がコメントに指摘しているように、この種のケースではExceptionsを使用する必要があります。ここに例があります。

function Add($post_id) 
{ 
    if(!is_numeric($post_id)) 
    { 
     throw new InvalidArgumentException('Argument should be numeric'); 
    } 

    $query = "..."; 
    $result = mysql_query($query, $this->link); 
    return $result; 
} 

try 
{ 
    $result = $obj->Add($post_id); 
} 
catch(InvalidArgumentException $e) 
{ 
    /* handle the invalid argument exception */ 
} 

if($result) 
{ 
    echo 'success'; 
} 
else 
{ 
    echo 'error'; 
} 

あなたがエラーコードを使用して主張すればまた、あなたがこれを使用する可能性があります：

function Add($post_id) 
{ 
    if(!is_numeric($post_id)) 
    { 
     throw new InvalidArgumentException('Argument should be numeric', 1181); 
    } 

    if($post_id <= 0) 
    { 
     throw new InvalidArgumentException('Argument should be larger than 0', 1182); 
    } 

    $query = "..."; 
    $result = mysql_query($query, $this->link); 
    return $result; 
} 

try 
{ 
    $result = $obj->Add($post_id); 
} 
catch(InvalidArgumentException $e) 
{ 
    switch($e->getCode()) 
    { 
     case 1181: 
      /* handle the invalid argument exception with code 1181 */ 
      break; 
     case 1182: 
      /* handle the invalid argument exception with code 1182 */ 
      break; 
     default: 
      /* handle other invalid argument exceptions */ 
      break; 
    } 
} 

最後に、他の人もコメントしているように、例外処理は、とは何の関係もない、としませんSQLインジェクションの防止に干渉します。