rowcount donne une mauvaise valeur

Version imprimable

Voir 40 message(s) de cette discussion en une page

Bonjour,

Je veux compter le nombre d'enregistrements de ma table répondant à ma condition ET pour lesquels le contenu d'une colonne donné doit être non nul:

Si je fais ceci, le nombre obtenu n'est pas le bon. Je pense que c'est dû au fait que ma variable n'est pas interprétée.
Code:

1 2 3 4 $sql = 'SELECT * FROM ma_table WHERE (' ."$id_Item". ' = '. $_SESSION["id_Item"]. ' AND '. $id_Item.' !=0) '; $resultat = $pdo->prepare($sql); $resultat->execute(); $nb_hits_non_nuls = $resultat->rowCount();
Si je modifie ainsi, j'obtiens une erreur Uncaught exception 'PDOException' with message 'SQLSTATE[42000]: Syntax error or access violation: 1064 Erreur de syntaxe.
Code:

1 2 3 4 $sql ="SELECT * FROM 'coursinfo_membres_cours' WHERE (`$id_Item` = ".$_SESSION['id_Item']." AND $id_Item !=0) "; $resultat = $pdo->prepare($sql); $resultat->execute(); $nb_hits_non_nuls = $resultat->rowCount();
Je vous remercie de bien vouloir m'aiguiller.

08/01/2019, 11h19
tyjez
Sauf erreur de ma part il faut d'abord faire un fetch pour pouvoir compter le nombre de ligne. (je ne suis pas sur)
Code:

1 2 3 4 $resultat->execute(); $resultat->fetch(PDO::FETCH_ASSOC); $nb_hits_non_nuls = $resultat->rowCount();
08/01/2019, 11h37
Denis Placé

Bonjour tyjez,

Merci pour ton intervention mais ça ne change rien.
08/01/2019, 12h16
tyjez

es-tu certain du fonctionnement de ta requête ?
j'aurai plutot fait

Code:

$sql ="SELECT * FROM 'coursinfo_membres_cours' WHERE (".$id_Item." = ".$_SESSION['id_Item']." AND ".$id_Item." !=0) ";
08/01/2019, 12h49
Denis Placé

Non, je suis pas sûr de ma requête.

Avec ta proposition, j'obtiens la même erreur

Citation:

Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[42000]: Syntax error or access violation: 1064 Erreur de syntaxe près de ''ma_table' WHERE (2001 = 2001 AND 2001 !=0)' à la ligne 1'
08/01/2019, 12h53
tyjez

et comme ca ?
c'est bizzare comme requete quand meme. tu as des noms de champs de ta table qui correspondent aux données qu'ils contiennent ?

Code:

$sql ="SELECT * FROM coursinfo_membres_cours WHERE (".$id_Item." = ".$_SESSION['id_Item']." AND ".$id_Item." !=0) ";

Citation:

Envoyé par Denis Placé

Non, je suis pas sûr de ma requête.

Avec ta proposition, j'obtiens la même erreur
08/01/2019, 12h57
Denis Placé

Merci encore pour ton implication.

Ta proposition fonctionne, comme ma première idée, c'est-à-dire en ne donnant pas le bon effectif ;-(
Curieux....
08/01/2019, 13h03
tyjez

J'aurai d'ailleurs, vue ta demande initiale, fait plutot ceci:

Code:

$sql ="SELECT * FROM coursinfo_membres_cours WHERE (".$id_Item." = ".$_SESSION['id_Item']." AND ".$id_Item." IS NOT NULL) ";

Citation:

Envoyé par Denis Placé

Merci encore pour ton implication.

Ta proposition fonctionne, comme ma première idée, c'est-à-dire en ne donnant pas le bon effectif ;-(
Curieux....
08/01/2019, 13h06
Denis Placé

Toujours le mauvais effectif....
08/01/2019, 13h27
tyjez

quels sont les champs de ta table ?
08/01/2019, 13h30
tyjez

on est bien d'accord que tu veux récupérer tous les lignes de la table coursinfo_membres_cours pour lesquelles le champ 2001 contient la valeur 2001 (selon ton exemple)
08/01/2019, 13h34
Denis Placé

Oui, et pour lesquels le contenu de cette même colonne est non nul.
PS: En fait, ici, c'est la colonne 2001 parce que la variable l'impose, ça peut être d'autres colonnes car il y en a beaucoup.
08/01/2019, 13h55
tyjez

En fait j'ai un peu du mal a comprendre ta logique de conception de table mais bon continuons de chercher ;)

si j'ai bien compris donc ta table 1 contient des colonnes qui peuvent s'appeler 2001 2002 2003 etc
et les lignes de ta table peuvent être par exemple pour la colonne 2001:
2001
2001
2001
null
2001
null
...
C'est bien ca ?

Alors, si tu veux la table, la voici.

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
59
60
61
62
63
64
65
66
67
68
69
70
71
CREATE TABLE `ma_table` (
  `_id` int(11) NOT NULL,
  `_pseudo` varchar(255) NOT NULL,
  `2001` decimal(5,2) NOT NULL,
  `2002` decimal(5,2) NOT NULL,
  `2003` decimal(5,2) NOT NULL,
  `2004` decimal(5,2) NOT NULL,
  `2005` decimal(5,2) NOT NULL,
  `2006` decimal(5,2) NOT NULL,
  `2014` decimal(5,2) NOT NULL,
  `2007` decimal(5,2) NOT NULL,
  `2008` decimal(5,2) NOT NULL,
  `2009` decimal(5,2) NOT NULL,
  `2010` decimal(5,2) NOT NULL,
  `2011` decimal(5,2) NOT NULL,
  `2012` decimal(5,2) NOT NULL,
  `2013` decimal(5,2) NOT NULL,
  `2015` decimal(5,2) NOT NULL,
  `2016` decimal(5,2) NOT NULL,
  `2017` decimal(5,2) NOT NULL,
  `2018` decimal(5,2) NOT NULL,
  `2019` decimal(5,2) NOT NULL,
  `2020` decimal(5,2) NOT NULL,
  `2021` decimal(5,2) NOT NULL,
  `2022` decimal(5,2) NOT NULL,
  `2023` decimal(5,2) NOT NULL,
  `2024` decimal(5,2) NOT NULL,
  `2025` decimal(5,2) NOT NULL,
  `2026` decimal(5,2) NOT NULL,
  `2027` decimal(5,2) NOT NULL,
  `2028` decimal(5,2) NOT NULL
) ENGINE=MyISAM DEFAULT CHARSET=utf8;
 
--
-- Contenu de la table `ma_table`
--
 
INSERT INTO `ma_table` (`_id`, `_pseudo`, `2001`, `2002`, `2003`, `2004`, `2005`, `2006`, `2014`, `2007`, `2008`, `2009`, `2010`, `2011`, `2012`, `2013`, `2015`, `2016`, `2017`, `2018`, `2019`, `2020`, `2021`, `2022`, `2023`, `2024`, `2025`, `2026`, `2027`, `2028`) VALUES
(27, 'adet', '3.12', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '3.16', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00'),
(28, 'Marvin', '10.57', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00'),
(29, 'lisa', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00'),
(30, 'tibow', '5.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00'),
(31, 'Laurianne', '5.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '13.16', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00'),
(32, 'Poam', '10.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00'),
(33, 'Roxannb', '10.63', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00'),
(34, 'kencous', '5.63', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00'),
(35, 'FlorianRuz', '10.63', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00'),
(36, 'ntm56000', '1.25', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00'),
(37, 'Jerem', '4.38', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00'),
(38, 'berfinguclu', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00'),
(39, 'rebeccab', '6.88', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00', '0.00');
 
--
-- Index pour les tables exportées
--
 
--
-- Index pour la table `ma_table`
--
ALTER TABLE `ma_table`
  ADD PRIMARY KEY (`_id`);
 
--
-- AUTO_INCREMENT pour les tables exportées
--
 
--
-- AUTO_INCREMENT pour la table `ma_table`
--
ALTER TABLE `ma_table`
  MODIFY `_id` int(11) NOT NULL AUTO_INCREMENT

Et selon la valeur prise par les variables $id_Item et $_SESSION['id_Item'], la colonne concernée change.

08/01/2019, 14h36
tyjez

Le probleme c'est que $id_Item et $_SESSION['id_Item'] ont l'air de retourner le même résultat.
D'après ce que j'ai compris en analysant tes données de table, il faudrait que ta requête te renvoie par exemple
WHERE 2001 = 3.12
Tu confirmes ?
Si c'est le cas il y a un problème lorsque tu alimentes ta $_SESSION['id_Item']
08/01/2019, 16h32
Denis Placé

En fait, les 2 variables prendront systématiquement la même valeur et cela correspondra à la colonne visée.
Pas toujours évident à cerner quand on ne connait pas le contexte mais c'est comme ça que ça fonctionne.
Merci encore pour ton implication.

1 pièce(s) jointe(s)

Je me permets d'intervenir.

Nommer un champ avec une valeur numérique c'est du jamais vu (en 30 ans de carrière) ! Ce n'est pas parce que c'est faisable que c'est à faire.

Question. Pourquoi avoir choisi une telle structure plutôt qu'une structure : id, pseudo, année, valeur ?

Car à ce que je vois pour chaque enregistrement dans ta table il y plus de 20 champs qui ne servent à rien. Il y aurait d'autres choses à dire.

Je pense que la requête fonctionnera si le champ est entouré par le caractère ` car un nombre n'est pas un nom de champ valable et le fameux caractère sert notamment à pouvoir donner à des champs des noms qui correspondent à des mots clés du SQL.

En effet si dans la clause where on a 2001 = 3.12 alors on n'obtiendra rien car 2001 est différent de 3.12 tout comme 5 est différent de 45.
Par contre si on a `2001` = 3.12 alors on obtiendra quelque chose car on comparera la valeur du champ nommé 2001 avec 3.12. C'est justement pour éviter ces désagréments qu'on ne nomme jamais un champ avec uniquement des chiffres.
Code:

1 2 $sql ="SELECT * FROM coursinfo_membres_cours WHERE (`".$id_Item."` = ".$_SESSION['id_Item']." AND `".$id_Item."` IS NOT NULL) ";
Pièce jointe 439809

S'il s'agit uniquement de compter les enregistrements un select count(*) as comptage from etc... suffit. Cela à l'avantage, entre autres, de consommer beaucoup moins de mémoire que de ramener toutes les données et de faire un rowCount.

Tu peux utiliser prepare avec bindParam. Mais comme tu as des noms de champs variables c'est moins intéressant.

Question pour des gens plus calés que moi. Est-ce qu'il y a un risque d'injection SQL si les noms des champs sont variables ?
Code:

1 2 3 4 5 6 $sql ="SELECT * FROM coursinfo_membres_cours WHERE (`".$id_Item."` = :id_item AND `".$id_Item."` IS NOT NULL) "; $resultat = $pdo->prepare($sql); $resultat->bindParam(':id_item', $_SESSION['id_Item']); $resultat->execute(); $nb_hits_non_nuls = $resultat->rowCount();
Pour ma curiosité. C'est un projet personnel (apprentissage) ou bien professionnel ?

09/01/2019, 07h57
Denis Placé

Bonjour badaze et merci pour ton post très complet,

Citation:

Envoyé par badaze

Question. Pourquoi avoir choisi une telle structure plutôt qu'une structure : id, pseudo, année, valeur ?
Car à ce que je vois pour chaque enregistrement dans ta table il y plus de 20 champs qui ne servent à rien. Il y aurait d'autres choses à dire.

Si, si, les 20 champs en question serviront. Pour l'instant, je n'en suis qu'en phase de test et de réflexion donc ils sont encore vides.
En fait, j'en suis venu là pour tenter d'éviter un case avec plus de 20 modalités. Je veux effectuer mon comptage sur une colonne donnée selon la valeur prise par ma variable de session.

Citation:

Envoyé par badaze

Je pense que la requête fonctionnera si le champ est entouré par le caractère ` car un nombre n'est pas un nom de champ valable et le fameux caractère sert notamment à pouvoir donner à des champs des noms qui correspondent à des mots clés du SQL.

Je viens d'essayer et cela ne change pas.

Citation:

Envoyé par badaze

S'il s'agit uniquement de compter les enregistrements un select count(*) as comptage from etc... suffit. Cela à l'avantage, entre autres, de consommer beaucoup moins de mémoire que de ramener toutes les données et de faire un rowCount.

Je vais essayer ça en effet.

Citation:

Envoyé par badaze

Pour ma curiosité. C'est un projet personnel (apprentissage) ou bien professionnel ?

C'est pour moi ;-)
09/01/2019, 09h19
tyjez
Je suis tout a fait de l'avis de badaze et je serais passé par deux tables et requête avec jointure.

Un première table lié au pseudo: id, pseudo... et tout ce qui est lié à la personne
Code:

1 2 3 4 5 INSERT INTO table1 (`_id`, `_pseudo`) VALUES (27, 'adet'), (28, 'Marvin') etc... ;
Une deuxième table lié à l'id du pseudo: id, pseudoId, année, valeur.
Code:

1 2 3 4 INSERT INTO table2 (pseudoId, annee, valeur) VALUES (27, 2001, 3.12), (27, 2002, 0.00); etc
et pour l'équivalent de ta requête
Code:

1 2 $sql ="SELECT COUNT(*) FROM table2 WHERE (annee = ".$_SESSION['id_Item']." AND annee IS NOT NULL) ";
09/01/2019, 10h25
Celira

Citation:

Envoyé par badaze

Question pour des gens plus calés que moi. Est-ce qu'il y a un risque d'injection SQL si les noms des champs sont variables ?

Tout dépend de la variable : si c'est une variable écrite en dur dans le script, on peut espérer que tu ne mets pas n'importe quoi dans ton script.
En revanche, si l'information vient de l'extérieur (d'un formulaire par exemple), il faudra protéger la requête en utilisant par exemple une liste des valeurs autorisées et/ou une fonction comme PDO::quote.

Voir 40 message(s) de cette discussion en une page