PHP ログイン機能

Question

phpでログイン希望を作成したいです。
新規会員登録をMySQLにしたいです。

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

Notice: Undefined variable: pdo in C:\Eclipse_for_PHP_Java\xampp\htdocs\yoyaku\signUp.php on line 32

Fatal error: Uncaught Error: Call to a member function prepare() on null in C:\Eclipse_for_PHP_Java\xampp\htdocs\yoyaku\signUp.php:32 Stack trace: #0 {main}thrown in C:\Eclipse_for_PHP_Java\xampp\htdocs\yoyaku\signUp.php on line 32

該当のソースコード

<?php
session_start();
require_once('config.php');
//データベースへ接続、テーブルがない場合は作成
try {
  $pdo = new PDO(DSN, username, password);
  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
  $pdo->exec("create table if not exists User_Deta(
      id int not null auto_increment primary key,
      username varchar(255),
      password varchar(255),
      created timestamp not null default current_timestamp
    )");
} catch (Exception $e) {
  echo $e->getMessage() . PHP_EOL;
}
//POSTのValidate。
if (var_dump($_POST['username'])) {
  $username = username_hash($_POST['username']);
  echo '入力された値が不正です。';
  return false;
}
//パスワードの正規表現
if (preg_match('/\A(?=.*?[a-z])(?=.*?\d)[a-z\d]{8,100}+\z/i', $_POST['password'])) {
  $password = password_hash($_POST['password'], PASSWORD_DEFAULT);
} else {
  echo 'パスワードは半角英数字をそれぞれ1文字以上含んだ8文字以上で設定してください。';
  return false;
}
//登録処理
try {
  $stmt = $pdo->prepare("INSERT INTO user_deta(username, password) VALUES(?, ?)");
  $stmt->execute([$username, $password]);
  echo '登録完了';
} catch (\Exception $e) {
  echo '登録済みのメールアドレスです。';
}

Answer 1

そもそも毎回create tableのお伺いたてるのって全く無駄です。
確実に用意したテーブルのみを使いましょう。

あとNoticeはきっとスコープの問題。
上記勘違い。既に指摘がある通りそもそもPDOのnewに失敗してて$pdoが存在しないものと思います。
というか同じロジック内でtry-catchをわけるメリットってほぼ皆無なのでひとくくりにしてしまっても差し支えないのでは。
※ただし先に指摘したように毎回CREATE TABLEでお伺いを立てることになり、初回以外CREATEされないのでこの実装が不要です。先にテーブル作っておけば済む話。

以下、実装に対する指摘

} catch (Exception $e) {

PDOならPDOExceptionを拾うべき。

return false;

exit()のほうが適切では。

ただ、今の実装だとブラウザの「戻る」押すしかないですけど、ここで止めるのは悪手のように思います。
関連性のないバリデーションは全て表示させるように実装した方がユーザーには優しいです。

if (var_dump($_POST['username'])) {

var_dump()の結果を判定して何がしたいのでしょう。
ちなみにPHPマニュアルの通り、voidで返却値がないので、例え定義されている変数を与えてもifはfalseとなって絶対にこのif内は通りません。

} catch (\Exception $e) {

始めの方には\入ってなかったけど。実装が統一されてない。

  echo '登録済みのメールアドレスです。';

いえ、「Exceptionが起きる＝重複のメールアドレス」とは限りません。
きちんとメールアドレスでSELECTかけてINSERTする前に弾きましょう。
これもバリデーションの範疇です。

Comments

[退会済みユーザー]

try～catchを抜けたら、$pdoが行方不明になってるな

[m.ts10806]

いやむしろTakumibooさんの指摘通りかもしれませんね。
実は最初のtryでException 吐いてるのに無視してるんじゃないかと。
試してみたら下記でも行方不明にならなかったので。
<?php
try{
$hoge = "hello";
}catch(Exeption $e){}
try{
echo $hoge;
}catch(Exeption $e){}

謎の実装は多いですけどね。

Answer 2

try {

$pdo = new PDO(DSN, username, password);

ここってusernameとpasswordは別として(伏せているだけですよね？)、このままのコードなのでしょうか。
であれば、DSNを指定すべき場所で「DSN」と変数でもなくそのまま入力していればそもそもエラーで接続されないかと思いますが。

PHP: PDO::__construct - Manual