php singleton数据库连接,这个代码是不好的做法？

Question

我正在尝试创建一个简单的使用单例类连接到mysql数据库并进行查询,代码工作正常,我没有遇到任何问题,但因为我是OOP的新手,我想知道这是否是不好的做法.

这是班级

class Database {
private $databaseName = 'dbname';
private $host = 'localhost';
private $user = 'user';
private $password = 'pass'; 
private static $instance; //store the single instance of the database

private function __construct(){
    //This will load only once regardless of how many times the class is called
    $connection = mysql_connect($this->host, $this->user, $this->password) or die (mysql_error());
    $db = mysql_select_db($this->databaseName, $connection) or die(mysql_error()); 
    echo 'DB initiated<br>';
}

//this function makes sure there's only 1 instance of the Database class
public static function getInstance(){
    if(!self::$instance){
        self::$instance = new Database();
    }
    return self::$instance;     
}

public function connect() { 
    //db connection
} 
public function query($query) {
    //queries   
    $sql = mysql_query($query) or die(mysql_error()); 
    return $sql;
}

public function numrows($query) {
    //count number of rows  
    $sql = $this->query($query);
    return mysql_num_rows($sql);
}


}

//Intantiate the class
$database = Database::getInstance();

当我想使用我会做的课时:

$query = "SELECT * FROM registrations";
echo $database->numrows($query);
$sql = $database->query($query);

Answer 1

单身人士是坏消息.

• 他们将全球国家引入计划.大多数程序员应该熟悉全球状态不好的原因.
• 它们引入了单例与使用它的任何类之间的紧密耦合.这意味着你不能重用有问题的类而不重用单例.
• 他们对依赖单例问题的类进行单元测试,因为你不能轻易地用mock替换单例.
• 它们鼓励编写样式,类尝试解析自己的依赖项.这很糟糕,因为它可以降低关于类具有哪些依赖关系的清晰度.
• PHP有一个Share Nothing Architecture,这意味着PHP单例根本不是真正的单例,任何时候都可以存在多个实例(每个打开请求一个).
• 如果您在以后突然发现您实际上需要多个单身人士提供的资源,会发生什么？这是比你想象的更常见的场景

你最好再看看依赖注入,因为它解决了上述问题.