Discussion :

[dancom5]

Bonjour et Salutations

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Contexte:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[dancom5]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[papajoker]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

selectID($id) { return $this->select('id'=>$id); }

[papajoker]

note :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

abstract class Model { ... }

Il faut mettre ta classe Model en abstraite :

new Model() provoquera alors une erreur

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

[dancom5]

Je le comprends comme cela :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[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)

[dancom5]

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

Si j'enlève le

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

'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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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;    
}

[dancom5]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[papajoker]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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);
    }

ça marche. mais est-ce suffisamment portable?

en fonction de ton cahier des charges ...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[dancom5]

Le tout dernier:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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é.

[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.

[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 ...

[dancom5]

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

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