Modification de la base de données

**Markos22** · 24/10/2016, 11h35

Bonjour,

Je ne comprends pas comment modifier les enregistrement d'une base de données à partir d'un formulaire php. Voici ce que j'ai essayé :

J’ai une base de données comprenant une liste de biens immobiliers.
Lorsque je clique sur un élément de ma liste de biens, j’envoie une requête GET à mon url. $_GET[‘bien’] contient l’id de mon bien (sa clé primaire auto-incrémentée). Un formulaire de saisie s'affiche alors qui me permet de saisir de nouvelles valeurs pour les champs de mon bien.

Lorsque je soumets le formulaire, je crée donc un objet BienDAO , qui va m’aider à modifier ma base de données.

Code :

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

$dao = new BienDAO() ;

Cet objet me renvoie une connexion à ma base de données. C’est l’objet qui gère les méthodes CRUD avec ma base de données : Je crée donc un bien qui reprend les valeurs des champs de l’enregistrement correspondant à l’id en question dans ma base de données :

Code :

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

$bien = $dao->trouver($_GET['bien']);

La méthode trouver() est définie ainsi :

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
public function trouver($id) {
 
        $id = (int) $id;
 
        $req = $this->bdd->query('SELECT * FROM biens WHERE id = '.$id);
        $donnees = $req->fetch();
 
        $bien_trouve = new Bien();
 
        foreach ($donnees as $attr => $valeur) {
            $setter = 'set'.ucfirst($attr);
 
            $bien_trouve->$setter($valeur);
        }
        return $bien_trouve;
 
    }

Je fais donc :

Code :

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

1
2
3
4
5
6
7
8
foreach ($_POST as $attr => $valeur) {
    $setter = 'set'.ucfirst($attr);
    if(method_exists($dao, $setter)) {
        $bien->$setter($valeur);
    }
}
$bien->setId((int) $_GET['bien']);
$dao->modifier($bien);

Quant à la méthode modifier() la voici :

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
public function modifier(Bien $bien) {
 
        $req = $this->bdd->prepare('UPDATE biens SET type = :type, date_modif = :date_modif, description = :description, code_postal = :code_postal, ville = :ville, voie = :voie, superficie = :superficie, montant = :montant WHERE id = :id');
 
        $req->bindValue(':type',$bien->getType(),PDO::PARAM_STR);
        $req->bindValue(':date_modif',date('Y-m-d'),PDO::PARAM_STR);
        $req->bindValue(':description',$bien->getDescription(),PDO::PARAM_STR);
        $req->bindValue(':code_postal',$bien->getCode_postal(),PDO::PARAM_INT);
        $req->bindValue(':ville',$bien->getVille(),PDO::PARAM_STR);
        $req->bindValue(':voie',$bien->getVoie(),PDO::PARAM_STR);
        $req->bindValue(':superficie',$bien->getSuperficie(),PDO::PARAM_INT);
        $req->bindValue(':montant',$bien->getMontant(),PDO::PARAM_INT);
        $req->bindValue(':id',$bien->getId(),PDO::PARAM_INT);
 
        $req->execute();
        $req->CloseCursor();
    }

Je ne comprends pas pourquoi la modification n’est pas effectuée dans la base de données. Mon objet Bien a toujours les mêmes attributs.

Merci de votre aide,

A bientôt !

**sabotage** · 24/10/2016, 13h19

Mon objet Bien a toujours les mêmes attributs.

Tu veux dire que les valeurs de l'objet "bien" ne sont pas modifiées au moment où tu appelles la méthode "modifier" ?

Dans ton formulaire tu envoies l'id en $_GET dans l'action en plus des autres valeurs en $_POST ?

**Markos22** · 24/10/2016, 14h02

Envoyé par sabotage

Tu veux dire que les valeurs de l'objet "bien" ne sont pas modifiées au moment où tu appelles la méthode "modifier" ?

Dans ton formulaire tu envoies l'id en $_GET dans l'action en plus des autres valeurs en $_POST ?

Oui, c'est exactement ça. Les valeurs de l'objet '$bien' ne sont pas modifiées, et, par conséquent, l'enregistrement en base de données non plus. L'id du bien est GET['bien']. Il est envoyé par clic sur la liste qui est composée de balises <a> comme ceci :

Code :

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

echo '	<a class="list-group-item list-group-item-action" href="form_modif.php?bien='.$bien->getId().'" id="bien-item-'.($bien->getId()).'"><strong>'.($key+1).'. '.$bien->getVoie().', '.$bien->getVille().'</strong><br/>'.$bien->getDescription().'</a>

Cette variable est bien transmise correctement. Je l'ai testée.

**sabotage** · 24/10/2016, 14h30

Mais dans le formulaire il passe comment cet id ?

**Markos22** · 24/10/2016, 14h48

Envoyé par sabotage

Mais dans le formulaire il passe comment cet id ?

Il est transmis en get par l'url, puisque les liens se terminent par href="form_modif.php?bien=xx", il est donc stocké dans la globale $_GET['bien'] et aurait dû être récupéré par $bien = $dao->trouver($_GET['bien']);, qui récupère le contenu de GET['bien'] (donc l'id), et va chercher en base de données l'enregistrement correspondant ("SELECT...WHERE id=...)

**sabotage** · 24/10/2016, 15h59

Tu as contrôlé ce qui se passait dans ta boucle de setter ?
Par exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
foreach ($_POST as $attr => $valeur) {
    $setter = 'set'.ucfirst($attr);
    echo 'setter : ' . $setter;
    if(method_exists($dao, $setter)) {
        $bien->$setter($valeur);
        echo ' trouvé<br/>';
    }
    else {
       echo ' non trouvé<br/>';
    }
}

**Markos22** · 24/10/2016, 16h13

Envoyé par sabotage

Tu as contrôlé ce qui se passait dans ta boucle de setter ?
Par exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
foreach ($_POST as $attr => $valeur) {
    $setter = 'set'.ucfirst($attr);
    echo 'setter : ' . $setter;
    if(method_exists($dao, $setter)) {
        $bien->$setter($valeur);
        echo ' trouvé<br/>';
    }
    else {
       echo ' non trouvé<br/>';
    }
}

Bonne question ! Apparemment, il n'a pas trouvé les méthodes....
(après avoir essayé : )

Merci, @sabotage ! C'était effectivement là : comme d'habitude, une fois qu'on a trouvé l'erreur, on se sent tout bête ! Ca fait 48 heures que je cherche et que je me désespère dessus, alors que je n'utilisais pas le bon objet. Ce n'est pas l'objet BienDAO qui possède les setters, mais l'objet métier "Bien". Donc : au lieu de

Code :

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