No enunciado, ele pede para criar uma sala, e quatro alunos, e estes com quatro notas. Só que, ao criar suas variáveis, você cria um aluno só:
var sala = [];
var aluno = []; // <- isso deveria representar apenas 1 aluno
Isso, em si, não representa um problema, você poderia guardar mais alunos neste mesmo array
, mas aí você não precisaria do array
sala
. Veja onde está o problema:
for (var i = 0; i < 4; i++) {
for (var j = 0; j < 5; j++) {
if (j == 0) {
aluno.push(prompt("Digite o nome")); // aluno recebe 4 nomes
} else {
aluno.push(prompt("Digite a nota")); // e 16 notas
}
}
sala.push(aluno); // o mesmo aluno é passado para sala, 4 vezes
}
Ou seja:
sala[0]
, sala[1]
, sala[2]
e sala[3]
terão referência para exatamente o mesmo objeto. Para um exemplo "visual" do que está ocorrendo, dê uma olhada neste fiddle.
Isso acontece porque em javascript, arrays são objetos, e portanto são passados como referência para funções e não como cópia do valor como é o caso de primitivos.
A solução do @joserogerio84 funciona, porque ele não está somente "zerando" o array a cada iteração, ele está criando um novo.
Veja um exemplo funcionando abaixo:
/*
* criar sala de aula com 4 alunos que possuem 4 notas
* e depois calcular a média de cada aluno.
* Sem objeto
*/
let sala = [];
let aluno = [];
let t = document.getElementById('table').getElementsByTagName('tbody')[0];
for (let i = 0; i < 4; i++) {
for (let j = 0; j < 5; j++) {
if (j == 0) {
aluno.push(prompt("Digite o nome"));
} else {
aluno.push(prompt("Digite a nota"));
}
}
sala.push(aluno); // mandamos a referencia desse aluno para sala
aluno = []; // isso cria um novo array com o mesmo nome
}
for (let i = 0; i < sala.length; i++) {
let linha = t.insertRow(t.rows.length);
for (let j = 0; j < sala[i].length; j++) {
saida(linha, j, sala[i][j]);
}
}
function saida(linha, numero, texto) {
linha.insertCell(numero).appendChild(document.createTextNode(texto));
}
table, th, td {
border: 1px solid black;
border-collapse: collapse;
}
th, td {
padding: 10px;
}
<table id="table">
<th>Nome</th>
<th>Nota</th>
<th>Nota</th>
<th>Nota</th>
<th>Nota</th>
</table>