正在从函数中删除全局 - Removing global from function

Removing global from function

本文关键字：删除全局函数 | 更新日期: 2023-09-27

我对PDO非常陌生。我创建了下面的函数，但我读到拥有全局数据库（$dbo）是个坏主意。代码也能工作并输出我想要的东西。关于如何改进/修复，有什么建议吗？

function langString($lang_id) { 
    global $dbo;
    $lang_result=$dbo->prepare("SELECT lang_string FROM lang WHERE lang_id=:lang_id");
    $lang_result->bindParam(":lang_id",$lang_id,PDO::PARAM_INT,3);
    if($lang_result->execute()){ 
        $lang_row = $lang_result->fetch(PDO::FETCH_OBJ);
        echo "<br><br>$lang_row->lang_string";
    }
}
echo langString(3);

将变量作为参数传递给函数

function langString($lang_id, $dbo) {   
    $lang_result=$dbo->prepare("SELECT lang_string FROM lang WHERE lang_id=:lang_id");
    $lang_result->bindParam(":lang_id",$lang_id,PDO::PARAM_INT,3);
    if($lang_result->execute()){ 
        $lang_row = $lang_result->fetch(PDO::FETCH_OBJ);
        echo "<br><br>$lang_row->lang_string";
    }
}
echo langString(3, $dbo);

接受答案的问题是，您最终会将每个函数作为参数来跟踪数据库。这会导致参数列表变长，而且当你得到更多的东西时，就会开始看起来一团糟。

另一种方法是制作一个Singleton，你可以调用它来获得数据库连接。例如，通常大型项目会使用它来访问数据库，但这里有一个简单的例子

class DatabaseProvider {
     private static $database;
     private $dbo;
     private function __construct() {
         //Here's where you do your PDO connection creation
         $this->dbo= new PDO()....;
     }
     public static function getDatabase() {
         if(!isset(self::$database) {
               self::$database = new DatabaseProvider();
         }
         return $database->dbo;
     }
}

然后您只需调用DatabaseProvider::getInstance()，它将返回pdo对象

使用包含文件。。。

在include中放入变量，甚至创建一个db对象，然后可以从任何地方调用它。

function langString($lang_id) { 
    include($_SERVER['DOCUMENT_ROOT']."/dbconnect.php"); //Put your vars in this document
    $lang_result=$dbo->prepare("SELECT lang_string FROM lang WHERE lang_id=:lang_id");
    $lang_result->bindParam(":lang_id",$lang_id,PDO::PARAM_INT,3);
    if($lang_result->execute()){ 
        $lang_row = $lang_result->fetch(PDO::FETCH_OBJ);
        echo "<br><br>$lang_row->lang_string";
    }
}
echo langString(3);