Discussion :

[persia]

Bonjour à tous,

J'aimerais calculer et afficher une moyenne seulement si une certaine condition s'applique. Pour faire simple je donne cette exemple:

Code :

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

1
2
3
$sql="SELECT t.auteur_id, t.cours_id,"; 
$sql.="CASE WHEN t.auteur_id ='3' THEN 'avg(t.score)' ELSE '' END ";
$sql.="FROM Auteur t";

Donc ici, j'aimerais afficher la moyenne d'un score seulement si auteur_id =3.

Mais j'ai toujours une erreur de syntaxe que j'ignore comment régler.

Merci pour votre aide.

[Climoo]

Je vois pas trop pourquoi utiliser un 'case' alors que tu n'as qu'une seule condition.
Je connais pas trop la syntaxe de ça ...
Sinon, c'est expliqué là :http://dev.mysql.com/doc/refman/5.0/...functions.html

A ta place, je tenterai un simple "IF(cond, then, else) "

Mais quand on y pense, je vois pas trop le sens de ta requête :
Supposons que l'on ait ça :
Table auteur
auteur_id | score
------------------
1 | 4
3 | 3
2 | 16
3 | 7
------------------

Que devrait afficher ta requête ?
Il te manque pas un truc genre group by par hasard...?

De plus, ta question porte davantage sur du SQL que du PHP, donc je pense que tu t'aies trompé de forum...

[ericd69]

salut,

dans un select, tu peux utiliser la version fonctionnelle de if que te propose climoo... mais là ton approche est mauvaise, tu n'auras pas la moyenne... il faut faire un group by...

pas la peine de surnommer ta table... non plus s'il n'y a qu'une table en jeu...

Code sql :

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

1
2
3
4
SELECT auteur_id, cours_id, avg(score)
FROM Auteur
group by auteur_id
where auteur_id=3

[persia]

Je vois pas trop pourquoi utiliser un 'case' alors que tu n'as qu'une seule condition.
Je connais pas trop la syntaxe de ça ...
Sinon, c'est expliqué là :http://dev.mysql.com/doc/refman/5.0/...functions.html

A ta place, je tenterai un simple "IF(cond, then, else) "

Mais quand on y pense, je vois pas trop le sens de ta requête :
Supposons que l'on ait ça :
Table auteur
auteur_id | score
------------------
1 | 4
3 | 3
2 | 16
3 | 7
------------------

Que devrait afficher ta requête ?

De plus, ta question porte davantage sur du SQL que du PHP, donc je pense que tu t'aies trompé de forum...

Ma requête devrait afficher 5 c'est à dire juste la moyenne de l'auteur avec id 3.

Le forum s'intitule PHP & MySQL, donc je suis sans le forum j'imagine, non ?

[persia]

ok, je vais donner un autre exemple qui peut être sera plus clair et va mieux expliquer mon besoin.

J'ai la table suivante:

Code :

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

1
2
3
4
5
6
7
id    note1  note2  timestamp
1     3        0        12 mars
2     5        0        12 mars
3     6        0        12 mars
4     5        6        15 mars
5     7        7        15 mars
6     4        0        15 mars

La colonne note2 a été ajouté dans la table le 15 mars. Quand j'ai ajouté la colonne note2, les 0 ont été ajoutés par défaut dans les dates avant le 15 mars, alors que c'est pas une vrai note qui doit être calculée dans la moyenne. Par contre le 0 de la note2 pour le id 6 est un vrai 0 car ca été ajouté à partir du 15 mars et doit être calculé dans la moyenne. Donc ce que je veux faire, c'est que la moyenne de la note2 se calcule à partir de 15 mars.

Autrement-dit, je dois ajouter dans ma requête une syntaxe du genre: si le timestamp est plus grand ou égal au 15 mars on calcule la moyenne de la note2, sinon on prends pas cette note en considération

J'espère que maintenant vous comprenez mieux mon besoin

[rawsrc]

Bonjour,

la structure de ta table est mauvaise et c'est la raison pour laquelle tu galères.
Tu comptes rajouter des colonnes noteX autant de fois que nécessaire ?
Si oui, cela va devenir vite ingérable.
tu devrais plutôt faire ainsi :
id | id_exam | id_eleve | note

La date devrait être rattachée à l'examen et pas à la note.

[persia]

Bonjour,

la structure de ta table est mauvaise et c'est la raison pour laquelle tu galères.
Tu comptes rajouter des colonnes noteX autant de fois que nécessaire ?
Si oui, cela va devenir vite ingérable.
tu devrais plutôt faire ainsi :
id | id_exam | id_eleve | note

La date devrait être rattachée à l'examen et pas à la note.

Bonjour Rawsrc,

En effet, ton approche est bonne et je l'utilise déjà dans d'autres projets, mais j'ai pas le choix de travailler avec la structure qui est déjà en place et c'est pour ça que je dois trouver un moyen pour contourner ça.

Non je ne comptes pas rajouter des colonnes noteX autant de fois que nécessaire, c'est seulement une fois et c'est une exception



Merci

[CinePhil]

Je pense que tu vas devoir calculer la moyenne à partir du 15 mars séparément et faire une union avec le reste.

Donne la structure de la table, un petit jeu de données et le résultat attendu.

[persia]

Je pense que tu vas devoir calculer la moyenne à partir du 15 mars séparément et faire une union avec le reste.

En effet, c'est exactement ce que je veux faire.

Voici ma table:

Code :

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

1
2
3
4
5
6
7
id   auteur_id  note1  note2    timestamp
1           3      7        0        12 mars
2           5      8        0        12 mars
3           6      5        0        12 mars
4           5      6        6        15 mars
5           7      3        7        15 mars
6           5      6        0        15 mars

Résultat attendu:

Code :

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

1
2
Moyenne note1       Moyenne note2
         5 .83                      4.33

Requête actuel:

Code :

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

1
2
3
4
SELECT t.id, t.auteur_id, r2.Moyenne1,r2.Moyenne2
FROM result t
JOIN (SELECT r.auteur_id, avg(r.note1) as "Moyenne1",avg(r.note2) as "Moyenne2",
FROM result r)r2;

Avec cette requête j'obtiens une moyenne de 2.17 pour la note 2 alors que c'est pas le résultat souhaité

Merci

[skuatamad]

Update les note2 inférieures au 15 mars à NULL, modifie la structure des colonnes notes en DEFAULT NULL et lors d'une prochaine création de colonne précise DEFAULT NULL.

Ensuite une simple requête :

Code :

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

select avg(note1), avg(note2) from la_table

fera l'affaire car NULL n'est pas pris en compte par les fonctions d'aggrégations comme AVG ou SUM...

Après ça reste de la bidouille, je rejoins totalement rawsrc sur le problème de modélisation de la table.

[persia]

Ok je vais essayer ca.Merci

[ericd69]

j'espère que cette fois c'est la bonne structure définitive...

n'oublie pas que les gens prennent sur leur temps pour réfléchir à ton problème et que prendre des pseudo exemple qui ne se ressemble pas du tout pour un même problème...

à supposer qu'il ne peut donc y avoir que 2 dates possible sur l'ensemble de la table, alors en une seule requête pas optimisée pour beaucoup de lignes:

Code sql :

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

1
2
select avg(if((select max(timestamp) from la_table)>timestamp,note1,note2))
from la_table

le mieux est de faire une procédure stockée, pour ne pas ré-exécuter la sous requête à chaque fois:

Code sql :

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

1
2
3
4
5
6
7
8
9
10
11
delimiter $$
drop procedure moyenne$$
create procedure moyenne()
begin
  declare date a;
  select max(timestamp) into a from la_table;
  select avg(if(a>timestamp,note1,note2)) from la_table;
end$$
delimiter ;
 
call moyenne();

[persia]

Update les note2 inférieures au 15 mars à NULL, modifie la structure des colonnes notes en DEFAULT NULL et lors d'une prochaine création de colonne précise DEFAULT NULL.

Ton approche à bien du sens et je vais opter pour car je trouve que c'est la meilleur solution qui réponds à mes besoins actuels.

Des fois on a pas le choix de faire avec ce qu'on a déjà en place.

[persia]

Merci à tous