Algo falla en mi select

[darkxer0x]

El script sencillito es este:

Código:

<form action="upload.php" name="formulario" method="post" enctype="multipart/form-data">
<SCRIPT language="JavaScript" type="text/javascript">
document.write("<label>Numero de Imagenes <select name=numero size=1 onSelect:imagenes()>");
for (var i = 1; i < 6; i++) {
	document.write("<option value=" + i + ">" + i);
}
document.write("</select></label>");
function imagenes()  {
	var n = formulario.numero.options[formulario.numero.selectedIndex].value

	for (var i = 1; i <= n; i++) {
	document.write("<p><label>Imagen Mediana " + i + " <input type=file name=mediana" + i + "></label></p>");
	}
}
</form>

Lo que hago es sacar tantos forms para subir archivos como especifique el usuario (hasta 5 como maximo). El problema es que una vez seleccionado no ocurre nada... He intentado poniendo onChange: y tampoco. Hay algo mal y no se me ocurre.
Muchas gracias

[caricatos]

Hola:

Debes usar onchange, y no valen 2 puntos, sino la igualdad:

onchange="imagenes()"

Pero ten cuidado al usar document.write()... prohibido con la página ya cargada... seguramente sea mejor usar innerHTML

saludos

[darkxer0x]

Hola.
Muchas gracias. Aunque un apunte:

prohibido con la página ya cargada

Parece que prohibido no, pero recarga la pagina con el nuevo codigo, muy molesto porque los demas formularios pueden borrarse.

seguramente sea mejor usar innerHTML

Pues has dado en el clavo. Aunque tiene su complicacion...

[JoseTejada]

ehmm :S, es primera vez q me atrevo a dar una respuesta ya q no soy muy experto en el JavaScript, pero espero que esto ayude...:
<form name="formulario">
<script language="JavaScript" type="text/javascript">
document.write("<label>Numero de Imagenes <select name=numero size=1 onChange='imagenes()'>");
document.write("<option selected></option>");
for (var i = 1; i < 6; i++) {
document.write("<option value=" + i + ">" + i +"</option>");
}
document.write("</select></label>");
function imagenes() {
var n = formulario.numero.options[formulario.numero.selectedIndex].value
texto = "";
for (var i = 1; i <= n; i++) {
texto += "<p><label>Imagen Mediana " + i + " <input type=file name=mediana" + i +

"></label></p>";
}
aqui.innerHTML = texto;

}
</script>
<div id="aqui"></div>

[darkxer0x]

Hola.
Gracias JoseTejada.
Tu script no me iba, pero todo parecia correcto. Hasta que me di que es lo que hacia que no fuera:

Código:

var n = formulario.numero.options[formulario.numero.selectedIndex]

Y habia que ponerlo con document. delante de formulario:

Código:

document.formulario.numero.options[document.formulario.numero.selectedIndex

Pero me has sacado de un apuro. Yo no he tratado nada con javascript, y todavia no me he acostumbrado.

Por cierto, el script final que funciona, por si alguien quiere poner algo como lo que yo hago, es este:

Código:

<form action="upload.php" method="post" name="formulario" enctype="multipart/form-data">
<SCRIPT language="JavaScript" type="text/javascript">
document.write('<label>Numero de Imagenes <select name="numero" size=1 onChange="imagenes()">');
document.write("<option selected></option>");
for (var i = 1; i < 6; i++) {
	document.write("<option value=" + i + ">" + i + "</option>");
}
document.write("</select></label>");
function imagenes()  {
var n = document.formulario.numero.options[document.formulario.numero.selectedIndex].value
texto = "";
for (var i = 1; i <= n; i++) {
texto += "<p><label>Imagen Mediana " + i + " <input type=file name=mediana" + i +"></label></p>";
}
document.getElementById('aqui').innerHTML = texto;

}
</script>
<div id='aqui'></div>
<p><input type="submit" value="Subir"></p>
</form>

Supongo que habrá formas mas elegantes de ahcerlo, pero bueno, esto por lo menos funciona de maravilla.

[caricatos]

Hola:

Tal vez intentar algo más "elegante" puede añadir una complejidad innecesaria, aunque con el DOM no sería tan difícil...

Código:

for (var i = 1; i <= n; i++) {
//texto += "<p><label>Imagen Mediana " + i + " <input type=file name=mediana" + i +"></label></p>";
p = document.createElement("p");
l = document.createElement("label");
l.appendChild(document.createTextNode("Imagen Mediana " + i + " "));
inp = document.createElement("input");
inp.type = "file";
inp.name = "mediana + i;
l.appendChild(inp);
p.appendChild(l);
document.getElementById("aqui").appendChild(p);
}
//document.getElementById('aqui').innerHTML = texto;

No sé si es más elegante, pero es el uso estandar del DOM, así que creo que como lo tienes está muy bien.

Lo único que creo que puedes simplificar es esta línea:

Código:

var n = document.formulario.numero.options[document.formulario.numero.selectedIndex].value

Por algo así:

Código:

var n = document.formulario.numero.value

Incluso podrías inicializar n en el propio bucle:

Código:

for (var i = 1, n = document.formulario.numero.value; i <= n; i++)

Saludos