Jointure et problème de sélection avec le WHERE

Version imprimable

2 pièce(s) jointe(s)

Bonjour,

J'ai une table qui comporte des messages, chaque enregistrements à un id expéditeur ainsi qu'un id destinataire.

Dans la table il peut donc y avoir plusieurs messages qui comporte le même expéditeur et le même destinataire si par exemple le membre 1 envoie 3 messages au membre 2 alors il y aura :
id_expediteur / id_destinataire
1 / 2
1 / 2
1 / 2
donc 3 messages.

le problème c'est que je suis en train de faire une boite de réception avec la liste des correspondants mais actuellement ma requête affiche justement l'ensemble des messages! je veux que ma requête "regroupe" les même id destinataire et expéditeur et affiche le pseudo du correspondant via une jointure. Je ne sais pas si j'ai été précis dans mon histoire ? :?

voici une image de mes enregistrements de ma table messages
Pièce jointe 200080

maintenant disons que je suis le membre avec l'id 1 (pseudo demo)
la photo affiche actuellement :
Pièce jointe 200081

Alors que j'attends un affichage de seulement 2 lignes car dans la photo phpmyadmin le membre 1 parle avec seulement 2 membres et pas 5 ! voila mon problème. Pas évident pour l'expliquer, je vous remercie pour votre aide :calim2: je pense qu'il faut modifier le where mais je ne sais pas comment :/
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 $user_id = $_SESSION['auth']->id; $reponse = $pdo->prepare("SELECT u.id id_correspondant, u.username pseudo_correspondant, m.id id_msg, m.id_expediteur id_exp, m.id_destinataire id_des, m.date_envoi date_msg FROM users u INNER JOIN messages m ON m.id_expediteur = u.id WHERE m.id_expediteur = $user_id OR m.id_destinataire = $user_id ORDER BY m.date_envoi DESC LIMIT 0, 20");

Ne met pas le detail des messages dans la requête puisque tu veux seulement les expediteurs/destinataires.
Il n'y a pas besoin d'une requête préparée puisque tu n'as pas de paramètres.
Code:

1 2 3 4 5 6 7 8 $reponse = $pdo->query("SELECT DISTINCT u.id id_correspondant, u.username pseudo_correspondant FROM users u INNER JOIN messages m ON m.id_expediteur = u.id WHERE m.id_expediteur = $user_id OR m.id_destinataire = $user_id ORDER BY m.date_envoi DESC LIMIT 0, 20");

ah je ne connaissais pas le SELECT DISTINCT 8O

merci infiniment :ccool:

sinon, je t’embête encore :oops:

Si je suis le membre demo (id 1) je voudrais que la requête affiche le pseudo de mon interlocuteur et non le mien, actuellement dans la deuxieme ligne la requete affiche "demo" alors que demo c'est moi :P

_________
Boîte de réception
Pseudo : test (Consulter la conversation)
Pseudo : demo (Consulter la conversation)
_________

Il faudrait que la requête affiche le pseudo "encoreuntest" c'est le pseudo du membre qui porte l'id 3..
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 <?php // Connexion à la base de données require_once 'inc/db.php'; $user_id = $_SESSION['auth']->id; $reponse = $pdo->query("SELECT DISTINCT u.id id_correspondant, u.username pseudo_correspondant FROM users u INNER JOIN messages m ON m.id_expediteur = u.id WHERE m.id_expediteur = $user_id OR m.id_destinataire = $user_id ORDER BY m.date_envoi DESC LIMIT 0, 20"); while ($donnees = $reponse->fetch()) { ?> Pseudo : <?php echo htmlspecialchars($donnees->pseudo_correspondant); ?> <a href="messages.php?idmbr=<?php echo $donnees->id_correspondant ?>">(Consulter la conversation)</a> <br> <?php } $reponse->closeCursor(); ?>

Une possibilité :

Code:

1
2
3
4
5
6
7
8
9
SELECT DISTINCT
	IF (m.id_expediteur = $user_id, m.id_destinataire, m.id_expediteur) id,
        IF (m.id_expediteur = $user_id, u2.username, u1.username) pseudo
FROM messages m
JOIN users u1 ON m.id_expediteur = u1.id
JOIN users u2 ON m.id_destinataire = u2.id
WHERE m.id_expediteur = $user_id OR m.id_destinataire = $user_id
ORDER BY m.date_envoi DESC
LIMIT 0, 20

06/02/2016, 03h01
bndd24

Ton code fonctionne parfaitement bien (sujet résolu grâce à toi) :merci: ! Je te remercie et je vais essayer de le décortiquer pour mon apprentissage :pc:

sinon dans ton message tu dis "une possibilité" ?

A un moment j'avai pensé faire une union plutôt qu'une jointure et des conditions ... je ne sais pas lequel est le plus efficace, il faudrait mesurer.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
SELECT DISTINCT id, username, date_envoi
FROM (
         SELECT u1.id, u1.username, m1.date_envoi
         FROM messages m1 JOIN users u1 ON m1.id_expediteur = u1.id
         WHERE m1.id_destinataire = $user_id 
 
         UNION
 
         SELECT u2.id, u2.username, m2.date_envoi
         FROM messages m2 JOIN users u2 ON m2.id_destinataire = u2.id
         WHERE m2.id_expediteur = $user_id 
)
ORDER BY date_envoi DESC
LIMIT 0, 20

oui je vois.. en tout cas j'ai essayé ton deuxième code mais il ne fonctionne pas (j'ai copié l'erreur ci-dessous) donc je garde le premier.. :zoubi:
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 $reponse = $pdo->query("SELECT DISTINCT IF (m.id_expediteur = $user_id, m.id_destinataire, m.id_expediteur) id, IF (m.id_expediteur = $user_id, u2.username, u1.username) pseudo FROM messages m JOIN users u1 ON m.id_expediteur = u1.id JOIN users u2 ON m.id_destinataire = u2.id WHERE m.id_expediteur = $user_id OR m.id_destinataire = $user_id ORDER BY m.date_envoi DESC LIMIT 0, 20"); $reponse = $pdo->query("SELECT DISTINCT id, username, date_envoi FROM ( SELECT u1.id, u1.username, m1.date_envoi FROM messages m1 JOIN users u1 ON m1.id_expediteur = u1.id WHERE m1.id_destinataire = $user_id UNION SELECT u2.id, u2.username, m2.date_envoi FROM messages m2 JOIN users u2 ON m2.id_destinataire = u2.id WHERE m2.id_expediteur = $user_id ) ORDER BY date_envoi DESC LIMIT 0, 20");
erreur du deuxième code pour infos :

Citation:

( ! ) Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[42000]: Syntax error or access violation: 1248 Every derived table must have its own alias' in C:\wamp\www\pag\liste_messages.php on line 37
( ! ) PDOException: SQLSTATE[42000]: Syntax error or access violation: 1248 Every derived table must have its own alias in C:\wamp\www\pag\liste_messages.php on line 37

ps : ligne 37 c'est : LIMIT 0, 20"); donc bizarre :mur:

et aussi pour info mon code d'affichage :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 while ($donnees = $reponse->fetch()) { ?> <?php echo htmlspecialchars($donnees->pseudo); ?> <a href="profils.php?idmbr=<?php echo $donnees->id ?>">Profil</a> <a href="messages.php?idmbr=<?php echo $donnees->id ?>">Consulter la conversation</a> <?php } $reponse->closeCursor(); ?>

06/02/2016, 19h06
sabotage
Comme indiqué dans l'erreur, il faut seulement mettre un alias.
Code:

1 2 ) u ORDER BY date_envoi DESC
06/02/2016, 19h14
bndd24

ok mais du coup dans le deuxième code il y a m1.date_envoi et m2.date_envoi je met quoi, je ne comprends pas trop le deuxième code :weird:

visuellement le premier code à l'air moins "lourd" que le deuxième.. mais bon après il faut mesurer oui, je ne sais pas du tout comment mesurer un code :roll:

1 pièce(s) jointe(s)

Re sabotage,

J'ai rajouté "date_lecture" après les deux IF et maintenant j'ai des doublons, pourquoi ? des conversations qui s'affichent en double dans la liste des conversation.

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
<div class="table-responsive">
	<table class="table table-striped table-hover">
 
<?php
 
// Connexion à la base de données
require_once 'inc/db.php';
$user_id = $_SESSION['auth']->id;
 
$reponse = $pdo->query("SELECT DISTINCT
	IF (m.id_expediteur = $user_id, m.id_destinataire, m.id_expediteur) id,
	IF (m.id_expediteur = $user_id, u2.username, u1.username) pseudo,
	date_lecture
	FROM messages m
	JOIN users u1 ON m.id_expediteur = u1.id
	JOIN users u2 ON m.id_destinataire = u2.id
	WHERE m.id_expediteur = $user_id OR m.id_destinataire = $user_id
	ORDER BY date_envoi DESC
	LIMIT 0, 30");
 
while ($donnees = $reponse->fetch())
{
	switch ($donnees->date_lecture)
	{
		case null : $lecture_msg = "msg non lu dans cette conversation"; break;
		case !null : $lecture_msg = "lu"; break;
	}
?>
		<tr>
			<th><?php echo htmlspecialchars($donnees->pseudo); ?></th>
			<th><?php echo "$lecture_msg" ?></th>
			<th><a href="profils.php?idmbr=<?php echo $donnees->id ?>">Profil</a> - <a href="messages.php?idmbr=<?php echo $donnees->id ?>">Consulter la conversation</a></th>
		</tr>
 
<?php
}
$reponse->closeCursor();
 
?>
 
	</table>
</div>

Pièce jointe 200232

8O

sans le "date_lecture" après les deux IF, aucun problème! mais dès que je rajoute le date_lecture j'ai des doublons.

08/02/2016, 14h52
sabotage

S'il y a plusieurs messages, il y a plusieurs dates de lecture, tu te retrouves donc avec plusieurs lignes.
08/02/2016, 15h05
bndd24

ok je vois.. mais si je veux avoir un seul affichage (une ligne par conversation) il faut que je procède de quelle façon ?

je veux que les "non lu dans cette conversation" et "lu" à coté de chaque conversations soit basé sur id_expediteur = $_GET['idmbr'] and id_destinataire = $user_id

mais de quelle façon dois-je l'écrire ?

trop compliqué le SELECT DISTINCT avec des IF j'y comprends rien! et pas faute de lire les docs :roll: avez-vous des docs à me conseiller sur le SELECT DISTINCT ou le UNION ?

comment rajouter date_lecture dans le select sans avoir de doublons comme sur la photo ci dessus svp

date_lecture basé sur la condition de : id_expediteur = $_GET['idmbr'] AND id_destinataire = $user_id
Code:

1 2 3 4 5 6 7 8 9 10 11 12 <?php $reponse = $pdo->query("SELECT DISTINCT IF (m.id_expediteur = $user_id, m.id_destinataire, m.id_expediteur) id, IF (m.id_expediteur = $user_id, u2.username, u1.username) pseudo, date_lecture FROM messages m JOIN users u1 ON m.id_expediteur = u1.id JOIN users u2 ON m.id_destinataire = u2.id WHERE m.id_expediteur = $user_id OR m.id_destinataire = $user_id ORDER BY date_envoi DESC LIMIT 0, 30"); ?>
je vous remercie pour votre aide

j'ai essayé de rajouter : m.date_lecture à la fin du premier IF mais j'ai une erreur, fonctionne pas!
je dois refaire un troisième IF sabotage ? pour éviter les doublons lors de la lecture d'un message. (cf photo ci dessus).
Code:

1 2 3 4 5 6 7 8 9 10 11 <?php $reponse = $pdo->query("SELECT DISTINCT IF (m.id_expediteur = $user_id, m.id_destinataire, m.id_expediteur, m.date_lecture) id, IF (m.id_expediteur = $user_id, u2.username, u1.username) pseudo FROM messages m JOIN users u1 ON m.id_expediteur = u1.id JOIN users u2 ON m.id_destinataire = u2.id WHERE m.id_expediteur = $user_id OR m.id_destinataire = $user_id ORDER BY date_envoi DESC LIMIT 0, 30"); ?>

09/02/2016, 08h24
sabotage

La vraie question est de savoir quelle date_lecture tu veux puisqu'il y en une pour chaque message entre les deux utilisateurs.
09/02/2016, 10h34
bndd24

Si de id_desninataire=$id_user à id_expediteur=$id_expediteur il y a une (ou même plusieurs) date non lu (=null) alors je veux pouvoir l'afficher et dire que dans cette conversation vous avez un message non lu.
Au fait, si tu veux c'est juste pour afficher un petit pictogramme à côté de chaque conversation dans la boîte de réception pour que l'utilisateur sache en un coup œil si il a des messages non lu dans cette conversation.
09/02/2016, 14h19
bndd24

j'ai encore essayé quelque chose mais raté !

Code:

WHERE (m.id_expediteur = $user_id OR m.id_destinataire = $user_id) AND (m.date_lecture IS NULL OR m.date_lecture IS NOT NULL)

j'ai toujours des doublons ! :mur: