Discussion :

[ziondog]

Bonjour à tous,
Etant nouveau dans le domaine de la programmation PL/SQL, j'aurais besoin de vos lumières.

Voici 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
14
15
16
17
18
19
20
21
22
23
24
25
26
27
CREATE OR REPLACE FUNCTION moyenneEtudiantModule(p_etudiant IN Etudiants.idEtudiant%TYPE, p_idModule IN Modules.idModule%TYPE) RETURN NUMBER
IS
moyenne NUMBER;
moyenne_note NUMBER;
moyenne_coefficient NUMBER;
 
BEGIN 
DBMS_OUTPUT.ENABLE;
 
FOR m IN(SELECT Ma.coefficientMatiere,N.note INTO moyenne_coefficient, moyenne_note
FROM Modules M, Matieres Ma, Etudiants E, Notes N
WHERE M.idModule = Ma.idModule
AND Ma.idMatiere = N.idMatiere
AND N.idEtudiant = E.idEtudiant
AND E.idEtudiant = p_etudiant
AND M.idModule = p_idModule)
 
LOOP
 
moyenne := moyenne + (moyenne_coefficient*moyenne_note);
 
END LOOP;
 
moyenne := moyenne / moyenne_coefficient;
RETURN (moyenne);
 
END;

Ensuite j'exécute ma fonction :

Code :

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

1
2
SELECT moyenneEtudiantModule('E6', 'M112')
FROM DUAL;

Mais la, rien ne se produit.
Normalement la moyenne devrait s'afficher, mais rien ne se passe. Ma requête SQL m'affiche bien 2 colonnes avec les notes des matières et leurs coefficients donc je ne vois pas d'où peut venir le problème.

Merci d'avance.

[mnitu]

Quel est le résultat de l'opération suivante

Code :

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

1
2
 
NULL + 3

[ziondog]

Merci de votre réponse,

Selon moi, cela donne NULL mais je ne vois pas trop ce qui pourrait être considéré comme NULL dans mon code.

Edit : je viens d'initialiser mes variables à 0 mais cela ne m'affiche toujours pas le résultat attendu.

[ziondog]

Désolé pour le double message,

j'ai réussi à régler mon erreur en initialisant à zéro et en changeant un petit peu 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
14
15
16
17
18
19
20
21
22
23
24
25
26
CREATE OR REPLACE FUNCTION moyenneEtudiantModule(p_etudiant IN Etudiants.idEtudiant%TYPE, p_idModule IN Modules.idModule%TYPE) RETURN NUMBER
IS
moyenne NUMBER:=0;
cpt_coeff NUMBER :=0;
 
BEGIN 
DBMS_OUTPUT.ENABLE;
 
FOR m IN(SELECT Ma.coefficientMatiere,N.note
FROM Modules M, Matieres Ma, Etudiants E, Notes N
WHERE M.idModule = Ma.idModule
AND Ma.idMatiere = N.idMatiere
AND N.idEtudiant = E.idEtudiant
AND E.idEtudiant = p_etudiant
AND M.idModule = p_idModule)
 
LOOP
moyenne := moyenne + (m.coefficientMatiere*m.note);
cpt_coeff := cpt_coeff + m.coefficientMatiere;
END LOOP;
 
moyenne := moyenne / cpt_coeff;
 
RETURN (moyenne);
 
END;

Encore merci !

[aieeeuuuuu]

Bonjour,

Il y a bien plus simple : le curseur est inutile, il suffit de faire une requête comme celle-ci (j'ai au passage normalisé les jointures, votre prof pourrait peut être se mettre à jour...)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
SELECT 
	SUM(Ma.coefficientMatiere * N.note) / SUM(Ma.coefficientMatiere) AS Moyenne
FROM Etudiants E
INNER JOIN Notes N
	ON	N.idEtudiant = E.idEtudiant
INNER JOIN Matieres Ma
	ON	Ma.idMatiere = N.idMatiere
INNER JOIN Modules M
	ON	M.idModule = Ma.idModule
WHERE	E.idEtudiant = p_etudiant
AND		M.idModule = p_idModule

[mnitu]

D'autre part quand vous analysez votre code vous constatez que vous calculez Somme(m.coefficientMatiere*m.note) divisez par Somme(m.coefficientMatiere). Or SQL est très performant dans ces types de calcul et la boucle en PL/SQL c'est juste une manière moins efficace de faire la même chose. Essayez de récrire votre fonction sans la boucle!

[ziondog]

Merci de vos réponses, je vais essayer de refaire le programme sans la boucle

Encore merci !