Discussion :

[gregtix]

Ce serait plutôt ça non ?

Code :

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

1
2
3
4
5
6
7
SELECT m.date, table_pronos.id_user, SUM( pts )+pts_bonus AS sumpts, SUM(IF(pts=3,1,0)) AS nbperfect,
FROM table_match
INNER JOIN table_pronos ON table_match.id = table_pronos.id_match
LEFT JOIN bonus b ON table_pronos.id_user=b.id_user
GROUP BY table_pronos.id_user
ORDER BY sumpts DESC,nbperfect DESC

Mais ça ne marche pas

[ericd69]

non je viens de voir que je me suis juste planté sur le sur-nommage dans le left join:

Code sql :

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

1
2
3
4
5
6
SELECT m.date, p.id_user, SUM( pts )+pts_bonus AS sumpts, SUM(IF(pts=3,1,0)) AS nbperfect
FROM table_match m
INNER JOIN table_pronos p ON m.id = p.id_match
LEFT JOIN bonus b ON b.id_user=p.id_user
GROUP BY p.id_user
ORDER BY sumpts DESC,nbperfect DESC

[gregtix]

ce que je voulais dire dans mon précédent message c'est que le champ id_user n'est pas sur la table match mais sur la table pronos.

Juste une petite question car je n'ai pas l'habitude d'utiliser des alias (m.date au lieu de table_match.date), c'est simplement pour des raisons d'optimisation ou il y a une subtilité que je n'ai pas compris ?

[ericd69]

ça compacte l'écriture... tu trimbales pas de longs noms de table ça fait mieux ressortir les colonnes des tables concernées...

tu peux avoir une jointure sur une table qui est elle même en jointure sur la table principale c'est pas un soucis...

j'ai rectifié la requête je pense qu'elle devrait marcher

[gregtix]

Merci pour l'info

Sinon pour la requète je pense qu'il y avait une virgule en trop (après nbperfect)

SELECT m.date, p.id_user, SUM( pts )+pts_bonus AS sumpts, SUM(IF(pts=3,1,0)) AS nbperfect
FROM table_match m
INNER JOIN table_pronos p ON m.id = p.id_match
LEFT JOIN bonus b ON b.id_user=p.id_user
GROUP BY p.id_user
ORDER BY sumpts DESC,nbperfect DESC

Par contre le résultat n'est toujours pas au rendez-vous... j'ai un résultat complètement hallucinant (20000 pts au lieu de 250 pour le 1er)

[ericd69]

tu as une , avant le from

[gregtix]

Nos messages se sont croisés, c'était bien la virgule

[ericd69]

oui comme j'avais repris ta requête j'avais pas fait gaffe à elle...

bon si ça marche enfin tu vas pouvoir passer en résolu...

[gregtix]

La virgule m'a permis de résoudre le message d'erreur mais comme je le précise dans mon précédent message je n'ai absolument pas le résultat désiré et la requète met plus d'une minutes en locale

[ericd69]

bon faut que j'essaye concretement...

ton temps d'exécution est hallucinant... tu as tant de ligne que ça?

file moi les create de tes tables

et un jeu de teste sous forme d'insert sur chaque... je suppose que c'est encore un bd de test que tu as...

si tu as correctement indexé tes tables ça devrait pas être si long...

et là on va avancer car de tête c'est pas évident... je suis pas un compilateur...

[gregtix]

ca marche je t'envoie ça par mp ...

[ericd69]

/* 0 rows affected, 157 rows found. Duration for 1 query: 19,610 sec. */

donc tu dois sérieusement te poser des question sur la la machine, la configuration de l'OS, d'une éventuelle VM et/ou du serveur mysql que tu utilises pour les tests...

le classement m'a l'air correct... le premier est :

2011-08-06 21:00:00,7,11210,2128

[CinePhil]

Juste en passant vite fait sur la requête proposée...
Toutes les colonnes du SELECT ne figurant pas dans une fonction de groupage doivent figurer dans le GROUP BY sous peine de récupérer pour les colonnes absentes des résultats aléatoires.

[ericd69]

regarde le miracle quand j'ai les tables et qu'on part pas d'un postulat faux:

Code sql :

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

1
2
3
4
5
6
7
8
9
select p.id_user as `users`,SUM(p.pts)+b1.bonus as 'total',SUM(IF(pts=3,1,0)) as `departage`
from league_pronos p
left join(
	select b.id_user as `id_user`,sum(bonus) as `bonus`
	from league_bonus b
	group by b.id_user
) b1 on b1.id_user=p.id_user
group by `users`
order by `total` desc,`departage` desc;

au passage, toutes tes tables sont à revoir...

[gregtix]

Ca me semble mieux sauf qu'il n'y a plus la table match dans cette requète et c'est celle-ci qui me permet de faire les filtres donc je ne peux voir si le résultat est correct.

Qu'entends tu par "regarde le miracle quand j'ai les tables et qu'on part pas d'un postulat faux"

Que faut-il améliorer au niveau des tables ? je ne suis pas développeur de formation donc je reconnais que tout n'est probablement pas optimisé

[ericd69]

normal je l'ai viré car elle n'as rien à y faire directement ou au pire on fait une jointure ou une sous-requête sur elle dans le where...

tout un tas de trucs son répétitif et ne servent à rien...

la saison d'un match n'a rien à faire ailleurs que dans la table match...

pire tu peux le générer à la volée vu que c'est juste dépendant de l'année de la date du match... donc saison peut (et doit) disparaitre dans toutes les table...

j'ai du faire une jointure pour le bonus car en fait tu n'as pas une seule entrée par utilisateur... donc il fallait sommer le bonus... n'est-ce pas à revoir...?

ensuite bien sur, placer les index sur les colonnes qui servent aux jointures...

[gregtix]

la saison d'un match n'a rien à faire ailleurs que dans la table match...

=> je suis tout à fait d'accord avec toi et cela fait partie de modification que je dois apporter mais cela implique de nombreuses requètes à modifier (il y a actuellement 66 tables dans la bdd)
Sur ce point j'ai prévu de créer une table saison qui contiendrait quelques infos me permettant d'accèder à certaine sinfos relatives à la saison. (date début du concours qui n'est pas forcément la même que le début du championnat, ....)

j'ai du faire une jointure pour le bonus car en fait tu n'as pas une seule entrée par utilisateur... donc il fallait sommer le bonus... n'est-ce pas à revoir...?

=> Cette table bonus me pose bien des soucis car il y a plusieurs concours et elle ne s'applique que pour le concours sur la Ligue 1 et de plus il se peut que certains participants ne répondent pas à la question qui permet d'obtenir ce bonus.

[gregtix]

cette fois ça marche avec quelques filtres en plus pour affiner ta requète