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