PDOを再利用する記述について

Question

失礼いたします。

PHPで開発をしており、今まで以下のようにPDOを使用しておりました。

class db {
  public static function dbc {
    $pdo = new PDO(....);
    return $pdo;
  }
}

例えば、viewに関することで、DBに接続する際には上のクラスを呼び出している形をそのたびクラス内でとっていました。
例えば(セキュリティチェック等無視します)、

class view {
  public function row {
   $db = db::dbc();
   $db->prepare('何らかのSQL');
   $db->bindValue(...);
   $db->execute();
  }
}

という感じです。
が、最近になって、例えばview以外のクラスまたはメソッドで、他にPDOを連続して使用する場合、何度も新規に接続するのは負荷が増えるのでは、と思うようになりました。
そこでPDOの再利用について調べると、stackoverflowに
・global, または$GLOBALSを使う（あくまで非推奨）
・__constructでクラスの呼び出し時に設定する
が見つかりました。
グローバルは極力使わない方が良いと思いますので、呼び出し時に設定する方が良いのかなとは思うのですが、これも本当に望んでいる形ではないのかなと思いました。

class view {
  private $db;
  public function __construct($pdo){
    $this->db = $pdo;
  }
  public function row{
    $db = $this->db;
    ....後は上記のコードと同じ
  }
}

ということになるのですが、本当はグローバルのような形が最も望ましいです。
引数に継ぐ形ではなく、class内、メソッド内で終わらせることができたらと思います。
__constructを他の機能に使っている場合、引数が$pdoのぶん2つになってしまいますので。
方法がない場合は、大人しく__constructを使おうと思っております。

皆さまが他に使用なさっている記述がございましたら、ご教授いただけませんでしょうか。
何卒、よろしくお願いいたします。

PHPの使用歴は1年程度です。

Answer 1

簡易なものであれば、こんな感じのシングルトンもどきを使ったりもします。

<?php

/**
 * ./classes/DB.php
 *
 * @since 2018/07/10
 */
class DB
{

	/**
	 * PDO Object
	 * @var PDO
	 */
	private static $instance = null;

	/**
	 * インスタンス取得
	 * @return PDO
	 */
	private static function getInstance()
	{
		if (!is_null(self::$instance)) {
			return self::$instance;
		}
		$dsn = "mysql:host=127.0.0.1;dbname=sample;charset=utf8mb4";
		$username = 'root';
		$password = 'password';
		$options = [
			PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION
			, PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC
		];
		self::$instance = new PDO($dsn, $username, $password, $options);
		return self::$instance;
	}

	/**
	 * select
	 * @param string $sql
	 * @param array $params
	 * @return array Recordset
	 */
	public static function select($sql, array $params = [])
	{
		$stmt = self::getInstance()->prepare($sql);
		$stmt->execute($params);
		return $stmt->fetchAll();
	}

}

Comments

[Newbi]

お忙しい中、ご回答ありがとうございます！

お見せいただいたコードを実行しましたら求めていた結果になりました！
また、実はKosuke_Shibuyaさまのコードと同じような形で、
class db {
private $pdo = false;
public static function dbc(){
if ( $this->pdo !== false )
return $this->pdo;

$pdo = new PDO(...);
$this->pdo = $pdo;
return $pdo;
}
}
のような感じで一度書いたのですが、
「 Using $this when not in object context 」というエラーが出て、出来ないのかと思いながら、調べている最中でした。
Kosuke_Shibuyaさまのコードにprivate staticとあったので、同じようにstaticで書いたら変数にアクセスできて驚きました。まだ、public, private, protected, staticなどをきちんと理解できてないのだなと思いました。
staticでアクセスした場合、書き換えた変数が初期化されないことにも驚きました。単に記述が省略できるだけかと思っていたので、勉強になりました。

改めまして、ご回答をありがとうございました！
今後も何卒、よろしくお願いいたします。

[退会済みユーザー]

PDOオブジェクトを使いまわさないと、トランザクション処理などをやろうとしたとき、lastInsertId を取得するときなど、困る場面がいっぱいありますので、「シングルトン」で実装するというのはある意味定番です。GoF デザインパターン といったキーワードで調べてみるといいでしょう。

[Newbi]

コメントに気が付きませんでした！
すみません。
デザインパターン、よく調べたいと思います。abstractもinterfaceもよく分かっていないため、勉強になります。
アドバイスいただきありがとうございました！

Answer 2

要件を満たす回答ではないです。ただ、私の意識を大きく変えた良い資料だったので、参考になれば^^

PHP7 で堅牢なコードを書く - 例外処理、表明プログラミング、契約による設計 / PHP Conference 2016

この資料は何度か改稿がされており、動画もあったはずです。
興味があれば、探してみると良いです。

Comments

[Newbi]

ご回答ありがとうございます！
スライド読ませていただきました。ちょうど例外処理に苦戦していたので、ありがたいです。。
チームで開発したことがないので、assertのメリットを活かしきれないとは思いますが、今後機会がありましたら活用させていただきたいと思います。
interfaceやabstractの役割をようやく掴みかけながら、いまは開発中のシステムでtry{}catch{}を少しでも減らせないかと苦戦しております苦笑
最初の設計が大事だなと痛感しました。
今後とも、ぜひよろしくお願いいたします。

[Newbi]

いや、すみません。
assertは個人開発でも便利ですね。一度読んで他の人が分かりやすいよう明示するものかと思ったのですが、評価式を示してくれて設定をすればその段階で落としてくれるので分かりやすいですね。エラー処理を目的に使うものではないですが、万が一に備えられるのはありがたいです。チーム開発ならなお分かりやすいかもしれません。
今後取り入れてまいりたいと思います。
ありがとうございます！