php ログインページ

Question

前提・実現したいこと

今登録ページで入力された情報をDBに登録し、ログインページで取得し、メインページで名前を表示していますが、もっと簡単でわかりやすくするにはどうするべきですか？

該当のソースコード

if(filter_input(INPUT_SERVER,'REQUEST_METHOD') === 'POST'){
  $name = filter_input(INPUT_POST, 'name');
  $mail = filter_input(INPUT_POST, 'mail');
  $pass = filter_input(INPUT_POST, 'pass');

  if($name === ''){
    $error['name'] = 'ユーザー名は入力必須です。';
  }
  if(strlen($name) > 10){
    $error['name'] = '名前は１０文字以内です。';
  }
  if($mail === ''){
    $error['mail'] = 'メールアドレスは入力必須です。';
  }
  if($pass === ''){
    $error['pass'] = 'パスワードは入力必須です。';
  }
  if(!empty($pass) && strlen($pass) < 5){
    $error['pass'] = 'パスワードは５文字以上です。';
  }

  if(count($error) === 0){
    
    try{
      
      $pdo = connect();
 
      $stmt = $pdo->prepare('SELECT * FROM users WHERE name = ? AND mail = ?');
      $params = [];
      $params[] = $name;
      $params[] = $mail;

      $stmt->execute($params);

      $rows = $stmt->fetchAll();
      
      foreach($rows as $row){
        $hash = $row['pass'];
        $NAME = $row['name'];
        
        if(password_verify($pass,$hash)){
          session_regenerate_id(true);
          $_SESSION['login'] = $NAME;
          header('Location:main.php');
          return;
        }
      }

    }catch(PDOException $e){
      $error['login'] = 'ログインに失敗しました。';
      echo $e->getMessage();
      // die();
    }
  }
}

<p>ようこそ<?php echo h($_SESSION['login'], ENT_QUOTES); ?>さん</p>

Comments

[退会済みユーザー]

高々五十行程度の短いコードの何がわかりにくいのですか？

[NEO_PLANETT-777]

簡単に判りやすくとは？
ソースを短くするとか、そんな事ですか？
質問の意図が、ちょっと判りかねます。

Answer 1

メインページで名前を表示していますが、もっと簡単でわかりやすくするにはどうするべきですか？

ログインページを作るならこのくらいの量が妥当な気がします。

ただ、コードの中身は気になるところがありますがね

Comments

[yuki911]

このぐらいが妥当でしたか。
コードの中身できになるところとはどこですか？

[m.ts10806]

$NAME　←ここだけなぜか大文字
connect();　←他人にはここの定義内容が分からない
session_regenerate_id　←他人にはどこでsession_start()しているか分からない

Answer 2

機能を関数として切り出して、適当な名前をつけると、読みやすくなります。