Discussion :

[bndd24]

Bonjour,

J'ai un problème avec mon code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
// Vérifie qu'une petite annonce existe.
$req = $pdo->prepare('SELECT a.idann, a.title, a.content, a.amount, a.datead, u.username, u.email, u.location, p.phone
FROM ad a
INNER JOIN users u ON a.idmbr = u.idmbr
INNER JOIN phone_users p ON a.idann = p.idmbr
WHERE a.idann = ?');
$req->execute([$idann]);
$ad = $req->fetch();
 
if (!$ad) {
    header('Location: /index.php');
    exit();
}

La requête fonctionne.
Sur mon site le champ du formulaire "phone" est facultatif (tout le monde ne souhaite pas communiquer un numéro).
Le problème c'est que si la requête ne trouve pas de résultat dans la table phone_users alors il rentre dans la condition !$ad juste en dessous.. alors que la petite annonce existe bien mais sans numéro...

merci pour votre aide

[sabotage]

Il faut faire un LEFT JOIN au lieu de INNER JOIN.
Par contre tu as fait ta jointure entre l'id de l'annonce et l'id du membre ... c'est un peu étrange.

[bndd24]

Merci pour ta réponse rapide.
Je n'ai pas bien compris pourquoi c'est étrange ? De quelle jointure, la première ou la deuxième ?

[bndd24]

c'est mieux la ?

Code :

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

1
2
3
4
5
6
7
$req = $pdo->prepare('SELECT a.idann, a.title, a.content, a.amount, a.datead, u.username, u.email, u.location, p.phone
FROM ad a
INNER JOIN users u ON a.idmbr = u.idmbr
LEFT JOIN phone_users p ON p.idmbr = u.idmbr
WHERE a.idann = ?');
$req->execute([$idann]);
$ad = $req->fetch();

[Celira]

En fait, ça dépend de la structure de ta table : est-ce qu'un numéro de téléphone est associé à un utilisateur ou à une annonce lorsque tu l'insères en base ?

[bndd24]

un numéro de téléphone est associé à un utilisateur.
(je ne stock pas les numéros de téléphone dans la même table que les infos perso du membre..)

[Celira]

Dans ce cas, il faut effectivement faire une liaison sur l'id du membre, et du coup, ta requête m'a l'air correcte.

[bndd24]

oui elle m'a l'air bien aussi mais je ne sais toujours pas ce que sabotage a voulu dire par "étrange" sans plus d'explication..

[sabotage]

Avoir une jointure entre l'id de l'annonce et l'id de l'utilisateur c'est peu étrange non ?

[bndd24]

?

Tu veux faire comment autrement pour récupérer les infos du membre ?

il s'agit d'une page qui affiche les détails d'une annonce. + les détails du membre qui a déposer l'annonce.

[Invité]

Je pense qu'il veut dire que lier l'id de l'annonce (de la table annonce) avec l'id du membre (de la table des numeros de tel) n'a pas beaucoup de sens car ce n'est pas ton annonce qui a un numero de telephone mais ton utilisateur. Je ne sais pas si c'est volontaire ou un bug mais il serait plus logique de lier la table utilisateur (id utilisateur) avec la table des telephones (id utilisateur).

[bndd24]

Et ce n'est pas le cas actuellement ?
p.idmbr = u.idmbr
.....

[sabotage]

Si mais ce n'est pas ce que tu avais dans la version d'origine.

[bndd24]

très juste