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

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

ただいまの
回答率

88.83%

issetで確認しないと、PHP Noticeが出てしまうので条件に加えました。でも、長ったらしいIF文になるので、関数を使って簡略なコードにしたい。

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,018

haln

score 13

前提

issetで確認しないと、PHP Noticeが出てしまうので条件に加えました。でも、長ったらしいIF文になるので、関数を使って簡略なコードにしたいです。
こちら、MySQLとPHPの2週間程度の知識しか持ち合わせていません。
まだ、開発を一度も完成させたことがありません。

修正したいコード

if(isset($_POST['hoge']) ? $_POST['hoge'] : NULL or isset($_POST['piyo']) ? $_POST['piyo'] : NULL) {
    updated_spamham_info($_POST['id'], $_POST['hoge'], $_POST['piyo']);
}

執筆者の考え(実現したいこと)

IF文の中に関数を入れてやればよいと考えました。

function foo($str) {
    $a = isset($str) ? $str : NULL;
    return $a;
}

if( foo($_POST['hoge']) or foo($_POST['piyo']) ) {
    updated_spamham_info($_POST['id'], $_POST['hoge'], $_POST['piyo']);
}


出力結果
Notice: Undefined index: hoge in C:\++++++\++++++\+++\++++\++++++++.php on line 6
Notice: Undefined index: hoge in C:\++++++\++++++\+++\++++\++++++++.php on line 6

試したこと

①関数が読み込まれてない可能性を疑い、関数名を変更してみました。
同じエラーが出力されたら、関数の置き方が違うかもしれない。
→エラー文が異なっていたので、関数の置き方は正しいと判断しました。

function bar($str) {
    $a = isset($str) ? $str : NULL;
    return $a;
}

if( foo($_POST['hoge']) or foo($_POST['piyo']) ) {
    updated_spamham_info($_POST['id'], $_POST['hoge'], $_POST['piyo']);
}


出力結果
Fatal error: Uncaught Error: Call to undefined function foo() in C:\++++++\++++++\+++\++++\++++++++.php:6 Stack trace: #0 {main} thrown in C:\++++++\++++++\+++\++++\++++++++.php:6 on line 6


②IF文の中に関数を入れるのが不適切だと考え、外に出しました。

function foo($str) {
    $a = isset($str) ? $str : NULL;
    return $a;
}

$a = foo($_POST['hoge']);
$b = foo($_POST['piyo']);
if( $a or $b ){
    updated_spamham_info($_POST['id'], $_POST['hoge'], $_POST['piyo']);
}


出力結果
Notice: Undefined index: hoge in C:\++++++\++++++\+++\++++\++++++++.php on line 1
Notice: Undefined index: piyo in C:\++++++\++++++\+++\++++\++++++++.php on line 2

補足情報

・開発環境

name version
Eclipse Version 2019-03 (4.11)
A5M2 SQL Mk-2 Version:1.14.1(x64 edition)
XAMPP Windows 7.3.4
PHP PHP 7.3.4
Apache Apache/2.4.39 (Win64)
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+2

変数が存在しない場合の警告を回避出来るのは empty() isset() といったものがあります。

$_POST['hoge']などに何かしら値がある場合、を判定するのであれば、下記で良いと思います。

if(! empty($_POST['hoge']) or ! empty($_POST['piyo'])) {
    updated_spamham_info($_POST['id'], $_POST['hoge'], $_POST['piyo']);
}

あるいは、 filter_input() を使っても良さそうです。

$hoge = filter_input(INPUT_POST, 'hoge');
$piyo = filter_input(INPUT_POST, 'piyo');
if($hoge or $piyo){
    updated_spamham_info($_POST['id'], $hoge, $piyo);
}

追記

最初に回答した際、下記の様に記載しておりました。

if(isset($_POST['hoge']) or isset($_POST['piyo'])) {
    updated_spamham_info($_POST['id'], $_POST['hoge'], $_POST['piyo']);
}

これはこれで同じような動作になるのですが、
$_POST['hoge'] と $_POST['piyo'] 両方に空文字列('') が入った場合、
trueと判定され、結果として関数が実行されます。 (isset関数はnull以外はtrueを返すため)
これは元のロジックと異なる動作となるので、! empty を利用した書き方に修正させていただきました。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/05/14 01:24 編集

    Eggpan様、回答ありがとうございます。
    一つ目のPHPを見て、ふと思ったのですが……。

    ◆修正したいコード
    $_POST['hoge'] = 'ほげ';
    $_POST['piyo'] = 'ぴよ';が入ってるとします。

    if(isset('ほげ') ? 'ほげ' : NULL or isset('ぴよ') ? $_POST['ぴよ'] : NULL) {
    updated_spamham_info($_POST['id'], $_POST['hoge'], $_POST['piyo']);
    }

    if('ほげ' or 'ぴよ') {
    updated_spamham_info($_POST['id'], $_POST['hoge'], $_POST['piyo']);
    }

    if(1) {
    updated_spamham_info($_POST['id'], $_POST['hoge'], $_POST['piyo']);
    }

    というようにに実行されると思うので……参考演算子が要らないですね。
    ◆当問題の解決策
    if(isset($_POST['hoge']) or isset($_POST['piyo'])) {
    updated_spamham_info($_POST['id'], $_POST['hoge'], $_POST['piyo']);
    }

    https://qiita.com/shinichi-takii/items/00aed26f96cf6bb3fe62
    ※上記のサイトで、isset以外にも、値があるかどうかの判定があるのを確認し、動き的にはissetがよいので、この形としました。

    何とも言いがたい誤りに気づけました。
    この程度のIF文の長さならば、当問題の長文を解消できそうです。
    感謝致します。

    キャンセル

  • 2019/05/14 01:57 編集

    http://web-engine.hatenadiary.com/entry/20160521/1463823327
    こちらのサイトを確認しますと、二つ目のPHPのようにするのが一番スマートな気がしてきました。
    後は自分で考え、どちらが好ましいか考えていこうと思います。回答感謝致します。

    キャンセル

  • 2019/05/14 09:54

    元コードが空文字列だとfalse判定なので、初めに書いたものが同様の動作になりそうですね。
    2つのパターン回答をだしておきながら動作が変わっている、というのは適切ではないので、
    後ほど3パターンとも併記するように修正させていただきたいとおもいます。

    私自身が間違えました通り、フォームの空文字列の判定は混乱しがちな部分です…。

    キャンセル

  • 2019/05/15 20:26 編集

    修正、感謝いたします……。改めて拝読します。

    キャンセル

0

画面の造りが分からないので概念的な表記になりますが…

if (submitされたか?) {
  // $_POST の入力チェック
  // $_POST を使った処理
}


のようにすれば、そんなに長ったらしくはならないと思います。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/05/14 02:08

    takasima20様、回答ありがとうございます。
    次は分かりやすい質問をあげれるように努めます。

    キャンセル

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

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

関連した質問

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

  • トップ
  • PHPに関する質問
  • issetで確認しないと、PHP Noticeが出てしまうので条件に加えました。でも、長ったらしいIF文になるので、関数を使って簡略なコードにしたい。