Optimización de función.

[erponcios]

Hola gente,

Una preguntilla,

¿Habría alguna forma de optimiziar/acortar la siguiente función?

Código PHP:

  function checkthis()
{
    
     var oCheckBox = document.getElementById('checkpromo'); 
 
     if (oCheckBox.checked == true) {
 
          document.getElementById('nuevapromo').style.visibility="visible";
          document.getElementById('promoexist').style.visibility="hidden";
    
     }else{
            
          document.getElementById('nuevapromo').style.visibility="hidden";
          document.getElementById('promoexist').style.visibility="visible";
        
     }
 
} 
   

Muchas gracias.

[marcopoloaz06]

Ps la acortarias asi pero podrías pasar algun parámetro que la acorte más, depende.

Código PHP:

  <script type='text/javascript'>
function checkthis(){
var oCheckBox=document.getElementById('checkpromo'); 
 var np=document.getElementById('nuevapromo').style.visibility;
  var pe=document.getElementById('promoexist').style.visibility;
 if(oCheckBox.checked==true){
    np="visible";  pe="hidden";
 }else{
     np="hidden";  pe="visible";
 }
}
</script> 
  

Zaludoz~

[caricatos]

Hola:

Tal vez pueda ponerse en 2 líneas:

document.getElementById('nuevapromo').style.visibi lity=(document.getElementById('checkpromo').checke d)?"visible":"hidden";
document.getElementById('promoexist').style.visibi lity=(document.getElementById('checkpromo').checke d)?"hidden":"visible";

Aunque las líneas pueden ser más pequeñas con un alias:
function tag(id) {return document.getElementById(id);}

Ahora sería:

tag('nuevapromo').style.visibility=(tag('checkprom o').checked)?"visible":"hidden";
tag('promoexist').style.visibility=(tag('checkprom o').checked)?"hidden":"visible";

También se podría usar una variable con el resultado del "chequeo)
var ok = tag('checkpromo').checked;

Ahora las 2 líneas serían:

tag('nuevapromo').style.visibility=(ok)?"visible": "hidden";
tag('promoexist').style.visibility=(ok)?"hidden":" visible";

También podría hacerse una asignación directa si tenemos un array de visibilidades:
var v = new Array(); v[false] = "hidden"; v[true] = "visible";

Con esto quedaría:

tag('nuevapromo').style.visibility=v[ok];
tag('promoexist').style.visibility=v[!ok];

Saludos

[erponcios]

Hola caricatos,

He usado esto y me va, el problema es que cuando lo desmarco no me vuelve a mostrar oriniexist.

Muchisimas gracias caricatos y marcopolaz.

Funciona correctamente.

Una última cosa, esta función también se podría optimizar?

Código PHP:

  function procesa(ac,valor){ 
 
    if (tag(ac).value.length >= 5) {
        
        tag(valor).disabled = false;
        
    } else {
        
        tag(valor).disabled = true;
        
    }
 
} 
   

Un saludo y muchisimas gracias.

[Kelpie]

Código PHP:

  function procesa(ac,valor){tag(valor).disabled=(tag(ac).value.length >= 5)?false:true;} 
   

[erponcios]

muchisimas gracias Kelipe.

[caricatos]

Hola:

Siempre que haya una asignación del tipo: (valor lógico) ? true:false;
... puede omitirse... así que en el caso de Kelpie (¡Hola !) solo habría que invertir la condición...

function procesa(ac,valor){tag(valor).disabled = (tag(ac).value.length < 5)}

Fíjate que se asigna una comparación equivalente a los valores lógicos true/false

Saludos

[Kelpie]

Esa es muy buena Caricatos, no la sabía.