Fatal error: Uncaught Error: Using $this when not in object context in ～のエラーを回避したい

Question

###前提・実現したいこと
MVCとクラスについて勉強中です。
とりあえずModel部分を作ろうとしています。
DB接続とデータ加工までをModelでやって、controllerで呼び出せるようにしたいです。

###発生している問題・エラーメッセージ

Fatal error: Uncaught Error: Using $this when not in object context in Model.php:53 Stack trace: #0 controller.php(11): DBconnect::getList() #1 {main} thrown in Model.php on line 53

###該当のソースコード

if(isset($_POST['item_id'])){
  $input_item_id = $_POST['item_id'];
} else{
  $input_item_id = null;
  //★エラーを表示させる？
}
if(isset($_POST['type'])){
  $input_item_type = $_POST['type'];
} else{
  $input_item_type = null;
  //★エラーを表示させる？
}
if(isset($_POST['maker'])){
  $input_item_maker_id = $_POST['maker'];
} else{
  $input_item_maker_id = null;
  //★エラーを表示させる？
}

class DBconnect
{
  public $db;
  private $host = 'xxx';
  private $dbname = 'xxx';
  private $dbuser = 'xxx';
  private $password = 'xxx';

  public function __construct()
  {
    $dsn = sprintf(
      'mysql:host=%s;dbname=%s;charset=utf8',
      $this->host, $this->dbname
    );
    try{
      $this->db = new PDO($dsn, $this->dbuser, $this->password);
    } catch (PDOException $e) {
      header('Content-Type: text/plain; charset=UTF-8', true, 500);
      exit($e->getMessage());
    }
  }

  // 商品リスト取得
  public function getList()
  {
    $sql = 'SELECT * FROM data where data.id = ?';
    $stmt = $this->db->prepare($sql); //ここでエラーが出ます
    $stmt->execute(['sample']);
    $rows = $stmt->fetchAll();
    return $rows;
  }
}

include(dirname(__FILE__) . "/model.php");
new DBconnect;
DBconnect::getList();

###試したこと
$stmt = $this->db->query($sql)
⇒ $stmt = self::$db->query($sql)
と書き換えてみましたが、Access to undeclared static propertyと怒られました。

###補足情報(言語/FW/ツール等のバージョンなど)
PHP7.1.8

Answer 1

DBconnect::getListはstaticでないメソッドなので、オブジェクトから呼ぶ必要があります。

$conn = new DBconnect;
$conn->getList();

Comments

[chibi144]

ありがとうございます。
ご提示頂いたとおりにcontroller側を書き換えたら無事動きました。

Answer 2

勉強中ということなので、こちらを紹介します。
PHP7 で堅牢なコードを書く - 例外処理、表明プログラミング、契約による設計 / PHP Conference 2016

「責務の再配置」あたりを参考にしてください。
もちろん、その他の部分も重要です。

ちなみに、質問には直接関係ないです＾＾；

Comments

[chibi144]

ありがとうございます。
さっそくhost以下を外に出しました。

[退会済みユーザー]

「戻り値の弱点」以降もサクッと読んでください。

このスクリプトには
setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
がないので、意図した動作を行っていないです。

[退会済みユーザー]

と思ったけど、
getList()
で、try catch してないからちゃんと動くかも＾＾；

でも、getList() もちゃんと try catch で記述してあげましょう！

[chibi144]

ひとまず外部に出した部分は下のとおりです。
$conf = array(
'host' => 'xxx,
'dbname' => 'xxx,
'dbuser' => 'xxx',
'password' => 'xxx',
);
$dboption = array(
PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
PDO::ATTR_EMULATE_PREPARES => false,
);
$dsn = sprintf(
'mysql:host=%s;dbname=%s;charset=utf8',
$conf['host'], $conf['dbname']
);
$pdo = new PDO($dsn, $conf['dbuser'], $conf['password'],$dboption);
$repo = new dataExport($pdo)

>>getList() もちゃんと try catch で記述してあげましょう！
コンストラクタでtry catchしている分が
getList()まで生きていると思っていたのですが、
具体的にはどんな風に書けばいいのかアドバイス頂けないでしょうか。

[退会済みユーザー]

「findALLメソッドの契約」に参考になる記述があります。
多分、まだ全体を読んでいない状態で質問していると思うので、調べながら読み込むとイイですよ。
私は最初にこの資料を見た時に衝撃を受けましたｗ

ちなみに、
setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
の記述がない場合、
catch (PDOException $e)
で拾える範囲は、かなり限定されます。
http://php.net/manual/ja/pdo.error-handling.php

ちょっと長いですけど、この辺も参考にすると良いです。
http://php.net/manual/ja/pdo.error-handling.php

Answer 3

staticで呼び出したいのかオブジェクトから呼びたいのかがはっきりしていないと思います。
DBconnectの$thisを活きとしたいなら下記のように書くべきですね。

$d = new DBconnect;
$d->getList();

ただ、これだけでは「Fatal error」が残りますが、それはまた別問題ということで。

追記：
Model.php 53行目とのことですが、提示されたDBconnectのソースは33行程度しかないですよね。
Model.php ソース全体も提示されたほうが良いかと思います。

Comments

[chibi144]

追記しました。
まるっと関係ないので割愛しておりましたが、確かに紛らわしかったですね。

[m.ts10806]

私の回答のみではまだちゃんと解決に至れていないようなので一度ベストアンサーを外されては？

[chibi144]

ご指摘ありがとうございます。
そのようにさせて頂きます。

Answer 4

Model.php:53行目が問題の箇所だと思いますが、DBConnect=Model.phpでしょうか？

Model.php の 53行目を確認してみましょう。