Ver Mensaje Individual
  #2 (permalink)  
Antiguo 21/07/2010, 05:40
Avatar de _cronos2
_cronos2
Colaborador
 
Fecha de Ingreso: junio-2010
Mensajes: 2.062
Antigüedad: 13 años, 11 meses
Puntos: 310
Respuesta: Tabla de nota y no recorre

Vayamos por partes, como diría Jack el destripador. Lo primero es bienvenido al foro. Lo segundo, usa los highlight de código (cuando vas a postear, arriba a la derecha en un select) para que los códigos se vean mejor. Ahora vamos a mirar el código.
Código Javascript:
Ver original
  1. function rango(id){
  2.  valor=parseInt(id.value,10);
  3.  notaini=window.document.EvAlu.nota.value;
  4.  cantFilas = document.getElementById("table").rows.length;
  5.  for(i=0;i<cantFilas;i++){
  6.   var x=document.getElementById('table').rows[i].cells
  7.   //¿para qué declaras esta variable si no la usas?
  8.   if (notaini<=valor){
  9.    //¿por qué pones 2 paréntesis? Y si valor siempre va a ser 0, no puede mayor o igual a 4, ¿qué se supone que hay que escribir en el input?
  10.    id.style.background="#0FF";
  11.   //los colores siempre es mejor ponerlos en hexadecimal por si acaso
  12.    id.style.color="#00F";
  13.    var o = document.createElement('input');
  14.    o.type = "text";
  15.    o.name = "lalala";
  16.    o.value = "Aprobados";
  17.    //como hay 4 ó 5 elementos con id 'no', no sabe a cuál te refieres, y por eso no funciona
  18.    //y en teoría el name tiene que ser igual al id
  19.    document.getElementById('no').appendChild(o);
  20.    document.getElementById('no').style.visibility = 'hidden';
  21.    // document.getElementById('no').selectedIndex[i] =1
  22.    // document.getElementById("no").length = 1;
  23.    // document.getElementById("no").options[0].value = "Aprobado";
  24.    // document.getElementById("no").options[0].text = "Aprobado";
  25.    //¿por qué todo esto está en forma de comentario?
  26.    //¿y qué significa 'document.getElementById('no').selectedIndex[i] =1'?
  27.    //selectedIndex es una propiedad del objeto, ¿qué pintan ahí esos corchetes?
  28.    return;
  29.   }else{
  30.    id.style.background="#F00";
  31.    id.style.color="#00F";
  32.    document.getElementById('no').selectedIndex[i] =2;
  33.    // document.getElementById("no").length = 1;
  34.    // document.getElementById("no").options[0].value = "Reprobado";
  35.    // document.getElementById("no").options[0].text = "Reprobado";
  36.    return;
  37.   }
  38.  }
  39.  id.style.background="#FFF";
  40.  id.style.color="#000";
  41.  return true;
  42. }
Creo que con eso será suficiente, si sigues teniendo problemas lo dices.
Saludos (:
__________________
" Getting older’s not been on my plans
but it’s never late, it’s never late enough for me to stay. "
Cigarettes - Russian Red