Comprendre la POO

Version imprimable

Comprendre la POO

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:

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:

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:

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

Merci !

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.

Citation:

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:

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

10/09/2010, 15h35
baggie

Merci beaucoup pour ces précisions ThomasR !

Citation:

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 ?

Voici ta classe créé avec un pattern singleton :

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

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/

10/09/2010, 16h45
baggie

Citation:

Envoyé par ThomasR

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

Merci !
10/09/2010, 16h56
ThomasR
Si tu veux faire référence à une variable déclarée globlement il faut préfixer global devant sa déclaration.
Code:

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
10/09/2010, 17h06
baggie

D'accord ! J'ai encore pas mal de choses à apprendre :)
Merci pour les conseils :D
14/09/2010, 16h57
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:

1 2 3 4 5 6 7 public static function instance() { if(self::$instance === NULL) self::$instance = new villeManage(); return self::$instance; }
14/09/2010, 17h40
ThomasR

Exact, merci.
15/09/2010, 10h44
baggie

De même ! :D