¿Esto es demasiado codigo para algo tan simple?

[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