rowCount mal placé ?

Version imprimable

Bonjour,

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

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
<?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.

04/10/2017, 12h00
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);.
04/10/2017, 12h11
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 ?
04/10/2017, 15h31
Celira
Code:

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.
04/10/2017, 15h34
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:

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

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:

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

04/10/2017, 16h15
Celira

Tu n'as pas bien compris : si execute plante, il va renvoyer une exception de toute manière. Donc, il va sauter directement au catch et ne rentrera pas dans la fonction de test du retour. Donc tu dois bien conserver ton catch, c'est le throw qui ne sert à rien.

Du coup, ça deviendrait:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
                $SQL = "UPDATE connectes SET timestamp=:time WHERE ip=:ip";
                try 
                    {
                        $stmt = $pdo->prepare($SQL);
                        $stmt->execute(array(
                            ':ip' => $ip,
                            ':time' => $time,
                        ));
 
                        $rows = $stmt->rowCount();
                        echo $rows;
                    }
 
                catch (PDOException $pdoE)
                    {
                        echo "Erreur MAJ".$pdoE->getMessage();
                    }

C'est ça ?

04/10/2017, 16h50
Celira

C'est bien ça :D

Du coup, problème :resolu: ?
04/10/2017, 20h04
Denis Placé

Yes, indeed !