PHP登录表单错误提示仅在两次提交相同错误后生效，如何解决？

核心问题是代码执行顺序错误：

• 你先渲染了HTML页面，此时输出的$_SESSION['status']是上一次请求存储的旧值
• 等HTML完全输出后，才执行POST请求的校验逻辑，把本次的错误信息写入$_SESSION['status']
  所以本次提交产生的错误，要等下一次页面加载（也就是第二次提交、或者刷新页面）时才会显示，就出现了需要连续两次提交才触发提示的问题。

1. 把POST请求处理逻辑移到代码最顶部，所有HTML输出之前执行
2. 错误提示渲染后立即清空SESSION中的状态值，避免下次页面加载重复显示
3. 补充header跳转后的exit逻辑，终止后续代码执行

修复后完整代码如下：

<?php
if (session_status() === PHP_SESSION_NONE) {
    session_start();
}

// POST处理逻辑移到最顶部
if (isset($_POST['submit'])) {
    $id = trim($_POST['id']);
    // 先清空上一次的状态
    unset($_SESSION['status']);
    
    if (empty($id)){
        $_SESSION['status'] = 'Enter a Driver Pin';
    } else {
        $success = false;
        $handle = fopen("users.csv", "r");
        while (($data = fgetcsv($handle)) !== FALSE) {
            if ($data[0] == $id) {
                $success = true;
                $_SESSION['displayname'] = $data[1];
                break;
            } 
        }
        fclose($handle);
        
        if ($success) {
            $_SESSION['allow'] = '';
            header('Location: ./');
            // 跳转后立即终止后续代码执行
            exit();
        } else {
            $_SESSION['status'] = 'Invalid Driver Pin - Try again';
        }
    }
}
?>
<html>
<head>
<title>Title</title>
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<link rel="stylesheet" href="style.css">
<link rel="apple-touch-icon" sizes="180x180" href="/apple-touch-icon.png">
<link rel="icon" type="image/png" sizes="32x32" href="/favicon-32x32.png">
<link rel="icon" type="image/png" sizes="16x16" href="/favicon-16x16.png">
<link rel="manifest" href="/site.webmanifest">
<link rel="mask-icon" href="/safari-pinned-tab.svg" color="#5bbad5">
<meta name="msapplication-TileColor" content="#da532c">
</head>
<body>
<h1>Page Title</h1>

<div class="smallcontainer">
  <form method="post" action="">
    
    <div class="row">
        
        <div class="col-20">
        <label for="id">Driver Pin</label>
        </div>
        <div class="col-80">
        <input type="text" name="id" >
        </div>
    </div>
    
    <div class="row">
        <div class="col-100 error">
            <?php 
            // 输出状态后立即清空，避免下次加载重复显示
            if (isset($_SESSION['status'])) {
                echo $_SESSION['status'];
                unset($_SESSION['status']);
            }
            ?>
        </div>
        <div class="col-100">
            <input type="submit" value="Submit" name="submit">
        </div>
    </div>
    
  </form>
</div>

</body>
</html>

• 空值判断使用trim后的$id变量，避免用户输入全空格被判定为有效输入
• 移除不安全的@错误抑制符，改用isset判断SESSION变量是否存在
• 若用户量较大，建议将用户存储从CSV替换为数据库，避免每次请求读文件的性能损耗
• 可以增加前端校验作为第一道防线，减少不必要的后端请求

内容的提问来源于stack exchange，提问作者Tim Apple