SQL - Apoyo Procedimiento

   
Vista:

Apoyo Procedimiento

Publicado por michel1n0 (5 intervenciones) el 11/05/2016 01:11:23
Estimados, he aquí nuevamente acudiendo por su apoyo, resulta que he estado haciendo un procedimiento,y en la parte del cursor noto que se demora demasiado, en hacer los UPDATE, tengo que aclarar que los registros seran algo de 150k aprox., favor me puedan indicar en dónde esta mi error.

- Cabe indicar que estoy usando MS Sql 2012
- Los registros son de 150k aprox.

he aquí el código:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
alter procedure prueba_fechas as
begin
declare
 
@valor3 varchar(20),
@empresa varchar(20),
@periodo numeric(4,0),
@tipo_comprobante varchar(20),
@correlativo numeric(8,0),
@secuencia numeric(9,0)
 
 
Create Table #tmp_tabla  (  tmp_valor3 varchar(20) null,
                            tmp_nuevo varchar(20) null,
                            tmp_empresa varchar(20) null,
                            tmp_periodo numeric(4,0) null,
                            tmp_tipo_comprobante varchar(20) null,
                            tmp_correlativo numeric(8,0) null,
                            tmp_secuencia numeric(9,0) null,
                            tmp_tipo_documento varchar(20) null,
                            tmp_referencia varchar(20) null,
                            tmp_idctacte varchar(20) null
	)
 
Create Table #Tmp_doc   (   tmp_fecha varchar(10) null,
                            tmp_empresa varchar(20) null,
                            tmp_tipo_documento varchar(20) null,
                            tmp_numero varchar(20) null,
                            tmp_idctacte varchar(20) null
    )
 
insert into #tmp_tabla
SELECT  VALOR3,
        '',
        empresa,
        periodo,
        tipo_comprobante,
        correlativo,
        secuencia,
        tipo_documento,
        referencia,
        case
            when len(aux_valor2)>0 then aux_valor2
            when len(aux_valor3)>0 then aux_valor3
        else null end
 
FROM FLEXLINE.CON_MOVCOM
WHERE EMPRESA = '001'
AND PERIODO = 2016
AND ESTADO = 'A'
AND LEN(VALOR3) > 0
and (len(aux_valor2)>0 or len(aux_valor3)>0)
 
 
insert into #tmp_doc
select
convert(varchar(10),fecha,103),
empresa,
substring(tipodocto,1,20),
numero,
idctacte
from flexline.documento
where empresa = '001'
and periodolibro like '2016__'
 
 
Declare cur_llena cursor FORWARD_ONLY for
 
select
b.tmp_fecha,
a.tmp_empresa,
a.tmp_periodo,
a.tmp_tipo_comprobante,
a.tmp_correlativo,
a.tmp_secuencia
from #tmp_tabla a, #tmp_doc b
where a.tmp_empresa = b.tmp_empresa
and a.tmp_tipo_documento = b.tmp_tipo_documento
and a.tmp_referencia = b.tmp_numero
 
open cur_llena
fetch cur_llena into   @valor3,
                       @empresa,
                       @periodo,
                       @tipo_comprobante,
                       @correlativo,
                       @secuencia
 
while(@@fetch_status = 0)
    begin
        update #tmp_tabla set tmp_nuevo = @valor3
        where   tmp_empresa = @empresa
        and     tmp_periodo = @periodo
        and     tmp_tipo_comprobante = @tipo_comprobante
        and     tmp_correlativo = @correlativo
        and     tmp_secuencia = @secuencia
 
        Fetch cur_llena Into   @valor3,@empresa,@periodo,@tipo_comprobante,@correlativo,@secuencia
    end
 
Close cur_llena
Deallocate cur_llena
 
Declare cur_Report cursor FORWARD_ONLY For
 
Select
tmp_valor3,
tmp_empresa,
tmp_periodo,
tmp_tipo_comprobante,
tmp_correlativo,
tmp_secuencia
from #tmp_tabla
where len(tmp_nuevo) = 0
 
 
open cur_report
fetch from cur_report into  @valor3,@empresa,@periodo,@tipo_comprobante,@correlativo,@secuencia
 
 
WHILE (@@FETCH_STATUS = 0 )
    begin
 
    if charindex('-',@valor3) = 0 and charindex('/',@valor3) = 0 and len(@valor3) = 8
    begin
        update #tmp_tabla set tmp_nuevo = substring(@valor3,1,2)+'/'+substring(@valor3,3,2)+'/'+substring(@valor3,5,4)
        where   tmp_empresa = @empresa
        and     tmp_periodo = @periodo
        and     tmp_tipo_comprobante = @tipo_comprobante
        and     tmp_correlativo = @correlativo
        and     tmp_secuencia = @secuencia
    end
 
    if charindex('-',@valor3) = 0 and charindex('/',@valor3) = 0 and len(@valor3) = 6
    begin
        update #tmp_tabla set tmp_nuevo = substring(@valor3,1,2)+'/'+substring(@valor3,3,2)+'/20'+substring(@valor3,5,5)
        where   tmp_empresa = @empresa
        and     tmp_periodo = @periodo
        and     tmp_tipo_comprobante = @tipo_comprobante
        and     tmp_correlativo = @correlativo
        and     tmp_secuencia = @secuencia
    end
 
    if charindex('-',@valor3) > 0 and charindex('/',@valor3) = 0 and len(@valor3) = 8
    begin
        update #tmp_tabla set tmp_nuevo = substring(@valor3,1,2)+'/'+substring(@valor3,4,2)+'/20'+substring(@valor3,7,2)
        where   tmp_empresa = @empresa
        and     tmp_periodo = @periodo
        and     tmp_tipo_comprobante = @tipo_comprobante
        and     tmp_correlativo = @correlativo
        and     tmp_secuencia = @secuencia
    end
 
    if charindex('-',@valor3) > 0 and charindex('/',@valor3) = 0 and len(@valor3) = 10
    begin
        update #tmp_tabla set tmp_nuevo = substring(@valor3,1,2)+'/'+substring(@valor3,4,2)+'/'+substring(@valor3,7,4)
        where   tmp_empresa = @empresa
        and     tmp_periodo = @periodo
        and     tmp_tipo_comprobante = @tipo_comprobante
        and     tmp_correlativo = @correlativo
        and     tmp_secuencia = @secuencia
    end
 
    if charindex('-',@valor3) = 0 and charindex('/',@valor3) > 0 and len(@valor3) = 8
    begin
        update #tmp_tabla set tmp_nuevo = substring(@valor3,1,2)+'/'+substring(@valor3,4,2)+'/20'+substring(@valor3,7,2)
        where   tmp_empresa = @empresa
        and     tmp_periodo = @periodo
        and     tmp_tipo_comprobante = @tipo_comprobante
        and     tmp_correlativo = @correlativo
        and     tmp_secuencia = @secuencia
    end
 
    if charindex('.',@valor3) > 0 and len(@valor3) = 9
    begin
        update #tmp_tabla set tmp_nuevo = substring(@valor3,1,2)+'/'+substring(@valor3,3,2)+'/'+substring(@valor3,5,4)
        where   tmp_empresa = @empresa
        and     tmp_periodo = @periodo
        and     tmp_tipo_comprobante = @tipo_comprobante
        and     tmp_correlativo = @correlativo
        and     tmp_secuencia = @secuencia
    end
 
    if charindex('-',@valor3) = 0 and charindex('/',@valor3) > 0 and len(@valor3) = 10
    begin
        update #tmp_tabla set tmp_nuevo = tmp_valor3
        where   tmp_empresa = @empresa
        and     tmp_periodo = @periodo
        and     tmp_tipo_comprobante = @tipo_comprobante
        and     tmp_correlativo = @correlativo
        and     tmp_secuencia = @secuencia
    end
 
 
Fetch CUR_report Into   @valor3,@empresa,@periodo,@tipo_comprobante,@correlativo,@secuencia
 
End
Close CUR_report
Deallocate CUR_report
 
 
select * from #tmp_tabla
drop table #tmp_tabla
drop table #tmp_doc
 
end

Saludos...

Michel1n0
Valora esta pregunta
Me gusta: Está pregunta es útil y esta claraNo me gusta: Está pregunta no esta clara o no es útil
0
Responder
Imágen de perfil de Isaias

Apoyo Procedimiento

Publicado por Isaias (691 intervenciones) el 11/05/2016 19:10:32
Esta muy rebuscado tu procedimiento, por principio de cuentas, deberias utilizar JOIN al unir tus tablas

from #tmp_tabla a, #tmp_doc b
where a.tmp_empresa = b.tmp_empresa
and a.tmp_tipo_documento = b.tmp_tipo_documento
and a.tmp_referencia = b.tmp_numero

¿Que hace exactamente tu procedimiento?
Valora esta respuesta
Me gusta: Está respuesta es útil y esta claraNo me gusta: Está respuesta no esta clara o no es útil
0
Comentar

Apoyo Procedimiento

Publicado por Frank Michael (5 intervenciones) el 13/05/2016 17:03:47
Hola, que tal, lo que pasa es que los usuarios han estado ingresando valores de fechas en los formatos que les plazca, esto significa que el sistema no validaba fechas ya que el campo es un varchar() y aceptaba todo, ahora se quiere un reporte con ese campo, pero con cierto formato, lo que me veo obligado a hacer un recorrido a ese campo y comparar formatos frecuentes y volverlos al mismo formato, por mi parte no tuve mas que decir a los usuarios el formato de ingreso, ya que el sistema no se puede modificar.

Probaré con lo que me dices de los JOIN, supongo que una actualización no me vendría mal.

Saludos...

michel1n0
Valora esta respuesta
Me gusta: Está respuesta es útil y esta claraNo me gusta: Está respuesta no esta clara o no es útil
0
Comentar
Imágen de perfil de Isaias

Apoyo Procedimiento

Publicado por Isaias (691 intervenciones) el 13/05/2016 17:23:52
Creo que estas tratando de inventar el agua purificada

SQL Server, tiene un servicio DQS Data Quality Services, que te permite hacer Data Cleasing (limpieza de datos), donde tu definies tus patron de caracteres y lo puedes modificar de forma automatica.

https://msdn.microsoft.com/es-es/library/ff877925%28v=sql.120%29.aspx

https://msdn.microsoft.com/en-us/library/ff877917.aspx

Ahora dices que estas almacenando FECHAS o datos de fecha en un VARCHAR(18), esa es una pesima practica en el desarrollo de sistemas, ya que a ese campo no le puedes aplicar funciones de fecha como DATEDIFF, DATEADD, DATEPART, etc., entonces me diras, pero puedo hacerle CAST/CONVERT, si, pero sigue siendo una pesima practica.

Creo que antes de insertar tus datos en dicho campo, deberias validar que trajera un formato de fecha.

ISDATE(tucampo)

Deberia darte un 1 si es verdadero o 0, si es falso, aunque si te mandan: "12:25", seria verdadero
Valora esta respuesta
Me gusta: Está respuesta es útil y esta claraNo me gusta: Está respuesta no esta clara o no es útil
0
Comentar