php - インスタンス変数とthisの利用方法

Question

追記

class Test
{

  public $hoge = "cccc";


    static public function getNew()
    {
      echo "aaaaa/";
      echo $this->hoge;    
    }
}

class Child extends Test
{
}

Child::getNew();

謎

上記のコードを実行すると、aaaaa/とだけ表示されます。何故、ccccは表示されないのでしょうか

やりたいこと

phpでオブジェクト指向を実践しています。PDOのdb接続と、selectをクラスで分割し、オブジェクト思考なコードを書く際、疑問が湧いたので質問します。書くの全体のコードは極めて正しく動くのですが。。

#疑問点

下記全体のコードの一部分、コンストラクトがありますが、$this->dbhというのはクラスDatabaseで作成した変数ですよね？newでDatabaseクラスのインスタンスを作って仕舞えば、インスタンス変数の$dbを利用せずとも、$thisが使え$this->dbhという風にclass Databaseの実装を利用することが可能なのでしょうか。

  function __construct()
    {
      $db=new Database;
      $this->dbh=$db->getDBH();
    }

実装

<?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();

Answer 1

$this　は自身のオブジェクトをさします。今回のコードではDataSelectクラスですね。
つまり、現状では$this->dbhにあたるものがDataSelectクラスにありません。
変数未宣言のエラーが出るのでは？

下記のように、対応する変数を宣言してやる必要があります。

class DataSelect
{
  public $dbh = null;

  function __construct()

//以下略

まとめサンプルコード

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

class DataSelect extends Database
{
    function __construct()
    {
        $this->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]

上記のコードはエラーにはならないのです。なぜか。自分も、dbhはどこを指してるんだろうと悩んでます。

[keys]

public $dbh = null;をつけても実行結果は変わりません。これであれば理解ができます。クラスDataSelectの$dbhと、クラスDatabaseの$dbhは別物だとわかるからです。でも、public $dbh = nullがなくてもエラーにはなりません。

[keys]

ちなみに、public $dbh = null;を追加せず、$this->dbh=$db->getDBH();を$this->dbhh=$db->getDBH();とかにするとちゃんとエラーになります。なので、クラスDataSelectで使用している$this->dbhは、確実にクラスDatabaseの$dbhだと思われます。

[退会済みユーザー]

phpはゆるゆるなんでNOTICEレベルは出るかもしれないけど、構造を簡単にかえれるんです。（笑）

phpはarrayをベースにオブジェクトをつくってるので‥

[keys]

DataSelectクラスからは、Databaseクラスの$dbh変数が見えています。

[keys]

んんんん、であれば、thisである必要がないと思われたので、試しに、$this->dbh=$db->getDBH();を$dbhhhhh=$db->getDBH();にしてみたらエラーになりました。どうしてここで、thisを使わなければいけないのかわかりません

[keys]

また、 下記のようにしても、エラーになります。

public $dbhhhhhh = null;
$db=new Database;
$this->dbhhhhhh=$db->getDBH();

[keys]

でも、クラスの変数はインスタンス化しなくても参照できるのに、クラスの関数を使うときは、インスタンス変数を用いてdb->getDBH();のようにしなければいけないっていうのは、なんだか不思議です

[keys]

何故、インスタンス化変数を利用せずに、$thisを使って別のクラスの変数を参照できるのか謎です。

[keys]

また、どうして$this->dbhという変数でなければ、$db->getDBH();からの返り値を受け取れないか、それも謎です

[m.ts10806]

あああー、なるほど。
質問の意図を理解できてなかったかもしれません。

質問本文
> コンストラクトがありますが、$this->dbhというのはクラスDatabaseで作成した変数ですよね？

いいえ違います。
正確にはDatabaseクラスのgetDBH()で生成した情報を保持している
DataSelectクラスの変数です。あくまで1変数に過ぎません。

$a = "test";
$b = $a;

echo $a;
echo $b;

として、出力結果は同じですが、「同一のもの」ではありません。
あくまで$bというただの変数がたまたま$aの情報を「代入」されただけであり、$aそのものではありません。
それと同じです。
（クラス側で宣言なしに値を突っ込むという横着な実装したことなかったので気づきませんでした・・・）

[m.ts10806]

つまり、

＞DataSelectクラスからは、Databaseクラスの$dbh変数が見えています。

これも違います。
あくまで「DataSelectクラスで生成された$dbhを参照している」だけです。
継承しているわけでも、Databaseクラスでパブリック変数として定義されているわけでもないのでDatabaseクラスの情報を参照しているわけではありません。

[m.ts10806]

例えば以下のようにする。

class DataSelect
{
public $dbhSelect = null;
function __construct()
{
$db=new Database;
$this->dbhSelect=$db->getDBH();
}

function test()
{
　　・・・
　　$stmt = $this->dbhSelect->prepare($sql);

----------
下記のようにしてエラーが出たのは実際に使っている先で $this->dbh->prepare($sql);としたからでは？
＞　$this->dbhhhhhh=$db->getDBH();

$thisは自身なので自身にはdbhというプロパティは存在しないわけですし。

[退会済みユーザー]

>$dbhhhhh=$db->getDBH();にしてみたらエラーになりました。どうしてここで、thisを使わなければいけないのかわかりません

そりゃ、$this->を書かないと、$dbhhhhhは関数内のローカル変数として扱われるからです。
$thisっていうのはインスタンスのメンバであることを指すのです。

で、asahina1979が書いている通り、PHPは動的にメンバ変数を作れます。

だから、class Dto {} みたいな宣言だけしておいて、
$dto = new Dto();とインスタンスを作って、
$dto->a = 1;
$dto->b = "hoge";
　：
みたいに、どんどんメンバ変数を作っていけます。
これが、PHPのとても便利なところです。

で、これらは全部publicな変数になります。
ですから、$this->dbhについてもpublicです。

そうすると、カプセル化もへったくれもなくなるので、
private $dbhと、class内できちんと宣言しておくことが望まれます。
というか、普通はclass Databaseあたりを基底クラスとして
class DataSelect extends Database {}にして、
protected $dbh;
ですね。

そんなわけで、天地がひっくり返っても誰かさんみたいに、
public $dbh = null;
とかやらないでください。

[keys]

なるほどおおお。こういう見解は的外れですか？

Databaseクラスの$db->getDBH();関数の返り値は、あくまでもDatabaseクラス内の変数に返さないといけないから、$this->dbhのようにしなければいけない。

$db->getDBH();という時点で、Databaseクラスはインスタンス化されているから、 $this->dbhがよくわからないけど、Databaseクラスの中のdbhってことで、目を瞑る。

[m.ts10806]

＞でも、クラスの変数はインスタンス化しなくても参照できるのに、クラスの関数を使うときは、インスタンス変数を用いてdb->getDBH();のようにしなければいけないっていうのは、なんだか不思議です

少し勘違いされています。
うまく伝えられるか分かりませんが、
$thisは自身の「オブジェクト」なので参照できています。
static宣言された変数であればself::{変数名} というような参照の仕方もできます。

もちろんインスタンス化することなく参照することも可能です。
メソッドをstaticにすれば良いのです。

class Database
{
static function getDBH()
{

すると

class DataSelect
{
function __construct()
{
　　　$this->dbh=Database::getDBH();

の、ように利用することができます。
※これもstaticにするメリットデメリットはきちんと調べてから利用した方が良いです。
確かPDOにもstaticなやり方ありましたよね。

[m.ts10806]

luckerさん
補足ありがとうございます。明示的なので別に「誰かさん」のように隠さなくても結構です。
ポンポン動的に増えるのは実装としては賢明ではありません。
publicとしたのは現在の質問者さんのレベルからひとまず宣言したものを使うようにしましょうという例として出しており、protectedが適当であるのは理解しております。
※前の質問で継承するときに親クラスで宣言するようにも伝えてます（「とか」にしてますが）

[m.ts10806]

＞Databaseクラスの$db->getDBH();関数の返り値は、あくまでもDatabaseクラス内の変数に返さないといけないから、$this->dbhのようにしなければいけない。

ちょっとどういう解釈でそうなったか分からないですが、
微妙に違うようにも思います。
luckerさんの仰るように、というか前の質問のように継承しておいて
Database側に子クラスも参照可能な形で宣言しておけばDataSelectクラスから$this->で利用可能となるわけですし。
継承せずに個別のクラスとするなら、DataSelectクラス側で用意しておいた任意の変数で受けるだけなので、別に変数名はdbhにする必要もなく本当に任意の変数名で良いです。

[m.ts10806]

ひとまず$this　は「（親クラスも含めた）自身のオブジェクトである」という観点で再構成してみると良いかもしれません。
Databaseクラスの情報を$thisで利用したいのであれば継承するしかありません。

[keys]

ありがとうございます。再勉強します。ご親切にありがとうございました。

[m.ts10806]

念のためまとめのコード載せておきますので、参考まで。

[keys]

ありがとうございます！参考になります、

[m.ts10806]

ちなみに冒頭のコードは

class Test
{

static $hoge = "cccc";

static public function getNew()
{
echo "aaaaa/";
echo self::$hoge;
}
}

とすると aaaaa/cccc と表示されます。
クラスをインスタンス化せずにstaticでメソッドを呼び出しているので呼び出しもstaticにしなければなりません。

エラー表示はOnにしていますか？最初の書き方だとエラーが必ず表示されますので
まずはそのエラーメッセージを解決の糸口としてください。
（あと前の質問でもコメントしたかもしれませんが、Eclipseなどのエラーを自動検出してくれる高機能エディタを利用してください。定義ミスなどはおおよそ実行前に見つかります）

[yambejp]

なんか私の書いた雑なソースで紛糾してしまって申し訳ないです。

[m.ts10806]

yambejpさん
あれはあれで考え方としてありだとは思います。
どういう役割分担をするかというので参考になります。