O problema é que você está somando as notas de todos os alunos, e além disso está dividindo pela quantidade de notas no meio do loop (o que não faz sentido, pois para calcular a média você só deve dividir depois de somar todas as notas - ou seja, deveria ser fora do for
, não dentro dele).
SeE se você quer a notamédia de cada aluno, tem que calcular vários totais diferentes, (um para cada matrícula). E a divisão só pode ser feita no final, depois que você somou tudo (e não dentro do for
).
let notasAlunos = [
{ matricula: "117", nome: "Joao", materia: "x1", nota: 78 },
{ matricula: "117", nome: "Joao", materia: "x8", nota: 80 },
{ matricula: "117", nome: "Joao", materia: "y5", nota: 48 },
{ matricula: "119", nome: "Pedro", materia: "w2", nota: 69 },
{ matricula: "119", nome: "Pedro", materia: "x2", nota: 90 },
{ matricula: "132", nome: "Joana", materia: "v3", nota: 77 },
{ matricula: "132", nome: "Joana", materia: "w2", nota: 76 }
];
// calcula a soma das notas de cada aluno (e também a quantidade de notas para cada um)
let MediaNotasAluno = {};
for (let notaAluno of notasAlunos) {
// se ainda não tem registro para esta matrícula, cria um novo
if (! MediaNotasAluno[notaAluno.matricula]) {
MediaNotasAluno[notaAluno.matricula] = {
'matricula': notaAluno.matricula, 'total': 0, 'qtd': 0
};
}
// atualiza o total e a quantidade de notas
MediaNotasAluno[notaAluno.matricula].total += notaAluno.nota;
MediaNotasAluno[notaAluno.matricula].qtd++;
}
// tendo o total e a quantidade, calcula a média para cada um
for// (letcria mum ofarray com os valores
MediaNotasAluno = Object.values(MediaNotasAluno);
for (let m of MediaNotasAluno) {
m.media = m.total / m.qtd;
// apaga a quantidade e o total, pois não preciso mais
delete m['qtd'];
delete m['total'];
}
// cria um array com os valores
MediaNotasAluno = Object.values(MediaNotasAluno);
console.log(MediaNotasAluno);
Mas como você quer um array, basta pegar os valores do objeto, usando Object.values
.
Depois, no segundo for
eu uso o total e a quantidade para calcular a média (e em seguida, apago o total e a quantidade, pois não vou precisar mais). Agora o resultado ainda é um objeto:
{
'117': { matricula: '117', media: 68.66666666666667 },
'119': { matricula: '119', media: 79.5 },
'132': { matricula: '132', media: 76.5 }
}
Mas como você quer um array, basta pegar os valores do objeto, usando Object.values
. O resultado será:
for (let m of Object.values(MediaNotasAluno)) {
m.media = Math.floor(100 * m.total / m.qtd) / 100;
delete m['qtd'];
delete m['total'];
}