PHP：数据库连接类构造方法

Question

我是 OOP 的新手。最初我是在类内和构造函数外定义变量并为它们赋值，但今天在 Java 的 OOP 课程之后，有人告诉我这是不好的风格，应该避免。

这是我模拟的原始 PHP 数据库连接类：

class DatabaseConnection {
    private $dbHost = "localhost";
    private $dbUser = "root";
    private $dbPass = "";
    private $dbName = "test";

    function __construct() {    
        $connection = mysql_connect($this->dbHost, $this->dbUser, $this->dbPass)
            or die("Could not connect to the database:<br />" . mysql_error());
        mysql_select_db($this->dbName, $connection) 
            or die("Database error:<br />" . mysql_error());
    }
}

以上认为可以吗？还是以下更好的方法？

class DatabaseConnection {
    private $dbHost;
    private $dbUser;
    private $dbPass;
    private $dbName;

    function __construct() {
        $this->dbHost = "localhost";
        $this->dbUser = "root";
        $this->dbPass = "";
        $this->dbName = "test";

        $connection = mysql_connect($this->dbHost, $this->dbUser, $this->dbPass)
            or die("Could not connect to the database:<br />" . mysql_error());
        mysql_select_db($this->dbName, $connection) 
            or die("Database error:<br />" . mysql_error());
    }
}

我应该关注什么以确保我正确理解 OOP？

Answer 1

首先：这是没有意义的。

您正在为 10 岁以上的 mysql_* 函数创建一个对象包装器。这个php扩展不再维护，弃用过程有already begun。您不应将此 API 用于 2012 年的任何新项目。

相反，您应该学习如何使用PDO 或MySQLi 并使用prepared statements。

也就是说..让我们看看你的代码：

• 构造函数应接收创建新实例所需的所有参数，参数不应在类定义中硬编码。如果您需要同时使用两个数据库怎么办？
• 创建连接时，应将其存储在对象的范围变量中。类似于$this->connection = mysql_conn... 的东西。相反，您将其存储在局部变量中，在构造函数完成后您“松散”。
• 您不应该对所有内容都使用private 变量。它们对扩展原始类的类不可见。除非是故意的，否则您应该为此选择protected。
• or die('..') 位最多。如果连接失败，请不要停止整个应用程序。相反，您应该抛出一个exception，然后可以在构造函数之外对其进行处理。

Answer 2

嗯，它还不会运行。您需要更改变量以使其与您的连接参数匹配：

$dbHost = "localhost";

应该是

$this->dbHost = 'localhost';

我通常根本不会将我的登录参数放在课程中。创建对象时，我会将它们传递给构造函数。使用外部配置文件，这样您就可以在多个构建中实际使用此类。 :)

更新::

好的，这里有一些小的 OOP 配置金块可以帮助您构建动态数据库类。

• 查看http://redbeanphp.com/ 它将允许您进行伪 ORM 风格的数据建模。超级容易安装，并且非常容易让您的数据库启动和运行。 http://redbeanphp.com/manual/installing

• 创建一个配置文件，其中包含常量、模板设置、常用函数和 AUTOLOADER 配置文件是在版本控制环境中工作的关键。 :)

• 将您的数据库类构建为抽象类http://php.net/manual/en/language.oop5.abstract.php

  abstract class Database
  {
    public function update()
    {
    }
  
    public function deactivate()
    {
    }
  
    public function destroy()
    {
    }
  
    //etc.
  }
  
  class MyAppObject extends Database
  {
  }
  

• 将所有类文件放入库文件夹中，然后将配置文件放入该库中。现在，为了让您的生活更轻松，您可以在需要时使用自动加载器功能使您的类栩栩如生，而无需包含任何特定的类。见下文：

  //note: this is never explicitly instantiated
  //note: name your files like this: MyAppObject.class.php  
  function my_fancypants_autoloader( $my_class_name )
  {
    if( preg_match( "%^_(Model_)%", $my_class_name ) ) return;
    require_once( "$my_class_name.class.php" );
  }
  spl_autoload_register( 'my_fancypants_autoloader' );
  

  • 现在您只需在 .php 文件中包含一个配置文件即可访问您的类。

希望这能为您指明正确的方向！祝你好运！

Answer 3

由于您只在__construct 方法中使用它们，因此您不需要它们作为类属性。只有$connection 必须保留以供以后使用恕我直言。

最好不要为这些参数提供任何默认值，而是让它们从外部设置。

$db = new DatabaseConnection("localhost", "user", "password", "db");

它们已经有大量的 PHP 工具，找到它们，阅读它们并从中学习。首先，使用PDO，Java 中正确的东西在 PHP 中并不总是正确的。

Answer 4

后者可能更好，但有一个调整：将一些参数传递给构造函数，即连接信息。

您的第一个示例仅在您有一个数据库连接并且您乐于对连接值进行硬编码（您不应该这样做）时才有用。第二个例子，如果你添加一个$name参数作为参数，可以用来连接多个数据库：

我是 OOP 的新手。最初我是在类内和构造函数外定义变量并为它们赋值，但今天在 Java 的 OOP 课程之后，有人告诉我这是不好的风格，应该避免。

class DatabaseConnection {
    private $dbHost;
    private $dbUser;
    private $dbPass;
    private $dbName;

    function __construct($config) {
        // Process the config file and dump the variables into $config
        $this->dbHost = $config['host'];
        $this->dbName = $config['name'];
        $this->dbUser = $config['user'];
        $this->dbPass = $config['pass'];

        $connection = mysql_connect($this->dbHost, $this->dbUser, $this->dbPass)
            or die("Could not connect to the database:<br />" . mysql_error());
        mysql_select_db($this->dbName, $connection) 
            or die("Database error:<br />" . mysql_error());
    }
}

所以使用这种风格，你现在有了一个更有用的类。

Answer 5

这是我的，效果很好：

class Database
{
   private static $_dbUser = 'user';
   private static $_dbPass = 'pwd';
   private static $_dbDB = 'dbname';
   private static $_dbHost = 'localhost';
   private static $_connection = NULL;

   /**
    * Constructor
    * prevents new Object creation
    */

   private function __construct(){
   }

   /**
    * Get Database connection
    * 
    * @return Mysqli
    */

   public static function getConnection() {
      if (!self::$_connection) {
     self::$_connection = @new mysqli(self::$_dbHost, self::$_dbUser, self::$_dbPass, self::$_dbDB);

         if (self::$_connection -> connect_error) {
            die('Connect Error: ' . self::$_connection->connect_error);
         }
      }
      return self::$_connection;
   }
}

通过将__construct 设为空，可以防止从任何地方实例化新类。然后，将函数设置为静态，所以现在我需要做的就是获得我的连接是Database::getConnection() 这一切都在include 文件中，在服务器上的受密码保护的文件夹中，并且只包含在每个类文件中。这还将在尝试另一个连接之前检查连接是否已经打开。如果已经建立了一个，则将打开的连接传递给该方法。

Answer 6

<?php

    class config
    {
        private $host='localhost';
        private $username='root';
        private $password='';
        private $dbname='khandla';

        function __construct()
        {   
            if(mysql_connect($this->host,$this->username,$this->password))
            {
                echo "connection successfully";
            }
        }
        function db()
        {
            mysql_select_db($this->$dbname);
        }
    }

    $obj=new config();
?>