新規会員登録でログインフォームが作動しません。

Question

新規会員登録フォーム入力→確認メール→リンク→ログインフォームの表示
という流れでコードを書いているのですが、メールの受信後にリンクを押しても
login.phpが作動しません。
コードに問題があるために作動しないと思うのですが、username passwordをDBと照合という内容で作りました。
コードの中でどこが不備か教えていただきたいです。
また、先頭部分にデバックのコードを追加したのですが、エラーが表示されません。
こちらに関しても何か良い方法があればお願いします。

<DB> CREATE TABLE `posts` ( `id` int(11) NOT NULL, `retweet_user_id` int(11) DEFAULT NULL, `content` varchar(255) NOT NULL, `picture` varchar(255) DEFAULT NULL, `datetime_posted` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP, `datetime_edited` datetime NOT NULL, `deleted` datetime NOT NULL, `created` datetime NOT NULL, `modified` datetime NOT NULL, `user_id` int(11) NOT NULL, `body` varchar(255) NOT NULL, `stamp` datetime NOT NULL ) ENGINE=InnoDB DEFAULT CHARSET=utf8;

<?php
session_start();
if(isset($_SERVER['REQUEST_METHOD']) && $_SERVER['REQUEST_METHOD'] === 'POST') {
ini_set('display_errors',1);
error_reporting(E_ALL);

if(isset($_SESSION['user'])!==''){
    header("Location:index.php");
  }

<?php
$SERVER = 'localhost';
$USER = 'root';
$PASS = 'root';
$DATABASE = 'microblog';
    
if (!($link = mysql_connect($SERVER,$USER,$PASS))){
    echo "<h3>Sorry, could not connect to database</h3>";
    exit;
}
 
mysql_select_db($DATABASE);

?>

<?php
    if(isset($_POST['login'])){
        $username = $mysql->real_escape_string($_POST['username']);
        $password = $mysql->real_escape_string($_POST['password']);
        $query = "SELECT * FROM users WHERE username ='$username'"; 
          
        $result = $mysqli->query($query);
	    if(!$result) {
	    print('Invalid username'.$mysqli->error);
	    $mysqli->close();// データベースの切断
	    exit();
	  }
        
	    while($row = $result->fetch_assoc()) {
	    $db_hashed_pwd = $row['password'];
	    $id = $row['id'];
	  }

	  // データベースの切断
	  $result->close();
        
        // ハッシュ化されたパスワードがマッチするかどうかを確認
	  if(password_verify($password, $db_hashed_pwd)){
	    $_SESSION['id'] = $id;
	    header("Location: index.php");// ログイン
	    exit;
	  } else {
	    echo "Sorry , We don't have your password ";
          
	  <?php }
	
    }
}
?>
コード

Answer 1

コードの中でどこが不備か教えていただきたいです。

どこがと言われれば、「全部」間違っています。
設計から見直して下さい。ログイン機能として必要な操作がいろいろ抜けています。

また、参考にしているモノも古いのではないでしょうか？
mysql_connect等は、古い php でしか動作しないです(現行の php ではすでに動作しない関数です)。

ちょっと駆け足ですけど、ログイン機能の設計に関しては以下の記事で触れられているので参考にすると良いです。
PHPでログイン機能を実装するチュートリアル #1

Answer 2

if(isset($_SESSION['user'])!==''){
    header("Location:index.php");
  }

<?php //閉じてないのに始めちゃダメ
$SERVER = 'localhost';

むしろ前も後ろもPHPコードなのでしたら閉じる必要ないと思います。

mysql_select_db($DATABASE);

//↓この下の空白も画面に出力されてしまいます。header()とか使うのだったら絶対NG
?>

<?php
    if(isset($_POST['login'])){

構文チェック機能の付いたエディタか、EclipseのようなIDEの利用を強くすすめます。

構文チェックだけで良ければpaiza.ioという手もありますがやはり手元のエディタをかえたほうがいいです。

以下蛇足。

---

慣習的な話なのですが、自身でつける変数名に全て大文字はやめておいたほうが良いです。
慣習として全て大文字は定数に使われます。

デフォルトで定数では大文字小文字を区別します。慣習的に、 定数は常に大文字で表記されます。

また$SERVERのような名前を付けてしまうとスーパーグローバル変数である$_SERVERと区別がつきにくくなります。
予約語との衝突も避けたいですね。

Comments

[amaturePy]

ありがとうございます。
phpのタグを一つでまとめていいのかなど初歩的すぎて調べてもでなかったので、大変参考になります。
エディターも考えてみます。現状はbracketを使用してます。
文字もなるべく小文字で対応します。

[m.ts10806]

Bracketsですね。私もよく使ってます。PHP構文チェック機能入れられないことはないですが簡単ではないようなので私はhtml,css,JavaScriptのみでしか使ってないです。
多少重たいもののEclipseがやはり手を出しやすいかなと。

[amaturePy]

承知しました。特に私みたいな初学者は変えた方が良さそうですね。一つハマると丸２日とか使ってしまうんで

[m.ts10806]

はい。構文エラーで毎回実行して確認では時間はもったいないと思います。
確認前にわかったほうが対策もとりやすいです。

[hentaiman]

初心者の頃から今に至るまでサクラエディタ(windows)一本でやってる人もココにイマスヨーｗ
javaの時だけeclipseに浮気したけど

[m.ts10806]

サクラはGrepでよく使いますね。職場によってはヒデ。
昔はTeraPadが異常に推されてた時代がありましたね。
UIキレイなのでEclipse使い始めるまでTepaEditorというのを使ってた時期もあります。

Answer 3

11行目の<?phpの書き間違えが動かない原因だね
コマンドライン使っているなら

php -l ファイル名.php

で構文チェックできるので活用してみるといいですよ

Comments

[amaturePy]

macなのでターミナルですが、これを構文チェックに使用する発想はなかったです！
sudoで権限も与えて試してみたのですが、ファイルを開いてくれません。

[hentaiman]

ファイル開かないわけないでしょうが、開けないファイルをどうやって編集してるのさ
せっかくなのターミナルになれるといいです。ls 打てばディレクトリ内のファイル見れるしmacの中のファイル構成を知る良い機会でしょう
macの良いところはターミナルとbrewだけなんだから有効活用しようよ

[amaturePy]

承知しました。サクラエディタ聞いたこと有ります。
php -l login.phpだとCoukd not open file となるので、ちょっと調べてみます。

[hentaiman]

ls と打って出てくるファイル一覧にlogin.phpはありますか？ないなら今いるディレクトリにはないので、正しいディレクトリにcdで移動しするか、フルパスでlogin.phpを指定すれば正常にコマンド打てますよ