PDOによるDB接続 - オブジェクト指向で書けません

Question

phpでオブジェクト指向の勉強をしています。試しにPDOによるDB接続をクラスで分け、可読性をあげたいと思っています。やりたいこととしては下記のように二つのクラスを定義します。

• dbに接続するクラス
• 上記を継承したdbのデータを取得するクラス

下記のように二つのクラスを定義して実行してみても、クエリが空のままとなってしまいます。子クラスでは親クラスを継承しているので、親クラスの実行の結果を引き継げると思ったのですが、正しくないのでしょうか。

アドバイスを頂ければ幸いです。

<?php

class Database
{   
    public function connect()
    {
      $user = "root";
      $pass = "user";
      $dbh = new PDO('mysql:host=localhost;dbname=db', $user, $pass);
    }
}


class DataSelect extends Database
{
    public function while_data()
    {
        $stmt = $dbh->query("SELECT * FROM tb");
        while ($row = $stmt->fetch()) 
        {
        $co1[] = $row['co1'];
        $co2[] = $row['co2'];
        }
        print_r($co1);
        print_r($co2);
    }
}

$Database = new Database();
$DataSelect = new DataSelect();

$Database->connect(); 
$DataSelect->while_data(); 

?>

Comments

[sazi]

丸投げでもないですし、分からないことを質問している分には、低評価になる理由がわからないので、相殺で高評価にしました。

[keys]

あわわわ... ご親切にどうもありがとうございます^^

Answer 1

いろいろ誤解している点があります。

• まず、メソッド内の$abcのような変数は、PHPでは（global宣言をした状況でなければ）ローカル変数です。オブジェクトに紐付いたインスタンス変数は、$this->abcのように読み書きする必要があります。
• $Databaseと$DataSelectは別個のオブジェクトなので、（きちんと$this->dbhに書き換えたとしても）$DataSelect->dbhは初期化されません。$DataSelect->connect()を呼ぶ必要があります。

また、Databaseクラスが「接続しないと使い物にならない」という前提であれば、コンストラクタ（リファレンス）を使うことで、インスタンス作成と接続を同時に行うことが可能となります。

Comments

[keys]

>また、Databaseクラスが「接続しないと使い物にならない」という前提であれば、コンストラクタ（リファレンス）を使うことで、インスタンス作成と接続を同時に行うことが可能となります。

なるほど...勉強になります。

Answer 2

質問に対しての直接的な回答ではないです

phpでオブジェクト指向の勉強をしています。試しにPDOによるDB接続をクラスで分け、可読性をあげたいと思っています。

この部分を見て、紹介したいスライドと動画があるので回答します。

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

[phpconfuk2017] PHP7で堅牢なコードを書く - 例外処理、表明プログラミング、契約による設計 和田 卓人

1ヶ月毎、3回ほど見ると、すげぇレベルアップした気になれます。

Comments

[keys]

ありがとうございます。拝見いたします

[退会済みユーザー]

迷走しているみたいですけど、スライドと講演、先に見たほうが良いですよ。

そもそもクラスのネーミングとして、`DataSelect`　ってのが良くないです。
参考：https://qiita.com/KeithYokoma/items/ee21fec6a3ebb5d1e9a8

命名を正しくすると、自然とそのクラスの責務を気にすることになり、スライドの内容が腑に落ちると思います。
まぁ、腑に落ちるまで、私は何回も読む必要がありましたが＾＾；

スライドの中に、このコードの行き着く先の理想形も書かれているので、講演の中身が理解できると、このコードはきれいに最終形にリファクタリング出来ます。

頑張ってください＾＾

Answer 3

extendは好き嫌いがでるので、extendせずに普通にclassを分けてもよいかと

<?PHP
class Database
{
  function getDBH()
    {
      try{
        $dsn = 'mysql:host=localhost; dbname=test1;charset=utf8;';
        $user = 'root';
        $password = 'root';
        $dbh = new PDO($dsn, $user,$password);
        $dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); 
        $dbh->setAttribute(PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_ASSOC);
        $dbh->setAttribute(PDO::ATTR_EMULATE_PREPARES,false);
        return $dbh;
      }catch(PDOException $e){
        die($e->getMessage());
      }
    }
}

class DataSelect
{
  function __construct()
    {
      $db=new Database;
      $this->dbh=$db->getDBH();
    }
  function test()
    {
      try{
        $sql="SELECT * FROM tb";
        $stmt = $this->dbh->prepare($sql);
        $stmt->execute([]);
        $rows=$stmt->fetchAll(PDO::FETCH_ASSOC);
        print_r($rows);
      }catch(PDOException $e){
        die($e->getMessage());
      }
    }
}
$myExec=new DataSelect;
$myExec->test();

Comments

[keys]

回答ありがとうございます。extendをした方が関係性が浮かび上がり易く、可読性が向上するかなと思ったのですが、どうなのでしょう。あと、try/catchって予期せぬエラーには便利だと思うのですが、いかんせん後で読んだ時に、邪魔っていうか... これ本当にあって意味あるかなって思ってしまいます。podの部分で予期せぬエラーが発生したことがありません。どう思いますか？

[keys]

constructは、毎回初期化されるという点であった方がいいと思うのですが、重要なのでしょうか。。。すみません。勉強不足で

[keys]

コードの部分ではく、コンストラクターというものについてです...

Answer 4

幾つかヒントを。

• 継承しているのでDatabase()クラスはnewしなくても良いです。

　つまりDataSelect()をnewすれば$DataSelect->connect()のような形で親クラスのメソッドを利用できます（※ただし親のprivateメソッドは子から利用できません）。
（参考：PHPマニュアル：継承　最初のサンプルプログラムでbarクラスからfooクラスのメソッドを利用している）

• 継承しているからと変数のスコープが延びるわけではないです。

　つまり子クラスでconnect()を実行する際に$dbhを返却して受け取るか、親クラスにpublicとかでもっておく必要があります。（参考：PHPマニュアル：プロパティ）

Comments

[keys]

＞継承しているからと変数のスコープが延びるわけではないです。
つまり子クラスでconnect()を実行する際に$dbhを返却して受け取るか、親クラスにpublicとかでもっておく必要があります。

おおおお、そうだったのですね。ありがとうございます。理解が深まりました。$dbhを返却して受け取れるようにします。

[m.ts10806]

返却するようにするより、publicな変数を親クラスに持っておいてそちらに保管し、
子クラスではその変数を参照するようにしたほうがオブジェクト指向っぽいです。
返却するだけだとただの関数とかわりません。

[keys]

お返事ありがとうございます。つまり、publicな変数というのは、staticで変数を作れば、グローバル変数のように複数のPHPファイルのどこからでも、クラス名さえ指定すれば使用可能ですということでしょうか。

http://uxmilk.jp/26442

[keys]

クラス変数とpublicな変数は違うものでしょうか

[m.ts10806]

>つまり、publicな変数というのは、staticで変数を作れば、グローバル変数のように複数のPHPファイルのどこからでも、クラス名さえ指定すれば使用可能ですということでしょうか。

確かに仰るとおりですが、「どこからでも使用可能としていいのか」という観点から考えてみましょう。
staticとすることで確かにnewすることなくどこからでも利用可能となりますが。

class a{
static public $test = "a";
}
echo a::test;

今回保管しようとしている情報は「接続情報」であるため、
「データベースに関する処理をするときだけ呼び出されるべき」と考えられます。
どこからでも関係なく呼び出されたら困るものです。
なので、staticは外して単に　public　とするべき、と思います。
また、newしても使わせない、あくまでDataBase関連のみだ　とするのであれば、
protected としておいても良いかと思います。

参考：http://php.net/manual/ja/language.oop5.visibility.php

> クラス変数とpublicな変数は違うものでしょうか

PHPにおいてはクラスで定義される変数を「プロパティ」と呼びます。
http://php.net/manual/ja/language.oop5.properties.php

クラス変数は「プロパティそのもの」で　public変数は「public指定がされたプロパティ」と言うべきでしょうか。
カテゴリと更に明細　といったレベルの違いはあります。

[m.ts10806]

これは仕様というかルールということになるのですが、
「このクラスを使う際には必ずDB接続が発生する」のであれば、
maisumakunさんの仰るように子クラスにコンストラクタを作っておいて親クラスの接続処理を呼び出しておくと楽かもしれません。
（子クラスのnew時には親クラスのコンストラクタは実行されません。）

[m.ts10806]

いっぱい情報があるので混乱しやすくなっているかもしれませんが、
一気に全てやろうとするのではなく、1つ1つ動作を確かめて「できる」「できない」を身につけていってください。
おそらくすべてやろうとするとチグハグなコードとなり「できない」の方が積み重なってしまい、悪循環に陥ってしまうので。

[keys]

お返事ありがとうございます。変数をstatic public $dbh のようにして、呼び出しの部分では、$stmt = Database::$dbh->query のようにしたのですが、どういうわけかPHP Parse error: syntax error, unexpected 'public' (T_PUBLIC), expecting :: (T_PAAMAYIM_NEKUDOTAYIM) inというエラーが発生してしまいます

[keys]

global $user　global $pass　global $dbhのようにすると、すると謎の syntax error, unexpected '=', expecting ',' or ';' in /という風に出ます

[m.ts10806]

コードがどうなってるか分からないですが、
プロパティを呼び出すときは
{オブジェクト}->dbh
のように頭の$は省略します。

そのエラーだと定義している場所にも問題があるように見えます。
現在のコード全体を質問本文に提示いただけますか？

シンタックスエラーは文法ミスによるものなので、エラーメッセージそのまま検索で結構解決策が出てきます。
また、エラーチェックを自動で行ってくれるIDE(Eclipseのような統合開発環境の高機能エディタ）を利用すると実行前にエラーがわかるので、活用を強く勧めます。
（あとglobalの使い方も何か怪しいです。）