¿Esto es demasiado codigo para algo tan simple?

[astut]

Hola, pues con este codigo cambio el valor, estilo y la propiedad disabled de algunos campos... lo que me gustaria saber es si uso demasiadas lineas pero esque no se hacerlo de otra forma, bueno aqui os dejo el codigo:

Código:

	function CfgCampos(grupo)
	{
		document.formulario_principal.bloqueo.disabled= false;
		document.formulario_principal.bloqueo.className= "";
		document.formulario_principal.tipo.disabled= false;
	  document.formulario_principal.tipo.value= "escudo";
		document.formulario_principal.ataque.disabled= false;
		document.formulario_principal.ataque.className= "";
		document.formulario_principal.defensa_fisica.disabled= false;
		document.formulario_principal.defensa_fisica.className= "";
		document.formulario_principal.defensa_magica.disabled= false;
		document.formulario_principal.defensa_magica.className= "";
		//Tomable
		if (grupo== 2)
		{
    	document.formulario_principal.tipo.value= "ninguno"; 
			document.formulario_principal.tipo.disabled= true; 
			document.formulario_principal.bloqueo.value= ""; document.formulario_principal.bloqueo.disabled= true;
			document.formulario_principal.bloqueo.className= "bloqueado";
			document.formulario_principal.ataque.value= ""; document.formulario_principal.ataque.disabled= true;
			document.formulario_principal.ataque.className= "bloqueado";
			document.formulario_principal.defensa_fisica.value= ""; document.formulario_principal.defensa_fisica.disabled= true;
			document.formulario_principal.defensa_fisica.className= "bloqueado";
			document.formulario_principal.defensa_magica.value= ""; document.formulario_principal.defensa_magica.disabled= true;
			document.formulario_principal.defensa_magica.className= "bloqueado";
		}
	}
</script>

¿Se podria reducir? xD thx

[caricatos]

Hola:

Se puede mejorar algo


Yo pondría algo así:

Código:

function CfgCampos(grupo)	{
	with	(document.formulario_principal)	{				bloqueo.disabled = (grupo == 2);
		bloqueo.className = (grupo == 1) ? "bloqueado" : "";
		tipo.disabled = (grupo == 2);
		tipo.value = (grupo == 2) ? "ninguno" : "escudo";
			...
	}
}

Saludos

[astut]

si pero eso seria para todos los campos del formulario =S, yo solo quiero modificar unoscuantos, habria que repetir eso poniendo los nombres de los campos no?= es decir nada no se puede xD

[derkenuke]

Pues no se me ha ocurrido nada, salvo que es menos óptimo hacer
document.formulario.campo.propiedad...;
mil veces; que hacer
variable = document.formulario.campo
variable.propiedad...; mil veces

De lo de simplificar el código.. mi respuesta habría sido la misma que la de caricatos ... es más simple hacer eso que casi cualquier otra cosa.

Hombre, si quieres puedes hacerte una función que sea
cambiarProp(elemento, propiedad, valorNuevo)
Pero yo creo que no te merece la pena, y quedará menos legible, menos óptimo, y más recargado.



Un saludo.

[caricatos]

Iniciado por astut

si pero eso seria para todos los campos del formulario =S, ...

¡Noooo! Solo debes modificar los que quieres... si se tratase de todos los campos sería tan fácil como hacer un bucle.

La idea es en vez de referenciar toda la jerarahía del objeto, posicionarse en la más baja jerarquía común: with(document.formulario_principal), y por cada elemento que haya que darle un valor condicional ponerlo con ese formato... y los valores lógicos asignarlo directamente con el condicional: [elemento].disabled = (grupo == 2)...

La idea de derkenuke es una alternativa, pero la instrucción with me parece más adecuada (lamentablemente es una estructura muy poco usada)... ;pensando:

Saludos