请求修复PHP基于JSON文件的登录验证代码

我帮你排查了代码里的几个核心问题，现在把完整的修复方案整理出来：

原代码的问题梳理 #

• 你没有从$_POST中获取用户提交的邮箱和密码，导致$email和$pass变量未定义
• foreach循环会不断覆盖$jsonemail和$jsonpass的值，最后只会保留最后一条用户数据，无法正确匹配所有用户
• HTML表单的密码输入框缺少name="pass"属性，PHP根本接收不到密码值
• 你的login.json格式不完整，外层缺少包裹的{}，会导致json_decode解析失败

第一步：修复HTML表单的密码输入框 #

给密码输入框添加name="pass"属性，这样PHP才能获取到提交的密码：

<div class="container">
  <div class="row">
    <div class="col-md-4"></div>
    <div class="col-md-4">
      <form method="post" action="<?php echo htmlspecialchars($_SERVER["PHP_SELF"]);?>">
        <div class="form-group">
          <label for="email">Email address</label>
          <input type="email" class="form-control" name="email" id="email" aria-describedby="emailHelp" required="require" placeholder="Enter email">
        </div>
        <div class="form-group">
          <label for="pass">Password</label>
          <!-- 新增name="pass"属性，让PHP能获取到密码值 -->
          <input type="password" class="form-control" name="pass" id="pass" required="require" placeholder="Password">
        </div>
        <button type="submit" class="btn btn-primary" name="submit" >Submit</button>
      </form>
    </div>
    <div class="col-md-4"></div>
  </div>
</div>

第二步：修复login.json格式 #

确保JSON是完整的对象格式，避免解析失败：

{
  "1": {
    "email": "test123@gmail.com",
    "pass": "123"
  }
}

第三步：修复PHP验证逻辑 #

这里修正了变量获取、循环验证逻辑，还添加了错误处理和基础安全优化：

<?php
if(isset($_POST['submit'])){
    // 1. 获取并清理用户提交的表单数据，避免空格干扰
    $email = trim($_POST['email']);
    $pass = trim($_POST['pass']);

    // 2. 读取并解析JSON文件，增加错误处理
    $json_file = 'login.json';
    if(!file_exists($json_file)){
        echo "系统错误：用户数据文件不存在";
        exit;
    }

    $json_string = file_get_contents($json_file);
    $parsed_json = json_decode($json_string, true);

    if($parsed_json === null){
        echo "系统错误：用户数据解析失败";
        exit;
    }

    // 3. 遍历所有用户数据，逐一验证登录信息
    $login_success = false;
    foreach($parsed_json as $user){
        if($user['email'] === $email && $user['pass'] === $pass){
            $login_success = true;
            break; // 找到匹配用户后立即退出循环，提升效率
        }
    }

    // 4. 根据验证结果跳转或提示错误
    if($login_success){
        header('location: view.php');
        exit; // 跳转后必须终止脚本，避免后续代码意外执行
    } else {
        echo "请检查你的邮箱或密码是否正确";
    }
}
?>

额外的安全建议（可选） #

• 绝对不要明文存储密码：现在的代码里密码是明文的，非常不安全，建议用password_hash()生成哈希值存储，登录时用password_verify()验证
• 可以添加邮箱格式验证，比如用filter_var($email, FILTER_VALIDATE_EMAIL)
• 考虑增加登录失败次数限制、验证码等功能，进一步提升安全性

内容的提问来源于stack exchange，提问作者MJ DEV