PHPからMySQLへのデータ送信のプログラミング構造

Question

プログラミング二か月目、MySQL数日目の初心者です。
見よう見まねで全ては理解できていない状態でプログラミングしました。

ユーザ登録画面です、この前のページで名前とメールアドレスを入力してもらい、DBにその情報を送る操作を打ちました。
Mysqlのファイルにデータが送信できることは確認済みです。
ただ、当方の書いたものだと名前とメアドが違う横列に入ってしまいます。
どこを統一すれば同一人物の情報だと認識して、DBに格納されるのでしょうか？

最初のif (!isset($_POST['Name']) || $_POST['Name'] === "" && !isset($_POST['Ad']) || $_POST['Ad'] === "")にするなど試したのですがうまくいきませんでした。
ご教授願います。

<?php
//name
if(empty($_POST)) {
	echo "<a href='projlogin.php'>ログイン画面</a>←こちらのページからどうぞ";
}else{
	//名前入力判定
	if (!isset($_POST['Name'])  || $_POST['Name'] === "" ){
		echo "名前が入力されていません。";
	}else{
		//プリペアドステートメント
		$stmt1 = $mysqli->prepare("INSERT INTO user_list (user_name) VALUES (?)");

		if($stmt1){
			//プレースホルダへ実際の値を設定する
			$stmt1->bind_param('s', $Name);
			$Name = $_POST['Name'];

			if($stmt1->execute()){
				echo htmlspecialchars($Name, ENT_QUOTES, 'UTF-8')."さんで登録いたしました。";
			}else{
				echo $stmt1->errno . $stmt1->error;
			}


			//ステートメント切
			$stmt1->close();
		}else{
			echo $mysqli->errno . $mysqli->error;
		}
	}

	//アド入力判定
	if (!isset($_POST['Ad'])  || $_POST['Ad'] === "" ){
		echo "アドレスが入力されていません。";
	}else{
		//プリペアドステートメント
		$stmt2 = $mysqli->prepare("INSERT INTO user_list (user_ad) VALUES (?)");

		if($stmt2){
			//プレースホルダへ実際の値を設定する
			$stmt2->bind_param('s', $Ad);
			$Ad = $_POST['Ad'];

			if($stmt2->execute()){
				echo "メールアドレスは".htmlspecialchars($Ad, ENT_QUOTES, 'UTF-8');
				echo "で登録いたしました。";
			}else{
				echo $stmt2->errno . $stmt2->error;
			}

			//ステートメント切断
			$stmt2->close();
		}else{
			echo $mysqli->errno . $mysqli->error;
		}
	}
}

// データベース切断
$mysqli->close();
?>

回答をもとに変更してみました。
以下のように記述しましたら、
1136Column count doesn't match value count at row 1
のエラーが出ました。
調べたところ、コラムの数が合わないのが原因のようなのですが、対処方法はありますでしょうか？
たしかにDBにはインデックスとしてauto_incrementでもう一つコラムは設定されています。

<?php
$name=filter_input(INPUT_POST,"Name");
$ad=filter_input(INPUT_POST,"Ad");

	//名前入力判定

		//プリペアドステートメント
		$stmt = $mysqli->prepare("INSERT INTO user＿list (user_name,user_ad) VALUES (?,?)");

		if($stmt){
			//プレースホルダへ実際の値を設定する
      $stmt->bind_param('s', $Name);
      $stmt->bind_param('s', $Ad);
      $Name = $_POST['Name'];
      $Ad = $_POST['Ad'];

			if($stmt->execute()){
				echo "お名前は".htmlspecialchars($Name, ENT_QUOTES, 'UTF-8')."さん<br>";
        echo "メールアドレスは".htmlspecialchars($Ad, ENT_QUOTES, 'UTF-8')."<br>";
        echo "で登録いたしました。";
			}else{
				echo $stmt->errno . $stmt->error;
			}

			//ステートメント切断
			$stmt->close();
		}else{
			echo $mysqli->errno . $mysqli->error;
		}



// データベース切断
$mysqli->close();
?>

Answer 1

回答者様の意見総合的に見て解決できました、ありがとうございます。
以下で解決

$name=filter_input(INPUT_POST,"Name");
$ad=filter_input(INPUT_POST,"Ad");

$stmt = $mysqli->prepare("INSERT INTO user_list (user_name,user_ad) VALUES (?,?)");
$stmt->bind_param('ss', $name,$ad);

今後の自身のためのメモ：insertと$stmt->bind_paramはまとめる、sは変数（打ち込みデータ）の数

Comments

[m.ts10806]

データ二個ならssでは。
いずれにしてもpdoに切り替えられた方が書きやすくなります。

[akoro]

ご指摘ありがとうございます。最重要箇所間違えておりました。

Answer 2

insertを2回送信していれば別レコードになるのは当然です。

下記のようなSQLを送信してください。

INSERT INTO user_list (user_name,user_ad) VALUES (?,?)

どちらも必須のようですが、必須にも関わらず一方だけでも登録できるのはDB設計上のミスと思います。
テーブル定義がどうなっているか分かりませんが、必須のカラムはNot Nullに設定してください。
そうするとSQLのエラーになるので今回のミスに気づけます。

Comments

[akoro]

insert intoを統一するのですねありがとうございます。
Nullになってました、Not nulに変えます、ありがとうございます。

[m.ts10806]

DBはキーが決まっている配列と思った方がイメージがわきやすいです。
質問者さんの現在のコードはPHPの配列作る処理だと下記です。

$arr = [];
$record1 =["name"=>"なまえ","mail"=>""];
$arr[] = $record1;
$record2 =["name"=>"","mail"=>"[e-mail]"];
$arr[] = $record2;

insertはその配列に1つのレコードを作る処理です。1つのinsertで1レコード挿入します。

[akoro]

なぜ検索で出てくる例はすべて配列なんだろうかと疑問に思っておりました。
考え方の解説ありがとうございます。
>insertはその配列に1つのレコードを作る処理です。1つのinsertで1レコード挿入します。
このアドバイスと、ほかの方のも参考に変えてみたのですが、また違うエラーが出てしまいました。

[m.ts10806]

> なぜ検索で出てくる例はすべて配列なんだろうか
fetchArray()など「配列で取り出す」機能を利用しているからだと思います。
SQL自体はあくまで結果を返しているだけです。
取り出し方によってはオブジェクトメンバー($rec->name )のような形にもなります。

[m.ts10806]

>このアドバイスと、ほかの方のも参考に変えてみたのですが、また違うエラーが出てしまいました。

変数は大文字と小文字を大別します。
aとAは別物 bとBは別物です。
定義したならどんなに短い次数でも手入力せずにコピペするのが基本です。

[akoro]

なるほど、ありがとうございます。

今回のエラーはタイプミスではなく、構造無知湯故の普通のミスだったのですが、コピペ意識して今後取り組みます。

[m.ts10806]

もう少しphpマニュアルをきちんと読んだ方がいいかなと。
ネットの記事を参考にするのも良いですが、そもそものphpの機能の仕様を理解して使うことが前提です

[akoro]

仕組を理解できるようにマニュアルも読みます、ありがとうございます。

Answer 3

$name=filter_input(INPUT_POST,"Name");
$ad=filter_input(INPUT_POST,"Ad");

$stmt = $mysqli->prepare("INSERT INTO user_list (user_name,user_ad) VALUES (?,?)");
$stmt->bind_param('s', $name);
$stmt->bind_param('s', $ad);

おなじレコードに登録するのですから一度に投入してください

Comments

[akoro]

ありがとうございます、試してみます。

[akoro]

>$stmt->bind_param('s', $name);
>$stmt->bind_param('s', $ad);
の箇所に
mysqli_stmt::bind_param(): Number of elements in type definition string doesn't match number of bind variables
というエラーがでてしまたのですが、解決策はありますでしょうか？

[akoro]

まとめて$stmt->bind_param('ss', $name,$ad);で解消しました！ありがとうございます。