Discussion :

[baggie]

Bonjour

J'essaie d'apprendre correctement la POO en PHP et je galère un petit peu. C'est pour cela que j'aimerai quelques conseils !

J'ai lu des cours et des tutoriels sur le net ainsi que sur un livre. Et avec ça j'ai écrit ceci :

ville.class.php

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
36
37
 
class Ville
{
 
    public $id;
    public $nom;
    public $description;
 
    function __construct($cid = null, $cnom = null, $cdescription = null)
    {
        if (!empty($cid))
        {
            $this->id = $cid;
        }
 
        if (!empty($cnom))
        {
            $this->nom = $cnom;
        }
 
        if (!empty($cdescription))
        {
            $this->description = $cdescription;
        }
    }
 
    public function nom()
    {
        return $this->nom;
    }
 
    public function description()
    {
        return $this->description;
    }
 
}

villeManage.class.php (avec les autres fonctions comme la modification, la suppression par exemple)

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
 
class villeManage
{
    protected $db;
 
    public function __construct()
    {
        $this->db = $db;
    }
 
    protected function addVille(Ville $ville)
    {
        $query = $this->db->prepare('INSERT INTO VILLE SET nom_ville = :nom, description_ville = :description');
 
        $query = bindValue(':nom', $ville->nom()); 
        $query = bindValue(':description', $ville->description());
 
        $query = execute();
    }
}

includes.php

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
 
include 'ville.class.php';
 
$connection = mysql_connect('serveur', 'user', 'pwd') or die ("<p class='error'>Sorry, we were unable to connect to the database server.</p>");
$database = "base";
mysql_select_db($database, $connection) or die ("<p class='error'>Sorry, we were unable to connect to the database.</p>");
 
function AfficheVille($cid = null, $cnom = null, $cdescription = null)
{
    $query = mysql_query("SELECT * FROM ville ORDER BY nom_ville ASC"); 
 
    $villeArray = array();
    while ($row = mysql_fetch_assoc($query))
    {
        $listVille = new ville($row["id_ville"], $row['nom_ville'], $row['description_ville']);
        array_push($villeArray, $listVille);
    }
    return $villeArray;
}

Tout d'abord, j'aimerai savoir si ce que j'ai écrit est "bien". Et deuxièmement, si ça suit (ou pas du tout) un schéma MVC.

Ensuite, quelque chose que j'ai du mal à voir, c'est : ajouter une ville -> on a addVille() comme fonction. Mais comment récupère-t-on les données que l'on aimerait bien mettre dans la base ? Via un formulaire tout "simple" sur la même page ? Avec des $_POST ?

Si vous pouviez m'éclaircir un peu plus sur la POO en PHP, sur le découpage des pages (entre les pages d'admin et les pages pour les visiteurs), ça me serait bien utile

Merci !

[ThomasR]

Bonjour,

La fonction AfficheVille devrait s'appeler RecupererVilles et devrait être inclue dans la classe villeManage (que j'aurais nommé villeProviders, ou villeFournisseur).

Ta classe ville devrait permettre de valoriser la ville et la description après la construction de l'objet.

Ta classe villeManage n'a aucun intérêt à être instanciable, elle devrait être statique ou exposer un singleton.

Via un formulaire tout "simple" sur la même page ? Avec des $_POST ?

Pas forcément sur la meme page.
Oui il faut récupérer à un moment les valeurs saisies dans le formulaire.

Code :

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

1
2
3
4
5
6
7
8
9
 
if($_SERVER['REQUEST_METHOD'] === 'POST'){
    $ville = new Ville(null, $_POST['nom_ville'], $_POST['desription_ville']);
    /* moi j'aurais fait : 
    $ville = new Ville($_POST['nom_ville'], $_POST['desription_ville']);
    ou $ville = new Ville();
    $ville->setNom($_POST['nom_ville']);
    */
    $ville->save();

[baggie]

Merci beaucoup pour ces précisions ThomasR !

Ta classe villeManage n'a aucun intérêt à être instanciable, elle devrait être statique ou exposer un singleton.

Est-ce que tu pourrais préciser ça s'il te plait ? Le singleton surtout ^^

Pour les autres précisions, ce sont donc surtout des problèmes de noms qui ne sont pas correspondants par rapport à ce que ça fait c'est bien ça ?

Ma page includes.php ne sert pour le moment à rien donc (comme afficheville() n'est pas à la bonne place).

Et enfin, je suis dans un schéma MVC ?

[ThomasR]

Voici ta classe créé avec un pattern singleton :

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
36
37
38
39
40
41
 
class villeManage
{
    protected $db;
    private static $instance;
    private function __construct()
    {
        // d'ou vient $db ???
        $this->db = $db;
    }
 
    protected function addVille(Ville $ville)
    {
        $query = $this->db->prepare('INSERT INTO VILLE SET nom_ville = :nom, description_ville = :description');
 
        $query = bindValue(':nom', $ville->nom()); 
        $query = bindValue(':description', $ville->description());
 
        $query = execute();
    }
 
    function RecupererVilles($cid = null, $cnom = null, $cdescription = null)
    {
        $query = mysql_query("SELECT * FROM ville ORDER BY nom_ville ASC"); 
 
        $villeArray = array();
        while ($row = mysql_fetch_assoc($query))
        {
            $listVille = new ville($row["id_ville"], $row['nom_ville'], $row['description_ville']);
            array_push($villeArray, $listVille);
        }
        return $villeArray;
    }
 
    public static function instance()
    {
        if(self::$instance === NULL)
            self::$instance = new villeManage();
        return self::$instance;
    }
}

Code :

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

1
2
 
$villes = villeManage::instance()->RecupererVilles();

D'où vient $db dans le constructeur ?

MVC c'est autre chose, c'est une méthode pour structurer la relation entre l'affichage et la logique métier, ici tu ne montres rien qui soit en rapport avec MVC. Il ne suffit pas de faire de la POO
http://tahe.developpez.com/web/php/mvc/

[baggie]

D'où vient $db dans le constructeur ?

MVC c'est autre chose, c'est une méthode pour structurer la relation entre l'affichage et la logique métier, ici tu ne montres rien qui soit en rapport avec MVC. Il ne suffit pas de faire de la POO
http://tahe.developpez.com/web/php/mvc/

Merci ! Je vais utiliser ce code alors.
$db est le nom de ma base qui est dans mon includes.php

Je sais pour MVC mais c'était pour savoir si ce que je faisais avait l'air de respecter ce schéma justement. J'ai téléchargé le PDF du cours, je vais le relire

Merci !

[ThomasR]

Si tu veux faire référence à une variable déclarée globlement il faut préfixer global devant sa déclaration.

Code :

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

1
2
3
4
5
 
private function __construct(){
    global $db;
    $this->db = $db;
}

mais idéalement il aurait fallu soit exposé ta $db au travers d'une fonction, d'une méthode, ou d'un singleton, soit le passer en paramètre du constructeur

[baggie]

D'accord ! J'ai encore pas mal de choses à apprendre
Merci pour les conseils

[Bugger24]

ThomasR, dans ton pattern singleton tu as oublié de mettre la fonction instance() en static. Sans ça, tu peux pas l'appelé avec villeManage::instance()

Le code corrigé :

Code :

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

1
2
3
4
5
6
7
 public static function instance()
    {
        if(self::$instance === NULL)
            self::$instance = new villeManage();
        return self::$instance;
    }

[ThomasR]

Exact, merci.

[baggie]

De même !