質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.23%

$_GETおよび$_POSTの値を判定するとき、noticeを出さないようにしたい。

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 2,366

muro

score 105

if($_GET["hoge"] == "foo") {
// 処理
}


$_GETおよび$_POSTの値が指定のものと同じかどうかif文で判定するとき、上記のようなコードでも動作しないことはないのですが、noticeが出てしまいます。

if(isset($_GET["hoge"]) && $_GET["hoge"] == "foo")) {
// 処理
}


よって、上記のようにissetも一緒に入れて、値そのものが空でないかどうかの判定も行っているのですが、if文が長くなってしまい、見づらいと感じてしまいます。
これよりも短く書いてnoticeを出さないようにする方法はないでしょうか。
なお、PHP7を利用しています。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

+3

if(isset($_GET["hoge"]) && $_GET["hoge"] == "foo")) {
// 処理
}


if (filter_input(INPUT_GET, 'hoge') == 'foo') {
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/12/13 14:25

    回答ありがとうございます。今回はこのコードにしたいと思います。
    filter_inputを知らなかったので調べてみましたが、$_GETなどだとスーパーグローバル変数に直接アクセスすることになるから危険ということもあるみたいで、今度から極力filter_inputを使っていくようにします。

    キャンセル

  • 2016/12/13 14:29

    > 「スーパーグローバル変数に直接アクセスすることになるから危険」
    セキュリティ的な観点というよりは、$_GET[('hoge'] = '値'; のように代入ができてしまうことで、バグの元になりやすいため。

    キャンセル

+1

PHP7らしく解決するならこんな書き方ができそうです。
http://qiita.com/hnw/items/dff62cd02c780b613d03

if(($_GET["hoge"] ?? null) == "foo") {
  // 処理
}

まぁfilter_inputのがPHP的には美しいかと思います。


Warning!

基本的に$_GETの中身は文字列が多いからこの問題は少ないと思いますが、
"yes" == 0の結果はtrueであることに注意してください。

文字列と数値の比較では、文字列側を先頭から評価して数値以外の文字列が出現したら評価終了。
つまり"yes"は数値が出てこないので0と同じ判定になります。
なので慣習として出来るだけ===を利用するようにしましょう。

if(($_GET["hoge"] ?? null) === "foo") {
  // 処理
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/12/13 14:28

    回答ありがとうございます。
    コードとしてはこの「??」を使う方法が短いのですが、パッと見どういう処理なのかわかりづらい気がするので、やはりfilter_inputが良さそうですね。
    あと、イコール3つによる判定は今までぜんぜん使っていませんでした。型も等しいかどうかの判定もしていくことにします。

    キャンセル

0

あまりお勧めしませんが、条件節でつかうだけなら@で処理しても良いかも

if(@$_GET["hoge"]==="foo"){・・・;}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/12/13 14:31

    回答ありがとうございます。
    アットマークをつけるとエラーが出力されないみたいですね。
    ただ、エラーが見えないとのちのち影響が出ないとも限らないので、いちおうエラー出力される方法で行きたいと思います。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.23%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る