質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.48%

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

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 1,980

score 199

 追記

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();
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

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

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/04/05 10:15 編集

    ちなみに冒頭のコードは

    class Test
    {

    static $hoge = "cccc";


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

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

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

    キャンセル

  • 2018/04/05 10:15

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

    キャンセル

  • 2018/04/05 10:18

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.48%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る