再帰処理がうまくいかない

Question

タイトル通り再起処理プログラムが動かないのです。かつ、何故このエラーが出るのかもわかりません。
木構造でデータを取り出すことを望んでいます。
どこに原因があるかご指摘いただきたいです。

<?php
  require('dbconnect.php');

 new Sample();

 class Sample {
 	public function __construct() {
 		$this->list_children();
 	}

 	public function list_children() {
  
  $stmt = $db->prepare("select * from t_bbs where deleted_at is null order by id desc");
  $stmt->execute();
  while($row = $stmt->fetch()){
    $rows[] = $row;
  }
  $parent_ids = [];
  foreach($rows as $row){
    $parent_ids[] = $row['parent_id'];
  }

  $row_bottom = [];
  foreach($rows as $row){
    if(!in_array($row['id'], $parent_ids)){
      $row_bottom[] = $row['id'];
    }
  }

  $categories = [];
  foreach($row_bottom as $row_id){
    $categories[] = $this->set_ids($row_id, $rows);
  }
  var_dump($categories);
}

private function set_ids($id, $rows, $args = []){
  if(is_null($id)){
    return array_reverse($args);
  }else{
    $args[] = $id;

    foreach($rows as $row){
      if($row['id'] == $id){
        return $this->set_ids($row['parent_id'], $rows, $args);
      }
    }
  }
 }
}
?>

<?php
  error_reporting(E_ALL & ~E_NOTICE);
  ini_set("display_errors",1);

  try {
  $db = new PDO('mysql:dbname=test_db;host=150.95.210.159;charset=utf8','mariasama','');
  $db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
} catch (PDOException $e) {
  echo 'DB接続エラー' . $e->getMessage();
}
 ?>

エラー文

Fatal error: Uncaught Error: Call to a member function prepare() on null in /var/www/html/sample.php:17 Stack trace: #0 /var/www/html/sample.php(8): Sample->list_children() #1 /var/www/html/sample.php(4): Sample->__construct() #2 {main} thrown in /var/www/html/sample.php on line 17

Comments

[yambejp]

ファイルが２つに別れている状況がわかりせん
下のほうがdbconnect.phpですか？

Answer 1

原因

どこに原因があるかご指摘いただきたいです。

エラーに問題の箇所が書いてあるので、まずはエラーを読んだりエラーで検索してみて下さい。

今回のケースだと

Fatal error: Uncaught Error: Call to a member function prepare() on null in /var/www/html/sample.php:17 Stack trace: #0 /var/www/html/sample.php(8): Sample->list_children() #1 /var/www/html/sample.php(4): Sample->__construct() #2 {main} thrown in /var/www/html/sample.php on line 17

のうちの
Fatal error: Uncaught Error: Call to a member function prepare() on null
あたりで検索すると山ほど対処方法が見つかります。（それ以降は個人の環境によって変わる部分なので、検索ワードにいれると精度が落ちる）

プログラミングの大部分の時間はデバッグに費やされることになるので、デバッグの仕方やエラーの調べ方自体を調べてみることをお勧めします。
PHP デバッグ 方法とかPHP エラー 読み方等で調べてみてください。

デバッグの流れ

検索しなくてもそのまま読むと
Call to a member function prepare() on null
prepare()がnullから呼ばれている

という感じのことが書いてあるので、ソースコードからprepare()を検索してみると
$stmt = $db->prepare("select * from t_bbs where deleted_at is null order by id desc");
の
$db->prepare(
の
$dbがnullになっていると推測されます。
実際、public function list_children() {が開始してから$dbはどこからも渡されておらず、急に上記の部分で湧いて出ていて、ここが問題だと特定できます。
(エラーが出ている部分の直前でvar_dump($db);としても確認できます)

解決方法

例えば、DB接続している部分

  try {
  $db = new PDO('mysql:dbname=test_db;host=150.95.210.159;charset=utf8','mariasama','');
  $db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
} catch (PDOException $e) {
  echo 'DB接続エラー' . $e->getMessage();
}

を
public function list_children() {
の直後あたりにコピーすればこのエラーは解決します。（が、とても残念な実装です）

その他の方法としては、
Sampleクラスのコンストラクタの引数に$dbを受けて、プロパティにセットして→メソッドからは$this->dbなどの形で呼ぶ
等が考えられます。

把握しないといけないこと

今回の問題で把握しないといけないことは、以下の2点に集約できそうです。
それぞれ、基本的な文法に属する部分なのでこれらの理解が浅いと後々多大な時間を無駄にすることになります、多少時間をかけてでもPHPマニュアルを読み込む&試すことをお勧めします。

1. 変数のスコープ

クラス、メソッド、関数、とグローバルスコープで同じ名前の変数がどう有効になるかを把握する必要があります。
変数のスコープのサンプルを実際に動かしてみて理解することをお勧めします。
スコープを把握することは超重要で、再起などの具体的な処理を考える場合は必ず把握している必要があります。

1. クラスの基礎

クラスの基礎
も実際にサンプルスクリプトを動かして把握してください。

Comments

[cacao86]

丁寧に回答ありがとうございます。
解決方法を参考にエラーは消えました。が、phpの基本が自分の中に落とし込めていないのでまずはそこから勉強していきたいと思います。

[tanat]

フィードバックありがとうございます。
> phpの基本が自分の中に落とし込めていないのでまずはそこから勉強していきたいと思います。

はい。この後に必ず必要になるところなので無駄にはならないかと思います。
最小限のサンプルコードを書いてみて、具体的に理解できないところがあれば改めて質問されると適切な回答が得られるかと思います。

Answer 2

dbconnect.phpで接続処理をしてるというのであれば
クラスを組んでclass Sample のコンストラクタから接続を
メンバ変数に確保しておいてください