conflit requête UPDATE

Version imprimable

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
 
if(isset($_POST['emailVerif'])){
 
    $email=htmlspecialchars($_POST['email']);
    $query=$db->query("SELECT email, nom, prenom, phone FROM clients WHERE email = '$email' ");
    $count= $query->rowCount(); 
    if($count == 1){
         while($cont=$query->fetch()){
             //echo $cont[1];
             $cl_email=$cont['email'];
             $cl_nom=$cont['nom'];
             $cl_prenom=$cont['prenom'];
             $cl_phone=$cont['phone'];
         }
    }else{ $mess='le client n\'existe pas';}
}
 
if(isset($_POST['action'])) {
    $nom=htmlspecialchars($_POST['nom']);
    $prenom=htmlspecialchars($_POST['prenom']);
    $email=htmlspecialchars($_POST['email']);
    $phone=htmlspecialchars($_POST['phone']);
    $dateDebut =htmlspecialchars($_POST['debut']);
    $dateFin =htmlspecialchars($_POST['fin']);
 
            if(!empty($email)&& !empty($nom)&& !empty($prenom)) {
                try{
                require("../includes/bdd/config.php");
                $query=$db->query("SELECT email, nom, prenom, phone FROM clients WHERE email = '$email' ");
                    $count = $query->rowCount(); 
                    if($count == 1){
                        //echo $nom;
                    //_____________________________ LE PROBLEME EST ICI
                        $req=(" UPDATE clients SET email='$email',phone='$phone', WHERE email=$email ");
                        $req->execute();
                    //__________________________________________________   
 
                    }else{
                        $req=$db->prepare("INSERT INTO clients  VALUES ('$email','$nom','$prenom', '$phone')");
                        $req->execute();
                    }
 
                    $req=$db->prepare("INSERT INTO reservation VALUES ('$titre', '$description','$dateDebut','$dateFin')");
                    $req->execute();
 
                    $msg="Evénement enregistré !";
 
                }catch(PDOEXCEPTION $e) {
                    echo $e;
                }
 
            }
            }
        }

Bonjour à tous,

je débute avec pdo et je suis autodidacte... Merci pour votre compréhension, j'essaye de m'améliorer chaque jour !

Je réalise une petite application d'administration : une table 'clients' où il ne peut pas y avoir de redondance (l'id unique au niveau de l'email).

- L'admin vérifie si l'email du client existe via un 'submit' d'un premier formulaire avec un seul 'input'. Si oui, remplissage d'un deuxième formulaire avec les infos de la table clients -> OK
- L'email existe, les infos se remplissent dans le form , il veut mettre à jour l'email du client qui a changé -> ça marche pas, php retourne une page blanche, ça foire au niveau de ' UPDATE clients ' ( quand j'enlève la requête et que je fais un echo '$nom', php me retourne bien la valeur de $nom ).

En fait, tout fonctionne sauf 'UPDATE' ( il n'y a pas de problème de connexion, ';' , ou autres fautes d'inattention )... j'ai fais des recherches mais je ne vois pas.

Merci pour vos lumières

Code:

1 2 3 4 //_____________________________ LE PROBLEME EST ICI $req=(" UPDATE clients SET email='$email',phone='$phone', WHERE email=$email "); $req->execute(); //__________________________________________________
Tu lui demandes un drôle de truc là.
Dans ton interface d'administration de ta base, quand tu testes la requête, tu dois avoir un message d'erreur du genre "on ne peut pas modifier un truc qui est au même moment un critère de filtre". En anglais, probablement, mais l'idée y est.
Autre chose, tu dis que ton identifiant cllient est le mail... pas bien logique. Ca impliquerait qu'il ne peut pas changer de mail.
Le plus logique, c'est un id autoincrémenté, un identifiant unique, du genre construit automatiquement nom_prenom_dateNaissance.
Des trucs qui ne changent pas quoi et tu auras peu de chance d'avoir deux fois le même.
Pourquoi un id autoincrementé ? parce que c'est bien pratique le jour où tu veux savoir quel est le dernier client par exemple...
Ensuite, tu pourras faire tous les update que tu veux en filtrant sur cet identifiant par définition impossible à modifier.

Dans une base idéale, tu as ta table client avec que des données du type 1 client = 1 enregistrement. (id, identifiant, nom, prenom,genre,date_naissance,date_inscription)
Et une autre table téléphone, reliant ton id client avec n téléphones possibles, une table mail reliant ton id client avec n mails possibles, une table compte reliant ton id client avec n comptes possibles etc.
Et après, tu remets tout ça dans une vue plus facile à appréhender pour nous.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 create or replace view v_clients as select client.id, client.identifiant, client.nom, client.prenom, client.genre, date_format(client.date_naissance,"%d/%m/%Y") as naissance, date_format(client.date_inscription,"%d/%m/%Y") as inscription, mail.libelle as email, telephone.libelle as tel from client inner join mail on client.id=mail.client_id /*ici au moins un mail est obligé sinon client disparaît*/ left outer join telephone on client.id=telephone.client_id /*ici on imagine que tous les clients n'ont pas forcément un tél*/
PS : ne pas mettre de order by dans une vue, ça ne fait que consommer pour rien.
Quand tu requêteras sur la vue, tu poseras ton filtre et ton order by.

15/03/2014, 16h34
WenPro

Merci Dendrite pour cette réponse rapide et précise,

j’ai compris le fait de séparer les données en plusieurs tables pour la maintenance et la notion d’UPDATE. Aussi, le fait qu’un email ne peut être l’id de la table.
J’ai de mauvaises bases et je dois les revoir… Je manque de temps.

Encore merci pour tes conseils, je vais tenter de les mettre en pratique et aussi lire ton tuto sur PHP ( ça ne me fera pas de tort non plus !).

Il y avait aussi en plus une grosse faute :
Code:

1 2 3 4 5 //_____________________________ LE PROBLEME EST ICI $req=(" UPDATE clients SET email='$email',phone='$phone', WHERE email=$email "); $req->execute(); //__________________________________________________
... l'oubli de la connexion préparée à la db :
$req=$db->prepare(" UPDATE clients SET email='$email',phone='$phone', WHERE email=$email ");) ... Je ne sais pas comment j'ai fait pour ne pas la voir ! (trop le nez dans le guidon).