フォーム登録画面で、もし同じユーザーネームが登録されたら、”○○ is already exist.” と表示させたいです。現在は、DBのユーザーネームをuniqueにして、ただ単にクエリーエラーにさせてるだけです。
それをきちんとプログラム内で、重複データ用のエラーを表示させたいです。
現在の動くプログラムはこんな感じです。
<?php
/* Program name: register.php
* Description: Program displays the blank form and checks
* all the form fields for blank fields.
*/
ini_set("display_errors","on");
error_reporting(E_ALL | E_STRICT);
ini_set("include_path","./includes");
include("dbinfo.inc");
if(isset($_POST['submitted']) and $_POST['submitted'] == "yes")
{
foreach($_POST as $field => $value)
{
if(empty($value))
{
$blank_data[] = $field;
}
else
{
$good_data[$field] = strip_tags(trim($value));
}
}
if(@sizeof($blank_data) > 0)
{
$message = "<p style='color: red; margin-bottom: 0;
font-weight: bold'>
You didn't fill in one or more required fields.
You must enter:
<ul style='color: red; margin-top: 0;
list-style: none' >";
/* display list of missing information */
foreach($blank_data as $value)
{
$message .= "<li>$value</li>";
}
$message .= "</ul>";
//extract($good_data);
echo $message;
include("form.inc");
exit();
}
foreach($_POST as $field => $value)
{
if(!empty($value))
{
//validations
$name_patt = "/^[A-Za-z0-9' -]{1,15}$/";
$email_patt = "/^.+@.+\\..+$/";
$passwd_patt = "/^[A-Za-z0-9' -]{1,15}$/";
$field = htmlspecialchars($field, ENT_QUOTES);
$value = htmlspecialchars($value, ENT_QUOTES);
if(preg_match("/username/i",$field))
{
if(!preg_match($name_patt,$value))
{
$error_array[] = "$value is not a valid Username";
}
}
if(preg_match("/email/i",$field))
{
if(!preg_match($email_patt,$value))
{
$error_array[] = "$value is not a valid Email";
}
}
if(preg_match("/password/i",$field))
{
if(!preg_match($passwd_patt,$value))
{
$error_array[] = "$value is not a valid password";
}
}
}
}
if(@sizeof($error_array) > 0)
{
$message = "<ul style='color: red; list-style: none' >";
foreach($error_array as $value)
{
$message .= "<li>$value</li>";
}
$message .= "</ul>";
echo $message;
@extract($clean_data);
include("form.inc");
exit();
}
else
{
$cxn = mysqli_connect($host,$user,$passwd,$dbname)
or die("Couldn't connect to server");
//foreach((array)$clean_data as $field => $value)
$clean_data = array();
foreach($_POST as $field => $value)
{
$clean_data[$field] = mysqli_real_escape_string($cxn,$value);
}
$date = date("Y-m-d H:i:s");
//echo date("Y/m/d H:i:s",$date);
//$sql = "INSERT INTO CustomerPhone (last_name,first_name,middle_name,phone)
//VALUES ('$clean_data[last_name]','$clean_data[first_name]',
// '$clean_data[middle_name]','$clean_data[phone]')";
?>
<form method="post">
<input type="hidden" name='$field' value="<?php echo htmlspecialchars($good_data['value']); ?>">
<input type="hidden" name="date" value="<?php echo $date; ?>">
</form>
<?php
//$status = "none";
/* loop that displays the form */
$sql = "INSERT INTO login (username,email,password,date)
VALUES ('".$clean_data['username']."','".$clean_data['email']."',
'".$clean_data['password']."','$date')";
//echo $clean_data['username'];
$result = mysqli_query($cxn,$sql)
or die("Couldn't execute query");
$signUpMessage = 'Registered. Your Username is '.$clean_data['username'].' Password is'.$clean_data['password'].' ';
echo $signUpMessage;
include("stored.inc");
}
}
else
{
include("form.inc");
}
?>
そして、エラーメッセージを出すために$clean_dataをinsert into したあとに、この条件文を入れました。
// userテーブルからemail項目を条件に1件取得するSQL文
$sql = "select * from login where username = :username";
// SQL発行準備
$stmt = $cxn->prepare($sql);
// SQL発行。条件のusernameは、引数$usernameの値。
$stmt->execute(array(":username" => $username));
// loginテーブルから1件データ取得
$login = $stmt->fetch();
// $usernameが存在していればtrue、そうでなければfalseを返す
return $login ? true : false;
if ($login == "true"){
echo ''.$username.' is already exist.' ;
}
そして、DBに既に存在するユーザーネームを打ち込むと、
Fatal error: Uncaught Error: Call to a member function execute() on boolean in /home/vol14_5/ihostfull.com/uoolo_19138640/htdocs/PHP/Ex5/E5/register.php:128 Stack trace: #0 {main} thrown in /home/vol14_5/ihostfull.com/uoolo_19138640/htdocs/PHP/Ex5/E5/register.php on line 128
と表示されます。
どのようにしたらいいでしょうか?
-
気になる質問をクリップする
クリップした質問は、後からいつでもマイページで確認できます。
またクリップした質問に回答があった際、通知やメールを受け取ることができます。
クリップを取り消します
-
良い質問の評価を上げる
以下のような質問は評価を上げましょう
- 質問内容が明確
- 自分も答えを知りたい
- 質問者以外のユーザにも役立つ
評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。
質問の評価を上げたことを取り消します
-
評価を下げられる数の上限に達しました
評価を下げることができません
- 1日5回まで評価を下げられます
- 1日に1ユーザに対して2回まで評価を下げられます
質問の評価を下げる
teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。
- プログラミングに関係のない質問
- やってほしいことだけを記載した丸投げの質問
- 問題・課題が含まれていない質問
- 意図的に内容が抹消された質問
- 過去に投稿した質問と同じ内容の質問
- 広告と受け取られるような投稿
評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。
質問の評価を下げたことを取り消します
この機能は開放されていません
評価を下げる条件を満たしてません
質問の評価を下げる機能の利用条件
この機能を利用するためには、以下の事項を行う必要があります。
- 質問回答など一定の行動
-
メールアドレスの認証
メールアドレスの認証
-
質問評価に関するヘルプページの閲覧
質問評価に関するヘルプページの閲覧
checkベストアンサー
+3
現在の動くプログラム
では mysqliモジュールを使用していますが、
$clean_dataをinsert into したあとに
追加したプログラムは PDOモジュールを使用しているかのような文になってることが、
エラーの原因です。
それ以前に、今、試みている方法では正しく
重複データ用のエラーを表示
することはできません。
$clean_dataをinsert into したあと
にselect * from login
を実行すると必ず1件以上のデータが存在するため、
常にエラーメッセージが表示されることになるからです。
代わりに、
DBのユーザーネームをunique
のままにした上で、以下のようにすると良いでしょう。
// IGNORE キーワードを追加
$sql = "INSERT IGNORE INTO login (username,email,password,date)
VALUES ('".$clean_data['username']."','".$clean_data['email']."',
'".$clean_data['password']."','$date')";
$result = mysqli_query($cxn, $sql)
or die("Couldn't execute query");
$affectedRows = mysqli_affected_rows($cxn);
if ($affectedRows === 1) {
$signUpMessage = 'Registered. Your Username is '.$clean_data['username'].' Password is'.$clean_data['password'].' ';
echo $signUpMessage;
} else if ($affectedRows === 0) {
echo $clean_data['username'] . ' is already exist.' ;
} else {
die('Something is wrong...');
}
https://dev.mysql.com/doc/refman/5.6/ja/insert.html
IGNORE キーワードを使用した場合、(中略)重複キーエラーは生成しません。
http://php.net/manual/ja/mysqli.affected-rows.php
直前の MySQL の操作で変更された行の数を得る
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
+1
エラーメッセージを読んでみてください。register.php の 128 行目で boolean の execute を呼んでいるが、そんなものはないと書いています。
返り値 ¶
もしデータベースサーバーが正常に文を準備する場合、 PDO::prepare() は PDOStatement オブジェクトを返します。 もしデータベースサーバーが文を準備できなかった場合、 PDO::prepare() は FALSE を返すか PDOException を発行します (エラー処理 の方法に依存します)。
投稿
-
回答の評価を上げる
以下のような回答は評価を上げましょう
- 正しい回答
- わかりやすい回答
- ためになる回答
評価が高い回答ほどページの上位に表示されます。
-
回答の評価を下げる
下記のような回答は推奨されていません。
- 間違っている回答
- 質問の回答になっていない投稿
- スパムや攻撃的な表現を用いた投稿
評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。
15分調べてもわからないことは、teratailで質問しよう!
- ただいまの回答率 88.34%
- 質問をまとめることで、思考を整理して素早く解決
- テンプレート機能で、簡単に質問をまとめられる
2017/05/12 03:00 編集
そうなんです、PDOとmysqliが頭の中でかなり混乱しています。
ちゃんと分けて考えられるよう、勉強します。