PHP：正确使用单例模式？

Question

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

这是一个合适的单身人士吗？此代码是否仅确保一个数据库连接？有什么办法可以测试吗？ （学会一个人钓鱼，他的余生都会有食物......）

我使用 redbean 作为我的 ORM，我是这样设置的：

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

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

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;
    }
}