Problème avec Update (PHP Objet)

Version imprimable

Bonjour,

J'ai un petit souci avec mon code PHP. Enfin, pas tout à fait le miens mais celui de mon professeur :) Je suis encore étudiante et je me suis mise au PHP que très récemment. Excusez donc du peu de compétence dont je risque de faire preuve ces prochains mois.

Notre projet est de créer un site web sur un sport de combat. Très gentiement, on nous a conseillé de nous pencher sur le PHP Objet. Si pour les fonctions DELETE/INSERT il n'y a aucun souci, je rencontre des difficultés pour l'UPDATE.

Voilà donc mes fichiers :

fichier controleur_pays

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
 
<?php
 
if (isset($_GET['action']))
{
	$action = $_GET['action'];
}
 
switch ($action)
{
(...) <--- je vous épargne l'inutile

case "modifier":
            include("./page/pays/modifier.php");
            break;
        case "traitement_modif":
            $pays = new Pays();
            $pays->setNomPays($_POST['NOMPAYS']);
            $pays->setNomNationalite($_POST['NOMNATIONALITE']);
            $pays->setIdPays($_POST['IDPAYS']);
            
            
            $pdo = new PdoPays();
            $pdo->update($pays);
            break;

(...)
}

fichier PdoPays

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
class PdoPays extends PdoConnect {
 
public function update(pays $pays) {
 
        $this->open();
        $sql = "UPDATE `pays` SET NOMPAYS= ?, NOMNATIONALITE= ? WHERE IDPAYS=?";
 
        $prep = $this->pdo->prepare($sql);
 
        $prep->bindValue(1, $pays->getNomPays(), PDO:: PARAM_STR);
        $prep->bindValue(2, $pays->getNomNationalite(), PDO:: PARAM_STR);
        $prep->bindValue(3, $pays->getIdPays(), PDO:: PARAM_INT);
 
 
        var_dump($prep);
        $prep->execute();
    }
}

fichier modifier

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
 
<?php
 
$IdPays = $_GET['IdPays'];
$oPDO = new PdoPays();
 
$pays = $oPDO->readOne($IdPays);
 
 
echo "<form id='formulaire' action='index.php?route=pays&action=traitement_modif' method='post'>";
 
echo "<label>modifiez un Nom de Pays</label><br />";
echo "<input type='text' name='NOMPAYS' value='" . $pays->getNomPays() . "' /><br />";
 
echo "<label>Entre un Nom de Nationalite</label><br />";
echo "<input type='text' name='NOMNATIONALITE' value='" . $pays->getNomNationalite() . "'/>";
 
 
echo "<input type='hidden' name='IDPAYS' value='" . $pays->getIdPays() . "' />";
echo "<input type='submit'  value='Valider'>";
 
echo "</form>";
 
echo "<br /><br /><br /><br /><br />";
?>

Si j'arrive à récupérer le pays et la nationalité déjà mise dans ma table, je n'arrive pas à les modifier. Lorsque je clique sur VALIDER ma requête donne ça - > object(PDOStatement)[4]
public 'queryString' => string 'UPDATE `pays` SET NOMPAYS= ?, NOMNATIONALITE= ? WHERE IDPAYS=?' (length=62)

Je ne comprends vraiment pas pourquoi... --" Help ?

08/04/2013, 21h26
sabotage

Ca t'affiche ça parce que tu as mis un var_dump().
As-tu contrôlé si ta mise à jour était réalisée ?
Si ce n'est pas le cas, il faut commencer par contrôler ce que valent les 3 variables que tu bind.
09/04/2013, 03h56
ABCIWEB

Et au passage pour la forme, dans le fichier "fichier controleur_pays" la définition de la variable "action" est incomplète. Comme il n'y a pas de valeur par défaut cela ne sert à rien et revient à faire directement : switch ($_GET['action'])

Bonjour,

Merci pour vos réponses ! Grâce à vous, j'ai résolu mon problème.
J'ai donc enlevé le var_dump ;)

Par ailleurs, je me permets de vous parler d'un autre problème. Voici la manière dont est organisé mon dossier :

http://img15.hostingpics.net/pics/80...chitecture.jpg

Les parties concernant les pays, les situations et les combattants sont les mêmes. Dans modifier.php de Combattants j'ai besoin de créer une liste déroulante avec le contenu de ma table pays (puisqu'un combattant appartient à un pays). Sauf que, comme c'est organisé, je n'y parviens pas.

Voici les fichiers pour les combattants :
modifier.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
38
39
40
41
42
43
44
45
 
<?php
$IdCombattant = $_GET['IdCombattant'];
$oPDO= new PdoCombattant();
 
$combattant= $oPDO->readOne($IdCombattant);
 
 
        echo "<form id='formulaire' action='index.php?route=combattant&action=traitement_modif' method='post'>";
 
	echo "<label>modifiez un nom</label><br />";
	echo "<input type='text' name='NOMCOMBATTANT' value='".$combattant->getNomCombattant()."' /><br />";
 
	echo "<label>Entre un prénom</label><br />";
	echo "<input type='text' name='PRENOMCOMBATTANT' value='".$combattant->getPrenomCombattant()."'/><br />";
 
        echo "<label>Entre un surnom</label><br />";
	echo "<input type='text' name='SURNOMCOMBATTANT' value='".$combattant->getSurnomCombattant()."'/><br />";
 
        echo "<label>Entrez l'allonge</label><br />";
	echo "<input type='text' name='ALLONGECOMBATTANT' value='".$combattant->getAllongeCombattant()."'/><br />";
 
        echo "<label>Entre une taille</label><br />";
	echo "<input type='text' name='TAILLECOMBATTANT' value='".$combattant->getTailleCombattant()."'/><br />";
 
        echo "<label>Entrez un poids</label><br />";
	echo "<input type='text' name='POIDSCOMBATTANT' value='".$combattant->getPoidsCombattant()."'/><br />";
 
        echo "<label>Entre une photo</label><br />";
	echo "<input type='text' name='PHOTOCOMBATTANT' value='".$combattant->getPhotoCombattant()."'/><br />";
 
        echo "<input type='hidden' name='IDCOMBATTANT' value='".$combattant->getIdCombattant()."' />";	
 
        echo 'Pays/Nationalité:';
        echo '<select name="IDPAYS" size="1">';
 
 
     //C'est là que je dois insérer une liste déroulante du contenu de la table pays
        echo '</select>';
 
    	echo "<input type='submit'  value='Valider'>";
 
	echo "</form>";
 
?>

pdoCombattant

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
 
<?php
 
class PdoCombattant extends PdoConnect {
 
    public function create(combattant $combattant) {
        try {
            $this->open();
            $sql = "INSERT INTO `combattant`(NOMCOMBATTANT, PRENOMCOMBATTANT, SURNOMCOMBATTANT,ALLONGECOMBATTANT,TAILLECOMBATTANT,POIDSCOMBATTANT,PHOTOCOMBATTANT, IDPAYS) VALUES(?, ?, ?, ?, ?, ?, ?, ?) ";
 
            $prep = $this->pdo->prepare($sql);
 
            $prep->bindValue(1, $combattant->getNomCombattant(), PDO:: PARAM_STR);
            $prep->bindValue(2, $combattant->getPrenomCombattant(), PDO:: PARAM_STR);
            $prep->bindValue(3, $combattant->getSurnomCombattant(), PDO:: PARAM_STR);
            $prep->bindValue(4, $combattant->getAllongeCombattant(), PDO:: PARAM_INT);
            $prep->bindValue(5, $combattant->getTailleCombattant(), PDO:: PARAM_INT);
            $prep->bindValue(6, $combattant->getPoidsCombattant(), PDO:: PARAM_INT);
            $prep->bindValue(7, $combattant->getPhotoCombattant(), PDO:: PARAM_STR);
            $prep->bindValue(8, $combattant->getIdPays(), PDO:: PARAM_INT);
 
            $prep->execute();
        } catch (PDOException $e) {
            $msg = 'ERREUR PDO dans ' . $e->getFile() . ' L.' . $e->getLine() . ' : ' . $e->getMessage();
            die($msg);
        }
    }
 
    public function readAll() {
        try {
            $this->open();
            $sql = "SELECT * From `combattant`";
            $prep = $this->pdo->prepare($sql);
 
            $prep->execute();
            $prep->setFetchMode(PDO::FETCH_ASSOC);
 
 
            return $prep;
        } catch (PDOException $e) {
            $msg = 'ERREUR PDO dans ' . $e->getFile() . ' L.' . $e->getLine() . ' : ' . $e->getMessage();
            die($msg);
        }
    }
 
    public function readOne($IdCombattant) {
        try {
            $this->open();
            $sql = "SELECT * From `combattant` Where IdCombattant=$IdCombattant";
            $prep = $this->pdo->prepare($sql);
 
            $prep->execute();
 
            $combattant = new Combattant();
            $ligne = $prep->fetch(PDO::FETCH_OBJ);
            $combattant->setIdCombattant($ligne->IDCOMBATTANT);
            $combattant->setNomCombattant($ligne->NOMCOMBATTANT);
            $combattant->setPrenomCombattant($ligne->PRENOMCOMBATTANT);
            $combattant->setSurnomCombattant($ligne->SURNOMCOMBATTANT);
            $combattant->setTailleCombattant($ligne->TAILLECOMBATTANT);
            $combattant->setPoidsCombattant($ligne->POIDSCOMBATTANT);
            $combattant->setPhotoCombattant($ligne->PHOTOCOMBATTANT);
            $combattant->setAllongeCombattant($ligne->ALLONGECOMBATTANT);
            $combattant->setIdPays($ligne->IDPAYS);
            return $combattant;
        } catch (PDOException $e) {
            $msg = 'ERREUR PDO dans ' . $e->getFile() . ' L.' . $e->getLine() . ' : ' . $e->getMessage();
            die($msg);
        }
    }
 
    public function delete(combattant $combattant) {
 
        try {
            $this->open();
            $sql = "DELETE FROM combattant WHERE IDCOMBATTANT=?";
 
            $prep = $this->pdo->prepare($sql);
 
            $prep->bindValue(1, $combattant->getIdCombattant(), PDO::PARAM_INT);
            $prep->execute();
        } catch (PDOException $e) {
            $msg = 'ERREUR PDO dans ' . $e->getFile() . ' L.' . $e->getLine() . ' : ' . $e->getMessage();
            die($msg);
        }
    }
 
    public function update(combattant $combattant) {
 
        $this->open();
        $sql = "UPDATE combattant SET NOMCOMBATTANT = ?, PRENOMCOMBATTANT= ?, SURNOMCOMBATTANT = ?, ALLONGECOMBATTANT = ?, TAILLECOMBATTANT = ?, POIDSCOMBATTANT = ?, PHOTOCOMBATTANT = ?, IDPAYS = ? WHERE IDCOMBATTANT= ?";
 
        $prep = $this->pdo->prepare($sql);
 
        $prep->bindValue(1, $combattant->getNomCombattant(), PDO:: PARAM_STR);
        $prep->bindValue(2, $combattant->getPrenomCombattant(), PDO:: PARAM_STR);
        $prep->bindValue(3, $combattant->getSurnomCombattant(), PDO:: PARAM_STR);
        $prep->bindValue(4, $combattant->getAllongeCombattant(), PDO:: PARAM_INT);
        $prep->bindValue(5, $combattant->getTailleCombattant(), PDO:: PARAM_INT);
        $prep->bindValue(6, $combattant->getPoidsCombattant(), PDO:: PARAM_INT);
        $prep->bindValue(7, $combattant->getPhotoCombattant(), PDO:: PARAM_STR);
        $prep->bindValue(8, $combattant->getIdCombattants(), PDO:: PARAM_INT);
        $prep->bindValue(9, $combattant->getIdPays(), PDO:: PARAM_INT);
 
        $prep->execute();
    }
 
    public function exist($surnom) {
        $this->open();
        $query = "SELECT * FROM combattant WHERE SURNOMCOMBATTANT='$surnom';";
        $statement = $this->pdo->query($query);
        return $statement->rowCount();
    }
 
}

controleur_combattant

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
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
 
<?php
include("./PDO/PdoConnect.php"); //On inclut la connexion à la base de données.
include("./PDO/PdoCombattant.php"); //On inclut les fonctions PDO (create / update etc...)
require("./class/Combattant.php"); //On requiere le fichier.
 
if (isset($_GET['action']))
{
	$action = $_GET['action'];
}
 
switch ($action)
{
	case "ajouter": //On inclut la page pour ajouter une situation.
            include("./page/combattant/ajouter.php");
            break;
	case "traitement_ajout": //On inclut la page qui insère la situation en base.
            $combattant = new Combattant();
            $combattant->setNomCombattant($_POST['NOMCOMBATTANT']);
            $combattant->setPrenomCombattant($_POST['PRENOMCOMBATTANT']);
            $combattant->setSurnomCombattant($_POST['SURNOMCOMBATTANT']);
            $combattant->setAllongeCombattant($_POST['ALLONGECOMBATTANT']);
            $combattant->setTailleCombattant($_POST['TAILLECOMBATTANT']);
            $combattant->setPoidsCombattant($_POST['POIDSCOMBATTANT']);
            $combattant->setPhotoCombattant($_POST['PHOTOCOMBATTANT']);
            $combattant->setIdPays($_POST['IDPAYS']);
            $oPDO= new PdoCombattant();
            $oPDO->create($combattant);
            break;
	case "lister":
            $oPDO = new PdoCombattant();
            $lesCombattants = $oPDO->readAll(); //ligne est un array contenant toutes les situations
            include("./page/combattant/lister.php");
            break;
        case "modifier":
            include("./page/combattant/modifier.php");
            break;
        case "traitement_modif":
            $combattant = new Combattant();
            $combattant->setIdCombattant($_POST['IDCOMBATTANT']);
            $combattant->setPrenomCombattant($_POST['PRENOMCOMBATTANT']);
            $combattant->setNomCombattant($_POST['NOMCOMBATTANT']);
            $combattant->setSurnomCombattant($_POST['SURNOMCOMBATTANT']);
            $combattant->setAllongeCombattant($_POST['ALLONGECOMBATTANT']);
            $combattant->setTailleCombattant($_POST['TAILLECOMBATTANT']);
            $combattant->setPoidsCombattant($_POST['POIDSCOMBATTANT']);
            $combattant->setPhotoCombattant($_POST['PHOTOCOMBATTANT']);
            $combattant->setIdPays($_POST['IDPAYS']);
 
            $pdo = new PDOfonction();
            $pdo->update($combattant);
            break;
        case "supprimer":
            $combattant = new Combattant();
            $combattant->setIdCombattant($_GET['IDCOMBATTANT']);
            $oPDO = new PdoCombattant();
            $oPDO->delete($combattant);
            break;
}

PS: Merci ABCIWEB pour ta correction ;)

09/04/2013, 17h38
ABCIWEB

Tu ne parviens pas à quoi ? Problème de méthode ou de syntaxe ? Essaies de préciser et montres nous ce que tu as essayé.

C'est un problème de méthode. J'ai bien réussi de cette manière :

ajouter.php (combattant)

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
42
43
44
45
46
47
48
49
50
51
52
53
54
<form id="validate_form" action="index.php?route=combattant&action=traitement_ajout" method="post">
 
    Prénom : <input type="text" name="PRENOMCOMBATTANT" id="prenom" class="validate[required]"/><span id="mess_exist"></span>
    <br/>
    Nom : <input type="text" name="NOMCOMBATTANT" id="nom" class="validate[required]"/><span id="mess_exist"></span>
    <br/>
    Surnom : <input type="text" name="SURNOMCOMBATTANT" id="surnom" class="validate[required]"/><span id="mess_exist"></span>
    <br/>
    Allonge : <input type="text" name="ALLONGECOMBATTANT" id="allonge" class="validate[required]"/><span id="mess_exist"></span>
    <br/>
    Taille : <input type="text" name="TAILLECOMBATTANT" id="taille" class="validate[required]"/><span id="mess_exist"></span>
    <br/>
    Poids : <input type="text" name="POIDSCOMBATTANT" id="poids" class="validate[required]"/><span id="mess_exist"></span>
    <br/>
    Photo : <input type="text" name="PHOTOCOMBATTANT" id="photo" class="validate[required]"/><span id="mess_exist"></span>
    <br/>
    <?php
    $conn = mysql_connect("localhost", "root", "");
    if (!$conn) {
        echo "Impossible de se connecter à la base de données : " . mysql_error();
        exit;
    }
 
    if (!mysql_select_db("ufc")) {
        echo "Impossible de sélectionner la base mydbname : " . mysql_error();
        exit;
    }
    $requete = "SELECT * FROM pays ORDER BY NOMPAYS"; //retourne tous les enregistrements correspondants à la requete
    $pays = mysql_query($requete);
 
    if (!$pays) {
        echo "Impossible d'exécuter la requête ($requete) dans la base : " . mysql_error();
        exit;
    }
 
    if (mysql_num_rows($pays) == 0) {
        echo "Aucune ligne trouvée, rien à afficher.";
        exit;
    } else {
        echo 'Pays/Nationalité:';
        echo '<select name="IDPAYS" size="1">';
        $row = mysql_fetch_assoc($pays);
 
 
        while ($row) {
            echo '<option value="' . $row["IDPAYS"] . '">' . $row["NOMPAYS"] . ' ' . $row["NOMNATIONALITE"] . '</option>';
            $row = mysql_fetch_assoc($pays);
        }
 
        echo '</select>';
    }
    ?>
    <input type="submit" id="valider" />
</form>

mais mon enseignant m'a dit de ne pas utiliser du php dans le formulaire. Il souhaite que j'appelle la fonction readOne de pays et que je récupère le résultat dans ajouter/modifier.

10/04/2013, 06h47
ABCIWEB

Le problème est qu'on n'est pas à la place de ton enseignant pour savoir ce qu'il veut exactement.

Et tu ne peux pas appeler la classe "pays" depuis ton contrôleur ?
12/04/2013, 15h03
Crema

Mon enseignant m'a proposé d'importer tous les Pdo et class sur mon index ;)
Après, je suis pas sûre que ça fonctionne.
20/04/2013, 10h13
Crema

Personne ? TT
J'ai besoin de faire un INNER JOIN entre ma table pays et combattant afin de pouvoir renseigner l'IdPays (clé étrangère) quand j'ajoute un combattant mais aussi pour pouvoir récupérer le pays du combattant que je veux modifier. J'ai tout essayé. N'importe quelle solution qui marche sera la bienvenue.