PDO、SQL文を２つ実行したいができない

Question

phpでPDOを使用して会員登録を行いたいのですが、その際２つのテーブルに情報を入れるときに$sql2の文は実行されるのですがsql3のほうが実行されないです。
PDOを初めて触っての作業なので調べながらやりましたがよくわかりません。
どこが間違っているのか教えていただきたいです。

Mysql使用しています

<?php
ini_set("display_errors", 1);
error_reporting(E_ALL);
//フォームからの値をそれぞれ変数に代入
$name = $_POST['name'];
$pass = $_POST['pass'];
try {
    $db = new PDO('*******);
}   catch (PDOException $e) {
    echo "データベース接続エラー　：".$e->getMessage();
}
$sql = "SELECT count(*) FROM user WHERE username=:name";
$stmt = $db->prepare($sql);
$stmt->bindValue(':name', $name);
$stmt->execute();
$member=0;
$member = $stmt->fetch();
$a=$member[0];
if ($a > 0) {
    $msg = '同じユーザー名が存在します。';
    $link = '<a href="signup.php">戻る</a>';
} else {
    $sql2 = "INSERT INTO user (username, passwd) VALUES (:name, :pass)";
    $stmt2 = $db->prepare($sql2);
    $stmt2->bindValue(':name', $name);
    $stmt2->bindValue(':pass', $pass);
    $stmt2->execute();
    $sql3 = "INSERT INTO userscore (name) values (:name)";
    $stmt3 = $db->prepare($sql3);
    $stmt3->bindValue(':name', $name);
    $stmt3->execute();
    $msg = '会員登録が完了しました';
    $link = '<a href="login_form.php">ログインページ</a>';
    ?>
<h1><?php echo $msg; ?></h1><!--メッセージの出力-->
<?php echo $link; ?>

Comments

[m.ts10806]

シンタックスハイライトで気づける部分もあるかとは思いますが、
> $db = new PDO('*******);

ここ閉じてないためエラーになりますよ。

[m.ts10806]

ただ、DBに変更を加える処理に対してException捕捉してないのはよろしくないのですし、処理結果による判定もないので実装が弱いです。
せめて、tryはDB処理全て入れてください。
あと、各メソッドの返却値を確認して処理を分岐させた方が良いです。

Answer 1

さすがに$db = new PDO('*******);の閉じ引用符忘れはギャグだと思うのだけど、
if ($a > 0) {に対する} else {の閉じる方がないのは文法エラーになっちゃうんじゃないかい？

つまらない文法エラーが有ると、冒頭の

ini_set("display_errors", 1);
error_reporting(E_ALL);

も無視されてしまうので、デバッグをちゃんとやるなら、
php.iniファイル上の該当箇所をデバッグ用に修正してください。
php.iniファイル上での記述の仕方は、こちらを参照：PHPのエラー表示設定について - Qiita

細かいことを言うと、
new PDO()する下りだけtry～catchしているんだが、
データベースにアクセスしている最中は例外は発生しやすいので、
全体的にtryブロック内に入れるべし。

さらには、複数のINSERT INTO文を実行しているけど、
片方だけ成功して片方が失敗したりするとデータの不整合を起こしちゃう危険性があるんじゃないかい？
失敗したときにINSERT INTO文をなかったコトにするためには、
PHP: PDO::beginTransaction - Manual
PHP: PDO::commit - Manual
PHP: PDO::rollBack - Manual
を駆使します。

try
＿new PDO()
＿try
＿＿beginTransaction()
＿＿INSERT INTO文実行1
＿＿INSERT INTO文実行2
＿＿commit()
＿catch
＿＿rollback()
catch

みたいな構造。このへんの流れは、ベテランでも何度も読み返すこちらの記事を参照あれ。
PHPでデータベースに接続するときのまとめ - Qiita