rejoindre la requête en retournant des résultats impairs

Question

J'utilise cette requête pour afficher une liste de chansons et montrer quelles chansons ont été cliquées comme favoris par un utilisateur.

$query = mysql_query(
    sprintf(" 
    SELECT 
     s.*, 
     UNIX_TIMESTAMP(`date`) AS `date`, 
     f.userid as favoritehash 
    FROM 
     songs s 
    LEFT JOIN 
     favorites f 
    ON 
     f.favorite = s.id 
     AND f.userid = %s", 
    $userhash) 
); 

Le tableau songs est configuré comme: id artist title duration etc., etc.

Le tableau favorites est configuré comme: id favorite userid

Le userid est une valeur de hachage stockée dans un cookie pour référencer un utilisateur unique .

La requête fonctionne correctement mais pour une raison quelconque, si je marque une chanson comme favorite dans un navigateur. Et puis marquer la même chanson que favori dans un autre navigateur pour simuler plusieurs utilisateurs, la chanson sera affichée deux fois ... une fois pour chaque fois qu'il est marqué favori, mais l'indicateur préféré un < 3 affichera toujours correctement.

Des idées?

Eh bien, il a fallu travailler via removign le sprintf() mais curieux de savoir pourquoi c'est si quelqu'un a des idées.

Answer 1

Vous utilisez sprintf et% s (string), mais vous n'êtes pas Enfermer la valeur de chaîne résultante entre guillemets. Si l'ID utilisateur est une chaîne, vous devez l'encadrer entre guillemets, sinon utilisez% d au lieu de% s. Comme cela fonctionne bien lorsque vous supprimez sprintf, cela semble être le problème.

Answer 2

J'ai eu un problème similaire je pense que si vous changez le & F.userid =% s à Où f.userid =% s il devrait le réparer ?.

Answer 3

Je crois que votre ON -clause est erroné.

essayer ceci:

ON f.favorite = s.id 
WHERE f.userid = %s 

Answer 4

Je crois que les suggestions précédentes ne fait vaincre le LEFT JOIN, l'amenant à agir en tant INNER JOIN; f.userid serait parfois NULL, et NULL se compare toujours comme false.

Je commencerais par regarder le contenu de la table favorites. Si je comprends votre schéma, vous voudrez peut-être établir une clé unique sur favorites sur favorite et userid pour vous assurer qu'un utilisateur donné ne peut que préférer chaque chanson une fois. Autrement dit, vous pouvez obtenir des lignes en double, et donc des résultats en double.

Answer 5

Puisque vous utilisez une 'jointure à gauche', je suppose que vous voulez une liste de toutes les chansons et que vous voulez que les favoris des utilisateurs soient facilement reconnaissables. à-dire quelque chose comme ceci:

song1 details | date | null 
song2 details | date | userhash (favorite) 
song3 details | date | null 

les opérations suivantes:

SELECT s.*, UNIX_TIMESTAMP(`date`) AS `date`, f.userid as favoritehash 
FROM 
    songs s 
LEFT JOIN 
    (SELECT userid, favorite FROM favorites WHERE userid = %s) f 
ON 
    f.favorite = s.id