Aby Twoja klasa działała statycznie, musisz zrobić kilka rzeczy.
Najpierw ustaw połączenie statyczne, np.
private static $connection;
Po drugie, dlaczego wszystkie podkreślenia?
define('DB_HOST', 'localhost');
define('DB_NAME', 'your_db_name');
define('DB_USER', 'username');
define('DB_PASS', 'password');
Po co w ogóle używać stałych klas? Po prostu użyj stałych, które już zdefiniowałeś.
Po trzecie, zgub konstruktora. Nie możesz oczekiwać, że utworzysz instancję tej klasy i używaj go statycznie. Wybrałbym leniwe podejście do połączenia
private static function getConnection() {
if (self::$connection === null) {
$dsn = sprintf('mysql:host=%s;dbname=%s;charset=utf8',
DB_HOST, DB_NAME);
self::$connection = new PDO($dsn, DB_USER, DB_PASS, array(
PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
PDO::ATTR_EMULATE_PREPARES => false,
PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC
));
}
return self::$connection;
}
Następnie twoje metody publiczne wywołałyby tę metodę wewnętrznie. Dostosuję również Twój dbDataArray
metoda pokazująca, jak zwrócić tablicę asocjacyjną
public static function dbDataArray($query, $params = array()) {
$stmt = self::getConnection()->prepare($query);
$stmt->execute($params);
return $stmt->fetchAll();
}