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

Question

#前提
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)

Answer 1

変数が存在しない場合の警告を回避出来るのは 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 を利用した書き方に修正させていただきました。

Comments

[haln]

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文の長さならば、当問題の長文を解消できそうです。
感謝致します。

[haln]

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

[Eggpan]

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

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

[haln]

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

Answer 2

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

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

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

Comments

[haln]

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