MySQL/PHP et PDO modification d'une table

Version imprimable

Bonsoir à toutes et à tous,
Je suis un néophyte complet pour MySql, j'ai lu sur les tutoriels qu'il valait mieux utiliser pdo et les requêtes préparées. Mais c'est bien au delà de mes capacités.
J'ai glané de-ci de-là de quoi écrire ma requête ci-dessous mais c'est visiblement insuffisant. J'ai collé en dessous les messages d'erreur que je reçois lorsque je tente le $_post.
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 //dans le <head></head> <?php function SauveCompet($Nom___PJ, $PrenomPJ, $PtCmpOcT, $PtCmpOcD, $Cpt_Conn, $Cpt_SvFr, $Cpt_Sens, $Cpt_Infl, $Cpt_Actn, $DateMmaJ) { $serveur = "127.0.0.1"; // Le serveur $nom_base = "adc"; // La Base de données $login = "MonLog"; // le login "" $pwd = "MonPsWrd"; // Le mot de passe "" $charset = "utf8"; // Le charset "utf8" //-------------------------------------------------------------------------------------------------------------------------------------------------------------------- // MA sauvegarde dans la base 'adc' table 'pj_pnj' champs compétences : `PtCmpOcT`, `PtCmpOcD`, `Cpt_Conn`, `Cpt_SvFr`, `Cpt_Sens`, `Cpt_Infl`, `Cpt_Actn`, `DateMmaJ` //-------------------------------------------------------------------------------------------------------------------------------------------------------------------- // connexion à MySQL en utilisant PDO try // PHP essaie d'exécuter les instructions à l'intérieur du bloc try. S'il y a une erreur, il rentre dans le bloc catch et fait ce qu'on lui demande (ici, on arrête l'exécution de la page en affichant un message décrivant l'erreur). { $bdd = new PDO('mysql:host='.$serveur.';dbname='.$nom_base.';charset='.$charset, $login, $pwd); } catch (Exception $e) // S'il y a une erreur, il rentre dans le bloc catch et fait ce qu'on lui demande (ici, on arrête l'exécution de la page en affichant un message décrivant l'erreur). { die('Erreur : ' . $e->getMessage()); // En cas d'erreur, on affiche un message et on arrête tout } // Si tout va bien, on peut continuer // On modifie une entrée dans la table pj_pnj // Utilisation d'une requète préparée $req = $bdd->prepare('UPDATE pj_pnj SET PtCmpOcT = :PtCmpOcT, PtCmpOcD = :PtCmpOcD, Cpt_Conn = :Cpt_Conn, Cpt_SvFr = :Cpt_SvFr, Cpt_Sens = :Cpt_Sens, Cpt_Infl = :Cpt_Infl, Cpt_Actn = :Cpt_Actn, DateMmaJ = :DateMmaJ WHERE Nom___PJ = :Nom___PJ AND PrenomPJ = :PrenomPJ'); $req->execute(Array( ':PtCmpOcT' => $PtCmpOcT, ':PtCmpOcD' => $PtCmpOcD, ':Cpt_Conn' => $Cpt_Conn, ':Cpt_SvFr' => $Cpt_SvFr, ':Cpt_Sens' => $Cpt_Sens, ':Cpt_Infl' => $Cpt_Infl, ':Cpt_Actn' => $Cpt_Actn, ':DateMmaJ' => $DateMmaJ )); } ?>
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 //dans le <body></body> <?php $Nom___PJ = $_POST['NomDuPj']; $PrenomPJ = $_POST['PrenomDuPj']; $DateMmaJ = $_POST['DateDeMaJ']; $PtCmpOcT = $_POST['PtCptOcT']; $PtCmpOcD = $_POST['PtCpOcDp']; $Cpt_Conn = str_replace("\n", "|", $_POST['CCnnChx']); $Cpt_SvFr = str_replace("\n", "|", $_POST['CSvFChx']); $Cpt_Sens = str_replace("\n", "|", $_POST['CSnsChx']); $Cpt_Infl = str_replace("\n", "|", $_POST['CInfChx']); $Cpt_Actn = str_replace("\n", "|", $_POST['CActChx']); SauveCompet($Nom___PJ, $PrenomPJ, $PtCmpOcT, $PtCmpOcD, $Cpt_Conn, $Cpt_SvFr, $Cpt_Sens, $Cpt_Infl, $Cpt_Actn, $DateMmaJ); ?>
Les messages d'erreur

Citation:

Notice: Undefined index: NomDuPj in D:\wamp\www\Cthulhu\_MIcptPJtt.php on line 92
Call Stack
# Time Memory Function Location
1 0.0004 160304 {main}( ) ..\_MIcptPJtt.php:0

Citation:

( ! ) Notice: Undefined index: PrenomDuPj in D:\wamp\www\Cthulhu\_MIcptPJtt.php on line 93
Call Stack
# Time Memory Function Location
1 0.0004 160304 {main}( ) ..\_MIcptPJtt.php:0

Citation:

( ! ) Notice: Undefined index: DateDeMaJ in D:\wamp\www\Cthulhu\_MIcptPJtt.php on line 94
Call Stack
# Time Memory Function Location
1 0.0004 160304 {main}( ) ..\_MIcptPJtt.php:0

Citation:

( ! ) Notice: Undefined index: PtCptOcT in D:\wamp\www\Cthulhu\_MIcptPJtt.php on line 95
Call Stack
# Time Memory Function Location
1 0.0004 160304 {main}( ) ..\_MIcptPJtt.php:0

Citation:

( ! ) Notice: Undefined index: PtCpOcDp in D:\wamp\www\Cthulhu\_MIcptPJtt.php on line 96
Call Stack
# Time Memory Function Location
1 0.0004 160304 {main}( ) ..\_MIcptPJtt.php:0

Citation:

( ! ) Warning: PDOStatement::execute(): SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens in D:\wamp\www\Cthulhu\_MIcptPJtt.php on line 53
Call Stack
# Time Memory Function Location
1 0.0004 160304 {main}( ) ..\_MIcptPJtt.php:0
2 0.0667 162000 SauveCompet( ) ..\_MIcptPJtt.php:102
3 0.0674 170296 execute ( ) ..\_MIcptPJtt.php:53

Je ne maitrise pas bien la langue de "j'expire"!
Mais je crois que çà dit que le nombre de paramètres ne correspond pas avec ...
des index indéfinis...
Voici la copie des champs de la table, je ne crois pas m'être trompé dans mes 'copier/coller' pour écrire la requête.
Code:

1 2 3 4 -- -- Contenu de la table `pj_pnj` -- INSERT INTO `pj_pnj` (`id_PjPnj`, `Nom___PJ`, `PrenomPJ`, `DatNaiss`, `SEXE_M_F`, `Nationlt`, `APParenc`, `CONstit_`, `DEXterit`, `FORce___`, `TAIlle__`, `EDUcatio`, `INTellig`, `POUvoir_`, `Prestanc`, `Enduranc`, `Agilite_`, `Puissanc`, `Corpulen`, `Connaiss`, `Intuitio`, `Volonte_`, `PtCmpOcT`, `PtCmpOcD`, `PtCmpPtT`, `PtCmpPtD`, `Aplomb__`, `SANteMen`, `PointVie`, `Blessure`, `PointMag`, `Impact__`, `Occupat1`, `Occupat2`, `Professn`, `Personna`, `NiveauVi`, `Cpt_Conn`, `Cpt_SvFr`, `Cpt_Sens`, `Cpt_Infl`, `Cpt_Actn`, `DateMmaJ`) VALUES
Merci d'avance à celles/ceux qui voudront bien éclairer ma lanterne.

Salut MelkInarian.

Citation:

Envoyé par MelkInarian

j'ai lu sur les tutoriels qu'il valait mieux utiliser pdo et les requêtes préparées.

Vous êtes français alors exprimez-vous correctement. On dit didacticiel et non tutoriel qui est un franglais.

Citation:

Le mot anglais tutorial, dont « tutoriel » est issu, déconseillé par les organismes de néologie français et québécois, est cependant également utilisé par certains francophones.

Pour répondre à votre question, oui, il est conseillé d'utiliser le PDO à la place de mysqli.

Ensuite, vous vous êtes trompé de forum qui est consacré exclusivement à MySql et non à MySql/php.
Je vais quand même répondre à votre question.

Voici la ligne pour vous connecter à votre base de données :

Code:

$bdd = new PDO('mysql:host='.$serveur.';dbname='.$nom_base.';charset='.$charset, $login, $pwd);

Le problème que je constate est que vous mettez le charset au mauvais endroit dans votre ligne ci-dessus.
Normalement, cela se met à la fin, dans la partie "option", donc après le mot de passe.

Voici un exemple qui fonctionne, pour se connecter à une base de données en php/pdo :
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 <?php /*-----------------------------------------------------*/ /* Informations pour la connexion à la base de données */ /*-----------------------------------------------------*/ $nom_du_serveur = "mysql:host=localhost;dbname=test"; $nom_utilisateur = "test"; $mot_de_passe = "test"; $options = array( PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES latin1", PDO::ATTR_CASE => PDO::CASE_LOWER, PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION, PDO::ATTR_PERSISTENT => false ); /*----------------------*/ /* Connexion au Serveur */ /*----------------------*/ try { $link = new PDO($nom_du_serveur, $nom_utilisateur, $mot_de_passe, $options); } catch (Exception $e) { echo "Connection à MySql impossible : ", $e->getMessage(); die(); } ?>
J'utilise dans ma base de données, le charset "latin1", c'est-à-dire "iso-8859-1".

Pour la ligne update, je n'aime pas trop cette forme d'écriture, en ce qui concerne les requêtes préparées. Je préfère l'écriture suivante :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 try { $ins = $link->prepare("INSERT INTO `tab_clients` (`nom`, `prenom`, `naissance`, `ville`, `datenreg`) VALUES (:param1, :param2, :param3, :param4, :param5)"); $ins->bindParam('param1', $nom); $ins->bindParam('param2', $prenom); $ins->bindParam('param3', $naissance); $ins->bindParam('param4', $ville); $ins->bindParam('param5', $datenreg); $ins->execute(); } catch ( Exception $e ) { $link->rollback(); echo "Une erreur est survenue dans la requète insert : ", $e->getMessage(); die(); }
C'est un peu plus bavard, mais au moins, cela à le mérite de ne pas être positionnel, comme dans l'usage du "array" que vous faites.

@+

Cela fait des heures que j'essaye de modifier la reqête de Artemus24 pour l'adapter à un update mais je n'y arrive pas.
J'ai toujour le message d'erreur suivant. (la ligne 75 correspond à la ligne 52 ci dessous.)

Citation:

( ! ) Fatal error: Uncaught exception 'PDOException' with message 'There is no active transaction' in D:\wamp\www\Cthulhu\_MIcptPJtt.php on line 75
( ! ) PDOException: There is no active transaction in D:\wamp\www\Cthulhu\_MIcptPJtt.php on line 75
Call Stack
# Time Memory Function Location
1 0.0004 170728 {main}( ) ..\_MIcptPJtt.php:0
2 0.0005 174048 SauveCompet( ) ..\_MIcptPJtt.php:235
3 0.0019 190104 rollBack ( ) ..\_MIcptPJtt.php:75

voici mon code après modification pour tenir compte des suggestions de Artemus24.
C'est peut-être au niveau du WHERE bien que le message d'erreur dise qu'il n'y a pas de transaction active.
La connexion à la base se fait je crois. le "echo "Connection à MySql réussie : ";", que j'ai mis après coup pour vérifier, s'affiche.
D'autre part, j'ai l'ensemble en UTF-8 (pages, tables) j'espère que le Latin1 ne me mettra pas le bazar, mais on verra çà plus tard.
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 <?php function SauveCompet($Nom___PJ, $PrenomPJ, $PtCmpOcT, $PtCmpOcD, $Cpt_Conn, $Cpt_SvFr, $Cpt_Sens, $Cpt_Infl, $Cpt_Actn, $DateMmaJ) { /*-----------------------------------------------------*/ /* Informations pour la connexion à la base de données */ /*-----------------------------------------------------*/ $nom_du_serveur = "mysql:host=localhost;dbname=adc"; $nom_utilisateur = "MonNom"; $mot_de_passe = "MonPsswrd"; $options = array( PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES latin1", PDO::ATTR_CASE => PDO::CASE_LOWER, PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION, PDO::ATTR_PERSISTENT => false ); /*---------------------------------------*/ /* Connexion au Serveur en utilisant PDO */ /*---------------------------------------*/ try { $link = new PDO($nom_du_serveur, $nom_utilisateur, $mot_de_passe, $options); echo "Connection à MySql réussie : "; } catch (Exception $e) { echo "Connection à MySql impossible : ", $e->getMessage(); die(); } try { $upd = $link->prepare("UPDATE pj_pnj SET (`PtCmpOcT`, `PtCmpOcD`, `Cpt_Conn`, `Cpt_SvFr`, `Cpt_Sens`, `Cpt_Infl`, `Cpt_Actn`, `DateMmaJ`) VALUES (:PtCmpOcT, :PtCmpOcD, :Cpt_Conn, :Cpt_SvFr, :Cpt_Sens, :Cpt_Infl, :Cpt_Actn, :DateMmaJ) WHERE Nom___PJ = :Nom___PJ AND PrenomPJ = :PrenomPJ"); $upd ->bindParam('PtCmpOcT', $PtCmpOcT); $upd ->bindParam('PtCmpOcD', $PtCmpOcD); $upd ->bindParam('Cpt_Conn', $Cpt_Conn); $upd ->bindParam('Cpt_SvFr', $Cpt_SvFr); $upd ->bindParam('Cpt_Sens', $Cpt_Sens); $upd ->bindParam('Cpt_Infl', $Cpt_Infl); $upd ->bindParam('Cpt_Actn', $Cpt_Actn); $upd ->bindParam('DateMmaJ', $DateMmaJ); $upd ->execute(); } catch ( Exception $e ) { $link->rollback(); echo "Une erreur est survenue dans la requète update : ", $e->getMessage(); die(); } } ?>

Salut MelkInarian.

En lisant votre message d'erreur, je constate que le mode transaction n'est pas activé.

Code:

There is no active transaction' in D:\wamp\www\Cthulhu\_MIcptPJtt.php on line 75

Essayez de mettre ceci (ce qui est en rouge) à l'ouverture de votre connexion :
Code:

1 2 3 4 5 6 $options = array( PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES latin1", PDO::ATTR_CASE => PDO::CASE_LOWER, PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION, PDO::ATTR_AUTOCOMMIT => false );
@+

Bonjour Artemus24,
C'est malheureusement pareil.

Citation:

Fatal error: Uncaught exception 'PDOException' with message 'There is no active transaction' in D:\wamp\www\Cthulhu\_MIcptPJtt.php on line 76
( ! ) PDOException: There is no active transaction in D:\wamp\www\Cthulhu\_MIcptPJtt.php on line 76
Code:

1 2 3 4 5 6 7 $options = array( PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES latin1", PDO::ATTR_CASE => PDO::CASE_LOWER, PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION, //PDO::ATTR_PERSISTENT => false PDO::ATTR_AUTOCOMMIT => false );

J'ai essayé de simplifier au maximum

Bonjour à toutes et à tous,
J'ai simplifié au maximum mon code

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
<?php
    function SauveCompet($id_PjPnj, $Nom___PJ, $PrenomPJ, $PtCmpOcT, $PtCmpOcD, $Cpt_Conn, $Cpt_SvFr, $Cpt_Sens, $Cpt_Infl, $Cpt_Actn, $DateMmaJ)
    {
    /*-----------------------------------------------------*/
    /* Informations pour la connexion à la base de données */
    /*-----------------------------------------------------*/
        $nom_du_serveur  = "mysql:host=localhost;dbname=adc";
        $nom_utilisateur = "MonUtil";
        $mot_de_passe    = "MonMotDePasse";
 
        $options         = array(
            PDO::MYSQL_ATTR_INIT_COMMAND => "set names utf8",
            PDO::ATTR_CASE               => PDO::CASE_LOWER,
        );
        /*---------------------------------------*/
        /* Connexion au Serveur en utilisant PDO */
        /*---------------------------------------*/
 
        try //  PHP essaie d'exécuter les instructions à l'intérieur du bloc try. S'il y a une erreur, il rentre dans le bloc catch et fait ce qu'on lui demande (ici, on arrête l'exécution de la page en affichant un message décrivant l'erreur).
        {
            $link = new PDO($nom_du_serveur, $nom_utilisateur, $mot_de_passe, $options); // Instanciation de l'objet PDO (notre connexion)
            echo "Connection à MySql réussie ! " . '<br />';
        }
        catch (Exception $e) // S'il y a une erreur, il rentre dans le bloc catch et fait ce qu'on lui demande (ici, on arrête l'exécution de la page en affichant un message décrivant l'erreur).
        {
            echo "Connection à MySql impossible : ", $e->getMessage();
            die();    // En cas d'erreur, on affiche un message et on arrête tout
        }
        // Si tout va bien, on peut continuer
 
        try {
            $update  = $link->prepare("UPDATE pj_pnj SET DateMmaJ=:DateMmaJ WHERE id_PjPnj=:id_PjPnj");
 
            $update->bindParam('DateMmaJ', $DateMmaJ);
 
            $update ->execute();
        }
        catch ( Exception $e )
        {
            $link->rollback();
            echo "Une erreur est survenue dans la requète update : ", $e->getMessage();
            die();
        }                
        $update->closeCursor();
        $link = null;    // et maintenant, je ferme la connexion PDO (http://php.net/manual/fr/pdo.connections.php)
    }
?>

et j'ai toujours cette erreur:

Citation:

Warning: PDOStatement::execute(): SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens in D:\wamp\www\Cthulhu\_MIcptPJtt.php on line 99
Call Stack
# Time Memory Function Location
1 0.0004 168712 {main}( ) ..\_MIcptPJtt.php:0
2 0.0005 172048 SauveCompet( ) ..\_MIcptPJtt.php:301
3 0.0015 179944 execute ( ) ..\_MIcptPJtt.php:99

Je n'ai jamais le message en français : echo "Une erreur est survenue dans la requète update : ", $e->getMessage();
Je crois que je vais devoir abandonner PDO et essayer de rentrer mes modifications "à l'ancienne" car là je suis désabusé.:traine:.
Si quelqu'un à une idée avant çà, je suis preneur.
Merci

31/01/2017, 18h00
sabotage

L'erreur est claire : il manque un paramètre.
En l'occurence :id_PjPnj
31/01/2017, 18h14
MelkInarian

:merci:Bonsoir sabotage,

Citation:

L'erreur est claire : il manque un paramètre.
En l'occurence :id_PjPnj

C'est clair ! Et toutes ces heures passées sans le voir.
"Il vaut mieux un qui sait que dix qui cherchent", disait le sage!
Merci