Discussion :

[Pandora]

Bonjour à tous,

Je modifie actuellement une interface pour la passer en PDO.
Une bonne partie fonctionne déjà, mais je bute face à un problème où je ne comprend pas la logique.

J'essaye de supprimer un enregistrement d'une table, pour celà je vérifie en premier qu'il existe , puis si c'est le cas je le supprime. Celà fonctionnait bien avant, mais depuis PDO il m'exécute les requetes dans le désordre.

Je veux dire lors de l'execution il supprime bien l'enregistrement puisque l'entrée a disparu de la table, mais il me fait une erreur, comme quoi l'enregistrement a disparu, alors qu'il est sensé vérifier avant de le supprimer son existence.

Je ne comprend pas vraiment tout.

J'utilise PDO sur postgresql.
Normalement j'utilise une classe qui étend PDO pour ma base, mais même en testant une requête sans cette classe le problème est le même.

Pouvez vous m'expliquer ce que je ne comprend pas, ou que je fais mal ?

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
 
try{
$test = new PDO('pgsql:host=127.0.0.1;dbname='.DATABASE_NAME,login,mdp);
$test->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);
$stm = $test->prepare("SELECT id FROM clinical_study WHERE id=:id");
$stm->bindValue('id',$_GET['id_clinical_study'],PDO::PARAM_INT);
$stm->execute();
 
if($stm->rowCount() != 1) echo "BAD";
else {
        echo "OK";
 
        $test2 = new PDO('pgsql:host=127.0.0.1;dbname='.DATABASE_NAME,login,mdp);
        $stm2 = $test2->prepare("DELETE FROM clinical_study WHERE id=:id");
        $stm2->bindValue('id',$_GET['id_clinical_study'],PDO::PARAM_INT);
        if($stm2->execute()) echo "DEL";
        else echo "PB";
}
} catch (PDOException $e) {
        print($e->getMessage());
}

A l'affichage : J'obtiens juste "BAD" et mon entrée a bien été supprimée.
Je ne vois jamais arriver "DEL" ou "PB" ou "OK".

Merci d'avance si quelqu'un peut m'expliquer.

[baderahmed]

Pour la plupart des bases de données, PDOStatement::rowCount() ne retourne pas le nombre de lignes affectées par une requête SELECT.
utilise fetchColumn :

Code :

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

1
2
 
if($stm->fetchColumn() > 0) echo "BAD";

[Pandora]

Malheureusement je viens de tester avec fetch et celà ne change rien.

La valeur est toujours supprimée avant d'être vérifiée.

(A savoir que si je supprime la partie delete que ce soit avec rowCount ou bien fetch la requete select fonctionne correctement).

[baderahmed]

il faut mettre la chaine ':id' au lieu de 'id'

Code :

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

1
2
3
4
 
$stm->bindValue(':id',$_GET['id_clinical_study'],PDO::PARAM_INT);
 
$stm2->bindValue(':id',$_GET['id_clinical_study'],PDO::PARAM_INT);

[Pandora]

Testé le changement de syntaxe des bind, et celà ne change rien.

Merci quand même d'essayer.

Si quelqu'un a une autre idée

[Djakisback]

Avec ce code, c'est impossible que "BAD" s'affiche et que la requête DELETE soit exécutée dans le même script :

Code :

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

1
2
3
4
5
6
7
8
9
10
if($stm->rowCount() != 1) echo "BAD";
else {
        echo "OK";
 
        $test2 = new PDO('pgsql:host=127.0.0.1;dbname='.DATABASE_NAME,login,mdp);
        $stm2 = $test2->prepare("DELETE FROM clinical_study WHERE id=:id");
        $stm2->bindValue(':id',$_GET['id_clinical_study'],PDO::PARAM_INT);
        if($stm2->execute()) echo "DEL";
        else echo "PB";
}

Tu es sûre que ce n'est pas quand tu vérifies manuellement dans ta BDD que tu te plantes et/ou que tu refresh dans ton navigateur ? (bref le code me semble correct, mis à part que tu peux utiliser le même objet PDO a priori)

[rawsrc]

Salut,

comme ça :

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
31
32
33
<?php
 
if ( ! empty($_GET['id_clinical_study']))
{
    $id = intval($_GET['id_clinical_study']);
    try
    {
        $pdo  = new PDO('pgsql:host=127.0.0.1;dbname='.DATABASE_NAME, $login, $mdp);
        $pdo->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);
        $sql  = 'SELECT id FROM clinical_study WHERE id=:id';
        $stmt = $pdo->prepare($sql);
        $stmt->bindValue('id', $id, PDO::PARAM_INT);
        $stmt->execute();
        $data = $stmt->fetchAll(PDO::FETCH_ASSOC);
 
        if (empty($data))
        {
            echo 'BAD';
        }
        else
        {
            echo 'OK';
            $sql  = 'DELETE FROM clinical_study WHERE id=:id';
            $stmt = $pdo->prepare($sql);
            $stmt->bindValue('id', $id, PDO::PARAM_INT);
            echo ($stmt->execute()) ? 'DEL' : 'PB';
        }
    }
    catch (PDOException $e)
    {
            echo $e->getMessage();
    }
}

Le ->rowCount() sur un select n'est pas garanti alors, vaut mieux s'en passer.

[sabotage]

Sur le problème je pense comme Djakisback que tu fais une erreur de lecture de tes résultats : tu ne peux pas avoir le if et le else en meme temps.

Sur le fond, ton select au début ne sert pas, tu peux faire le DELETE même si l'enregistrement n'existe pas. rowCount peut te dire si une ligne a été supprimée ou non.

[Pandora]

En lisant vos remarques, j'ai sorti la requête et je l'ai mise dans une page seule hors du contexte de l'interface.
Et effectivement dans ce cas là la requête a un comportement normal.

Le problème est donc en amont quelque chose doit influer sur la page avant l'include, ou peut-être la page incluse deux fois de suite (ce qui m'a l'air cohérent avec le comportement.). Je vais donc regarder de ce côté là.

(Pour le select qui ne sert pas il est là apparement pour vérifier que l'utilisateur n'a pas changé à la main le numéro dans l'url, je n'ai pas vraiment investigué sur les raisons, vu que je ne suis pas auteur du code originel)

Merci à tous d'avoir pris le temps de jeter un oeil et de m'avoir "rassurée" sur mon bout de code qui me rendait un peu chèvre à force de tourner en rond.