跨PHP文件调用自定义函数时遇数据库连接变量未定义错误求助

嘿，这个问题我之前也碰到过，本质是PHP变量作用域的锅！

你看，当你在functions.php里直接调用get_option_list时，$db是在当前文件的全局作用域里，函数能直接拿到；但当你在my_view.php里引入functions.php再调用函数时，函数内部默认只能访问自己的局部变量，找不到全局的$db，所以就出现了那些“未定义变量”和“参数为null”的错误。

下面给你几个可行的解决办法，按推荐程度排序：

方法1：把$db作为参数传入函数（最推荐） #

这种方式让函数不依赖全局状态，代码更干净、易维护：

先修改functions.php里的函数：

<?php 
include_once("db_connect.php"); 

### 生成省市县下拉选项的函数 ### 
function get_option_list($db, $table, $col_id, $col_value, $sel=0){ 
    $SQL = "SELECT * FROM $table"; 
    // 注意mysqli_error要传入$db连接对象，之前你漏了这个参数
    $rs = mysqli_query($db,$SQL) or die(mysqli_error($db)); 
    $option_list = "<option value='0'>Please select...</option>"; 
    while($data = mysqli_fetch_assoc($rs)){ 
        $option_list.= "<option value='$data[$col_id]'>$data[$col_value] </option>"; 
    } 
    return $option_list; 
} 

// 内部调用时也要记得传入$db哦
//echo get_option_list($db, "city","city_id","city_name"); 
//echo get_option_list($db, "state","state_id","state_name"); 
//echo get_option_list($db, "country","country_id","country_name"); 
?>

然后修改my_view.php的调用代码：

<?php 
include_once("functions.php"); 
?> 
<form> 
    <select> 
        <?php echo get_option_list($db, "city","city_id","city_name"); ?> 
    </select> 
</form>

方法2：在函数里声明$db为全局变量（简单但不推荐） #

这种方式快速解决问题，但会让函数绑定全局状态，后续扩展或复用函数时容易出问题：

修改functions.php的函数：

function get_option_list($table,$col_id,$col_value,$sel=0){ 
    global $db; // 告诉函数要使用全局作用域的$db变量
    $SQL = "SELECT * FROM $table"; 
    $rs = mysqli_query($db,$SQL) or die(mysqli_error($db)); 
    // 剩下的代码和原来一样...
}

方法3：封装数据库连接为单例类（长期优雅方案） #

如果你的项目后续会变大，建议把数据库连接封装成类，避免到处传递$db：

新建Db.php文件：

<?php
class Db {
    // 单例实例，确保整个项目只有一个数据库连接
    private static $instance;

    // 私有构造方法，防止外部new这个类
    private function __construct() {}

    // 获取数据库连接实例的方法
    public static function getInstance() {
        if (!self::$instance) {
            self::$instance = mysqli_connect("localhost","root","","sms1") or die(mysqli_connect_error());
        }
        return self::$instance;
    }
}
?>

然后修改functions.php：

<?php 
include_once("Db.php"); 

function get_option_list($table,$col_id,$col_value,$sel=0){ 
    // 获取数据库连接实例
    $db = Db::getInstance();
    $SQL = "SELECT * FROM $table"; 
    $rs = mysqli_query($db,$SQL) or die(mysqli_error($db)); 
    // 剩下的代码不变...
} 
?>

这样不管在哪个文件调用函数，都不用操心$db的问题了～

哦对了，你原来的mysqli_error()调用少传了$db参数，上面的所有方案里都已经修正了这个小问题，不然即使解决了作用域问题，还是会报错的。

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