Classe database sqlite

Version imprimable

Bonjour tout le monde,

Je démarre le POO en PHP. Je cherche déjà à connecter la base de données en sql.

J'ai ce code mais je pense que j'ai mal adapté.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 <?php namespace app; use PDO; class Database{ try { $DB = new PDO('sqlite:calibreDatabase.db'); $DB->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); } catch (Exception $e) { die('Erreur : ' . $e->getMessage()); } }

Voici la doc PHP sur la POO à lire, tout y est détaillé : https://www.php.net/oop5

Quand quelque chose ne va pas, donne le message d'erreur.

La syntaxe de ta classe est invalide, tes instructions doivent être placées dans des méthodes.

Ton try/catch ne devrait pas contenir de die(). En effet, par défaut une exception non rattrapée met fin au script, mais elle doit aussi pouvoir être rattrapée par l'utilisateur pour un traitement adéquat, ce que tu empêches.

Si tu veux encapsuler PDO il existe plusieurs manières de faire.

Par héritage, avec extends, MyDatabase va hériter de toutes les méthodes de PDO, et tu peux en ajouter d'autres si besoin :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 class MyDatabase extends PDO { public function __construct() { parent::__construct('sqlite:calibreDatabase.db'); // On appelle le constructeur de l'objet PDO parent $this->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); $this->setAttribute(PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_OBJ); } } try { $db = new MyDatabase(); $data = $db->query('...')->fetchAll(); print_r($data); } catch (Exception $e) { // On laisse la possibilité à l'utilisateur de gérer l'exception comme il veut // Affichage de l'exception et sortie du script print_r($e); exit; }

27/11/2021, 18h15
night18

Merci beaucoup !
Dans le code que tu m'as donné, $parent n'est pas reconnu ? VSC m'indique que c'est une erreur.
27/11/2021, 19h16
Séb.

Citation:

Dans le code que tu m'as donné, $parent n'est pas reconnu ? VSC m'indique que c'est une erreur.

Et sinon ça fonctionne ? Voir la config de VSC, je ne pourrai pas t'aider.
28/11/2021, 12h30
Toufik83

Bonjour,

parent sans dollar, ceci indique que le constructeur doit hériter du constructeur parent PDO.
28/11/2021, 14h21
Séb.

Citation:

Envoyé par Toufik83

parent sans dollar, ceci indique que le constructeur doit hériter du constructeur parent PDO.

Oups en effet, je corrige la coquille. Merci !
28/11/2021, 22h38
night18

Merci à tous les 2, effectivement ça fonctionne ! Un grand merci.
29/11/2021, 14h48
night18

Excusez-moi à tous les 2, mais finalement je bute encore pour appeler la classe.
J'ai créé des classes pour faire mes requêtes sql puis manipuler les données comme je veux. Comme faire la liaison avec ma classe database que nous venons d'écrire ?
29/11/2021, 16h54
Toufik83

Bonjour,

Montres nous la classe que tu veux utiliser avec celle qu'on vient d'écrire, et explique un peu ce que tu cherches à faire exactement.

Citation:

Envoyé par Toufik83

Bonjour,

Montres nous la classe que tu veux utiliser avec celle qu'on vient d'écrire, et explique un peu ce que tu cherches à faire exactement.

Voici ma classe livre, permettant de récupérer les données de la base (id, titre, auteur).
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 <?php namespace app\table; use Database; class livre{ public static function getlast(){ return $data::getdb()->query(" SELECT * FROM books ", __CLASS__); } public function __get($key){ $method = 'get' . ucfirst($key); $this->$key = $this->$method(); return $this->$key; } public function getUrl(){ return 'index.php?p=livre&id=' . $this->id; } }
L'idée est de travaillé sur une base de données existante, externe à l'application. Il s'agit de visualiser les données. La base étant très complexe avec beaucoup de données différentes et n'ayant rien à voir entre elles, je pensais intéressant de séparer l'accès a la base avec une classe, puis écrire d'autres classes chacune correspondant à des objets. Ici, les livres. Le code sql ici est volontairement simple mais s'étend en vrai sur plusieurs dizaine de lignes.

30/11/2021, 09h30
escartefigue
Bonjour,

Je ne répondrai pas sur la partie PHP que je ne maîtrise pas, mais concernant la requête, il ne faut jamais utiliser SELECT * dans un traitement :
- select * est instable : toute modification de la table ou de la vue impactera la requête
- select * est couteux : toutes les colonnes sont ramenées alors que la plupart du temps, un traitement n'a besoin que d'une partie des colonnes.
  Le transport des données inutiles charge le réseau au détriment des performances de votre traitement et aussi des traitements qui s'exécutent en parallèle.
Remplacez donc SELECT * par la liste des colonnes utiles ;)
30/11/2021, 12h22
night18

Merci, je note le conseil ! Ma requête SQL est bien plus longue et j'ai noté les titres des colonnes :)

Bonjour,

La classe livre n'a aucun rapport avec les requêtes sql, elle doit s'occuper seulement de l'objet livre pas plus.

il faudrait créer une classe dbManagerLivre héritante de livre, c'est cette classe qui doit gérer les requêtes sql :

Classe Database :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
<?php 
namespace Database;
use \PDO;
class Database extends \PDO{
    public function __construct()
    {
        parent::__construct('sqlite:calibreDatabase.db'); // On appelle le constructeur de l'objet PDO parent
        $this->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        $this->setAttribute(PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_OBJ);
    }
}

Classe livre :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
 
<?php 
 
namespace app\table;
class livre{
	protected $id,$name,$titre;
	public function __construct($data){
		/*Pour instancier un nouveau Livre manuellement lorsqu'on en a besoin */
		if(is_array($data)){
			foreach($data as $prop=>$value){
				$method = 'set' . ucfirst($prop);
				if(method_exists($this,$method))
					$this->{$method}($value);
			}
		}
	}
	//getters
        public function getId(){return $this->id;}
	public function getName(){return $this->name;}
	public function getTitre(){return $this->titre;}
 
        //setters
	protected function setId($id){$this->id=$id;}
	protected function setName($name){$this->name=$name;}
	protected function setTitre($titre){$this->titre=$titre;}
 
	public function __get($key){
           $method = 'get' . ucfirst($key);
           $this->$key = $this->$method();
           return $this->$key;
        }
	public function __set($name,$key){
          $method = 'set' . ucfirst($name);
          $this->{$method}($key);
       }
       public function getUrl(){
          return 'index.php?p=livre&id=' . $this->id;
       }
 
}

Classe dbManagerLivre :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
<?php
namespace app\dbManagers;//il faut que cette classe existe dans le dossier app/dbManagers ! 
use Database;
use app\table\livre; 
 class dbManagerLivre extends livre{
	private static $db;
        public static function getAll(){
          return self::$db->query("
            SELECT id,name,titre
            FROM books
           ")
           ->fetchAll(\PDO::FETCH_CLASS,parent::class,["id","name","titre"]);
        }
 
        public function __construct(livre $lv=null,Database $db){
		self::$db=$db;
       }
}

Page index de teste :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
 
<?php
/*Activer l'affichage des erreurs/avertissements*/
ini_set('display_errors', 1);
ini_set('display_startup_errors', 1);
error_reporting(E_ALL);
 
$db=new Database();
$livre=new livre(['id'=>10,'name'=>"name livre 1","titre"=>"titre livre 1"]);
$dbMng=new dbManagerLivre($livre,$db);
 
$allBooks=$dbMng::getAll();
foreach($allBooks as $book){
	echo '<br />book id :'.$book->getId()."<br />"
	.'book name :'.$book->getName()."<br />"
	.'book titre :'.$book->getTitre().'<br />';
}
/*Affiche : 
book id :1
book name :name 1
book titre :titre 1

book id :2
book name :name 2
book titre :titre 2
*/

Remarque : j'ai modifié les namespaces parce que j'ai créer un autoloader classique, ce qui n'est pas le cas pour toi...

[EDIT] :
Il est aussi utile de récupérer l'objet Database d'une façon statique afin de l'utiliser sans avoir à l'instancier avec le mot clé new...

30/11/2021, 21h57
night18

1 pièce(s) jointe(s)

Un grand merci. Après plusieurs ajustements suite à des erreurs, une dernière apparait :

Pièce jointe 609497
30/11/2021, 22h04
Toufik83

L'erreur dit que la méthode setTitle() n'existe pas dans la classe livre, tu n'as qu'à remplacer setTitre() par setTitle() et n'oublis pas aussi le getter.
30/11/2021, 22h34
night18

Parfait ça fonctionne !! Un grand grand merci !!