在php中制作mysql单例数据库类的任何更简单/更好的方法？

Question

这是我正在使用的：

<?php
final class Database {
    private static $oDb;
    public static function init() {
        if(self::$oDb == NULL)
        {
            self::$oDb = mysql_connect('localhost', 'mysql_user', 'mysql_password') or die(mysql_error());
            mysql_select_db('mysql_db_name', self::$oDb) or die (mysql_error());;
        }
        return self::$oDb;
    }
    public function query($sql)
    {
        return mysql_query($sql) or die(mysql_error());
    }
}
?>

用法：

$oDb = Database::init();
$sql = foo;
$oDb->query($sql);

假设我只希望它连接并执行这一个查询功能，我应该对这个类进行任何改进吗？内存还是代码效率？

另外，有没有一种有效的方法可以从配置文件中获取数据库凭据？我知道我不能在课堂上使用包含。

Answer 1

我通常在这种情况下使用惰性初始化，并且只有一个公共方法（在这种情况下），并带有一个私有构造函数来防止外部实例化（根据单例模式）：

class Database {
  private static $instance;
  private $conn;

  private function Database() {
    // do init stuff
    require_once('dbconfig.php'); // contains define('DB_USER', 'webuser'); etc...
    $this->conn = mysql_connect(DB_HOST, DB_USER, DB_PASS); // do error checking
  }

  public static function getInstance() {
    if(!self::$instance) {
      self::$instance = new Database();
    }
    return self::$instance;
  }

  public static function query($sql) {
    $instance = self::getInstance();
    return mysql_query($sql, $instance->conn);
  }
}

然后您可以在需要使用它的任何时候致电$dbHandle = Database::getInstance()。或者在这种情况下，由于定义了静态查询方法，因此您可以使用 Database::query("select * from xx;"); 而无需调用任何类型的 init。

Answer 2

这很简单，可以正常工作。

您可以将您的凭据传递给 init();

include(config.php);
$oDb = Database::init( DB_HOST, DB_NAME, DB_USER, DB_PASSWORD );
$sql = foo;
$oDb->query($sql);

Answer 3

您可以在类中的函数内使用包含

<?php
final class Database {
    private static $oDb;
    public static function init() {
        if(self::$oDb == NULL)
        {
            include('config.php')
            self::$oDb = mysql_connect(DB_HOST, DB_USER, DB_PASS) or die(mysql_error());
            mysql_select_db(DB_NAME, self::$oDb) or die (mysql_error());;
        }
        return self::$oDb;
    }
    public function query($sql)
    {
        return mysql_query($sql) or die(mysql_error());
    }
}
?>

或者你可以只传递变量...

<?php
final class Database {
    private static $oDb;
    public static function init($host, $user, $pass, $name) {
        if(self::$oDb == NULL)
        {
            self::$oDb = mysql_connect($host,$user,$pass) or die(mysql_error());
            mysql_select_db($name, self::$oDb) or die (mysql_error());;
        }
        return self::$oDb;
    }
    public function query($sql)
    {
        return mysql_query($sql) or die(mysql_error());
    }
}
?>

或者您可以将凭据存储在 php.ini 文件中

<?php
final class Database {
    private static $oDb;
    public static function init($db_name) {
        if(self::$oDb == NULL)
        {
            self::$oDb = mysql_connect() or die(mysql_error());
            mysql_select_db($db_name, self::$oDb) or die (mysql_error());;
        }
        return self::$oDb;
    }
    public function query($sql)
    {
        return mysql_query($sql) or die(mysql_error());
    }
}
?>

php.ini 文件：

mysql.default_host="host"
mysql.default_user="user"
mysql.default_password="password"

Answer 4

对于单例类，Dan Breen 遵循的模型是最干净且非常常见的。但是，在这种情况下，我还允许 getInstance 方法接受一些参数，以便您可以在实例化时覆盖默认配置，或者只获取引用而不创建连接（这两种用例都会不时发生)。

数据库.php

require_once("../path/to/config/database.php");

class Database {
  private static $instances = array();

  private function Database($host, $user, $password, $name) {
    // do init stuff
  }

  public static getInstance(
    $host=DB_HOST, $user=DB_USER, $password=DB_PASSWORD, $name=DB_NAME
  ) {
    $key = strtolower($host . $user . $password . $name);

    if ( !$self::instances[$key] ) {
      $self::instances[$key] = new Database($host, $user, $password, $name);
    }
    return $self::instances[$key];
  }
}

..config/database.php:

define("DB_HOST", "localhost");
define("DB_USER", "mrsqlguy");
define("DB_PASS", "!!!");
define("DB_NAME", "just_another_wordpress");

编辑：我已将其更改为更像一个享元，以确保您只为每个连接位置/数据库获取一个实例。这解决了您的顾虑并保持了一定程度的灵活性。