2

OOP PHP5 コードを作成しようとしています。しかし、私の試みは不器用だと思います。これらは私の質問です：

  • データベース構成情報を含めるためのより良い、より無駄のない方法はありますか?
  • 作成するすべての関数で $db = new Db() を宣言しなければならないことをなんとか回避できますか?
  • Mysqli_database.php の代わりに PEAR をデータベース抽象化レイヤーとして使用する必要がありますか?

mysqli_database.php

<?php
class Db {
    private $connection;

    private function open_connection() {
        if (file_exists('config.inc.php')) {
            require('config.inc.php');
        } else {
            require('../config.inc.php');
        }
        try
        {
            $this->connection = mysqli_connect($dbhost,$dbuser,$dbpass,$dbname);
        }
        catch (Exception $e)
        {
            throw $e;
        }
    }
    private function close_connection() {
        try
        {
            mysqli_close($this->connection);
        }
        catch (Exception $e)
        {
            throw $e;
        }
    }
    public function query($query) {
        try
        {
            $this->open_connection();
            $result = mysqli_query($this->connection,$query);
            return $result;
        }
        catch (Exception $e)
        {
            throw $e;
        }
        $this->close_connection();
    }
    public function fetchArray($query) {
        $row = mysqli_fetch_assoc($query);
        return $row;
    }
    public function count_rows($query) {
        $row = mysqli_num_rows($query);
        return $row;
    }
    public function rows_affected() {
        $row = mysqli_affected_rows($this->connection);
        return $row;
    }
    public function created_id() {
        $row = mysqli_insert_id($this->connection);
        return $row;
    }
}
?>

Test_data.php

<?php
class Test_data {
    public function show_text() {
        $db = new Db();
        $sql = $db->query("SELECT * FROM test_table");
        $row = $db->fetchArray($sql);
        echo 'This is the output: '.$row['text'];
    }
}
?>

config.inc.php

<?php
$dbname     = 'database_name';
$dbhost     = 'localhost';
$dbuser     = 'database_user';
$dbpass     = 'database_password';
?>

includes.php

<?php
require_once('config.inc.php');
require_once('Mysqli_database.php');
$db = new Db();
$test_data = new Test_data();
?>

index.php

<?php
require_once('includes.php');
$test_data->show_text();
?>
4

3 に答える 3

3

構成情報は好みの問題です。ただし、構成オブジェクトに保存し、別のファイルからグローバル変数を含めるよりもオブジェクト指向の方法で取得する方がよいでしょう。

シングルトン パターンを使用して、新しいオブジェクトの作成を回避できます。

選択するレイヤーが抽象化されているほど、あるデータベースから別のデータベースへの移動が容易になります。PDOもご覧ください。

于 2009-08-17T16:46:58.463 に答える
3

データベース接続を確立するための非常に一般的なパターンがいくつかあります。Singleton、Factory、および場合によっては Registry です。

これがどのように見えるかです。

<?php

class DbConn
{
  const CONN_DEV_1  = 'dev.db1';
  const CONN_PROD_1 = 'prod.db1';
  const CONN_DEV_2  = 'dev.db2';
  const CONN_PROD_2 = 'prod.db2';

  protected static $instances = array();

  protected $conn;

  public static function factory( $database, $env )
  {
    $connectionName = "$env.$database";
    if ( !isset( self::$instances[$connectionName] ) )
    {
      switch ( $connectionName )
      {
        case self::CONN_DEV_1:
          $dbname = 'dev1';
          $dbhost = 'localhost';
          $dbuser = 'database_user';
          $dbpass = 'database_password';
          break;
        case self::CONN_PROD_1:
          $dbname = 'prod1';
          $dbhost = 'some.server';
          $dbuser = 'database_user';
          $dbpass = 'database_password';
          break;
        case self::CONN_DEV_2:
          $dbname = 'dev2';
          $dbhost = 'localhost';
          $dbuser = 'database_user';
          $dbpass = 'database_password';
          break;
        case self::CONN_PROD_2:
          $dbname = 'prod2';
          $dbhost = 'some.server';
          $dbuser = 'database_user';
          $dbpass = 'database_password';
          break;
        default:
          throw new Exception( 'Unrecognized database connection!' );
      }
      self::$instances[$connectionName] = new self( $dbhost,$dbuser,$dbpass,$dbname );
    }
    return self::$instances[$connectionName];
  }

  private function __construct( $dbhost, $dbuser, $dbpass, $dbname )
  {
    $this->conn = mysqli_connect( $dbhost, $dbuser, $dbpass, $dbname );
  }

  /* all your other methods here */
}

そして使用中 

$db1 = DbConn::factory( 'db1', 'dev' );

明らかに、ここでのポイントは$env、現在のアプリケーション構成からの値を取得することです。

使用法に関しては、通常、関数/クラスにデータベース接続を渡す/渡す必要があり、接続自体を確立する責任を負わせないようにします。これにより、結合が緩くなります。あなたの例を使用するには：

<?php
class Test_data
{
    protected $db;

    public function __construct( DbConn $db )
    {
        $this->db = $db;
    }

    public function show_text()
    {
        $sql = $this->db->query( "SELECT * FROM test_table" );
        $row = $this->db->fetchArray($sql);
        echo 'This is the output: '.$row['text'];
    }
}
?>
于 2009-08-17T16:57:00.923 に答える
2

各関数でインスタンスをキャプチャすることに関しては、このような DB オブジェクトは、Singleton パターンに与えられた事実上の例です。ここは本当によく似合います。

大まかな例を次に示します。

class DB
{

   // Private Constructor so external code cannot directly instantiate
   private function __construct() {  

   }

   public static function instance() {
     static $instance = false;

     if (!$instance)
       $instance = new DB();

     return $instance;

   }

}

また、(異なるデータベースに対して) 複数の DB 接続を開きたい場合の小さなバリエーションは、次のようなインスタンス メソッドになります。

public static function instance($dsn) {
   static $instances = array();

   if (!isset($instances[$dsn]))
       $instances[$dsn] = new DB($dsn);

   return $instances[$dsn];

}
于 2009-08-17T18:59:00.473 に答える