Нет, здесь все неправильно.
Если коротко, то
1. Класс Database выкинуть на помойку.
2. Все exit() в коде выкинуть на помойку
3. Все try..catch туда же
В итоге у нас останется
class article {
public function __construct($pdo)
$this->pdo = $pdo;
}
public function get_all_db() {
$sql = "SELECT `articles_id`, `title`, `keywords`, `description` FROM `articles`";
return $this->pdo->query($sql)->fetchAll();
}
public function get_one_db($id){
$sql = "SELECT `articles_id`, `title`, `keywords`, `description` FROM `articles` WHERE articles_id = ?";
$stmt = $this->pdo->prepare($sql);
$stmt->execute([$id]);
return $stmt->fetch();
}
}
Все удобно, компактно и безопасно.
Кода получается меньше чем в mysqli и mysql вместе взятых.
