Ver Mensaje Individual
  #1 (permalink)  
Antiguo 13/07/2010, 09:50
th3r0rn
 
Fecha de Ingreso: noviembre-2007
Mensajes: 504
Antigüedad: 17 años, 4 meses
Puntos: 2
Que opinan de mi codigo y como mejorarlo

Hola, m gustaria le hecharan un vistaso a mi codigo y me dijeran que puedo mejorar de el ya que yo lo siento muy "sucio".
Explico:
Este codigo lo que haxce es ejecutar un procedure que busca datos por tiempo real, es decir la hora actual, pero si no se encuentran datos en esa hora lo que hace es cerrar la conexion con la DB y abrir otra para consultar datos por la hora anterior actual, y ara esto siempre y cuando no encuentre datos en la hora actual:
Código ASP:
Ver original
  1. '########################################## L15 SUR #######################################################################################
  2.     linea15s="L15s"
  3.         SET conexion15s=Server.CreateObject("ADODB.Connection")
  4.     conexion15s.Open "PROVIDER=SQLOLEDB; DATA SOURCE=VALP-MESSRV02;UID=sa;PWD=messrv;DATABASE=wasp "
  5.             SET registros15s=Server.CreateObject("ADODB.RecordSet")
  6.          registros15s.open"EXEC monitor '"&fecha&"','"&linea15s&"','"&horamia&"'",conexion15s
  7.        
  8.          SET conexionAVG15s=Server.CreateObject("ADODB.Connection")
  9.     conexionAVG15s.Open "PROVIDER=SQLOLEDB; DATA SOURCE=VALP-MESSRV02;UID=sa;PWD=messrv;DATABASE=wasp "
  10.             SET registrosAVG15s= Server.CreateObject("ADODB.RecordSet")
  11.        
  12.         registrosAVG15s.open"SELECT AVG(PESO)as promedio,o.linea from Optimas as o INNER JOIN pesos as p on o.codigo =p.codoptima WHERE fecha >='"&fecha&"' AND hora='"&horamia&"' AND o.linea='L15S' GROUP BY LINEA",conexionAVG15s
  13.    
  14.         do while not registrosAVG15s.eof
  15.         promedio=cint(registrosAVG15s.fields("promedio"))
  16.                
  17.     if registros15s.eof then
  18.     conexion15s.close
  19.     SET conexion15s=Server.CreateObject("ADODB.Connection")
  20.     conexion15s.Open "PROVIDER=SQLOLEDB; DATA SOURCE=VALP-MESSRV02;UID=sa;PWD=messrv;DATABASE=wasp "
  21.             SET registros15s=Server.CreateObject("ADODB.RecordSet")
  22.     registros15s.open"EXEC monitor '"&fecha&"','"&linea15s&"','"&hora2&"'",conexion15s
  23.         response.write("EXEC monitor '"&fecha&"','"&linea15s&"','"&hora2&"'")
  24.         do while not registros15s.eof
  25.        
  26.                     target=cint(registros15s.fields("target"))'## Definicion de variable Target
  27.                     OP=(promedio-target)/target  '### Definicion de operacion de la variable OP para ser mostrada en la tabla
  28.                     saving=(PROMEDIO-target)*NumBolsas '### Definicion y operacion de la variable SAVING para mostrar en la tabla
  29.         response.write("<tr>")
  30.         response.write("<td>"&registros15s.fields("linea")&"</td>")
  31.         response.write("<td>"&registros15s.fields("peso")&"</td>")
  32.         response.write("<td>"&registros15s.fields("target")&"</td>")
  33.         response.write("<td>"&registros15s.fields("marca")&"</td>")
  34.         response.write("<td>"&registros15s.fields("vaso")&"</td>")
  35.         'response.write("<td>"&registros6n.fields("fecha")&"</td>")
  36.         response.write("<td>"&fechahoy&" "&hora&":"&minuto&"</td>")
  37.         response.write("<td><h3>"&registros15s.fields("totality")&"</h3></td>")
  38.         response.write("<td>"&registrosAVG15s.fields("promedio")&"</td>")
  39.         if OP<=0 then response.write("<td>"&op&"<img src='http://www.pact-oil.co.uk/images/error.gif'</a></td>") else response.write("<td>"&op&"<img src='http://www.hospitalsierrallana.com/imagenes/gif_ok.gif'</a></td>") end if
  40.             if saving <0 then
  41.         response.write("<td bgcolor= '#FF0000'>  "&saving&" KG</td>")  
  42.         elseif saving >0 then  
  43.         response.write("<td bgcolor='#00FF00'>"&saving&" KG</td>")
  44.         elseif saving=0 then
  45.         response.write("<td>"&saving&" KG</td>")
  46.         end if
  47.         response.write("</tr>")
  48. registros15s.movenext
  49.     loop
  50.        
  51.         conexion15s.close
  52.     else
  53.    
  54.         do while not registros15s.eof
  55.        
  56.                     target=cint(registros15s.fields("target"))'## Definicion de variable Target
  57.                     OP=(promedio-target)/target  '### Definicion de operacion de la variable OP para ser mostrada en la tabla
  58.                     saving=(PROMEDIO-target)*NumBolsas '### Definicion y operacion de la variable SAVING para mostrar en la tabla
  59.         response.write("<tr>")
  60.         response.write("<td>"&registros15s.fields("linea")&"</td>")
  61.         response.write("<td>"&registros15s.fields("peso")&"</td>")
  62.         response.write("<td>"&registros15s.fields("target")&"</td>")
  63.         response.write("<td>"&registros15s.fields("marca")&"</td>")
  64.         response.write("<td>"&registros15s.fields("vaso")&"</td>")
  65.         'response.write("<td>"&registros6n.fields("fecha")&"</td>")
  66.         response.write("<td>"&fechahoy&" "&hora&":"&minuto&"</td>")
  67.         response.write("<td><h3>"&registros15s.fields("totality")&"</h3></td>")
  68.         response.write("<td>"&registrosAVG15s.fields("promedio")&"</td>")
  69.         if OP<=0 then response.write("<td>"&op&"<img src='http://www.pact-oil.co.uk/images/error.gif'</a></td>") else response.write("<td>"&op&"<img src='http://www.hospitalsierrallana.com/imagenes/gif_ok.gif'</a></td>") end if
  70.             if saving <0 then
  71.         response.write("<td bgcolor= '#FF0000'>  "&saving&" KG</td>")  
  72.         elseif saving >0 then  
  73.         response.write("<td bgcolor='#00FF00'>"&saving&" KG</td>")
  74.         elseif saving=0 then
  75.         response.write("<td>"&saving&" KG</td>")
  76.         end if
  77.         response.write("</tr>")
  78. registros15s.movenext
  79.     loop
  80.        
  81.         conexion15s.close
  82.        
  83. end if
  84. registrosAVG15s.movenext
  85.     loop
  86.        
  87.         conexionAVG15s.close
Saludos