質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

Q&A

解決済

1回答

549閲覧

ログインページにて、正しいpw、アドレスを入力してもエラーが出る

haru-

総合スコア19

MySQL

MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

0グッド

1クリップ

投稿2020/05/04 00:11

編集2020/05/04 00:49

ログインページを作っています

現在ログインページを作ってます。
入力欄が空白場合はエラーを出し、入力されているものはDBに登録してあるデータと不一致であればエラーを出す。
一致していれば次の画面に飛ぶというふうにしたいです。

しかしプログラムを実装すると、正しいpw、アドレスを入力してもエラー('failde')が表示されます。
正しいpw、アドレスを入力してもエラーが出る原因を教えてください。

該当のソースコード

index.php

php

1<?php 2require('dbconnect.php'); 3session_start(); 4$db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); 5 6if(!empty($_POST)) { 7 //空白の場合 8 if($_POST['email'] === '') { 9 $error['email'] = 'blank'; 10 } 11 if($_POST['password'] === '') { 12 $error['password'] = 'blank'; 13 } 14 15 16 if($_POST['email'] !== '' && $_POST['passwprd'] !== '') { 17 //不一致の場合 18 $membersEmail = $db->prepare('SELECT * FROM members WHERE email=?'); 19 $membersEmail->execute(array( 20 $_POST['email'] 21 )); 22 $memberEmail = $membersEmail->fetch(); 23 24 if($_['email'] !== $memberEmail) { 25 $error['login'] = 'failde'; 26 } 27 28 $membersPw = $db->prepare('SELECT * FROM members WHERE password=?'); 29 $membersPw->execute(array( 30 sha1($_POST['password']) 31 )); 32 $memberPw = $membersPw->fetch(); 33 34 if($_POST['password'] !== $memberPw) { 35 $error['loginPw'] = 'failde'; 36 } 37} 38 39 //エラーがない場合 40 if(empty($error)) { 41 $_SESSION['id'] = $_POST; 42 header('location:index.php'); 43 exit(); 44 } 45 46 47 48 } 49 ?> 50 51<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> 52<html xmlns="http://www.w3.org/1999/xhtml"> 53<head> 54<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" /> 55<link rel="stylesheet" type="text/css" href="style.css" /> 56<title>ログインする</title> 57</head> 58 59<body> 60<div id="wrap"> 61 <div id="head"> 62 <h1>ログインする</h1> 63 </div> 64 <div id="content"> 65 <div id="lead"> 66 <p>メールアドレスとパスワードを記入してログインしてください。</p> 67 <p>入会手続きがまだの方はこちらからどうぞ。</p> 68 <p>&raquo;<a href="join/">入会手続きをする</a></p> 69 </div> 70 <form action="" method="post"> 71 <dl> 72 <dt>メールアドレス</dt> 73 <?php if($error['email'] == 'blank'): ?> 74 <p class="error"><?php print 'アドレスを入力してください。'; ?></p> 75 <?php endif; ?> 76 <?php if($error['login'] == 'failde'): ?> 77 <p class="error"><?php print 'アドレスが違います'; ?></p> 78 <?php endif; ?> 79 <dd> 80 <input type="text" name="email" size="35" maxlength="255" value="<?php echo htmlspecialchars($_POST['email']); ?>" /> 81 </dd> 82 <dt>パスワード</dt> 83 <?php if($error['password'] == 'blank'): ?> 84 <p class="error">pwを入力してください</p> 85 <?php endif; ?> 86 <?php if($error['loginPw'] == 'failde'): ?> 87 <p class="error"><?php print 'pwが違います'; ?></p> 88 <?php endif; ?> 89 <dd> 90 <input type="password" name="password" size="35" maxlength="255" value="<?php echo htmlspecialchars($_POST['password']); ?>" /> 91 </dd> 92 <dt>ログイン情報の記録</dt> 93 <dd> 94 <input id="save" type="checkbox" name="save" value="on"> 95 <label for="save">次回からは自動的にログインする</label> 96 </dd> 97 </dl> 98 <div> 99 <input type="submit" value="ログインする" /> 100 </div> 101 </form> 102 </div> 103 <div id="foot"> 104 <p><img src="images/txt_copyright.png" width="136" height="15" alt="(C) H2O Space. MYCOM" /></p> 105 </div> 106</div> 107</body> 108</html> 109

dbconnect.php

php

1<?php 2try { 3$db = new PDO('mysql:dbname=mini_bbs;host=localhost;charset=utf8','root', 'root'); 4} catch (PDOException $e) { 5echo 'DB接続エラー: ' . $e->getMessage(); 6} 7?>

イメージ説明

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

m.ts10806

2020/05/04 00:25

ツッコミどころも少なくないのですけど、 これだけでは何とも言えません。 テーブル定義、データも提示してください。 データの登録処理も必要です。
haru-

2020/05/04 00:49

DBの情報と接続時の処理を追加しました。 ご確認お願いします。
m.ts10806

2020/05/04 00:52

なるほど「気づけばプロ並み」で学習ですか。 となると・・・書籍をかえるところからですね。 この書籍を参照したと思われる質問が多く投稿されています。 つまり、それだけ書籍の質がよろしくないということになります。 PHPマニュアルを活用できるようになるだけでも違うと思いますが・・・・
haru-

2020/05/04 00:56

あ!すみません!「気づけばプロ並み」はおっしゃる通りでしたので 書籍は変えてUdemyで学習しています。 たまたま名前が思い浮かばなかったので、この名前を使っただけです。 ちなみにUdemyのコードと全く同じで一度学習して 二週目なので自分なりにコードを変えて書いていたところエラーが出て つまずいてしまいました。
m.ts10806

2020/05/04 01:26

んーなるほど。 にしてもHTML4の構成なのでそれはそれで古いですね。
guest

回答1

0

ベストアンサー

ざっと読んで気になるところ。

if($_['email'] !== $memberEmail) {

上記、参照できる配列ですか?
エラー出る設定にして確かめてください。

PHPのエラー表示設定について

他にもいろいろありますが、IDとパスワードを個別に確認しているのもよろしくないかと思います。
IDは重複できないとしても、パスワードが同じになることはあり得ますから、1件に絞れません。(DB設計次第)

暗号化方式についても、再検討の余地はあります。
CRYPTREC が「SHA-1 の安全性低下について」を公開

グーグル、SHA-1衝突攻撃に成功--同一ハッシュ値の2つのPDFも公開

SHA-1の新しい脆弱性について(2020/01/07報告)

PHPにおいてはpassword_hash()でハッシュ化し、password_verify()で照合するのが望ましいです。

投稿2020/05/04 01:36

編集2020/05/04 01:49
m.ts10806

総合スコア80850

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

haru-

2020/05/04 01:58

POSTが抜けていますね…。 エラーの表示はphp.iniでdisplay_errors = Onにしているのですが さらに設定し直した方が良いですか? 確かにIDとパスワードを個別よくないですね! ありがとうございます! 暗号化方式も書き換えてみます!
m.ts10806

2020/05/04 02:18

指定してるならエラーでるんじゃないかなと思いました。
haru-

2020/05/04 02:28

エラー出る時もあるのですが、今回のように出ない時もあります…。 もう一度エラー設定を見直してみます。 そしてIDとパスワードを個別にせずに実装したところ上手くいきました! ありがとうございます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問