Totalmente de acuerdo. Es casi imposible detectar nada en ese código. Aún sin llegar a lo que comenta Agustín (y hay que llegar a ello), hay que tratar de ser más prolijos. Por ejemplo, ¿para qué son todas esas cadenas vacías que concatenas en la últimas líneas? ¿Para qué conviertes 18 en cadena pudiendo colocarlo directamente en el resto de la cadena? ¿Por qué introduces valores numéricos como cadenas teniendo que hacer una doble conversión?
Además, armar las consultas SQL línea a línea y sobre la marcha es algo que debería estar "prohibido". Código confuso y de difícil mantenimiento garantizado.
LineComment Saludos
|