SQLSTATE[42000]: Syntax error or access violation: 1064

**EfferNath** · 23/11/2023, 09h58

Bonjour à tous,

pourriez vous m'aider à savoir d'ou provient l'erreur que j'ai dans ma requête SQL.

Erreur :

SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'AND m1.user1read = 'no' AND Effer_Utilisateur.id = m1.user2) OR (m1.user2 = ...' at line 1

Code de la requête :

Code :

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

1
2
3
4
5
6
7
8
9
    function getMessageNonLu($idMembre){
        $pdo = monPDO::getPDO();
        $SQL = "SELECT m1.id, m1.title, m1.timestamp, count(m2.id) as reps, Effer_Utilisateur.id as userid, Effer_Utilisateur.login FROM Effer_Messagerie as m1, Effer_Messagerie as m2, Effer_Utilisateur WHERE ((m1.user1 = $idMembre AND m1.user1read = 'no' AND Effer_Utilisateur.id = m1.user2) OR (m1.user2 = $idMembre AND m1.user2read = 'no' AND Effer_Utilisateur.id = m1.user1)) AND m1.id2 = '1' AND m2.id = m1.id GROUP BY m1.id ORDER BY m1.id desc";
        $stmt = $pdo->prepare($SQL);
        $stmt->execute();
        $resultat = $stmt->fetchAll(PDO::FETCH_ASSOC);
        $stmt->closeCursor();
        return $resultat;
    }

Format de la BDD : ( InnoDB )
Nom : sql.png
Affichages : 131
Taille : 12,3 Ko

Merci à vous

**unanonyme** · 23/11/2023, 10h52

Bonjour,

c'est sûrement parce que vous avez oublié d'utiliser
des paramètres nommés.

https://www.php.net/manual/fr/pdostatement.execute.php

Bonne journée.

**escartefigue** · 23/11/2023, 12h20

Bonjour,

Syntaxiquement, la requête est fausse, car le GROUP BY est incohérent avec le SELECT : toutes les colonnes ne faisant pas l'objet d'un agrégat doivent être dans le GROUP BY.
Même si, en fonction du paramétrage, MySQL peut accepter cette incohérence (et je précise que tout autre SGBD rejetterait la requête), fonctionnellement ça n'a aucun sens.

Ensuite, depuis la norme SQL 1995, les jointures ne se font plus dans la clause WHERE, mais avec l'opérateur JOIN.

Réécrite comme il convient, votre requête devient :

Code SQL :

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
SELECT M1.id
     , M1.title
     , M1.timestamp
     , count(M2.id) as reps
     , EU.id as userid
     , EU.login 
FROM        Effer_Messagerie as M1
inner join  Effer_Messagerie as M2
   on M2.id=M1.id
inner join Effer_Utilisateur as EU
   on UE.id = M1.user1
   or UE.id = M1.user2
WHERE (    (M1.user1 = $idMembre AND M1.user1read = 'no') 
       OR  (M1.user2 = $idMembre AND M1.user2read = 'no')
      ) 
AND M1.id2 = '1' 
GROUP BY M1.id 
       , M1.title
       , M1.timestamp
       , EU.id 
       , EU.login 
ORDER BY M1.id desc

**EfferNath** · 23/11/2023, 13h29

Bonjour et merci de vous être intérréssé à mon souci,

Merci aussi pour les explications, et la présentation correcte d'une requête SQL

J'ai fait 2 petites modifications et cela me paraît pleinement fonctionnel ( je vous direz à l'usage )
- une petite coquille ou vous avez mis UE au lieu de EU pour la table Effer_Utilisateur
- vous avez regroupé ces 2 conditions alors que je pense qu'elle ne devait pas l'être

Code :

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

1
2
3
4
5
6
inner join Effer_Utilisateur as EU
   on UE.id = M1.user1
   or UE.id = M1.user2
WHERE (    (M1.user1 = $idMembre AND M1.user1read = 'no') 
       OR  (M1.user2 = $idMembre AND M1.user2read = 'no')
      )

que j'ai remplacé par

Code :

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

1
2
3
inner join Effer_Utilisateur as EU
WHERE ((M1.user1 = $idMembre AND M1.user1read = 'no' AND EU.id = M1.user2)
OR (M1.user2 = $idMembre AND M1.user2read = 'no' AND EU.id = M1.user1))

Merci beaucoup pour votre aide et le temps passé sur mon problème

**escartefigue** · 23/11/2023, 14h36

Désolé pour l'inversion EU/UE, j'ai écrit ça un peu vite et sans pouvoir tester la requête

Concernant l'autre point, puisque le OR est déporté dans la condition de jointure, je pense qu'il est inutile de le maintenir dans le WHERE.
Testez ma version pour en avoir la coeur net