nullの場合if文が反応しないようにしたい

Question

php7.1.8を使っています。

ajaxのgetでデータを送信してphpで受けています。
データを受けているかどうかの判断をif文で書いているのですが、if文の書き方が良くないのかif文をtrue判断してしまいます。

var bango=$('#bango').val();
$.ajax({
	type: "GET",
	url: "test1.php",
	data: {
		bango: bango
	}
});

$bango=null;
if(isset($_GET['bango'])){
  $bango = $_GET['bango'];
}

$sql=" select * from test "
    ." where chk=1"
if(!is_null($bango)){
  $sql=$sql." and bango=:bango"
}
$prepared = $db->prepare($sql);
if(!is_null($bango)){
  $prepared->bindValue(':bango, $bango, PDO::PARAM_STR);
}

bangoにデータを渡していないのにsql文にbangoの抽出が追加されてしまいます。
nullのときはbangoの抽出（and bango=:bango）をSQL文に追加しないようにしたいです。

if文の判断が良くないのでしょうか。

教えてください。
お願いします。

Comments

[KoichiSugiyama]

リファレンス（https://www.php.net/manual/ja/types.comparisons.php）によると、isset()はNULLか未定義でないかぎりTRUEになるようですが、ajaxでどういう送信の仕方をしているかもソースを開示してもらった方が回答がつきやすいと思います。

[m.ts10806]

bango=null;
↑ここ本当にこう書いてますか？
このままならエラーかと思います

[chintao1224]

bango=null;でエラーにはなってないです。ちなみにbango="";にして、if(!empty($bango)){にしても駄目でした。上記コードと同じで、SQL文にand bango=:bangoが追加されてしまいます。

[m.ts10806]

<?php
もないのでそもそもPHPとして動かないということなのですけど。

[m.ts10806]

>bangoにデータを渡していないのに

下記コードが本当なら、渡してますね。
$.ajax({
type: "GET",
url: "test1.php",
data: {
bango: bango
}
});

ただ、コードがつまみすぎて誰も再現確認できないことは問題です。
コードは一通り、全体の流れが分かるものを提示してください。
このままではJavaScriptもPHPも動きません。

[m.ts10806]

しつこいようですが、コードは一通り、全体の流れが分かるものを提示してください。
このままではJavaScriptもPHPも動きません。
HTMLもないわけですから。

[chintao1224]

すみませんでした。記入ミスでした。申し訳ありませんでした。

[m.ts10806]

<?php
もないのでそもそもPHPとして動かないということなのですけど。

Answer 1

bango=
だけ渡されたときに(つまりempty)どうする設計なのか次第ですが、ひとまずarray_key_existsまたはfilter_inputで対応してみてください。

Comments

[chintao1224]

コードを追記しました。やりたいことは、bangoをnullなどで初期化して、issetで判断しデータを受け取り、if(!is_null($bango))などで判断し受け取れてない場合SQL文を抽出から外したいということです。

[m.ts10806]

実行的ないコードを提示されても的確なアドバイスは出来ませんが、
bango　変数
$bango　変数

意味が全く違います。
あとこの手の処理でissetで判定は基本的にしないですね。
issetをPHPマニュアルで確認すれば明白です。
それに、bangoというキー、渡してるし・・

[m.ts10806]

今こういう状態なのは認識されてますでしょうか。

<?php
$ar = [];
$ar["test"] = "";
var_dump(isset($ar["test"])); // bool(true)

[chintao1224]

すみません、初心者に近いもので。受け取るときはissetは使わないということでしょうか。

[chintao1224]

すみません、データが渡されるときbangoだけ渡しているんですが、配列として渡されるのでしょうか。

[m.ts10806]

デバッグしてください。
初心者に近い＝初心者からは抜け出している
のでしたらデバッグは知っているはずです。

[m.ts10806]

>データが渡されるときbangoだけ渡しているんですが、配列として渡されるのでしょうか。

Ajaxの基本からやってください。
何が渡されるか確認するやり方はそう難しくはなく調べればどこでもあります。
私が指摘しているのはisset()の役割を勘違いしているということです。
初心者抜け出しているならPHPマニュアルは適切に確認できるはずです。

[chintao1224]

わかりました。ajaxの基本からやります。bangoがnullの場合SQLに追加させたくないので、nullでない判定ができるif文だけ教えてください。デバッグした結果、bangoにはnullが入っていました。if文はif($bango!=null)やif($bango!==null)やif(!($bango==null))やif(!($bango===null))すべて駄目でした。

[m.ts10806]

>nullでない判定ができる

ですから、「nullである」という確認できてないからじゃないですか？
デバッグしてください。
あと、emptyとnullは同じではないです。
data: {
bango: bango
}
と書いてる時点でnullの可能性消えてませんか？
HTML内に#bangoが存在しなければnullの可能性はありますが、存在するなら何かしら渡されます。HTMLないと何ともいえませんが、input系の要素であれば、「未入力」はnullではなく「empty」です。

Answer 2

まずはクライアント側のbangoの取得状況

• #bangoが存在しない
• #bangoがvalueを持てない(例えばdiv)
• #bangoがvalueを持っていない
• #bangoのvalueが""
• #bangoがvalueを持っており、空ではない

のように分類できます
そうすると以下のように場合分けができます

var bango=$('#bango').val();
var data={};
if(typeof bango!=="undefined" && bango!=="") Object.assign(data,{bango});

これをサーバーサイドで処理する場合

$bango=filter_input(INPUT_GET,"bango",FILTER_VALIDATE_REGEXP,["options"=>["regexp"=>"/\S/"]]);

のようにします。$bangoの値は

• bangoが送られてこないときはnull
• bangoが空や空白文字だけの場合はfalse
• ちゃんと送られてくるときはその値

となりますので、その条件で場合分けをしてください