acortar script

[arcanisgk122]

Código Javascript:

function calcula(dat1, dat2){
            var ope = dat1.options[dat1.selectedIndex].value
            document.form1.dat2.value = ope;
            document.form1.p+dat2.disabled=this.value!='';
            }
 

y la llamada la hago:

Código HTML:

onchange="calcula(this.options.selectedIndex,'des')"
 

no me funciona alguna idea???

[linuxzero]

Probá en el código javascript usar dat1 en vez de dat1.selectedIndex cuando seteas la variable ope.

[arcanisgk122]

bueno si he logrado hacerlo funcionar pero tengo otro inconveniente es que el script es repetitivo y quería ver si se puede acortar a una sola función:

Código Javascript:

function calcula1(data){
        var pfrs= data.options[data.selectedIndex].value;
        document.form1.frs.value = pfrs;
        document.form1.pfrs.disabled=this.value!='';
    }
    function calcula2(data){
        var pdes= data.options[data.selectedIndex].value;
        document.form1.des.value = pdes;
        document.form1.pdes.disabled=this.value!='';
    }
    function calcula3(data){
        var psab= data.options[data.selectedIndex].value;
        document.form1.sab.value = psab;
        document.form1.psab.disabled=this.value!='';
    }
    var global= new Array(0, 0, 0, 0, 0);
    function statpoint(elemSelect, indiceArray){
        var pointset= parseInt(elemSelect.options[elemSelect.selectedIndex].text);
        var datpast= global[indiceArray];
        var atrset= document.getElementById("atr");
        var atrpoint= parseInt(atrset.value);
        atrpoint=atrpoint-datpast;
        atrset.value=atrpoint+pointset;
        global[indiceArray]=pointset;
    }
 

Código HTML:

<select title="Selecciona los puntos de fuerza - ALERTA (ESTA OPCION SE BLOQUEARA)" id="pfrs" name="pfrs" size="1" onchange="calcula1(this);statpoint(this, 0)">
    <option value="10" selected="selected">0</option>
    <option value="11">10</option>
    <option value="12">20</option>
    <option value="13">30</option>
    <option value="14">45</option>
    <option value="15">60</option>
    <option value="16">80</option>
    <option value="17">100</option>
    <option value="18">125</option>
    <option value="19">150</option>
    <option value="20">200</option>
</select>
<input type="text" name="frs" id="frs" value="10" maxlength="3" size="4" readonly="readonly"/>
<select title="Selecciona los puntos de destreza - ALERTA (ESTA OPCION SE BLOQUEARA)" id="pdes" name="pdes" size="1" onchange="calcula2(this);statpoint(this, 1)">
    <option value="10" selected="selected">0</option>
    <option value="11">10</option>
    <option value="12">20</option>
    <option value="13">30</option>
    <option value="14">45</option>
    <option value="15">60</option>
    <option value="16">80</option>
    <option value="17">100</option>
    <option value="18">125</option>
    <option value="19">150</option>
    <option value="20">200</option>
</select>
<input type="text" name="des" id="des" value="10" maxlength="3" size="4" readonly="readonly"/>
<select title="Selecciona los puntos de saber - ALERTA (ESTA OPCION SE BLOQUEARA)" id="psab" name="psab" size="1" onchange="calcula3(this);statpoint(this, 2)">
<option value="10" selected="selected">0</option>
    <option value="11">10</option>
    <option value="12">20</option>
    <option value="13">30</option>
    <option value="14">45</option>
    <option value="15">60</option>
    <option value="16">80</option>
    <option value="17">100</option>
    <option value="18">125</option>
    <option value="19">150</option>
    <option value="20">200</option>
</select>
<input type="text" name="sab" id="sab" value="10" maxlength="3" size="4" readonly="readonly"/>
<input name="atr" id="atr" value="0" readonly="readonly" />
 

[arcanisgk122]

he logrado convertir el script en una sola llamada a funcion pero no logro simplificarlo creo que 1 llamada a funcion para cada elemento es mucho.

Código Javascript:

var global= new Array(0, 0, 0, 0, 0);
        
        function calcula1(data, elemSelect, indiceArray){
            var pfrs= data.options[data.selectedIndex].value;
            document.form1.frs.value = pfrs;
            var pointset= parseInt(elemSelect.options[elemSelect.selectedIndex].text);
            var datpast= global[indiceArray];
            var atrset= document.getElementById("atr");
            var atrpoint= parseInt(atrset.value);
            atrpoint=atrpoint-datpast;
            atrset.value=atrpoint+pointset;
            global[indiceArray]=pointset;
            document.form1.pdes.disabled=this.value!='';
            }
        function calcula2(data, elemSelect, indiceArray){
            var pdes= data.options[data.selectedIndex].value;
            document.form1.des.value = pdes;
            var pointset= parseInt(elemSelect.options[elemSelect.selectedIndex].text);
            var datpast= global[indiceArray];
            var atrset= document.getElementById("atr");
            var atrpoint= parseInt(atrset.value);
            atrpoint=atrpoint-datpast;
            atrset.value=atrpoint+pointset;
            global[indiceArray]=pointset;
            document.form1.pdes.disabled=this.value!='';
            }
        function calcula3(data, elemSelect, indiceArray){
            var psab= data.options[data.selectedIndex].value;
            document.form1.sab.value = psab;
            var pointset= parseInt(elemSelect.options[elemSelect.selectedIndex].text);
            var datpast= global[indiceArray];
            var atrset= document.getElementById("atr");
            var atrpoint= parseInt(atrset.value);
            atrpoint=atrpoint-datpast;
            atrset.value=atrpoint+pointset;
            global[indiceArray]=pointset;
            document.form1.psab.disabled=this.value!='';
            }
 

alguna idea?

[arcanisgk122]

solo me hace falta que alguien me diga como paso esto:

Código Javascript:

document.form1.frs.value = pfrs;
 

donde "frs" deve ser dinamico ya que los id cambian y lo manejo de esa forma
asi podria trabajar con

Código Javascript:

document.form1.'variable'.value = pfrs;
 

alguna idea??

[linuxzero]

No lo manejes segun el id del campo, sino por el class del campo, es la única forma de trabajar en forma dinámica, despues te fijarás la lógica a aplicar.

Saludos!

[arcanisgk122]

Iniciado por linuxzero

No lo manejes segun el id del campo, sino por el class del campo, es la única forma de trabajar en forma dinámica, despues te fijarás la lógica a aplicar.

Saludos!

algún ejemplo sencillo que pueda adaptar???

[linuxzero]

En js puro no estoy muy seguro, en jquery seria algo asi:

Como no manejas segun un id sino por class, tenes que recorrer todos los inputs que tengan esa clase y obtener el value, para eso lo haces asi en jquery:

Código Javascript:

$('.classInput').each(function(){
    alert($(this).val());
});
 

Este es un ejemplo de como accedes a cada input segun una clase, y como la clase la podes repetir no habria problema en cuantos tengas, total los recorre todos.

Esto es como para que tengas una idea de como trabaja, si necesitas js puro, lo buscamos ;)

[arcanisgk122]

ok el problema es que no puedo recorrerlo todos, ya que necesito que solo se active en el elemento usado y no en todos ademas no son input son select option.
creo que es mas complicado cambiar todo el script funcional a class que solo cambiar esto

Código Javascript:

document.form1.'variable'.value = pfrs;
document.form1.[p+'variable'].disabled=this.value!='';
 

actualmente varieble contendria la cadena frs, entonces el elemento al que apunta deria el que tiene la id="frs" asi lo veo yo, mas sin envargo en la misma funcion debe modificar el atributo de el select option que tiene id="pfrs" por eso utiliso [p+'variable'] p+frs = pfrs

el cual no se como es la sintaxis correcta para que funcione.

la funcion la estoy llamando de esta forma:

Código Javascript:

onchange="calcula1(this,this, 0,'frs')