Como optimizar este script para sumar horas?

[fermdp]

Mirando el FAQ http://www.forosdelweb.com/showpost....0&postcount=14 modifique el script para sumar hasta 100 valores que traigo de una base de datos, pero lo hice de una manera un poco torpe porque no se hacerlo de otra, es decir el en script cada vez que es necesario listo "TODOS LOS CAMPOS DE TEXTO" esto funciona pero me queda un script gigantezco, lo que yo quiero es sabiendo cuantos registros me tira la db, por ejemplo 36, que me sume esos 36 campos y no tener que escribir semejante pagina, imaginen que la db me de unos 300 registros para sumar !!! actualmente lo soluciono con el siguiente ejemplo, y limito la consulta a 100 resultados, funciona, pero quisiera optimizar el tema !!!

Código PHP:

  function sumaTiempos(t1, t2, t3, t4, t5, t6, t7, t8, t9, t10, t11, t12, t13, t14, t15, t16, t17, t18, t19, t20, t21, t22, t23, t24, t25, t26, t27, t28, t29, t30, t31, t32, t33, t34, t35, t36, t37, t38, t39, t40, t41, t42, t43, t44, t45, t46, t47, t48, t49, t50, t51, t52, t53, t54, t55, t56, t57, t58, t59, t60, t61, t62, t63, t64, t65, t66, t67, t68, t69, t70, t71, t72, t73, t74, t75, t76, t77, t78, t79, t80, t81, t82, t83, t84, t85, t86, t87, t88, t89, t90, t91, t92, t93, t94, t95, t96, t97, t98, t99, t100){   
    var secs1 = stringToSeconds(t1);   
    var secs2 = stringToSeconds(t2);  
    var secs3 = stringToSeconds(t3);  
    var secs4 = stringToSeconds(t4);  
    var secs5 = stringToSeconds(t5);  
    var secs6 = stringToSeconds(t6);  
    var secs7 = stringToSeconds(t7);  
    var secs8 = stringToSeconds(t8);  
    var secs9 = stringToSeconds(t9);  
    var secs10 = stringToSeconds(t10);  
    var secs11 = stringToSeconds(t11);  
    var secs12 = stringToSeconds(t12);  
    var secs13 = stringToSeconds(t13);  
    var secs14 = stringToSeconds(t14);  
    var secs15 = stringToSeconds(t15);  
    var secs16 = stringToSeconds(t16);  
    var secs17 = stringToSeconds(t17);  
    var secs18 = stringToSeconds(t18);  
    var secs19 = stringToSeconds(t19);  
    var secs20 = stringToSeconds(t20);  
    var secs21 = stringToSeconds(t21);  
    var secs22 = stringToSeconds(t22);  
    var secs23 = stringToSeconds(t23);  
    var secs24 = stringToSeconds(t24);  
    var secs25 = stringToSeconds(t25);  
    var secs26 = stringToSeconds(t26);  
    var secs27 = stringToSeconds(t27);  
    var secs28 = stringToSeconds(t28);  
    var secs29 = stringToSeconds(t29);  
    var secs30 = stringToSeconds(t30);  
    var secs31 = stringToSeconds(t31);  
    var secs32 = stringToSeconds(t32);  
    var secs33 = stringToSeconds(t33);  
    var secs34 = stringToSeconds(t34);  
    var secs35 = stringToSeconds(t35);  
    var secs36 = stringToSeconds(t36);  
    var secs37 = stringToSeconds(t37);  
    var secs38 = stringToSeconds(t38);  
    var secs39 = stringToSeconds(t39);  
    var secs40 = stringToSeconds(t40);  
    var secs41 = stringToSeconds(t41);  
    var secs42 = stringToSeconds(t42);  
    var secs43 = stringToSeconds(t43);  
    var secs44 = stringToSeconds(t44);  
    var secs45 = stringToSeconds(t45);  
    var secs46 = stringToSeconds(t46);  
    var secs47 = stringToSeconds(t47);  
    var secs48 = stringToSeconds(t48);  
    var secs49 = stringToSeconds(t49);  
    var secs50 = stringToSeconds(t50);  
    var secs51 = stringToSeconds(t51);  
    var secs52 = stringToSeconds(t52);  
    var secs53 = stringToSeconds(t53);  
    var secs54 = stringToSeconds(t54);  
    var secs55 = stringToSeconds(t55);  
    var secs56 = stringToSeconds(t56);  
    var secs57 = stringToSeconds(t57);  
    var secs58 = stringToSeconds(t58);  
    var secs59 = stringToSeconds(t59);  
    var secs60 = stringToSeconds(t60);  
    var secs61 = stringToSeconds(t61);  
    var secs62 = stringToSeconds(t62);  
    var secs63 = stringToSeconds(t63);  
    var secs64 = stringToSeconds(t64);  
    var secs65 = stringToSeconds(t65);  
    var secs66 = stringToSeconds(t66);  
    var secs67 = stringToSeconds(t67);  
    var secs68 = stringToSeconds(t68);  
    var secs69 = stringToSeconds(t69);  
    var secs70 = stringToSeconds(t70);  
    var secs71 = stringToSeconds(t71);  
    var secs72 = stringToSeconds(t72);  
    var secs73 = stringToSeconds(t73);  
    var secs74 = stringToSeconds(t74);  
    var secs75 = stringToSeconds(t75);  
    var secs76 = stringToSeconds(t76);  
    var secs77 = stringToSeconds(t77);  
    var secs78 = stringToSeconds(t78);  
    var secs79 = stringToSeconds(t79);  
    var secs80 = stringToSeconds(t80);  
    var secs81 = stringToSeconds(t81);  
    var secs82 = stringToSeconds(t82);  
    var secs83 = stringToSeconds(t83);  
    var secs84 = stringToSeconds(t84);  
    var secs85 = stringToSeconds(t85);  
    var secs86 = stringToSeconds(t86);  
    var secs87 = stringToSeconds(t87);  
    var secs88 = stringToSeconds(t88);  
    var secs89 = stringToSeconds(t89);  
    var secs90 = stringToSeconds(t90);  
    var secs91 = stringToSeconds(t91);  
    var secs92 = stringToSeconds(t92);  
    var secs93 = stringToSeconds(t93);  
    var secs94 = stringToSeconds(t94);  
    var secs95 = stringToSeconds(t95);  
    var secs96 = stringToSeconds(t96);  
    var secs97 = stringToSeconds(t97);  
    var secs98 = stringToSeconds(t98);  
    var secs99 = stringToSeconds(t99);  
    var secs100 = stringToSeconds(t100);  
    var secsSuma = secs1 + secs2 + secs3 + secs4 + secs5 + secs6 + secs7 + secs8 + secs9 + secs10 + secs11 + secs12 + secs13 + secs14 + secs15 + secs16 + secs17 + secs18 + secs19 + secs20 + secs21 + secs22 + secs23 + secs24 + secs25 + secs26 + secs27 + secs28 + secs29 + secs30 + secs31 + secs32 + secs33 + secs34 + secs35 + secs36 + secs37 + secs38 + secs39 + secs40 + secs41 + secs42 + secs43 + secs44 + secs45 + secs46 + secs47 + secs48 + secs49 + secs50 + secs51 + secs52 + secs53 + secs54 + secs55 + secs56 + secs57 + secs58 + secs59 + secs60 + secs61 + secs62 + secs63 + secs64 + secs65 + secs66 + secs67 + secs68 + secs69 + secs70 + secs71 + secs72 + secs73 + secs74 + secs75 + secs76 + secs77 + secs78 + secs79 + secs80 + secs81 + secs82 + secs83 + secs84 + secs85 + secs86 + secs87 + secs88 + secs89 + secs90 + secs91 + secs92 + secs93 + secs94 + secs95 + secs96 + secs97 + secs98 + secs99 + secs100;   
    return secondsToTime(secsSuma);   
   }   
 
function calcT3(){   
with (document.frm)   
t.value = sumaTiempos(t1.value, t2.value, t3.value, t4.value, t5.value, t6.value, t7.value, t8.value, t9.value, t10.value, t11.value, t12.value, t13.value, t14.value, t15.value, t16.value, t17.value, t18.value, t19.value, t20.value, t21.value, t22.value, t23.value, t24.value, t25.value, t26.value, t27.value, t28.value, t29.value, t30.value, t31.value, t32.value, t33.value, t34.value, t35.value, t36.value, t37.value, t38.value, t39.value, t40.value, t41.value, t42.value, t43.value, t44.value, t45.value, t46.value, t47.value, t48.value, t49.value, t50.value, t51.value, t52.value, t53.value, t54.value, t55.value, t56.value, t57.value, t58.value, t59.value, t60.value, t61.value, t62.value, t63.value, t64.value, t65.value, t66.value, t67.value, t68.value, t69.value, t70.value, t71.value, t72.value, t73.value, t74.value, t75.value, t76.value, t77.value, t78.value, t79.value, t80.value, t81.value, t82.value, t83.value, t84.value, t85.value, t86.value, t87.value, t88.value, t89.value, t90.value, t91.value, t92.value, t93.value, t94.value, t95.value, t96.value, t97.value, t98.value, t99.value, t100.value);   
}  
   

Gracias al usuario derkenuke modifiqué parte del código de la primer función por esto

Código PHP:

  function sumaTiempos() { 
for(var i=0, dev=0; i<sumaTiempos.arguments.length; i++) 
dev+=stringToSeconds(sumaTiempos.arguments[i]); 
return dev; 
}  
   

Ahorrando muchas líneas de código y sin importar la cantidad de argumentos pasados,lo que me faltaría es optimizar la función calcT3 para no tener que poner por ejemplo los 100 valores de los campos (t1.value, t2.value, t3.value etc) se entiende, buscar alguna manera que diga función sumartiempo (sumame todos los campos del form)

Desde ya les estoy muy agradecido!!!

[fermdp]

Alguien tendra algun ejemplo o explicacion?
Desde ya muchas gracias!

[HackmanC]

Hola,

Posiblemente:

Código javascript:

...
function sumaTiempos() {
    var dev=0;
    for(var i=0; i<document.Valores.elements.length; i++)
    if (document.Valores.elements[i].type == "text")
        dev+=stringToSeconds(document.Valores.elements[i].value);
    return dev;
}
//-->
</script>
</head>
<body>
<form id="Valores" name="Valores" method="post" action="">
    <input type="text" name="Valor[]" />
    <input type="text" name="Valor[]" />
    <input type="text" name="Valor[]" />
...
 

Saludos,

[buzu]

Creo que es una buena sugerencia hackman. Sin embargo, considerando la cantidad de elemenentos que pueden ser manejado, creo que el for quedaría mejor así:

for(var i=0, var l=document.Valores.elements.length; i< l; i++)

De sa forma logras optimizar la función.
También he oido que document.Valores puede dar problemas. Una forma que es más segura es document.forms.Valores Pero en realidad de esto no estoy completamente seguro.

[HackmanC]

Hola,

Seguramente que sí.

En NN6 se usaba de esta forma : document.forms['Valores']

Muchos saludos !

ps:

Habría que investigar cual es la más compatible entre todos los browsers, existen muchas formas de llegar a los elementos del formulario.

[fermdp]

Muchísimas gracias! la parte de la función sumaTiempos la entendí y me parece facil de implementar, pero me faltaría ver como optimizar o simplificar la última funcion calcT3

Código PHP:

  function calcT3(){    
with (document.frm)    
t.value = sumaTiempos(t1.value, t2.value, t3.value, t4.value, t5.value, t6.value, t7.value, t8.value, t9.value, t10.value, t11.value, t12.value, t13.value, t14.value, t15.value, t16.value, t17.value, t18.value, t19.value, t20.value, t21.value, t22.value, t23.value, t24.value, t25.value, t26.value, t27.value, t28.value, t29.value, t30.value, t31.value, t32.value, t33.value, t34.value, t35.value, t36.value, t37.value, t38.value, t39.value, t40.value, t41.value, t42.value, t43.value, t44.value, t45.value, t46.value, t47.value, t48.value, t49.value, t50.value, t51.value, t52.value, t53.value, t54.value, t55.value, t56.value, t57.value, t58.value, t59.value, t60.value, t61.value, t62.value, t63.value, t64.value, t65.value, t66.value, t67.value, t68.value, t69.value, t70.value, t71.value, t72.value, t73.value, t74.value, t75.value, t76.value, t77.value, t78.value, t79.value, t80.value, t81.value, t82.value, t83.value, t84.value, t85.value, t86.value, t87.value, t88.value, t89.value, t90.value, t91.value, t92.value, t93.value, t94.value, t95.value, t96.value, t97.value, t98.value, t99.value, t100.value);    
}  
   

Si alguién me puede orientar les estaré agradecido!!!

[HackmanC]

Hola,

Iniciado por fermdp

Muchísimas gracias! la parte de la función sumaTiempos la entendí y me parece facil de implementar, ...

Iniciado por fermdp

... pero me faltaría ver como optimizar o simplificar la última funcion calcT3

La función calcT3 ya no la necesitas, porque la reemplaza completamente la función sumaTiempos



Saludos,

[HackmanC]

Hola,

Bueno, talves no la reemplaza completamente,
Pero no tengo ni idea que es : t.

Código PHP:

  function calcT3(){   
t.value = secondsToTime(sumaTiempos());
}  
   

Saludos,

[HackmanC]

Hola,

Posiblemente:

Código javascript:

<script type="application/javascript">
<!--
function stringToSeconds(s) {
    return new Number(s);
}
function secondsToTime(s) {
    return s;
}
function sumaTiempos() {
    var dev=0;
    for(var i = 0; i < document.forms.Valores.elements.length; i++)
    if (document.Valores.elements[i].type == "text" &&
            document.Valores.elements[i].name == "Valor[]")
        dev += stringToSeconds(document.forms.Valores.elements[i].value);
    return secondsToTime(dev);
}
function calcT3() {   
    document.getElementById("Resultado").value = sumaTiempos();
    return false;
}
//-->
</script>
</head>
<body>
<form id="Valores" name="Valores" method="post" action="" onsubmit="return calcT3();">
    <input type="text" name="Valor[]" />
    <input type="text" name="Valor[]" />
    <input type="text" name="Valor[]" />
    <input type="submit" name="Boton" id="Boton" value="Submit" />
    <input type="text" name="Resultado" id="Resultado" disabled="disabled" />
</form>
 

Saludos.

[neiker]

Si los tiempos estan en la base de datos, porque no pedis en la consulta SQL que los devuelva sumados? (Con SUM)

Fijate estas 2 webs:
techonthenet.com/sql/sum.php
sql-tutorial.net/SQL-SUM.asp


Saludos!

[buzu]

Eso... por que dejar que JS haga un trabajo que MySQL (supongo que es lo que estás usnado) Puede hacer sin problemas. Que es lo que quieres lograr?

[fermdp]

Muchísimas gracias a todos. Al final logre un mi cometido usando parte de las respuestas y soluciones dadas por ustedes, dado que la opción de realizar la suma con mysql no me parecía adecuada por ser muy compleja la consulta ue realizo y completa también la página y no quiero sumarle tarea a mysql para obtener el mejor tiempo posible en la respuesta, dado que muchas veces el resultado de la consulta es una cantidad impotante de datos y no quiero tener un cuello de botella en alguno de los procesos, me parece que esta opción que les muestro a modo de ejemplo cumple bien su cometido, este es el resultado:

Código javascript:

<html>
<head>
  <script language="JavaScript">
<!--
function padNmb(nStr, nLen){
    var sRes = String(nStr);
    var sCeros = "0000000000";
    return sCeros.substr(0, nLen - sRes.length) + sRes;
   }
 
   function stringToSeconds(tiempo){
    var sep1 = tiempo.indexOf(":");
    var sep2 = tiempo.lastIndexOf(":");
    var hor = tiempo.substr(0, sep1);
    var minu = tiempo.substr(sep1 + 1, sep2 - sep1 - 1);
    var sec = tiempo.substr(sep2 + 1);
    return (Number(sec) + (Number(minu) * 60) + (Number(hor) * 3600));
   }
 
   function secondsToTime(secs){
    var hor = Math.floor(secs / 3600);
    var minu = Math.floor((secs - (hor * 3600)) / 60);
    var sec = secs - (hor * 3600) - (minu * 60);
    return padNmb(hor, 2) + ":" + padNmb(minu, 2) + ":" + padNmb(sec, 2);
   }
 
 function sumaTiempos() {
    var dev=0;
    for(var i = 0; i < document.forms.formulario_horas.elements.length; i++)
    if (document.forms.formulario_horas.elements[i].type == "text" &&
            document.forms.formulario_horas.elements[i].name == "tiempo")
        dev += stringToSeconds(document.forms.formulario_horas.elements[i].value);
    return secondsToTime(dev);
}
function calcular() {   
    document.getElementById("resultado").value = sumaTiempos();
    return false;
}
 
//-->
</script>
</head>
 <body onLoad="calcular()">
  <form name="formulario_horas">
   Hora1 (hh:mm:ss): <input type="text" name="tiempo" value="12:30:15"><br>
   Hora2 (hh:mm:ss): <input type="text" name="tiempo" value="3:41:18"><br>
   Hora3 (hh:mm:ss): <input type="text" name="tiempo" value="3:42:18"><br>
   Hora4 (hh:mm:ss): <input type="text" name="tiempo" value="3:49:18"><br>
   Hora5 (hh:mm:ss): <input type="text" name="tiempo" value="1:44:18"><br>
   Hora6 (hh:mm:ss): <input type="text" name="tiempo" value="00:00:1"><br>
   Hora7 (hh:mm:ss): <input type="text" name="tiempo" value="00:00:00"><br>
   Hora8 (hh:mm:ss): <input type="text" name="tiempo" value="00:00:14"><br>
   <hr>
   Suma (hh:mm:ss): <input type="text" name="resultado" id="resultado" value="">
  </form>
</body>
</html>
 

Nuevamente muchisimas gracias a todos!