Singleton de connexion et Le R du CRUD

Version imprimable

Bonjour et Salutations :java:

Bon, j'ai réaliser un singleton pour la connexion de base de données, le tout fonctionne bien pour cette partie.

Pour le R du CRUD, aussi. Cependant, j'aimerais bien savoir si c'est fait correctement et si je peux le rendre plus portable.

Le paramètre de ma fonction est $id et j'aimerais si c'est adéquat de mettre $id au

Code:

bindValue(':id', $id, PDO::PARAM_INT);

Contexte:
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 <?php require_once 'pdo_mysql.php'; class Categorie extends pdo_mysql { private $conn; private $id; public function __construct() { $this->conn = pdo_mysql::pdo_connection(); } public function select($id) { $stmt = $this->conn->prepare("SELECT * FROM categorie WHERE id=:id"); $stmt->bindValue(':id', $id, PDO::PARAM_INT); $stmt->execute(); $rows = $stmt->fetchAll(PDO::FETCH_ASSOC); return $rows; } } $test = new Categorie(); $c = $test->select(170); foreach ($c as $v) { echo $v['cat'] .'<br>'; } ?>
Je sais que je peux ajouter des "setter" et des "getter" mais je verrai pour aller plus loin. Le $id fait référence au paramètre de ma fonction et me semble que c'est supposé être comme cela. J'ai tors? Si oui pourquoi.

S'il y a lieu de rendre la chose portable, j'aimerais bien avoir un exemple puisque je n'ai aucune idée comment chercher dans les recherches.

Ça serait vraiment apprécié, ça fait un bout que je suis sur le même genre de problématique.

23/10/2013, 10h29
papajoker
Bonjour,
je n'ai pas grande idée de ce que tu désires faire :(

Mais pour avoir une classe + générale :
1) mettre le nom de la table en propriété
Code:

1 2 protected $tableName='tablename'; stmt = $this->conn->prepare('SELECT * FROM '.$this->tablename.' WHERE id=:id');
2) renommer ta classe Categorie en Model
3) créer classe Categorie extend de Model
dans lea nouvelle classe juste changer le nom de la classe
protected $tableName='categorie';

---------------------------
dans methode select()
retourner un objet ! pourquoi un array :(
return $stmt->fetch(PDO::FETCH_OBJ);

---------------------------
un select + général
Code:

1 2 3 public function select(array $conditions) { ... } // declaration $c= $test->select( array('id'=>44) ); // utilisation $c= $pizza->select( array('prix'=>4) );

Pour le 1 et 2, j'ai bien compris.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
class Model extends pdo_mysql {
 
    private $conn;
    private $id;
 
    public function __construct() {
        $this->conn = pdo_mysql::pdo_connection();
    }
}
 
 
class Categorie extends Model {
 
    protected $table='categorie';
 
    public function select($id) {
        $stmt = $this->conn->prepare("SELECT * FROM $this->table WHERE id=:id");
        $stmt->bindValue(':id', $id, PDO::PARAM_INT);
        $stmt->execute();
        $rows = $stmt->fetchAll(PDO::FETCH_OBJ);
        return $rows;
    }
}

Ça deviendrait quelque chose du genre?

J'ai compris que select() change mais je n'ai pas compris de quelle manière.

23/10/2013, 11h09
papajoker

Citation:

Envoyé par dancom5

J'ai compris que select() change mais je n'ai pas compris de quelle manière.

1) mettre la methode select() dans la classe Model() car maintenant elle est commune a tous tes futurs modèles.

en poo il faut etre le + général possible pour aller vers le cas particulier, et toi, ta requete est un cas particulier :(
--------- aucune obligation ----------
2) ici tu passes comme requete sql WHERE id=XXX
moi je te propose de passer la chaine 'id' en parametre pour avoir une méthode select() plus générique ("WHERE prix=44")

3) apres tu peux donc ecrire une methode

Code:

selectID($id) { return $this->select('id'=>$id); }
23/10/2013, 11h12
papajoker

abstract

note :

Code:

abstract class Model { ... }

Il faut mettre ta classe Model en abstraite :

new Model() provoquera alors une erreur :)

-------------------------
abstract class Model extends pdo_mysql ????
pourquoi ?

Je le comprends comme cela :

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
<?php
 
require_once 'pdo_mysql.php';
 
abstract class Model extends pdo_mysql {
 
    private $conn;
    protected $table='categorie';
    private $id;
 
    public function __construct() {
        $this->conn = pdo_mysql::pdo_connection();
    }
 
    public function select($id) {
        $stmt = $this->conn->prepare("SELECT * FROM $this->table WHERE id=:id");
        $stmt->bindValue(':id', $id, PDO::PARAM_INT);
        $stmt->execute();
        $rows = $stmt->fetchAll(PDO::FETCH_OBJ);
        return $rows;
    }
}
 
class Categorie extends Model {
 
    public function selectID($id) { 
        return $this->select('id'=>$id); 
    }
 
}
 
$test = new Categorie(); 
$c = $test->selectID(170);
 
?>

ps :
Parse error: syntax error, unexpected T_DOUBLE_ARROW in line 27

23/10/2013, 11h39
papajoker

:( protected $table='categorie'; :(
quelle TRES TRES mauvaise idée de la déplacer !

alors que c'est la seule chose pour l'instant qui devrait être dans categorie !!!!
On met dans la classe abstraite tout ce qui est commun et justement la seule chose non commune ici, c'est le nom de la table :)

------------------------
pour select(), laisse tombé si il est trop difficile pour toi de réécrire complètement cette méthode. C'est juste passer un tableau en parametre et reutiliser ce tableau pour générer une requete : que du pur php (rien a voir avec la poo)

PS, ta dernière réponse, je ne l'avais pas vue...

Si j'enlève le

Code:

'id'

dans select(), ça m'affiche qqchose.

On dirait comme une étape de plus pour afficher. A moins que ça me permettrait d'afficher un id ou juste tout afficher.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 class Categorie extends Model { public function selectID($id) { return $this->select($id); } } $test = new Categorie(); $c= $test->selectID(170); foreach ($c as $a) { echo $a->cat; }

Tiens, au final, j'ai fait comme ceci :

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
<?php
 
require_once 'pdo_mysql.php';
 
abstract class Model extends pdo_mysql {
 
    private $conn;
    private $id;
 
    public function __construct() {
        $this->conn = pdo_mysql::pdo_connection();
    }
 
    public function select($id) {
        $stmt = $this->conn->prepare("SELECT * FROM $this->table WHERE id=:id");
        $stmt->bindValue(':id', $id, PDO::PARAM_INT);
        $stmt->execute();
        $rows = $stmt->fetchAll(PDO::FETCH_OBJ);
        return $rows;
    }
}
 
class Categorie extends Model {
    protected $table='categorie';
}
 
$test = new Categorie(); 
$c= $test->select(170);
 
foreach ($c as $a) {
    echo $a->cat;    
}
 
 
?>

ça marche. mais est-ce suffisamment portable?

PS, ça aurait été bien de tout afficher sans spécifier le id dans la même requête.

Citation:

Envoyé par dancom5

PS, ça aurait été bien de tout afficher sans spécifier le id dans la même requête.

a toi de faire la methode getAll()
Code:

1 2 3 4 5 6 /* Modele */ public function getAll() { $stmt = $this->conn->prepare("SELECT * FROM $this->table"); $stmt->execute(); return $stmt->fetchAll(PDO::FETCH_OBJ); }
Citation:

Envoyé par dancom5

ça marche. mais est-ce suffisamment portable?

en fonction de ton cahier des charges ...
Code:

1 2 3 4 class Pizza extends Model { protected $table='pizzas'; public function getAllByCategorie($categorie) { } }

Le tout dernier:
Code:

1 2 3 4 5 6 7 class Pizza extends Model { protected $table='categorie'; public function getAllByCategorie($categorie) { } } $test = new Pizza(); $c = $test->getAllByCategorie("cat");
Celui-là, je ne sais pas. Dans ma base c'est id et cat que j'ai seulement
avec la table categorie. Je comprends pas vraiment le sens de l'utilité.

23/10/2013, 12h24
dancom5

J'ai compris que dans le fond, c'est ajouter une fonction pour tout afficher.

Mais le cahier des charges, celui-la, je le comprends pas.
23/10/2013, 12h26
papajoker

lol
je te parlais de portable vers d'autres tables, ici tu utilises Categorie avec que 2 champs donc tres tres limité. peut-être ajouter un ORDER BY ..

le cahier des charges : avec mon exemple :
afficher toutes les pizzas d'une catégorie particulière ...
23/10/2013, 12h28
dancom5

ha ok. Je vais regarder plus en profondeur sur le sujet de abstract PHP.

Hey, Merci pour ton aide bien apprécié. :ccool: