ログインページにアクセスした時のエラー

会員登録用のページにて、[login.php]のページにアクセスすると必ず現れるエラーがあります。
Notice: Undefined variable: error in /Applications/MAMP/htdocs/myportfolio2/login.php on line 32

動作はうまくいくのですが、上記のようなエラー文が出るのが気になります。
どのように対処すればよろしいでしょうか？よろしくお願いします。

login.php
1<?php
2require("dbconnect.php");
3require("function.php");
4session_start();
5
6if (!empty($_POST)) {
7    $stmt = $dbh->prepare("SELECT * FROM members WHERE email=:email");
8    $stmt->bindValue(':email', $_POST['email']);
9    $stmt->execute();
10    $member = $stmt->fetch();
11
12    if (password_verify($_POST['password'], $member['password'])) {
13      $_SESSION['id'] = $member['id'];
14      $_SESSION['name'] = $member['name'];
15      header ("Location:index.php"); exit();
16  } else {
17    $msg = "メールアドレスもしくはパスワードが間違っています。";
18}
19}
20?>
21
22<!DOCTYPE html>
23<html lang="ja">
24  <head>
25    <meta charset="UTF-8">
26    <title>ログイン</title>
27  </head>
28  <body>
29    <h1>ログイン</h1>
30    <a href="signup.php"><p>会員登録がお済みでない方はコチラ</p></a>
31    <form action="" method="post">
32      <p><?php echo $msg; ?></p>
33      <dl>
34        <dt>メールアドレス</dt>
35        <dd>
36          <input type="text" name="email" size="35" maxlength="255">
37        </dd>
38
39        <dt>パスワード</dt>
40        <dd>
41          <input type="password" name="password" size="35" maxlength="255">
42        </dd>
43
44        <dt>ログイン情報の記録</dt>
45        <dd>
46          <input id="save" type="checkbox" name="save" value="on"><label for="save">次回からは自動的にログインする</label>
47        </dd>
48      </dl>
49      <div>
50        <input type="submit" value="ログインする">
51      </div>
52    </form>
53  </body>
54</html>

AbeTakashi

2021/08/13 05:51

$msgに対するエラーとのことですが、それは文章で何と書いてありますか？また、その意味を翻訳してみたり、ネットで調べたりしたでしょうか？　たぶん、それをすればエラーの原因はすぐに分かるはずです。何も考えず、調べずだとエラーは解決できません。意味を翻訳しても分からない、ネットで調べても分からないのであれば、その内容を質問文に追加してください。

m.ts10806

2021/08/13 05:56 編集

https://teratail.com/help/question-tips#questionTips3-4-2 https://qiita.com/cannorin/items/eb062aae88bfe2ad6fe5

m.ts10806

2021/08/13 05:59

エラーレベルは正確に認識できるようになった方が良いです。「プログラム実行にどれだけの影響があるか」というものなので、全部「エラー」としてしまうとプログラミングの理解進みません。 https://www.php.net/manual/ja/errorfunc.constants.php

suke1001

2021/08/13 06:15

Abe Takashiさんありがとうございます。一応、翻訳して色々試したのですが答えに辿り着くことができませんでした。ちなみに、[Notice: Undefined variable: msg in /Applications/MAMP/htdocs/myportfolio2/login.php on line 32]と書かれておりました。

suke1001

2021/08/13 06:19

m.ts10806さんいつもお教えありがとうございます。参照の記事を読ませてもらいもう一度解決できるように試してみます。

AbeTakashi

2021/08/13 06:26

「Undefined variable」ですよ？翻訳したらそのまま悩まれていることの答えが書いてあるんです。「$msgがUndefined variableということはどういうことか？」ってちゃんと考えましたか？これとif文の仕組みが分かれば答えにたどり着けるはずです。もう少し頑張って考えてみてください。ちゃんと考えれば必ず答えにたどり着けます。

m.ts10806

2021/08/13 06:41

質問は編集できます。

suke1001

2021/08/13 06:44

編集させてもらいました。

m.ts10806

2021/08/13 07:16

そのものの解決策を提示したわけではないので、どう解決したのか、フィードバックください。 https://teratail.com/help/question-tips#questionTips4-2 「今後に向けた」ことも書いてるので、その部分を理解できてるかどうかで全然違ってきます。むしろ質問しなくても解決できるようになるくらいのことが書いてあります。

suke1001

2021/08/13 07:40

了解しました。今、自己解決方法についてフィードバック書きました。おっしゃる通りで、難しく考えすぎたためにこのようになってしまいました。質問をする前にもっと考えます。

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

回答2件

ベストアンサー

$msgという変数、
$_POSTがemptyの時は定義されてません。

同様の状況を再現できるミニマムコード。

php
1<?php
2if(false){
3    $hoge = "false";
4}
5echo $hoge;

NOTICEはエラー関係の定義済み定数より実行時の警告。エラーを発しうる状況に遭遇したことを示す。ただし通常のスクリプト実行の場合にもこの警告を発することがありうる。 とのことで、実際のスクリプト実行（というより遂行）に影響がないものです。
Undefined variableの意味自体は調べれば分かります。
エラーメッセージの読み方と対処, 検索や質問の原則

このように学習時期に直面する問題は「エラーを読むこと」「デバッグをすること」「公式ドキュメントを確認すること」で大抵解決できます。
初学者が真っ先に覚えるべき作業です（逆に覚えられない人はずっと初心者と言っても過言ではない）

「どうすべきか」は、あとは「どういう設計か」によります。

例えば「画面初期アクセス時は表示しない」ならpタグごと分岐かけて表示させないようにすべきでしょうし。

ちなみに「画面にURLでアクセス」もしくは「リンクによる遷移」は基本GETです。
あとは変数内容や分岐でデバッグしながら「現在の各情報の状態」を調べてみてください。

蛇足：
PHPは結構ゆるい言語なのでNOTICEで済んでいます。
Javaなどの言語ではそもそもコンパイルエラーで実行できません。

投稿2021/08/13 06:53

編集2021/08/13 06:57

m.ts10806

総合スコア80861

if文の仕組みをもう一度調べなおしてコードを修正したところ、該当のエラー分が消えました。

全くその部分に問題があると思っておらず、全体的な構造を変えたり、違う修正ばかり行っていたため解決に至りませんでした。ミニマムコードを今後活用して足元のエラーに気付けるようにしていきたいです。

GETとPOSTの仕組みに関しては今から修正します。

解決に至るまでに、色々とご指導ありがとうございました。

login.php
1
2<?php
3require("dbconnect.php");
4require("function.php");
5session_start();
6
7if (!empty($_POST)) {
8  if ($_POST['email'] != '' && $_POST['password'] != '') {
9    $stmt = $dbh->prepare("SELECT * FROM members WHERE email=:email");
10    $stmt->bindValue(':email', $_POST['email']);
11    $stmt->execute();
12    $member = $stmt->fetch();
13
14    if (password_verify($_POST['password'], $member['password'])) {
15      $_SESSION['id'] = $member['id'];
16      $_SESSION['name'] = $member['name'];
17      header ("Location: index.php"); exit();
18    } else {
19      $msg = 'ログイン失敗しました';
20    }
21  } else {
22    $msg = 'メールアドレスとパスワードを入力してください。';
23  }
24} else {
25  $msg = '';
26}
27?>
28
29
30
31<!DOCTYPE html>
32<html lang="ja">
33  <head>
34    <meta charset="UTF-8">
35    <title>ログイン</title>
36  </head>
37  <body>
38    <h1>ログイン</h1>
39    <a href="signup.php"><p>会員登録がお済みでない方はコチラ</p></a>
40    <form action="" method="post">
41      <p><?php echo $message; ?></p>
42      <dl>
43        <dt>メールアドレス</dt>
44        <dd>
45          <input type="text" name="email" size="35" maxlength="255">
46        </dd>
47
48        <dt>パスワード</dt>
49        <dd>
50          <input type="password" name="password" size="35" maxlength="255">
51        </dd>
52
53        <dt>ログイン情報の記録</dt>
54        <dd>
55          <input id="save" type="checkbox" name="save" value="on"><label for="save">次回からは自動的にログインする</label>
56        </dd>
57      </dl>
58      <div>
59        <input type="submit" value="ログインする">
60      </div>
61    </form>
62  </body>
63</html>

投稿2021/08/13 07:38

編集2021/08/13 07:52

suke1001

総合スコア7

AbeTakashi

2021/08/13 08:10

解決して良かったです。ちなみに「Undefined variable PHP」でGoogleで検索すると、以下のページが一番上に表示されるはずです。参考） https://hacknote.jp/archives/21606/ ご自身で解決した$msgをもれなく定義する方法での解決方法もありですし、isset()を使う方法もありでしょう。どちらが良いかはケースバイケースです。エンジニア転職を目指してるとのことでしたので、あえて直球で答えないようにしていましたが、本気でエンジニアを目指すのであれば、今後に向けてある程度の自己解決能力を身につける必要があると思います（いちいち他人に聞いていては仕事になりません）。ミニマムコードでの検証も大事ですが、それ以前にエラーメッセージでしっかり検索し、エラーメッセージの意味を考えるのも重要かと思います（ミニマムコードでの検証よりも時間かからない可能性が高い）。これからも色々困難な壁があるかもしれませんが、頑張ってください。

suke1001

2021/08/13 08:23

Abe Takashiさん暖かいお言葉ありがとうございます。独学で勉強しているのでつまるとすぐにテンパってしまい、ミスを自分で気づけなかったりします。そんな、甘い考えではダメだとさらに感じましたので、よく考えてから質問をするように気をつけます。お時間取らせてしまいすみませんでした、ありがとうございます。

m.ts10806

2021/08/13 08:37

$msgを冒頭で初期化すればわざわざelse書く必要ありません。

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

あなたの回答