Codigo mal en funcion con variables?

[Jomer_zgz]

Muy buenas de nuevo (Hoy me estoy luciendo con la cantidad de preguntas, joer)

Tengo una funcion en la que quiero definir unas variables, pero realmente no se si lo estoy haciendo bien:

Código:

function cambio(boton,campo,variable){
	eval("var "+variable+" = document.forms.form1."+campo+".value;");
	switch(variable) {
	case "0":
	eval("document.forms.form1."+boton+".value='Si ';");
	eval("document.forms.form1."+campo+".value = '1';");
	break;
	case "1":
	eval("document.forms.form1."+boton+".value='No';");
	eval("document.forms.form1."+campo+".value = '2';");
	break;
	default:
	eval("document.forms.form1."+boton+".value='--';");
	eval("document.forms.form1."+campo+".value = '0';");
	break;
	}
	}

(Te suena el codigo de switch caricatos? )

Código:

<input type="button" name="btn01" id="btn01" onClick="cambio(btn01,cod_uno,var_uno)" value="--" class="WG519">

Alguna idea??

Saludos de nuevo y gracias!

[Jomer_zgz]

He probado a poner también el switch con eval, y sigue igual :(

[caricatos]

Iniciado por Jomer_zgz

Muy buenas de nuevo (Hoy me estoy luciendo con la cantidad de preguntas, joer)
...
(Te suena el codigo de switch caricatos? )
...

¡Je, je !... claro que me suena.

Las referencias a subobjetos y atributos en una página, aparte de hacerse con puntos, se puede hacer mediente cadenas de texto entrecorchetadas...

document.forms.form1.campo.value;
es igual que
document.forms.form1["campo"].value;

así que solo tienes que quitar esos odiosos "eval" (no me gustan nada) y poner tus variables entre corchetes...

Y las variables deben ser parte del objeto window, así que te arreglo la primera línea, y te dejo el resto como tarea...

window[variable] = document.forms.form1[campo].value;

Lo que no tengo muy claro es el contenido del switch... creo que deberia ser:
switch(window[variable])

Bueno, ya nos contarás.

Saludos

[Jomer_zgz]

Puessssssss me temo que me sigue dando error.

Me dice que document.forms.form[...].value no esta definido. (Me sustituye la variable por 3 puntos suspensivos).

El codigo:

Código:

function cambio(boton,campo,variable){
	window[variable] = document.forms.form1[campo].value;
	switch(window[variable]) {
	case "0":
	document.forms.form1[boton].value='Si ';
	document.forms.form1[campo].value = '1';
	break;
	case "1":
	document.forms.form1[boton].value='No';
	document.forms.form1[campo].value = '2';
	break;
	default:
	document.forms.form1[boton].value='--';
	document.forms.form1[campo].value = '0';
	break;
	}
	}

Y el boton, como ya puse (no he cambiao nada):

<input type="button" name="btn01" id="btn01" onClick="cambio(btn01,cod_uno,var_uno)" value="--">

La verdad, no sé qué estoy haciendo mal... ains

[caricatos]

Hola:

Dentro de la función se supone que pones textos y no variables u objetos, así que tendrías que entrecomillarlos:

<input type="button" name="btn01" id="btn01" onClick="cambio('btn01', 'cod_uno', 'var_uno')" value="--">

Saludos

[Jomer_zgz]

Evidentemente, tenías razón. No me había percatado de ello. Los estaba interpretando igualmente como variables.

Muchas gracias.

[caricatos]

Hola:

Me alegro de que sea eso...

Saludos