データベースへの追加

Question

先日に引き続きログインシステムの作成を練習してます。
フォームでユーザー名とパスワードを入力して移動すると、
catch部分で設定したエラーが返ってきます。（データベースに追加もされてません。）
ATOMで見ると、new,ATTR_ERRMODE,ERRMODE_EXCEPTION,INSERT INTO,
VALUESの部分が動作してないようです。（文字が赤い）
スペルミスや、足りない部分を教えていただきたいです。

<?php


try{
$staff_name = $_POST['name'];
$staff_pass = $_POST['pass'];

$staff_name = htmlspecialchars($staff_name,ENT_QUOTES,'UTF-8');
$staff_pass = htmlspecialchars($staff_pass,ENT_QUOTES,'UTF-8');

$dsn = 'mysql:dbname = shop;host=localhost;charset=utf-8';
$user = 'root';
$password= '';
$dbh = new PDO($dsn,$user,$password);
$dbh->setAttribute(PDO::ATTR_ERRMODE , PDO::ERRMODE_EXCEPTION);

$sql = 'INSERT INTO mst_staff(name,password) VALUES (?,?)';
$stmt = $dbh->prepare($sql);
$data[] = $staff_name;
$data[] = $staff_pass;
$stmt->execute($data);

$dbh = null;

print $staff_name;
print 'さんを追加しました。<br />';

}
catch (Exception $e){
print 'ただいま障害発生中です。ご迷惑おかけします。';
exit();

}

?>
<a href="staff_list.php">戻る</a>

Comments

[kei344]

質問文のコードはコードブロックで囲んでいただけませんか？ ```（バッククオート3つ）で囲み、前後に改行をいれるか、コードを選択して「<code>」ボタンを押すとコードブロックになります。

[退会済みユーザー]

知りませんでした。申し訳ないです、バッククオート追加しました。

Answer 1

データベースを以下のように定義して、

CREATE TABLE `mst_staff`(
    `id` int(11) UNSIGNED NOT NULL AUTO_INCREMENT PRIMARY KEY,
    `name` varchar(255) NOT NULL DEFAULT '',
    `password` varchar(255) NOT NULL DEFAULT ''
)ENGINE = InnoDB DEFAULT CHARSET = utf8mb4 COLLATE = utf8mb4_unicode_ci;

以下のようにしたところ、無事動作しました。どこを修正したかはコードにコメントとして書いてあるので、確認してみてください。

<?php
try {
    $staff_name = $_POST["name"];
    $staff_pass = $_POST["pass"];

    $staff_name = htmlspecialchars($staff_name, ENT_QUOTES, "UTF-8");
    $staff_pass = htmlspecialchars($staff_pass, ENT_QUOTES, "UTF-8");
    $dsn = "mysql:dbname=shop;host=localhost;charset=utf8";// dbname = のスペースを取り除き、utf-8のハイフンを取る。
    $user = "root";
    $password = "password";
    $dbh = new PDO($dsn, $user, $password);
    $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

    $sql = "INSERT INTO `mst_staff` (`name`, `password`) VALUES (?, ?)";// 修正。詳細は質問文のコードと見比べたらわかると思います。
    $stmt = $dbh->prepare($sql);
    $data[] = $staff_name;
    $data[] = $staff_pass;
    $stmt->execute($data);
    $dbh = null;
    print $staff_name;
    print "さんを追加しました。<br />";
} catch (Exception $e) {
    print "ただいま障害発生中です。ご迷惑おかけします。";
    exit();
}
?>
<a href="staff_list.php">戻る</a>

Answer 2

詳細なエラーがわからないことにはどうにもならないので、

catch (Exception $e){ 
print 'ただいま障害発生中です。ご迷惑おかけします。'; 
echo $e->getMessage();
exit();

}

こんな感じにすると、例外が発生した原因が表示されるので、その内容を読んで対処してみてください。
以下はPHPマニュアルの該当のページです。
http://php.net/manual/ja/language.exceptions.php

Comments

[退会済みユーザー]

さっそく打ち込んでみましたが、
Fatal error: Call to undefined method PDOException::getMassage() in C:\xampp\htdocs\staff\staff_add_done.php on line 39
と、getMassageがそのまま「おかしい」と返ってきてしまいました・・・。
XAMMP環境で、MySQLのローカルサーバーも立ち上げてるのに
PHPで設定したサーバーエラーのメッセージが返ってきます・・・

[Z-TALBO]

getMassageではなく、getMessageですよ。

[退会済みユーザー]

申し訳ないです、打ち直しました。
SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'INSERTINTOmst_staff(name,password)VALUES('ろくまる','ec1f850d934f440cfa8e4a1' at line 1
というエラーが返ってきました。

[tanat]

You have an error in your SQL syntax;
なので、SQLが間違っていると言われています。
エラーメッセージで検索したり、機械翻訳でいいのでエラーを読んでみてください。

SQLが明らかに間違っているので、
'INSERTINTOmst_staff(name,password)VALUES('ろくまる','ec1f850d934f440cfa8e4a1'
をよく見て、意図したSQLになっていないことを確認して修正してみてください。

[退会済みユーザー]

やっぱりそうですか！
後でまた調べながら修正してみます、ありがとうございます。

[退会済みユーザー]

無事に動作しました、ありがとうございますm(_ _)m