Sinceramente, nem tentei entender qual o problema do seu código, eu acho mais simples reescrever o cálculo da média (que aliás, me parece desnecessariamente complicado).
A ideia é simples: para cada aluno você pega o array de notas, soma elas e divide pelo tamanho do array. Depois insere isso no array de médias, assim:
function Aluno(nome, qtfaltas, notas) {
this.nome = nome;
this.qtfaltas = qtfaltas;
// *** OBS: mudei o nome para "notas", pois são várias notas e não uma só ***
this.notas = notas;
}
let Aluno1 = new Aluno("Jose Silva", 0, [0, 10, 20]);
let Aluno2 = new Aluno("Paulo Santos", 5, [0, 1, 2]);
let Aluno3 = new Aluno("Jose Santos", 10, [0, 5, 8]);
let Aluno4 = new Aluno("Antonio Silva", 4, [0, 7, 9]);
let Aluno5 = new Aluno("Maria dos santos", 6, [0, 3, 4]);
let listaAlunos = [Aluno1, Aluno2, Aluno3, Aluno4, Aluno5];
const medias = [];
for (const aluno of listaAlunos) {
let soma = 0;
for (const nota of aluno.notas) {
soma += nota;
}
medias.push({nome: aluno.nome, media: soma / aluno.notas.length });
}
console.log(medias);
Só isso. Nada de map
, forEach
, find
, etc. E também mudei o nome da variável notas
, afinal, ela é um array que pode ter várias notas, e colocar o nome no plural deixa isso mais claro. Pode parecer um detalhe bobo, mas dar nomes melhores ajuda muito na hora de programar.
Ah, também mudei listalunos
para listaAlunos
(faltou um "A" ali). De novo, pode parecer preciosismo, mas arrumar esses pequenos detalhes no início pode fazer diferença no longo prazo.
E em vez de for..of
(que pode não funcionar em browsers mais antigos), você também poderia usar o for
tradicional:
const medias = [];
for (let i = 0; i < listaAlunos.length; i++) {
const aluno = listaAlunos[i];
let soma = 0;
for (let j = 0; j < aluno.notas.length; j++) {
soma += aluno.notas[j];
}
medias.push({ nome: aluno.nome, media: soma / aluno.notas.length });
}
Claro que você pode até poderia usar map
para este caso, afinal para cada elemento do array de alunos você quer criar um elemento correspondente contendo o nome e a média:
// para cada aluno, retorne um objeto com o nome e a média
const medias = listaAlunos.map(function(aluno) {
let soma = 0;
for (const nota of aluno.notas) {
soma += nota;
}
return { nome: aluno.nome, media: soma / aluno.notas.length };
});
Claro que tem o custo extra do callback, mas para poucos arrays pequenos a diferença é imperceptível. De qualquer forma, este é um caso em que o map
é mais apropriado do que forEach
(apesar de ambos percorrerem o array, há diferenças: map
retorna outro array com os resultados, que é exatamente o que você quer).
Por fim, vi que em algumas linhas você usou ponto-e-vírgula no final e em outras não. Eu sugiro que seja consistente e use sempre. Pode parecer "frescura", e sei que o JavaScript "aceita" o código sem ponto e vírgula e "funciona", mas isso evita algumas situações bizarras que podem ocorrer se você não usá-los, como essa e essa (veja mais sobre isso aqui).