problema con variable numerica

leoalvis · #1 (**permalink**) 18/08/2017, 11:51

Hola buenas tardes: tengo inconvenientes en mostrar el promedio de una lista de estudiantes que se van registrando uno a uno me arroja el valor NaN. Ademas, extrañamente las funciones de mostrarMayor y mostrarMenor no funcionan correctamente. Si se ejecuta una la otra no me envia resultados. En mi opinion creo que es la variable numerica que se captura en el arreglo estudiante. Pero ya he revisado e intentado muchas cosas pero no se que mas hacerle.. Agradezco la ayuda.. adjunto el html y javascript respectivo...

Código HTML:

<!DOCTYPE html>
<html>
  <head>
    <meta charset="utf-8">
    <title></title>
    <script src="js/ejercicio.js"></script>
    <link rel="stylesheet" href="css/ejercicio.css">
  </head>
  <body>

    <h1>Registro Estudiante</h1>
    <b>Codigo</b>
    <br>
    <input type="text" id="txtCodigo">
    <br>
    <b>Nombre</b>
    <br>
    <input type="text" id="txtNombre">
    <br>
    <b>Nota:</b>
    <br>
    <input type="text" id="txtNota">
    <br>
    <br>
    <button type="button" id="btnRegistrar">Registrar Estudiante</button>
    <button type="button" id="btnPromedio">Registrar Promedio</button>
    <button type="button" id="btnNotaMayor">Mostrar Nota Mayor</button>
    <button type="button" id="btnNotaMenor">Mostrar Nota Menor</button>
    

  </body>
</html>

Código Javascript:

Ver originalwindow.onload=function(){
  let estudiante=new Array();
  let codigo;
  let nombre;
  let nota;
 
 document.getElementById("btnRegistrar").addEventListener("click",registrar);
  document.getElementById("btnPromedio").addEventListener("click",mostrarPromedio);
  document.getElementById("btnNotaMayor").addEventListener("click",mostrarMayor);
  document.getElementById("btnNotaMenor").addEventListener("click",mostrarMenor);
 
 
  function registrar(){
    codigo=document.getElementById("txtCodigo").value;
    nombre=document.getElementById("txtNombre").value;
    nota=parseFloat(document.getElementById("txtNota").value);
    estudiante.push({'codigo':codigo,'nombre':nombre,'nota':nota});
    document.getElementById("txtCodigo").value="";
    document.getElementById("txtNombre").value="";
    document.getElementById("txtNota").value="";
 
    mostrarTabla();
 
    
 
  }
 
  var tabla=document.createElement("table");
 
  function mostrarTabla(){
 
 
    tabla.setAttribute("border","1");
 
    var fila;
    var celda;
 
    for (var i = 0; i < estudiante.length; i++) {
 
      fila=document.createElement("tr");
      celda=document.createElement("td");
 
      celda.appendChild(document.createTextNode(estudiante[i].codigo));
      celda.appendChild(document.createTextNode(estudiante[i].nombre));
      celda.appendChild(document.createTextNode(estudiante[i].nota));
    }
 
    fila.appendChild(celda);
    tabla.appendChild(fila);
 
    document.body.appendChild(tabla);
 
 
  }
 
  function mostrarPromedio(){
 
    var sum=0;
 
    for(var i=0;i<estudiante.lenght;i++){
      sum+=estudiante[i].nota;
    }
 
    var prom=sum/i;
 
    alert("El promedio de la clase es "+prom);
  }
 
  function mostrarMayor(){
    var salida="";
    var mayor=estudiante[0].nota;
 
    for(var i=0;i<estudiante.length;i++){
        if(mayor<estudiante[i].nota){
          mayor=estudiante[i].nota;
          alert("El estudiante con mayor nota es "+estudiante[i].nombre+
                                                            " con una nota de "+mayor);
        }
    }
 
 
  }
 
  function mostrarMenor(){
    var salida="";
    var menor=estudiante[0].nota;
 
    for(var i=0;i<estudiante.length;i++){
        if(estudiante[i].nota<menor){
          menor=estudiante[i].nota;
          alert("El estudiante con menor nota es "+estudiante[i].nombre+
                                                            " con una nota de "+menor);
        }
 
    }
 
  }
 
 
}

emilio_viguri · #2 (**permalink**) 18/08/2017, 13:31

Bueno ya intnetaste especificar tus varibles dandoles la propiedad segun sea

for(var i=0;i<estudiante.lenght;i++){
sum+=estudiante[i].nota;
}

var prom=parseFloat(sum/i);

parseInt o parseFloat según el tipo de valor enviado, ya sea entero o con decimales según corresponda.

calculo = saldo - cobro + parseInt(valpago);// enteros

o bien
calculo = saldo - cobro + parseFloat(valpago); // con decimales

leoalvis · #3 (**permalink**) 18/08/2017, 13:41

Cita:

Iniciado por emilio_viguri

Bueno ya intnetaste especificar tus varibles dandoles la propiedad segun sea

for(var i=0;i<estudiante.lenght;i++){
sum+=estudiante[i].nota;
}

var prom=parseFloat(sum/i);

parseInt o parseFloat según el tipo de valor enviado, ya sea entero o con decimales según corresponda.

calculo = saldo - cobro + parseInt(valpago);// enteros

o bien
calculo = saldo - cobro + parseFloat(valpago); // con decimales

Hola gracias por la sugerencia.. si ya lo hice... pero no me funciona.. hay algo en el elemento estudiante[i].nota que creo esta mal.. pero no se me ocurre otra forma de calcular el promedio de la lista de estudiantes que no sea traves de un arreglo y comparar la nota mayor y menor. Lo extraño es que ya esta parseada deberia funcionarme pero no se por que no lo hace.

Alexis88 · #4 (**permalink**) 18/08/2017, 14:31

La propiedad que obtiene el total de elementos de un arreglo es length, no "lenght". Ese error se encuentra en el bucle que usas en la función mostrarPromedio(). Solo cambia eso y obtendrás el resultado que esperas.

leoalvis · #5 (**permalink**) 18/08/2017, 15:04

Cita:

Iniciado por Alexis88

La propiedad que obtiene el total de elementos de un arreglo es length, no "lenght". Ese error se encuentra en el bucle que usas en la función mostrarPromedio(). Solo cambia eso y obtendrás el resultado que esperas.

Que error tan bobo, pude haberme pasado horas y horas tratando de ver errores de lógica cuando la respuesta era tan elemental de sintaxis... Solucionado, De verdad muchas gracias.