Discussion :

[bndd24]

Bonjour,

Je viens de passer mon après-midi sur une seule requête et la je bute sur la solution à adopter dans un cas précis.

Je programme une messagerie interne pour mon espace membre, le fil de la conversation est sur une page.

voici la requête qui affiche le fil de la conversation :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
$reponse = $pdo->query('SELECT
	m.id idmsg,
	m.message msg,
	m.date_envoi dateen,
	m.date_lecture datelu,
	u.id id_expediteur,
	u.username pseudo_expediteur
FROM users u
INNER JOIN messages m
ON m.id_expediteur = u.id
WHERE m.id_destinataire = \'1\' OR m.id_expediteur = \'1\';
ORDER BY date_envoi DESC LIMIT 0, 10');

les 1 dans la ligne du WHERE représente l'id du membre actuellement connecté (et qui affiche donc la page du fil de la conversation).

Mon problème :
Disons que le membre avec l'id 1 souhaite afficher sa conversation avec le membre possédant l'id 2 (via une variable GET) le nom de la page appelée par le membre 1 ressemble donc a ceci : /messages.php?idmbr=2 (jusqu'ici facile^^)
Mais actuellement la requête affiche les messages à partir du moment ou elle trouve un enregistrement possédant l'id 1 du membre dans les champs id_expediteur ou id_destinataire.

donc si le membre possédant l'id 1 envoie un message au membre possédant l'id 4, le message s'affichera (car la requête trouvera dans le champ id_expediteur l'id 1) ! et sa je ne veux pas car le fil de la conversation concerne uniquement le membre possédant l'id 1 a celui possédant l'id 2.. je ne sais pas si j'ai été très précis dans mon explication ? en même temps c'est difficile d'expliquer^^ donc je pense que le WHERE est a modifier mais de quelle façon ? à base de OR et AND non ? mais comment le mettre en place :/

en gros je veux que la requête fasse :
id_expediteur / id_destinataire
1 / 2 : affiche
2 / 1 : affiche
1 / 3 : affiche PAS.
..

je vous remercie j'attends vos réponses avec impatience car j'y arrive vraiment pas

[sabotage]

C'est exactement ce que tu as écris : 1 et 2, ou 2 et 1

Code sql :

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

WHERE (m.id_destinataire = 1 AND m.id_expediteur = 2) OR (m.id_expediteur = 2 AND m.id_expediteur = 1)

Et il n'y a pas besoin de guillemets sur des nombres.

[bndd24]

au moment ou tu as posté la réponse j'avais trouvé la solution^^ avec le code :

Code sql :

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

WHERE (m.id_destinataire = 1 OR m.id_expediteur = 1) AND (m.id_destinataire = 2 OR m.id_expediteur = 2);

mais ton code fonctionne aussi :

Code sql :

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

WHERE (m.id_destinataire = 1 AND m.id_expediteur = 2) OR (m.id_expediteur = 2 AND m.id_expediteur = 1);

du coup j'utilise ton code ou le mien ?
lequel est le plus "conventionnel" ?

[sabotage]

Le mien est plus logique

Avec le tiens on arrive à des combinaisons impossibles comme

Code :

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

m.id_destinataire = 1  AND m.id_destinataire = 2

[bndd24]

oops.. minute.. je reviens sur mon message, ton code ne fonctionne pas, bizarre mais il ne fonctionne pas :/

voici ma bdd :


avec mon code :
j'ai 4 enregistrements affiché donc EXACT (voir image plus haut).

avec ton code :
j'ai 1 enregistrement affiché donc faux. il affiche uniquement l'enregistrement portant l'id 16 ! je ne comprends pas pourquoi ??

[sabotage]

Youps j'ai mis deux fois expediteur :

Code sql :

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

WHERE (m.id_destinataire = 1 AND m.id_expediteur = 2) OR (m.id_destinataire = 2 AND m.id_expediteur = 1);

[bndd24]

exact^^ ça marche nickel et en effet ton code est meilleur car avec le mien on arrive à des combinaisons impossibles ! j'avoue, je suis vaincu

sinon j'ai une petite question supplémentaire :

Je viens de mettre les deux variables en place mais j'ai un message d'erreur..

( ! ) Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[42S22]: Column not found: 1054 Unknown column '$user_id' in 'where clause'' in C:\wamp\www\pag\messages.php on line 48
( ! ) PDOException: SQLSTATE[42S22]: Column not found: 1054 Unknown column '$user_id' in 'where clause' in C:\wamp\www\pag\messages.php on line 48

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
 
require_once 'inc/db.php';
$user_id = $_SESSION['auth']->id;
$idcorrespondant = $_GET['idmbr'];
 
$reponse = $pdo->query('SELECT
	m.id idmsg,
	m.message msg,
	m.date_envoi dateen,
	m.date_lecture datelu,
	u.id id_expediteur,
	u.username pseudo_expediteur
FROM users u
INNER JOIN messages m
ON m.id_expediteur = u.id
WHERE (m.id_destinataire = $user_id AND m.id_expediteur = $idcorrespondant) OR (m.id_destinataire = $idcorrespondant AND m.id_expediteur = $user_id);
ORDER BY date_envoi DESC LIMIT 0, 10');
 
while ($donnees = $reponse->fetch())
{
?>

as tu une idée ?

[sabotage]

Les variables ne sont pas interprétés entre guillemets simples et il faut utiliser une requête préparée pour protéger la requête quand on place des données externes :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
$reponse = $pdo->prepare("SELECT
	m.id idmsg,
	m.message msg,
	m.date_envoi dateen,
	m.date_lecture datelu,
	u.id id_expediteur,
	u.username pseudo_expediteur
FROM users u
INNER JOIN messages m
ON m.id_expediteur = u.id
WHERE (m.id_destinataire = $user_id AND m.id_expediteur = ?) OR (m.id_destinataire = ? AND m.id_expediteur = $user_id);
ORDER BY date_envoi DESC LIMIT 0, 10");
$reponse->execute(array($_GET['idmbr'], $_GET['idmbr']));

[bndd24]

exact ! merci pour ton code, je viens de faire la même chose pour déposer un message :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
if(!empty($_POST)){
	// Connexion à la base de données
	require_once 'inc/db.php';
	$user_id = $_SESSION['auth']->id;
 
	// Insertion du message à l'aide d'une requête préparée
	$req = $pdo->prepare('INSERT INTO messages (id_expediteur, id_destinataire, message, date_envoi) VALUES('.$user_id.', ?, ?, NOW())');
	$req->execute(array($_GET['idmbr'], $_POST['message']));
 
	$_SESSION['flash']['success'] = 'Message envoyé !';
}

il fonctionne mais est-ce "améliorable" ? il faut aussi que je m'occupe de la sécurité des données pour le champ message avec un htmlspecialchars..