DBに登録済みのusernameも登録されてします

Question

データベースに登録されているusernameは登録されないようにしたい。

発生している問題

エラーは出ないんですが、すでに登録されているusernameまでも登録されてしまいます；
登録されているusernameとpasswordを２つ両方入力したら、ちゃんと最初のif文が処理されます。
それ以外はすべて登録されてしまいます、、、

### 該当のソースコード
<?php
session_start();
if(empty($_POST["username"]) || empty($_POST["password"])) {
    header("Location: signin.php?err=2");
    exit();
}

//重複チェック
require_once("dbconnect.php");
//データベース内のusernameを取得
$sql = "SELECT * FROM users WHERE username=:username AND password=:password";
$stmt = $pdo->prepare($sql);
$stmt->bindValue(":username", $_POST["username"], PDO::PARAM_STR);
$stmt->bindValue(":password", $_POST["password"], PDO::PARAM_STR);
$stmt->execute();
$row = $stmt->fetch(PDO::FETCH_ASSOC);


//DB内に登録されているusernameと重複していない場合は、登録する
if(empty($row['username'])) {
	$sql = "INSERT INTO users(username, password) VALUES(:username, :password)";
	$stmt = $pdo->prepare($sql);
	$stmt->bindValue(":username", $_POST["username"], PDO::PARAM_STR);
	$stmt->bindValue(":password", $_POST["password"], PDO::PARAM_STR);
	$stmt->execute();
	header("Location: done.php");
} else {
  header("Location: insert.php");
	exit();
}
?>
```ここに言語名を入力
php

試したこと

isset関数やcount関数を使ってみましたが、php勉強中の初心者なのであまりよくわからなくて、困っています。
よろしくおねがいします！

Answer 1

そもそもですが「重複は許さない」のでしたら、エラーが出るようにDB側に制約つけておいたほうが良いです（キーワード「主キー制約 ユニーク制約」）。
PDO利用ならそれ用のエラーハンドリングも導入してください

パスワードも平文はよろしくないかと。幸いPHPにはpassword_hash()で暗号化したパスワードを保存してpassword_verify()で検証するという流れがあるので採用してください。

Comments

[m.ts10806]

低評価された方は理由をコメントください。

Answer 2

こんにちは。

重複チェックの時にDBの中にすでに存在しないかチェックする際の、SELECT文が

$sql = "SELECT * FROM users WHERE username=:username AND password=:password";

となっていますが、これだと検索条件の中に、
「ユーザー名とパスワードが一致する物」が存在するかどうかになってしまいますので、
同じユーザー名で登録しようとしても、パスワードさえ異なればOKということになってしまいます。

存在チェックの際はユーザ名のみで確認するとよいのではないでしょうか？
$sql = "SELECT * FROM users WHERE username=:username";

※パラメタから:passwordがなくなるので、このselect文に対応する
$stmt->bindValue(":password", $_POST["password"], PDO::PARAM_STR);
も不要になります。

Comments

[rumichan]

わわわわあすごい数時間悩んでいたことが一瞬で解決して感動してます
ありがとうございます！；；；
修正したところ、ちゃんと認識されました！
ありがとうございます。