Skip to main content
adicionou 123 caracteres ao conteúdo
Fonte Link
hkotsubo
  • 66,9mil
  • 25
  • 87
  • 170

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'];
}

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.

Se quer a nota de cada aluno, tem que calcular vários totais diferentes, 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) {
    if (! MediaNotasAluno[notaAluno.matricula])
        MediaNotasAluno[notaAluno.matricula] = { 'matricula': notaAluno.matricula, 'total': 0, 'qtd': 0 };
    MediaNotasAluno[notaAluno.matricula].total += notaAluno.nota;
    MediaNotasAluno[notaAluno.matricula].qtd++;
}

// tendo o total e a quantidade, calcula a média para cada um
for (let m of Object.values(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);

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'];
}

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).

E se você quer a mé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
// cria um array 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'];
}
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). O resultado será:

for (let m of MediaNotasAluno) {
    m.media = Math.floor(100 * m.total / m.qtd) / 100;
    delete m['qtd'];
    delete m['total'];
}
Fonte Link
hkotsubo
  • 66,9mil
  • 25
  • 87
  • 170

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.

Se quer a nota de cada aluno, tem que calcular vários totais diferentes, 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).

Uma forma de fazer é guardar os totais de cada aluno e a respectiva quantidade de notas em um objeto. Depois você calcula a média para cada um:

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) {
    if (! MediaNotasAluno[notaAluno.matricula])
        MediaNotasAluno[notaAluno.matricula] = { 'matricula': notaAluno.matricula, 'total': 0, 'qtd': 0 };
    MediaNotasAluno[notaAluno.matricula].total += notaAluno.nota;
    MediaNotasAluno[notaAluno.matricula].qtd++;
}

// tendo o total e a quantidade, calcula a média para cada um
for (let m of Object.values(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);

No primeiro for eu crio um objeto contendo as matrículas, e para cada uma eu vou somando as notas e atualizando a quantidade de notas. Depois deste primeiro loop, o objeto estará assim:

{
  '117': { matricula: '117', total: 206, qtd: 3 },
  '119': { matricula: '119', total: 159, qtd: 2 },
  '132': { matricula: '132', total: 153, qtd: 2 }
}

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á:

[
  { matricula: '117', media: 68.66666666666667 },
  { matricula: '119', media: 79.5 },
  { matricula: '132', media: 76.5 }
]

Se você quer arredondar os valores para ter apenas duas casas decimais, pode trocar o cálculo para:

for (let m of Object.values(MediaNotasAluno)) {
    m.media = Math.floor(100 * m.total / m.qtd) / 100;
    delete m['qtd'];
    delete m['total'];
}

Assim a primeira média será 68.66.