PHPでif文のstrcmp()のエラーについて

前提・実現したいこと

顧客登録画面で入力した値とDBに同じ値があったらコメントで教えてくれるプログラムを作ってデバッグしているのですが、片方と一致する値を入力しても「異なる文字列です」と出てきます。

引用符などを変えたりいろいろ試してみたのですが、全く変わりません。

どこが原因なのでしょうか。

発生している問題・エラーメッセージ

エラーメッセージ

該当のソースコード

<?php
	$dsn = 'mysql:dbname=example;host=localhost';
	$user = '';
	$password = '';
	try{
		$dbh = new PDO($dsn, $user, $password);
		$stmt = $dbh->prepare('select * from example');
		$stmt->execute(array());
		if ( strcmp('C_nm', $_GET['C_num']) == 0 ) {
    		echo "同じ文字列です1\n";
		} else {
    		echo "異なる文字列です1\n";
		}
		if ( strcmp("C_phn",  $_GET['C_phon']) == 0 ) {
    		echo "同じ文字列です2\n";
		} else {
    		echo "異なる文字列です2\n";
		}
		if ( strcmp('C_add', $_GET['C_sa']) == 0 ) {
    		echo "同じ文字列です3\n";
		} else {
    		echo "異なる文字列です3\n";
		}
	}
	catch (PDOException $e){
		print('Error:'.$e->getMessage());
		die();
	}
	$C_name = $_GET['C_name'];
	$C_num = $_GET['C_num'];
	$C_phon = $_GET['C_phon'];
	$C_sa = $_GET['C_sa'];
	$dbh = null;
?>

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
 "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="ja" lang="ja">
<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width">
<title>顧客登録</title>
<link rel="stylesheet" href="ser_style.css">
</head>
<body>
<div class="content">
	<h1>以下の情報を登録しますが、よろしいでしょうか？</h1>
	<form action="c_regi_ans.php" method="get">
		<div class="control">
			<label for="C_name">顧客名</label>
			<input readonly id="C_name" type="text" name="C_name" value="<?php print($C_name);?>">
		</div>
		<div class="control">
			<label for="C_num">顧客番号</label>
			<input readonly id="C_num" type="number" name="C_num" value="<?php print($C_num);?>">
		</div>
		<div class="control">
			<label for="C_phon">顧客電話番号</label>
			<input readonly id="C_phon" type="tel" name="C_phon" value="<?php print($C_phon);?>">
		</div>
		<div class="control">
			<label for="C_sa">顧客住所</label>
			<input readonly id="C_sa" type="text" name="C_sa" value='<?php print($C_sa);?>'>
		</div>
		<div class="control">
			<input type="submit" value="確定">
		</div>
	</form>
	<form action="c_regi.php" method="get">
		<div class="control">
			<input type="submit" value="戻る">
		</div>
	</form>
</div>
</body>
</html>

試したこと

ここに問題に対して試したことを記載してください。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

行動規範の内容に同意します

回答2件

ベストアンサー

たとえば以下についてですが、

if ( strcmp('C_nm', $_GET['C_num']) == 0 ) {

これだと、入力された文字列が固定の文字列「C_nm」であるか否かを比較していて、データベースから読みこんだ値との比較にはなっていません。データベースから読み込んだ値は使われていません。

投稿2021/01/27 11:13

ockeghem

総合スコア11705

noizi.

2021/01/27 11:39

回答ありがとうございます。＄をつけろということですか？＄を付けてやってみた結果、下記のようなエラーが出てしまいます。 Warning: Undefined variable $C_nm in C:\xampp\htdocs\customer\c_regi_veri.php on line 9 異なる文字列です1 Warning: Undefined variable $C_phn in C:\xampp\htdocs\customer\c_regi_veri.php on line 14 異なる文字列です2 Warning: Undefined variable $C_add in C:\xampp\htdocs\customer\c_regi_veri.php on line 19 異なる文字列です3

noizi.

2021/01/27 11:40

引用符をつけてると「異なる文字列です」と出てきます。

noizi.

2021/01/27 11:42

ちなみにif文のところを下記に書き換えた上で実行しました。 if ($C_nm === $_GET['C_num']) { echo "同じ文字列です1\n"; } else { echo "異なる文字列です1\n"; } if ($C_phn === $_GET['C_phon']) { echo "同じ文字列です2\n"; } else { echo "異なる文字列です2\n"; } if ($C_add === $_GET['C_sa']) { echo "同じ文字列です3\n"; } else { echo "異なる文字列です3\n"; }

ockeghem

2021/01/27 11:46

違います。fetchメソッドなどを使って、検索した値をとりださなければなりません。その処理が抜けています。

noizi.

2021/01/27 12:01

fetchメソッドですか。初めて知りました。ありがとうございます。

ockeghem

2021/01/27 12:08

比較などの処理をする前に、単に検索した文字列を表示するプログラムを試したほうがよいと思います。

noizi.

2021/01/27 12:24

検索した文字列を表示するプログラムは作ってちゃんと動いています。

ockeghem

2021/01/27 12:26

検索した文字列を表示するプログラムの中で、fetchあるいは他の方法でデータを取り出しているはずです。その処理を文字列比較する前の箇所に記述してください。

noizi.

2021/01/27 12:29

また、fetchメソッドの使い方として以下の通りでよろしいんでしょうか？ $dbh = new PDO($dsn, $user, $password); $stmt = $dbh->prepare('select * from example'); $stmt->execute(array()); $all = $stmt->fetchAll(); if ( strcmp('$all', $_GET['C_num']) == 0 ) { echo "同じ文字列です1\n"; } else { echo "異なる文字列です1\n"; }

noizi.

2021/01/27 12:32

fetchallで値を全て代入するとあったのでallに代入してつかってみました

ockeghem

2021/01/27 12:32

色々違います。検索結果を表示するプログラムの中でforeachなどのループ処理をしていると思います。それと同じ流れで、比較をします。ただ、全部のデータを取り出すと、データ量が多い場合に処理できなくなるので、通常はSELECT文のWHERE句を追加して、SQL側で検索をするのが普通です。

noizi.

2021/01/27 12:33

コメントありがとうございます。やってみます

行動規範の内容に同意します