PHP报错Notice: Undefined variable: login 代码问题求助

这个报错的核心原因是PHP的变量作用域规则：你在全局作用域定义的$login数据库连接变量，默认无法在自定义函数proses_hoby()内部访问到，函数会把它当成一个未定义的局部变量。

下面给你两种实用的解决方法，从简单到规范排序：

方法1：使用global关键字声明全局变量 #

在函数内部开头，用global关键字告诉PHP要使用外部的全局$login变量，这样函数就能正常调用数据库连接了：

<?php 
$host = "localhost"; 
$user = "root"; 
$pass = ""; 
$dabname = "tutorial"; 
// 初始化数据库连接（注意：mysqli_connect第四个参数已经指定了数据库，下面的mysqli_select_db可以省略）
$login = mysqli_connect($host, $user, $pass, $dabname) or die('Could not connect to mysql server.');
// 移除多余的mysqli_select_db调用
// mysqli_select_db($login, $dabname) or die('Could not select database.');

$nama_karyawan = $_POST['nama_karyawan']; 
$umur_karyawan = $_POST['umur_karyawan']; 

function proses_hoby($id_karyawan) {
    // 声明要使用全局的$login变量
    global $login;
    if (isset($_POST["rincian_hoby"])) { 
        $hoby = $_POST["rincian_hoby"]; 
        reset($hoby); 
        while (list($key, $value) = each($hoby)) {
            // 这里就可以正常使用$login执行数据库操作了，比如插入数据
            // 示例：$query = "INSERT INTO hoby (id_karyawan, hoby) VALUES ('$id_karyawan', '$value')";
            // mysqli_query($login, $query);
        }
    }
}

⚠️ 注意：这种方法虽然简单，但在大型项目中不推荐，全局变量容易导致代码耦合度高，难以维护。

方法2：将数据库连接作为参数传入函数（推荐） #

这是更符合现代编程规范的做法，把$login作为参数传递给proses_hoby函数，明确依赖关系，代码更清晰：

<?php 
$host = "localhost"; 
$user = "root"; 
$pass = ""; 
$dabname = "tutorial"; 
$login = mysqli_connect($host, $user, $pass, $dabname) or die('Could not connect to mysql server.');

$nama_karyawan = $_POST['nama_karyawan']; 
$umur_karyawan = $_POST['umur_karyawan']; 

// 修改函数，添加$login参数
function proses_hoby($id_karyawan, $login) {
    if (isset($_POST["rincian_hoby"])) { 
        $hoby = $_POST["rincian_hoby"]; 
        reset($hoby); 
        while (list($key, $value) = each($hoby)) {
            // 正常使用传入的$login连接
            // 示例：$query = "INSERT INTO hoby (id_karyawan, hoby) VALUES ('$id_karyawan', '$value')";
            // mysqli_query($login, $query);
        }
    }
}

// 调用函数时传入$login
// proses_hoby($some_id, $login);

这种方法的优势是函数的依赖关系明确，避免了全局变量带来的潜在问题，也更容易调试和维护。

另外补充两个小优化点：

1. 你代码中的mysqli_select_db($login, $dabname)是多余的，因为mysqli_connect的第四个参数已经指定了要连接的数据库，不需要再次调用选择。
2. 注意处理SQL注入风险，建议使用预处理语句（mysqli_prepare）来执行数据库操作，避免直接把用户输入拼接到SQL语句中。

内容的提问来源于stack exchange，提问作者Tehnik Nusantara Jakarta