0

我对 PHP 非常熟悉，但我刚刚开始用它做一些面向对象的东西。我想建立一个单例数据库连接，但遇到了问题和错误。“致命错误：允许的内存大小为 134217728 字节用尽（试图分配 523800 字节）”我知道我不应该用完内存我正在运行的查询是

$con = getConnection();
$stmt = $con->prepare("SELECT gene_name,jgi_protein_id FROM   jgi_transcriptid_proteinid_match where our_protein_id = ?");

这是该类的代码。

class Connection
{
    // Store the single instance of connection 
    private static $connection;

    private function __construct()
    {
        $connection = new mysqli(HOSTNAME, DBUSER, PASSWORD, DBNAME);

        if ($connection->connect_errno)
            die("Failed to connect to MySQL: (" . $connection->connect_errno . ") " . $connection->connect_error);
    }

    public static function getInstance() 
    { 
        if (!self::$connection) 
            self::$connection = new Connection(); 

        return self::$connection; 
    }

    public function prepare($query) 
    {
        $statement = $this->prepare($query);
        return $statement; 
    }
}

我正在使用 mysqli 作为数据库的东西。

4

1 回答 1

5

这段代码有几个问题：

无限递归 

public function prepare($query) 
{
  $statement = $this->prepare($query);
  return $statement; 
}

引用局部变量，而不是静态变量
代码可能应该引用self::$connection. 但是，基于课程，我不确定，因为self::$connection在 中以不同的方式使用getInstance()，我看不到任何地方都被称为。

private function __construct()
    {
        $connection = new mysqli(HOSTNAME, DBUSER, PASSWORD, DBNAME);

        if ($connection->connect_errno)
            die("Failed to connect to MySQL: (" . $connection->connect_errno . ") " . $connection->connect_error);
    }

命名混淆
类名为Connection，它包含一个名为 的静态变量$connection，该变量存储该类的单例实例。构造函数包含另一个$connection，而是一个mysqli连接。

重构类 - 未经测试的修复
下面的类未经测试，仅用于说明目的。使用风险自负。

class Connection { // 存储类的单个实例 private static $instance; // 存储 mysqli 连接 private $connection;

public function __construct() {
    // NOTE: it would be better to pass connection parameters as arguments
    $this->connection = new mysqli(HOSTNAME, DBUSER, PASSWORD, DBNAME);

    if ($this->connection->connect_errno)
        die("Failed to connect to MySQL: (" . $this->connection->connect_errno . ") " . $this->connection->connect_error);
}

public static function getInstance() { 
    if(empty(self::$instance)) {
        self::$instance = new Connection(); 
    }

    return self::$instance; 
}

public function prepare($query) {
    $statement = $this->connection->prepare($query);
    return $statement; 
}

}

于 2012-11-30T18:34:45.013 回答