Discussion :

[bndd24]

Bonjour,

Voici mon code pour vérifier si un enregistrement existe.

Mon code fonctionne mais faut t'il plutôt privilégier fetchColumn ?

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
$idann = (int) $_GET['idann'];
 
$req = $pdo->prepare('SELECT a.idann, a.content, a.level, a.location, a.amount, u.username, u.email
FROM ad a
INNER JOIN users u ON a.idmbr = u.idmbr
WHERE a.idann = ?');
$req->execute([$idann]);
$ad = $req->fetch();
 
if (!$ad) {
    header('Location: index.php?p=ad/search');
    exit();
}

et je ne comprends pas bien aussi le rowcount, http://php.net/manual/fr/pdostatement.rowcount.php regardez l'exemple numéro 2 "Comptage des lignes retournées par une requête SELECT"

Je cite la doc php :

Pour la plupart des bases de données, PDOStatement::rowCount() ne retourne pas le nombre de lignes affectées par une requête SELECT. À la place, utilisez PDO::query() pour faire une requête SELECT COUNT(*), puis utilisez PDOStatement::fetchColumn() pour récupérer le nombre de lignes retournées. Votre application peut ainsi effectuer la bonne action.

donc je ne comprends pas, mon code fonctionne oui, mais est-ce qu'il existe une méthode plus "propre" pour vérifier qu'un enregistrement existe ?

merci pour votre aide.

[Invité]

Bonjour,

Code sql :

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

SELECT COUNT(id) AS nbr_lignes WHERE ...

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
$req = $pdo->prepare("SELECT COUNT(a.idann) AS nbre_lignes
FROM ad a
INNER JOIN users u ON a.idmbr = u.idmbr
WHERE a.idann = ? ;");
$req->execute([$idann]);
$ad = $req->fetch();
 
$nbreLignes = $ad['nbre_lignes'];
 
if ($nbreLignes!=1) {
    header('Location: index.php?p=ad/search');
    exit();
}

N.B.

1/ Je n'ai jamais utilisé fetchColumn() !

2/ Malgré l'avertissement, j'utilise aussi (sans problème jusqu'ici) rowCount() pour mes requêtes SELECT.

[bndd24]

ok donc quasi la même chose que mon code mais avec COUNT dans la requête..

bizarre que la doc php effectue la même tache en 2 étapes .. alors que ton code et le mien en une seule. Du coup comment choisir, php laisse trop de liberté sur la façon de faire.. et selon moi ce n'est pas toujours bon !

Façon de faire de la doc php :

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
<?php
$sql = "SELECT COUNT(*) FROM fruit WHERE calories > 100";
if ($res = $conn->query($sql)) {
 
   /* Récupère le nombre de lignes qui correspond à la requête SELECT */
   if ($res->fetchColumn() > 0) {
 
      /* Effectue la vraie requête SELECT et travaille sur le résultat */
      $sql = "SELECT nom FROM fruit WHERE calories > 100";
      foreach ($conn->query($sql) as $row) {
      print "Nom : " .  $row['NOM'] . "\n";
      }
   }
   /* Aucune ligne ne correspond -- faire quelque chose d'autre */
   else {
      print "Aucune ligne ne correspond à la requête.";
   }
}
 
$res = null;
$conn = null;
?>

[Invité]

1/ Comme je t'ai dit, je n'ai (jusqu'ici) jamais eu de souci avec rowCount() :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
<?php
$sql = "SELECT nom FROM fruit WHERE calories > 100";
$res = $conn->query($sql)
$nbr_res = $res->rowCount();
 
if ($nbr_res > 0) {
 
   foreach( $row = $res->fetch() )
   {
   ....
   }
}

2/ je n'utilise ""SELECT COUNT(id)..." que pour vérifier qu'un enregistrement existe (pour savoir si une "article" existe, par exemple, avant d'en afficher le détail), sans avoir besoin de récupérer des infos (champs).

Après... chacun son style, chacun sa méthode et ses habitudes


Ne pas négliger de passer par une requête préparée quand elle contient des valeurs "externes" (passées par l'utilisateur, en POST ou GET) !

[ABCIWEB]

Salut,

La doc sur rowCount est placée dans un contexte général. Dans l'univers des systèmes de gestion de bases de données, une minorité d'entre eux ont implémentés la fonction rowCount. C'est ce que dit la doc en montrant une méthode universelle qui fonctionnera quelque soit le sgbdd. Cela dit dans le contexte du web avec php on utilise très souvent mysql ou apparenté et ces systèmes renvoient la bonne valeur pour cette fonction. Donc c'est très simple si tu ne connais pas la base de donnée qui sera utilisée avec ton code, évites d'utiliser rowCoount, par contre si c'est pour une bdd dont tu sais qu'elle gère cette fonction c'est quand même plus pratique que de faire deux requêtes.

Après quand il s'agit simplement de savoir si un enregistrement existe, l'important est de ne pas collecter des champs qui ne servent à rien si tu ne les exploitent pas par la suite. C'est ce processus qui prend du temps pour la requête et c'est pour cela que la doc php montre le cas typique d'une requête avec SELECT COUNT(*) qui ne sélectionne aucun champ.

[bndd24]

oui en effet, je comprends le fonctionnement mais j'ai du mal à le mettre en pratique dans mon cas car j'ai une problématique supplémentaire. Est-ce que dans mon cas il faut mieux récupérer les infos au même moment que la verif ou bien dans une autre requête plus bas comme dans l'exemple sur la doc ? Même si bien-sûr la function rowCount est implémenté dans mysql.

Je pense qu'il est préférable de vous montrer le code complet de ma page (mon code fonctionne) :

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
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
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
<?php
session_start();
 
$idann = (int) $_GET['idann'];
 
//--- vérifie qu'un enregistrement existe ET récupère les infos.
$req = $pdo->prepare('SELECT a.idann, a.content, a.location, a.amount, a.dateann, a.phone, u.username, u.email
FROM ad a
INNER JOIN users u ON a.idmbr = u.idmbr
WHERE a.idann = ?');
$req->execute([$idann]);
$ad = $req->fetch();
 
//--- si pas de résultat..
if (!$ad) {
    header('Location: index.php?p=ad/search');
    exit();
}
//--- end
 
if (!empty($_POST)) {
 
    if (empty($_POST['email']) || !filter_var($_POST['email'], FILTER_VALIDATE_EMAIL)) {
        $_SESSION['flash']['danger'] = 'Votre email n\'est pas valide';
    }
 
    if (empty($_SESSION['flash'])) {
        //mail($_POST['email'], 'sujet du mail', "contenu du mail");
        $_SESSION['flash']['success'] = 'Votre message a été envoyé à l\'annonceur !';
        header('Location: index.php?p=ad/view&idann='.$idann.'');
        exit();
    }
}
 
$title_page = 'Fiche';
 
?>
 
<h1>Fiche</h1>
 
<div class="row">
    <div class="col-md-8">
        <div class="panel panel-default">
            <div class="panel-body">
                <?= htmlspecialchars($ad->content); ?>
            </div>
            <!-- List group -->
            <ul class="list-group">
                <li class="list-group-item">Ville : <?= htmlspecialchars($ad->location); ?></li>
                <li class="list-group-item">Tarif : <?= $ad->amount; ?> €</li>
            </ul>
            <div class="panel-footer">Mise en ligne le <?= htmlspecialchars($ad->dateann); ?></div>
        </div>
    </div>
    <div class="col-md-4">
        <h2>Contacter <?= htmlspecialchars($ad->username); ?></h2>
 
        <p class="lead"><span class="glyphicon glyphicon-earphone" aria-hidden="true"></span> <?= implode(' ', str_split($ad->phone,2)); ?></p>
 
        <form action="" method="post">
            <div class="form-group">
                <label class="sr-only" for="name">Nom et prénom</label>
                <input type="text" name="name" value="<?= empty($_POST['name']) ? '' : htmlspecialchars($_POST['name'], ENT_QUOTES); ?>" class="form-control" id="name" placeholder="Nom et prénom">
            </div>
            <div class="form-group">
                <label class="sr-only" for="email">Votre email</label>
                <input type="text" name="email" value="<?= empty($_POST['email']) ? '' : htmlspecialchars($_POST['email'], ENT_QUOTES); ?>" class="form-control" id="email" placeholder="Votre email">
            </div>
            <div class="form-group">
                <label class="sr-only" for="content">Votre message</label>
                <p id="helpBlock" class="help-block">Pensez à indiquer vos coordonnées téléphoniques pour que l'annonceur puisse vous contacter facilement.</p>
                <textarea name="content" class="form-control" id="content" rows="6" placeholder="Votre message" aria-describedby="helpBlock"><?= empty($_POST['content']) ? '' : htmlspecialchars($_POST['content'], ENT_QUOTES); ?></textarea>
            </div>
            <button type="submit" class="btn btn-primary">Envoyer</button>
        </form>
    </div>
</div>