Discussion :

[Denis Placé]

Bonjour,

J'exécute ce script destiné à mettre à jour une table.

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
18
19
20
21
22
23
24
25
26
27
28
29
30
<?php
        include "connexion.php";
        $ip=$_SERVER['REMOTE_ADDR'];
        $time=time();
 
                $SQL = "UPDATE connectes SET timestamp=:time WHERE ip=:ip";
                try 
                    {
                        $stmt = $pdo->prepare($SQL);
                        $stmt->execute(array(
                            ':ip' => $ip,
                            ':time' => $time,
                        ));
 
                        if (!$stmt->execute())
                            {
                                throw new PDOException('Erreur Execute requete');
                            }
                        else
                            {
                                $rows = $stmt->rowCount();
                                echo $rows;
                            }
                    }
 
                catch (PDOException $pdoE)
                    {
                        echo "Erreur MAJ".$pdoE->getMessage();
                    }
?>

La MAJ s'effectue correctement mais le script m'affiche 0 alors que je m'attends à trouver 1, c'est-à-dire le nombre de lignes mises à jour.
Merci de m'indiquer où je fais mon erreur.

[Celira]

Ce n'est pas parce que la requête s'exécute correctement que quelque chose est modifié en base.
Par exemple, pour une requête UPDATE ma_table SET mon_champ=0, rowCount va retourner le nombre de lignes modifiées, donc le nombre de lignes pour lesquelles le champ avait une valeur différente de 0 avant la requête.
Dans ton cas, c'est peut-être le where qui fait obstacle : si il n'y a aucune ligne qui a la bonne valeur de "ip" en base (autrement dit, à la 1ere connexion), il n'y aura aucune modification.
Tu peux ajouter une vérification par un select count et faire une insertion au lieu d'une modification si la ligne n'existe pas.

Au passage, il n'est pas nécessaire de faire le lancement des PDOException à la main, il te suffit de les activer à la connexion : $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);.

[Denis Placé]

Bonjour Celira,

Merci pour ton intervention.
Une ligne existait avant et son timestamp a bien été modifié donc dans ce cas on devrait avoir $rows =1, non ?

Edit: Merci également pour le commentaire sur le lancement des PDOException à la main mais je ne le comprends pas totalement. Sachant que mon fichier connexion.php comporte ces lignes, qu'est-ce que je pourrais supprimer dans le script présent ici pour qu'il fonctionne de la même façon ?

[Celira]

Code :

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

1
2
3
4
 if (!$stmt->execute())
{
    throw new PDOException('Erreur Execute requete');
}

execute va lancer une exception si il y a une erreur, donc inutile de la lancer à la main.

[Celira]

En écrivant ça, je vais de réaliser que tu exécutais deux fois la requête : une fois avec des paramètres et une 2e fois dans le if. Du coup, le rowCount doit prendre le compte de la 2e exécution.
Simplifie :

Code :

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

1
2
3
4
5
6
7
8
$stmt = $pdo->prepare($SQL);
$stmt->execute(array(
    ':ip' => $ip,
    ':time' => $time,
));
 
$rows = $stmt->rowCount();
echo $rows;

[Denis Placé]

Celira,

C'est clair et ça fonctionne
Merci beaucoup.

Edit: J'y verrais comme inconvénient qu'un message d'erreur serait moins explicite qu'avec un try/catch dont le catch serait

Code :

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

1
2
3
4
                catch (PDOException $pdoE)
                    {
                        echo "Une erreur s'est produite dans la MAJ du timestamp: <br/>".$pdoE->getMessage();
                    }