ログインに必要なPHPの syntax error の解決方法が分からない。

Question

予てから、LAMPを使ってPHPのCRUDアプリを作成してきておりました。
現在は一般ユーザーが閲覧できるページと、ログインページを経由しないとアクセスできないユーザー編集ページとに分けようとしているところです。

しかし、ログイン機能を実装したところ、ログインに直接関わってくるphpファイルのコードで「syntax error」が出てしまい、その記述をどう解消していいかが分からずにいます。

該当のコードは以下のとおりです。

// login.php

<?php
session_start();

// サーバー接続用のファイルを読み込む
include('dbconnect.php');

$mail = htmlspecialchars($_POST['mail'], ENT_QUOTES, "UTF-8");

$sql = "SELECT * FROM users WHERE mail = :mail";
$stmt = $dbh->prepare($sql);
$stmt->bindValue(':mail', $mail);
$stmt->execute();
$member = $stmt->fetch();
//指定したハッシュがパスワードにマッチしているかチェック
if (password_verify($_POST['pass'], $member['pass']) {
    //DBのユーザー情報をセッションに保存
    $_SESSION['id'] = $member['id']; //←この「;」がおかしいと出る
    $_SESSION['name'] = $member['name']; //←とするならばここも
    $msg = 'ログインしました。';
    $link = '<a href="index.php">ホーム</a>';
} else {
    $msg = 'メールアドレスもしくはパスワードが間違っています。';
    $link = '<a href="login.php">戻る</a>';
}
?>

<h1><?php echo $msg; ?></h1>
<?php echo $link; ?>

<?php
  // dbconnect.php

$host = 'localhost';  //ホスト名を入力して下さい。
$dbname = 'crud'; //データベースサーバー名を入力して下さい。
$username = 'root';  //ユーザーネームを入力して下さい。
$password = '';  //パスワードを入力して下さい（ない場合は入力不要）。

$dsn = "mysql:host={$host}; dbname={$dbname}; charset=utf8";

try {
    $dbh = new PDO($dsn, $username, $password);
} catch (PDOException $e) {
    $msg = $e->getMessage();
}

login.phpのファイルでエラーの原因となっている「;」は、或いはそこを修正した後の周辺のコードのまとめ方としては、どう記述するのがふさわしいでしょうか。

ごく基礎的な内容かもしれませんが、畏れながら、ご回答お願いします。

Comments

[m.ts10806]

エラー全文をコピペで提示してください

[m.ts10806]

ただ、htmlspecialchars()を画面出力しない情報に対して行うのは間違いです。

Answer 1

if (password_verify($_POST['pass'], $member['pass']) {

) が足りない。

分解すると分かります。
if() の中に
password_verify()
が入っているわけです。

コーティングにIDE使うと未然に防げる、もしくは実行前に判明する程度のミスかと思います。

Comments

[thyda.eiqau]

:handshake:

[m.ts10806]

b

[Nishin]

ご回答ありがとうございます。

確かによく見ると、ifから{の間で、「(」の数に対して「)」が一つ足りていませんでした。
これを$member['pass'])の後ろに付ければ良かったのですね。

コーディングはIDEのPaizaCloudを使って実行しており、「syntax error」も実装時に出たのですが、（一刻も早く次の機能を実装したいが為とはいえ）あろうことか大半をコピペしてしまっていたが為に発生した、云わば自業自得な問題だったと、認識しております。

[m.ts10806]

書いたとおりにしか動かないので、
動くように 間違いのない手順を確立してください。そのほうが早いです。

Answer 2

if (password_verify($_POST['pass'], $member['pass']) {
閉じカッコ足りなくないですか。