Validar campo creados dinamicamente

[mfdp10]

Holasss
Tengo esta funcion para crear campos de texto dinamicamente:

<script type="text/javascript">
ncanti=1

function addIngre() {
var newCantidad = document.createElement('input');
newCantidad.type = 'text';
newCantidad.name = 'cantidades'+ncanti;
newCantidad.size = '5';
ncanti=ncanti+1

document.getElementById('fs').appendChild(newCanti dad);
}
</script>

...y en el form1

<input name="button" type="button" onClick="addIngre(); return false;" value="Agregar">

hasta aqui todo bien
Luego quiero validar estos campos, con el que llamo a otra funcion con un boton en el mismo form1:

<INPUT type="button" value="Enviar datos" name="enviar" onclick="noVacio()">

y esta es la funcion de validar que saque del faq con alguna pequeña modificacion

<script language="JavaScript">
function noVacio() {
var i;
var n = ncanti;
var bError = false;

for (i = 0; i < n; i++){
bError = bError || (eval("document.form1.cantidades" + i + ".value == ''"));
if (bError){
alert("Debe rellenar el/los campo/s cantidad");
eval("document.form1.cantidades" + i + ".focus()");
break;
}
}
if (!bError) alert("Ok");
}
</script>

El problema es que me da error y no se pq
A ver si hay alguna solucion....
Gracias

[JavierB]

Hola mfdp10

Prueba cambiando:

eval("document.form1.cantidades" + i + ".value == ''")

por:

document.form1['cantidades'+i].value == ""

Saludos,

[mfdp10]

Me sigue dando error
Lo que no dije fue que el primer input, o sea "cantidades0" lo cargo con la pagina y los que se crean dinamicamente son los siguientes.
Por eso ncanti en la funcion de validar
Y si no genero ningun campo dinamicamente, o sea dejo el que se carga con la pagina, sí que funciona el validar.
Creo que el problema esta cuando agrego algun campo con la funcion addIngre()
Que por otro lado he comprobado ue los campos se generan bien, no solo pq me los muestra, sino pq -sin validarlos- los recojo bien cuando ejecuto el submit

[JavierB]

Hola de nuevo.

He probado tu código y no me da ningún error Dinos que error te da y en que línea.

Saludos,

[mfdp10]

Hola JavierB

me da el siguiente error

Línea: 23
Car: 3
Error: document.form1.cantidades0.value es nulo o no es un objeto
Codigo:0

Saludos

[JavierB]

Rehola.

Supongo que le has puesto nombre al campo que cargas con la página

<input type="text" name="cantidades0" />

¿Se puede ver la página en algún sitio?

Saludos,

[mfdp10]

Recontra hola
Si, se lo he puesto, auque en algunas pruebas lo he quitado.... por probar solo los dinamicos...
Lo puedes ver aqui http://www.cocinarbien.com/dinamicos.asp
gracias

[JavierB]

Sorprendente

Con Firefox funciona bien (por eso te dije que no me daba error) pero con IE falla. Parece ser que el explorer no se entera de que se ha asignado un name dinamicamente. Voy a estar atento a este mensaje a ver si a alguien se le ocurre como solucionarlo. Si se me ocurre algo a mí... volveré.

Saludos,

[nicolaspar]

Que tal?
No me ha dado error, de todas maneras agregaria debajo de la asignacion obj.name el id del mismo:
function addIngre() {
var newCantidad = document.createElement('input');
newCantidad.type = 'text';
newCantidad.name = 'cantidades'+ncanti;
newCantidad.id = 'cantidades'+ncanti;
newCantidad.size = '5';
ncanti=ncanti+1
document.getElementById('fs').appendChild(newCanti dad);
}
Lo mismo para el input inicial:
<input name="cantidades0" id="cantidades0" type="text" size="5">
No se si lo solucione ya que como dije no vi error, pero al menos es lo correcto;)

[mfdp10]

Eres un mago nicolaspar
poniendo el id a cada campo funciona perfecto
Muchas gracias para ti tb JavierB
Saludos

[mfdp10]

hola otra vez

Ahora el error me lo da cuando quiero validar el campo cantidades, para que solo se escriban numeros...

var i;
var n = ncanti;
var bError = false;

for (i = 0; i < n; i++){
bError = bError || (eval("document.form1['cantidades'+i].value == ''"));
if (bError){
alert("Escriba por lo menos 1 caracter en el campo \"Cantidad\" .");
eval("document.form1['cantidades'+i].focus()");
return (false);
}
}
var checkOK = "0123456789";
var checkStr = document.form1['cantidades'+i].value;
var allValid = true;
var decPoints = 0;
var allNum = "";
for (t = 0; t < checkStr.length; t++) {
ch = checkStr.charAt(t);
for (j = 0; j < checkOK.length; j++)
if (ch == checkOK.charAt(j))
break;
if (j == checkOK.length) {
allValid = false;
break;
}
allNum += ch;
}
if (!allValid) {
alert("Escriba sólo dígitos en el campo \"Cantidad\".");
document.form1['cantidades'+i].focus();
return (false);
}


Línea: 101
Car: 3
Error: document.form1[...].value es nulo o no es un objeto
Codigo:0

ademas me gustaria que se puedan meter en ese campo dos decimales

Saludos

[mfdp10]

Hola otra vez
Por si a alguien le vale lo de validar que solo sean numeros, ademas de comprobar que no este vacio, hice lo siguiente... no se si es lo correcto, pero me funciona

for (i = 0; i < n; i++){
bError = bError || (eval("document.form1['cantidades'+i].value == ''"));
var checkOK = "0123456789.,";
var checkStr = document.form1['cantidades'+i].value;
var allValid = true;
var decPoints = 0;
var allNum = "";
for (t = 0; t < checkStr.length; t++) {
ch = checkStr.charAt(t);
for (j = 0; j < checkOK.length; j++)
if (ch == checkOK.charAt(j))
break;
if (j == checkOK.length) {
allValid = false;
break;
}
allNum += ch;
}
if (!allValid) {
alert("Escriba sólo números en el campo \"Cantidad\" .");
document.form1['cantidades'+i].focus();
document.form1['cantidades'+i].select();
return (false);
}
if (bError){
alert("Escriba por lo menos 1 caracter en el campo \"Cantidad\" .");
eval("document.form1['cantidades'+i].focus()");
return (false);
}
}

[mfdp10]

Bueno..... solo me falta poder eliminar dinamicamente los inputs que se crean con la funcion addIngre().... con lo que lo intento de esta forma y me limpia todo... algo es algo..

function removeIngre()
{
var aBorrar = document.getElementById('fs');
aBorrar.parentNode.removeChild(aBorrar);
}

pq es donde se generan los inputs <fieldset id="fs">

ahora bien si quiero eliminar desde el ultimo hasta el primero que cree, pruebo con esto...

function removeIngre()
{
var aBorrar = document.getElementById('cantidades'+ncanti-1);
aBorrar.parentNode.removeChild(aBorrar);
}

me da error:
Linea: tal
Car: 3
Error: Se requiere un objeto
Codigo: 0

no se como hacerlo.... una ayuda porfa
Saludos