ユーザーの新規登録ができない

前提・実現したいこと

PHPのMVCモデルにて、ユーザーの新規登録を実装したい。

現在、PHPのMVCを勉強中で、ユーザーの新規登録やログイン機能の実装中です。
しかし、ユーザーの新規登録ができないため、ユーザーの新規登録ができるように実装したいです。

userのデータベース構造

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

新規登録フォーム(signup_form.php)

<!DOCTYPE html>
<html lang="en">
<head>
  <meta charset="UTF-8">
  <meta http-equiv="X-UA-Compatible" content="IE=edge">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <link href="https://cdn.jsdelivr.net/npm/bootstrap@5.0.0-beta1/dist/css/bootstrap.min.css" rel="stylesheet" integrity="sha384-giJF6kkoqNQ00vy+HMDP7azOuL0xtbfIcaT9wjKHr8RbDVddVHyTfAAsrekwKmP1" crossorigin="anonymous">
  <link rel="stylesheet" href="/css/user.css">
  <title>worldcup</title>
</head>
<body>
  <h2>ユーザー登録</h2>
  <div class="signup-wrapper">
    <form action="register.php" method="POST" class="edit-form">
      <div class="mb-3">
        <label for="email" class="form-label">Eメール</label>
        <input type="email" class="form-control" id="email" name="email" value="" placeholder="Eメールアドレスを入力してください。">
      </div>
      <div class="mb-3">
        <label for="password" class="form-label">パスワード</label>
        <input type="password" class="form-control" id="password" name="password" value="" placeholder="パスワードを入力してください。">
      </div>
      <div class="mb-3">
        <label for="password_conf" class="form-label">パスワード確認用</label>
        <input type="password" class="form-control" id="password_conf" name="password_conf" value="" placeholder="同じパスワードを入力してください。">
      </div> 
      <button type="submit" class="btn btn-primary">新規登録</button>
    </form>
  </div>
</body>
</html>

登録画面(register.php)

<?php
ini_set('display_errors', "On");

require_once(ROOT_PATH .'/Models/User.php');


$err = [];

if(!$email = filter_input(INPUT_POST, 'email')) {
  $err[] = 'メールアドレスを入力してください。'; 
}

$password = filter_input(INPUT_POST, 'password');

if(!preg_match("/\A[a-z\d]{8,100}+\z/i",$password)) {
  $err[] = "パスワードは英数字で8文字以上100文字以内で入力してください。";
}

$password_conf = filter_input(INPUT_POST, 'password_conf');
if($password !== $password_conf) {
  $err[] = "確認用パスワードと異なっています。";
}

$email = $_POST['email'];
$password = password_hash($_POST['password'], PASSWORD_DEFAULT);

if(count($err) === 0) {
  $user = new User();
  $hasCreated = $user->createUser($email, $password);

  if(!$hasCreated){
    $err[] = '登録に失敗しました';
  }
}

var_dump($email);
var_dump($password);

?>

<!DOCTYPE html>
<html lang="en">
<head>
  <meta charset="UTF-8">
  <meta http-equiv="X-UA-Compatible" content="IE=edge">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <link href="https://cdn.jsdelivr.net/npm/bootstrap@5.0.0-beta1/dist/css/bootstrap.min.css" rel="stylesheet" integrity="sha384-giJF6kkoqNQ00vy+HMDP7azOuL0xtbfIcaT9wjKHr8RbDVddVHyTfAAsrekwKmP1" crossorigin="anonymous">
  <link rel="stylesheet" href="/css/user.css">
  <title>worldcup</title>
</head>
<body>
  <?php if(count($err) > 0): ?>
    <?php foreach($err as $e): ?>
      <p><?php echo $e ?></p>
    <?php endforeach ?>
  <?php else: ?>
    <h2>ユーザー登録完了</h2>
  <?php endif ?>
  <a href="signup_form.php">ユーザー登録へ戻る</a>
</body>
</html>

User.php(ユーザーの新規登録のロジック)

<?php
require_once(ROOT_PATH .'/Models/Db.php');

class User extends Db {
  public function __construct($dbh = null) {
    parent::__construct($dbh);
  }

  public function createUser($email, $password) {
    $sql = 'INSERT INTO users(email, password, role) VALUES (:email, :password, :role)';
      $sth = $this->dbh->prepare($sql);
      $sth->bindValue(':email', $email, PDO::PARAM_STR);
      $sth->bindValue(':password', $password, PDO::PARAM_STR);
      $sth->bindValue(':role', 1, PDO::PARAM_INT);
      $result = $sth->execute();
      return $result;
  }
}

データベースの接続(Db.php)

<?php
  require_once(ROOT_PATH .'/database.php');

  class Db {
    protected $dbh;

    public function __construct($dbh = null) {
      if(!$dbh) {
        try {
          $this->dbh = new PDO(
            'mysql:dbname='.DB_NAME.
            ';host='.DB_HOST, DB_USER, DB_PASSWD
          );
        } catch (PDOException $e) {
          echo "接続失敗:".$e->getMessage()."\n";
          exit();
        }
      } else {
        $this->dbh = $dbh;
      }
    }
    public function get_db_handler(){
      return $this->dbh;
    }
  }

試したこと

このようにeメールとパスワードはsignup_form→registerに送信できていること確認ずみ。

INSERT INTO users(email, password, role) VALUES ('test02@gmail.com', 'aaaa1111', 1)
みたいな感じでSQL文を打てば、手動で登録はできること確認済み。

phper.k

2021/03/17 10:38 編集

> ユーザーの新規登録ができないため、ユーザーの新規登録ができるように実装したいです新規登録ができないとはどのような状態を言っていますか？ - DBを直接のぞくとデータが入っていない。 - 作成したはずのユーザーでログインできないなどなど。

m.ts10806

2021/03/17 11:54

設計の確認を。あと型とか桁とか気にしてください。書いたとおりに動いてます。

行動規範の内容に同意します

回答4件

ベストアンサー

データベースのvarchar(32)に対して保存しようとしてるパスワードが長すぎるから。

PDOは初期状態ではエラーが分かりにくいので例外を発生させたほうがいい。
Db.phpのnew PDO後に。

php
1$this->dbh = new PDO(
2    'mysql:dbname='.DB_NAME.
3    ';host='.DB_HOST, DB_USER, DB_PASSWD
4);
5$dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

https://www.php.net/manual/ja/pdo.error-handling.php

投稿2021/03/17 10:59

退会済みユーザー

総合スコア0

退会済みユーザー

2021/03/17 13:20

ご指摘ありがとうございます！実装できました。確かに文字数会ってなかったです。

行動規範の内容に同意します

emailとpasswordの文字数制限を逆にしちゃった感じかな？
それにしても皆様のツッコミが辛い…

投稿2021/03/17 11:50

退会済みユーザー

総合スコア0

hentaiman

2021/03/17 12:32

各解答者に意見があるのならそれぞれのコメント欄へどうぞ。

退会済みユーザー

2021/03/17 22:57

> emailとpasswordの文字数制限を逆にしちゃった感じかな？これに突っ込めってことなんですかね？ email は 254 or 360 password は 60 or 255 あたりが最大 or 推奨サイズになるので、そこに合わせるのが良いかと。参考： https://stackoverflow.com/questions/1297272/how-long-should-sql-email-fields-be https://www.php.net/manual/ja/function.password-hash.php

退会済みユーザー

2021/03/17 23:21

> hentaiman様特に無いですよーあるとしたらむしろ質問者さんへですかね。 DBに新規登録できるコードを持ってるはずなのに、なぜまた新規登録で質問してるかなぁ。って感じです。皆さんのやさしさが彼？に届かないのが辛い。と言い換えときますね。これからもまた既にできることで質問立てるんだろうなぁ、と。不毛だなぁって。

退会済みユーザー

2021/03/17 23:56

> te2ji様いつも補足情報をありがとうございます！ > email は 254 or 360 ご提示された参考URLのベストアンサー？を鵜呑みにされてしまった感じでしょうか？それよりも評価が1.8倍ほどついているコメントを読まれましたか？ 1000万件のデータのうち最長が89だったようです。ピーク値が19で38を超えるとほぼいないとのこと。理論値で語られるのはエンジニアっぽいなぁと感じますが… 実際の運用を考慮すると昔よりもドメインは増えてますし、そこまでは要らないかなぁと愚考いたします。 32だと少し少ないかなと思いますが、「32文字以下のメールアドレスをご登録ください」という注意書き程度で十分だと思います。ピークが19ってことは「みんな好きで長くしたいわけじゃない」ということでしょうから。 > password は 60 or 255 パスワードをハッシュ化するとDBで確認する際にいつも長くなっていたので、どれくらい長くなるのか調べようと思ってました。 URLありがとうございます！実際の運用を考えるとそこまで必要ではないかなぁという気もいたしますが。「ランダム生成した場合、32文字程度でスパコンじゃない限りクラックされない」といったような情報を数年前に見た覚えがあります。クラックされるまで無制限接続とかまず許可しないでしょうから、理論上限で設定しなくても良さそうであると愚考いたします。 32文字でも僕が選択するものですと100で足りそうかなと。参考URL　Qiita　hashアルゴリズムとハッシュ値の長さ一覧 https://qiita.com/KEINOS/items/c92268386d265042ea16

hentaiman

2021/03/18 00:03

> 32文字でも僕が選択するものですと100で足りそうかなと。足りるかたりないかより、DBで制約を掛けたいかどうかで決めれば良いかとそしてDBによる制約が不要な場合は255以下に収めるかそれを超えるかで決めたら良い ※質問内容の情報を元に考えた場合

退会済みユーザー

2021/03/18 00:37 編集

> 1000万件のデータのうち最長が89だったようです。ピーク値が19で38を超えるとほぼいないとのこと。そちらのデータはあまり意味ないです。たとえば、gamil では `+`や`.`を使用することで、受信専用のエイリアスアドレス(?)を作ることが出来ますが、サービスサイトのドメイン名やメモを追加してそのサービス専用のアドレスとして登録する人も一定割合います。そういったことを許容している巨大なサービスがある以上、トレンドによっては登録メールアドレスがいつ巨大化するかわからないので、最終的にはそもそものメールアドレスの仕様で判断するのが適切かと考えます。あと、ドメイン部が巨大化するサービスの登場も怖いですね。いくつかのシナリオが考えられますが、無くはない未来です。 > 32文字でも僕が選択するものですと100で足りそうかなと。パスワードのハッシュはストレッチングが前提となるので、元の文字長は関係ないです。全くの余談ですが、最近のトレンドとしてパスワード管理ツールを使用したパスワード生成や salt pepper を加味すると、32 文字の前提も現実的な数字ではないです。

退会済みユーザー

2021/03/18 01:38

> hentaiman様質問者さんの実装を見るに、60での生成になるようなので100で問題ないかと。数値を入れ違えたのであれば100で制限するつもりだったんでしょうし、どちらにしろ問題ないかと。まぁ、平文の100と誤認している可能性はありますが…

hentaiman

2021/03/18 01:54

そういう理由を元に示した数値ではないけど余計な事言って失礼しました

退会済みユーザー

2021/03/18 02:13

> te2ji様例外のご提示をありがとうございます！エイリアスの存在は存じておりますが、それを許容するかどうかはサービスの提供者側が決めることかと。そこまでリテラシーが高い人だとコスト的に赤になりそうかなぁと思います。メール32文字は確かに少ないのでせめて倍は欲しいところですが、理論値上限までカバーする必要性は感じません。ウケ狙いの長いドメインは出てくるかもしれませんが、弾くべき利用者であると愚考いたします。 > パスワードのハッシュはストレッチングが前提となるので、元の文字長は関係ないです。ご教授ありがとうございます！教えていただいておいて恐縮なのですが、自分で試し、自分の目で確かめてみました。適当なハッシュ化ツールで1文字と100文字で比較してみましたが、確かに同じ桁数になりますね。ただ、同じ桁数で全てを表すことはできないと思うので、ハッシュ化前の文字列が何文字くらいまでであれば利用に耐えるのかと考えていたんですが、長くなりそうなので自分で調べることにします。以下蛇足なのですが、質問者さんはおそらくコメントを読むことが無いでしょうから、直接教えてあげていただけると非常に助かります。

退会済みユーザー

2021/03/18 02:17

> hentaiman様僕も失礼いたしました。 SQLやDB操作は得意ではないと何度か伝えているのですが、いまだに回答依頼が飛んでくるんですよ… 放置するとteratailシステム的にどうなるのか分からないため、昔覚えた錆びてる知識を引っ張り出してコメントしてるんですが… 非常に辛いです。スルーするなら依頼するなよ、と。

K_3578

2021/03/18 02:33

横槍失礼します。回答依頼って別に受けなくても良かったような･･･。私も過去に来ましたが、専門外だったので回答しませんでした。この質問者あんまり素行良くないですし気になるようならプロフィールで回答依頼の設定を「相互フォローユーザーのみ」にしてこの質問者のフォロー外しても良いかと。お節介かもしれませんが･･･。

退会済みユーザー

2021/03/18 02:48 編集

> K_3578様あなたは神ですか！深く感謝いたします！即効でプロフィールの設定を見直してみますっありがとうございますっ！追記　相互フォローのみになってましたので、そっと外しておきました。同じ地域の方だったので「できるだけお手伝いしようかなぁ」とかなんとか思ってたんですがさすがに無差別はちょっとキツイｗ

K_3578

2021/03/18 02:45

amiya-seさんが殆どの方はフォローされたら返すタイプのユーザーさんのようなので多分これだったらある程度は防げるのかな･･･と。予めプロフィールに回答依頼来ても答えれない範囲明記したりするとちゃんとプロフィール見てくれる質問者さんからは配慮して頂けるかと。自分は、答えれない範囲の場合は追記修正依頼の箇所に依頼されたがこの範囲は専門外であるとコメント残すようにしてます。(追記･修正依頼の欄の使い方としては良くないかもしれませんが･･･。)

退会済みユーザー

2021/03/18 02:57

> K_3578様基本的にはウエルカムでございますよっ助けたいと思う方、議論できそうな方、スキルがかなり高そうな方をフォローしておりますです。ちょっとしたすれ違いで言葉(OHANASHI)のラリーは問題無いんですが、無差別依頼で反応良かった人のみってのはちょっとｗ初めて回答依頼がきて嬉しかったのもあったかもです。※あとは同地域的な？自分は回答するときに「あまり得意ではないんですが…」とつけていたような気がします。人間扱いされていないようなのでまぁ、ハズシでいいやーと思いますた。※猫ですがっ

K_3578

2021/03/18 03:09

>amiya-seさん＞自分は回答するときに「あまり得意ではないんですが…」とつけていたような気がします。該当回答拝見させて貰ったのですが主観的には少し気付きづらいかな？と思いました。まぁ質問者だったら頂いた回答の内容を全部読むのは当然だと思いますが･･･。＞人間扱いされていないようなので質問者も自分を人間だと思い込んでる兎なので大丈夫でしょう（？）完全に蛇足ですが、私はこれ何なんですかね･･･？サボテン？

『🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中！

＼teratail特別グッズやAmazonギフトカード最大2,000円分が当たる！／

前提・実現したいこと

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

試したこと

関連した質問