PHP登录表单遇isset相关报错及登录无响应问题求助

问题原因分析 #

1. 「Undefined array key 'login'」警告：
   页面首次加载时（非表单提交状态），$_POST数组中不存在login键，直接使用$_POST['login']会触发PHP 8+的未定义数组键警告，同时可能导致后续登录逻辑判断异常。

2. 登录后不显示欢迎信息：
   原代码的表单action为./，若当前文件不是站点首页，表单提交后会跳转到首页而非处理登录逻辑的login.php，导致登录逻辑未执行；另外，未检查$_POST['login']是否存在的逻辑错误，也会让登录代码块无法正常触发。

修复方案 #

1. 修正$_POST['login']的判断逻辑 #

将登录判断条件改为先检查login键是否存在，避免警告并保证逻辑正确性：

// 原代码第9行
// if(!isset($_SESSION['user_id']) && $_POST['login']) {
// 修改为：
if(!isset($_SESSION['user_id']) && isset($_POST['login'])) {

2. 明确表单提交路径 #

将表单action指定为login.php，确保提交后能触发登录处理逻辑：

// 原表单代码
// <form action='./' method='POST'>
// 修改为：
<form action='login.php' method='POST'>

3. （重要优化）替换不安全的MD5加密 #

MD5已被证实存在安全漏洞，建议后续改用PHP内置的password_hash()和password_verify()处理密码。若暂时需兼容现有MD5存储的密码，可先保留原逻辑，但需尽快迁移。

修复后的完整代码 #

<?php
session_start();

require('DatabaseObject.php');
require('databaseVars.php');

$database = new DatabaseObject($host, $username, $password, $database);
    
// 修复：先检查$_POST['login']是否存在，避免警告
if(!isset($_SESSION['user_id']) && isset($_POST['login'])) {
    $username = $database->clean($_POST['username']);
    $password = $database->clean($_POST['password']);
        
    $result = $database->query("SELECT `id`, `username`, `password` 
                                FROM `Users` 
                                WHERE `username` = '$username' LIMIT 1");
    
    try{
        if ($database->num_rows == 0){
            throw new Exception ("User does not exist.");
        }
            
        $user = $database->fetch($result);
            
        // 注意：MD5不安全，建议后续替换为password_verify
        if(md5($password) != $user['password']){
            throw new Exception ("The username and password combination cannot be found.");
        }
            
        $_SESSION['user_id'] = $user['id'];
    } catch(Exception $e){
        echo $e->getMessage();
    }
}

echo "<h3 style = 'text-align:center'>ArenaScape</h3>";

if(isset ($_SESSION['user_id'])){
    echo "Welcome to ArenaScape!";
    
}else{ 
    // 修复：明确指定表单提交到login.php
    echo "<form action='login.php' method='POST'>
        Username: <input type='text' name='username'/><br />
        Password: <input type='password' name='password'/><br />
        <input type='submit' name='login' value='Login'/>
    </form>";
}
?>

额外注意事项 #

• 确保session_start()在所有输出之前执行，你的代码已满足该要求。
• 检查DatabaseObject类的clean()方法是否能有效防止SQL注入，更安全的方式是使用预处理语句。
• 登录成功后可添加跳转逻辑（如header("Location: ./"); exit;），避免用户刷新页面重复提交表单。

内容的提问来源于stack exchange，提问作者Peter Cloutier-Jabalee