Как исправить конструктор класса для подключения к Базе данных через PDO?

Question

[Алексей Денисов]

Подскажите как исправить конструктор для работы с Базой Данных (mysqli) для работы с PDO?

class ClientData extends Dbconfig {	
    protected $hostName;
    protected $userName;
    protected $password;
    protected $dbName;
	
    private $table = 'people';
    private $dbConnect = false;
	
	
	// Констурктор - Заполняет поля и выполняет подключение к Базе данных
    public function __construct(){
        if(!$this->dbConnect){ 		
	    $database = new dbConfig();            
            $this -> hostName 	= $database -> serverName;
            $this -> userName 	= $database -> userName;
            $this -> password 	= $database -> password;
	    $this -> dbName 	        = $database -> dbName;
			// Подключение к базе данных
            $conn = new mysqli($this->hostName, $this->userName, $this->password, $this->dbName);
            if($conn->connect_error){
                die("Error failed to connect to MySQL: " . $conn->connect_error);
            } else{
                $this->dbConnect = $conn;
            }
        }
    }
  	
	public function clientList(){		
		$sqlQuery 	= "SELECT * FROM ".$this->table." ";		
		$result 	= mysqli_query($this->dbConnect, $sqlQuery);
		$numRows 	= mysqli_num_rows($result);
		// Дальнейшие Действия
	}
}

Делаю так, но не работает:

class ClientData extends Dbconfig {
	
	public $table = 'people';
	public $dbConnect = false;
	
	public function __construct(){		
	if(!$this->dbConnect){        
	    $host 		= "localhost";
        $user 		= "admin";
        $password 	= "";
        $dbname 	= "forum";
        $charset 	= "utf8";
        
        $dsn = "mysql:host=$host;dbname=$dbname;charset=$charset";
      
		try {
				$pdo	 = new PDO($dsn, $user, $password);
			}
			catch (PDOException $e) {
				echo "Connection failed: " . $e->getMessage();
			}
			$this->dbConnect = $pdo;
		}	
	}
	
	public function clientList(){
		
		$sqlQuery 	= "SELECT * FROM ".$this->table." ";
		
		//после добавления этой строки - выдает ошибку 500
		$result 	= $this->dbConnect->query($sqlQuery);
		// Дальнейшие Действия
	}
}

И не могу понять, подключения нет, или запрос не верный, не могу проверить переменную $dbConnect
Так как Объект этого класса запускается через JS

В примере который я написал, данные для подключения занес сразу в переменные.
Без создания Объекта на основе класса dbConfig() $database = new dbConfig();

Comments

[Дмитрий]

Слов "не работает" не бывает. Бывает сообщение или об ошибке, или я жду такие данные, а получаю другие. Какой вариант у вас?

[Алексей Денисов]

Выдает ошибку 500 через инструменты разработчика вижу в браузере,
как будто синтаксическая ошибка.

Вот и не пойму, или подключения нет или не инициализирована переменная
$dbConnect

$pdo  = new PDO($dsn, $user, $password);
$this   -> dbConnect = $pdo;

Если этой строки, то и ошибки нет и вывод Пустой соответственно, как проверить произошло ли подключение?

И еще вот такой запрос смущает:
$result = $this->dbConnect->query($sqlQuery);

Обращение к полям класса через This - будет ли корректно работать

[Дмитрий]

Я бы выкинул этот конфиг, а в конструктор передавал бы готовый объект pdo

[Алексей Денисов]

Дмитрий, Согласен, но как это сделать не могу понять, не могли бы "накидать" примерный план, и интересно - почему этот не работает, не пойму в чем причина. Если прописать не PDO, а mysqli - то все работает

[Дмитрий]

Алексей Денисов,

class Db
{
    private PDO $pdo;

    public function __construct(PDO $pdo)
    {
        $this->pdo = $pdo;
    }

    public function getData(array $args = [])
    {
        $data = $this->pdo->prepare(...);

        return $data->execute($args);
    }
}

[Александр Кубинцев]

500 ошибка - это не информативно. Включите отображение ошибок или смотрите лог ошибок (без понятия какое у вас окружение)

[Vitsliputsli]

Читайте ошибки из лога, там доступно будет написано в чем проблема.
И удалите бессмысленный и вредный try catch - ошибка и так упадет в лог, а продолжать работу при мертвой базе скорее всего бессмысленно.