PHP：正确使用单例模式吗？

Question

我尝试学习 PHP OOP，并对如何创建一个全局数据库类在我的项目中使用进行了一些研究。据我所知，最合适的可用模式是单例，它可以确保始终只存在一个数据库连接。然而，由于这是我第一次使用单例模式，我不确定我是否做对了。

这是一个合适的单身人士吗？这段代码只能确保一个数据库连接吗？有什么方法可以测试这个吗？ （授人以渔，终生有食……）

我使用红豆作为我的 ORM，以下是我的简单设置方式：

require_once PLUGINPATH.'rb.php';
$redbean= R::setup("mysql:host=192.168.1.1;dbname=myDatabase",'username','password');

我根据此源制作了以下脚本，作为我自己的单例数据库类；

class database {
    private $connection = null;

    private function __construct(){
        require_once PLUGINPATH.'rb.php';
        $this->connection = R::setup("mysql:host=192.168.1.1;dbname=myDatabase",'username','password');
    }

    public static function get() {
        static $db = null;

        if ( $db === null )
        $db = new database();

        return $db;
    }

    public function connection() {
        return $this->connection;
    }
}

谢谢！

Answer 1

实例变量必须是类的静态成员。我还没有测试过这段代码，但它应该可以工作。连接可能也应该是静态的。使用此代码，您将拥有数据库类的一个实例和连接的一个实例。可以在连接不静态的情况下执行此操作，但使其静态将确保只有一个连接实例。我还将类的字面名称更改为 php 魔术常量。这使您的代码更易于维护。如果您以后更改类的名称，则不必在代码中查找旧类名称的所有文字实例。现在看来这似乎有点矫枉过正，但相信我，当你从事越来越复杂的项目时，你会欣赏它的价值。

class database {
    private static $connection = null;
    private static $db;

    private function __construct(){
        require_once PLUGINPATH.'rb.php';
        self::$connection = R::setup("mysql:host=192.168.1.1;dbname=myDatabase",'username','password');
    }

    public static function get() {
        if ( !(self::$db instanceof __CLASS__) ) {
            $klass = __CLASS__; // have to set this to a var, cant use the constant with "new"
            self::$db = new $klass();
        }

        return self::$db;
        }

    public function connection() {
        return self::$connection;
    }
}

Answer 2

你是单身人士几乎是正确的。

私有成员（没有双关语）$connection 也需要是静态的。您也可以选择以下内容：

class database {
    private static $instance = NULL;
    protected $conn;

    private function __construct() {
        $this->conn = mysql_connect( ... );
    }

    public static function init() {
        if (NULL !== self::$instance) 
            throw new SingletonException();
        self::$instance = new database();
    }

    public static function get_handle() {
        if (NULL === self::$instance)
            ; // error handling here
        return self::$instance->conn;
    }
}