Mi consulta me da registros repetidos:

[th3r0rn]

Hola, tengo el siguiente procedimiento:

Código SQL:

ALTER PROCEDURE [dbo].[report]
@finicio datetime,
@ffin datetime,
@linea nvarchar(5),
@tubo nvarchar(5),
@hora nvarchar(15) AS
BEGIN
    SET Nocount ON
SELECT o.Linea,o.Tubo,h.Marca,h.Target,p.Peso,p.Vaso,O.Codigo,p.Hora,p.Fecha
    INTO #checho
    FROM pesos AS p, optimas AS o,htargets AS h
    WHERE o.codigo = p.codOptima AND o.linea = @linea 
    AND o.tubo = @tubo AND p.fecha  >=@finicio AND p.fecha<@ffin
    AND h.fecha >=@finicio AND h.fecha<@ffin
    AND p.hora= @hora AND h.codoptima = o.codigo AND h.hora= @hora
    SELECT *,(SELECT COUNT(1)total FROM #checho) AS Totality FROM #checho ORDER BY fecha
 

El cual lo ejecuto de esta manera:

Código SQL:

EXEC report '20110216','20110217','L17S','D','0:00 - 0:30'
 

Pero me da MUCHOS datos repetidos, entonces el numero total de registros que me retorna es muy grande.
Entonces cree una consulta con casi las mismas condiciones para probar si la aplicacion esta guardando los datos repetidos pero no es asi, puesto la consulta es esta:

Código SQL:

SELECT * FROM PESOS WHERE codoptima=97 
AND fecha >='20110216' 
AND fecha <'20110217' AND hora='0:00 - 0:30'
ORDER BY fecha
 

Y me retorna un numero diferente de registros y los registros no estan repetidos:
Dejo captura de pantalla:
http://ompldr.org/vN2xtdw

[leonardo_josue]

Hola th3r0rn:

Seguramente te falta definir alguna relación entre tus tablas, de tal manera que se está realizando un producto cartesiano. De hecho el la imagen que pones la columna VASO tiene valores diferentes, creo que ahí está el problema. haz un select * para que revises sobre qué tabla se está realizando el producto.

Y otra cosa. muchas veces hemos insistido en la utilización de la cláusula JOIN (INNER, LEFT, RIGHT según sea el caso) en lugar de hacer las uniones en el WHERE... haz la prueba y verás una enorme diferencia en el performance de la consulta:

Código:

--Utilizando JOIN

select * from Tabla1 INNER JOIN tabla2 on Tabla1.id = Tabla2.id

--Sin utilizar JOIN

select * from Tabla1, Tabla2 where Tabla1.id = Tabla2.id

Saludos
Leo

[th3r0rn]

Hola Leo,
No entiendo a que te refieras a hacer un selec * para revisar sobre que tabla se esta realizando el producto. Pdrias darme mas detalles de esto? puesto que no entiendo a que te refieras.
Gracias y saludos

[leonardo_josue]

Hola th3r0rn:

En tu select estás tomando sólo algunos de los campos de tus tablas, pero me imagino que además de estos tienes algunos más. Al no mostrar todos los campos que intervienen en tu tabla puede resultar complicado determinar si te falta declara alguna relación entre tus tablas. Te pongo un ejemplo para ver si se entiende:

Código SQL:

DECLARE @Tabla1 TABLE (id1 INT, id2 INT, descripcion VARCHAR(50))
INSERT INTO @Tabla1 VALUES(1, 1, 'uno')
INSERT INTO @Tabla1 VALUES(2, 2, 'dos')
INSERT INTO @Tabla1 VALUES(3, 3, 'tres')
 
DECLARE @Tabla2 TABLE (id1 INT, id2 INT, descripcion VARCHAR(50))
INSERT INTO @Tabla2 VALUES(1, 1, 'UNO-UNO')
INSERT INTO @Tabla2 VALUES(1, 2, 'UNO-DOS')
INSERT INTO @Tabla2 VALUES(1, 3, 'UNO-TRES')
INSERT INTO @Tabla2 VALUES(2, 1, 'DOS-UNO')
INSERT INTO @Tabla2 VALUES(2, 2, 'DOS-DOS')
INSERT INTO @Tabla2 VALUES(2, 3, 'DOS-TRES')
INSERT INTO @Tabla2 VALUES(3, 1, 'TRES-UNO')
INSERT INTO @Tabla2 VALUES(3, 3, 'TRES-TRES')
 
SELECT T1.id1, T1.id2, T1.descripcion FROM @Tabla1 T1 
INNER JOIN @Tabla2 T2 ON T1.id1 = T2.id1
 

Si ejecutas este script te regresa lo siguiente:

Código:

id1|id|descripcion|descripcion
---------------------------------------
1|1|uno
1|1|uno
1|1|uno
2|2|dos
2|2|dos
2|2|dos
3|3|tres
3|3|tres

Si observas aquí se están repitiendo los registros, pero como hice sólo un select sobre los campos de la primer tabla no puedo determinar donde está el error. En estos casos es conveniente que hagas un select *, para obtener la información de todas las tablas que se involucran en la consulta:

Código SQL:

SELECT * FROM @Tabla1 T1 
INNER JOIN @Tabla2 T2 ON T1.id1 = T2.id1
 

Con esto obtienes lo siguiente:

Código:

id1|id2|descripcion|id1|id2|descripcion
----------------------------------------------------
1|1|uno|1|1|UNO-UNO
1|1|uno|1|2|UNO-DOS
1|1|uno|1|3|UNO-TRES
2|2|dos|2|1|DOS-UNO
2|2|dos|2|2|DOS-DOS
2|2|dos|2|3|DOS-TRES
3|3|tres|3|1|TRES-UNO
3|3|tres|3|3|TRES-TRES

Creo que con esto es más claro que NO ES QUE SE REPITAN LOS REGISTROS, sino que CADA REGISTRO ES ÚNICO, pero al no declararse la relación con el campo id2 de las tablas se está realizando un producto cartesiano.

De tal manera que la consulta correcta debería ser así:

Código SQL:

SELECT * FROM @Tabla1 T1 
INNER JOIN @Tabla2 T2 ON T1.id1 = T2.id1 AND T1.id2 = T2.id2
 

Código:

id1|id2|descripcion|id1|id2|descripcion
----------------------------------------------------
1	1	uno	1	1	UNO-UNO
2	2	dos	2	2	DOS-DOS
3	3	tres	3	3	TRES-TRES

Espero que se entienda más o menos el problema y ojalá que te sea de ayuda

Saludos
Leo.

[th3r0rn]

Leo,
Quiero hacer el join pero me da muchos errores:

Código SQL:

CREATE PROCEDURE [dbo].[reportes]
@finicio datetime,
@ffin datetime,
@linea nvarchar(5),
@tubo nvarchar(5),
@hora nvarchar(15) AS
BEGIN
    SET Nocount ON
SELECT optimas.Linea,optimas.Tubo,htargets.Marca,htargets.Target,pesos.Peso,pesos.Vaso,Optimas.Codigo,pesos.Hora,pesos.Fecha
    INTO #checho
    FROM pesos INNER JOIN optimas ON optimas.codigo=pesos.codoptima
    WHERE optimas.codigo = pesos.codOptima AND optimas.linea = @linea 
    AND optimas.tubo = @tubo AND pesos.fecha  >=@finicio AND pesos.fecha<@ffin
    AND htargets.fecha >=@finicio AND htargets.fecha<@ffin
    AND pesos.hora = @hora AND h.codoptima = optimas.codigo AND htargets.hora= @hora
    SELECT *,(SELECT COUNT(1)total FROM #checho) AS Totality FROM #checho ORDER BY fecha
  
END
 

Te dejo una captura de pantalla a continuacion:
http://ompldr.org/vN202YQ

[iislas]

th3r0n

(con el permis)

Los mensaje de error son bien claros, hay algunos objetos (vistas o tablas) que no se reconocen en tu SELECT, por ejemplo: htargets

[leonardo_josue]

Tal como lo comenta el compañero iislas el error es claro, ya que no estás especificando en ningún lado la tabla htargets ni en el FROM ni en el INNER JOIN, sólo estás manejando las tablas pesos (en el from) y optimas (en el join)

Te recomiendo que leas cualquier manual de fundamentos de SQL para saber la sintáxis de los JOINS o pregúntale a Santa Wikipedia o a San Google, te aseguro que ellos pueden tener la respuesta.

para unir tres o más tablas la sintaxis es más o menos la siguiente (Solo pongo la parte del FROM-JOIN)

FROM Tabla1
INNER JOIN Tabla2 ON Tabla1.campo = Tabla2.campo
INNER JOIN Tabla3 ON Tabla1.campo = Tabla3.campo
***
***
INNER JOIN Tablan ON Tabla1.campo = Tablan.campo

es decir UN INNER JOIN POR CADA TABLA QUE ESTÉ INVOLUCRADA...

Saludos
Leo.

[th3r0rn]

Hola, perdon
ya lo he puesto bien, pero aun asi sigue repitiendome los registros :(

Código SQL:

USE [wasp]
GO
/****** Object:  StoredProcedure [dbo].[report]    Script Date: 02/28/2011 11:05:53 ******/
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
 
 
 
ALTER PROCEDURE [dbo].[reportes]
@finicio datetime,
@ffin datetime,
@linea nvarchar(5),
@tubo nvarchar(5),
@hora nvarchar(15) AS
BEGIN
    SET Nocount ON
SELECT optimas.Linea,optimas.Tubo,htargets.Marca,htargets.Target,pesos.Peso,pesos.Vaso,Optimas.Codigo,pesos.Hora,pesos.Fecha
    INTO #checho
    FROM pesos INNER JOIN optimas ON optimas.codigo=pesos.codoptima
    INNER JOIN htargets ON pesos.codoptima = htargets.codoptima
    WHERE  optimas.linea = @linea 
    AND optimas.tubo = @tubo AND pesos.fecha  >=@finicio AND pesos.fecha<@ffin
    AND htargets.fecha >=@finicio AND htargets.fecha<@ffin
    AND pesos.hora = @hora AND htargets.codoptima = optimas.codigo AND htargets.hora= @hora
    SELECT *,(SELECT COUNT(1)total FROM #checho) AS Totality FROM #checho ORDER BY fecha
  
END
 

[leonardo_josue]

Compañer@ th3r0rn:

Creo que no haz terminado por entender donde está el error, si revisas el primer post que puse te hago el siguiente comentario

te falta definir alguna relación entre tus tablas, de tal manera que se está realizando un producto cartesiano

En la consulta que pusiste NO VEO NINGUNA RELACIÓN ADICIONAL, es decir, lo único que hiciste es hacer las uniones con JOIN, pero dejaste exactamente las mismas condiciones.

¿Qué relación falta? pues eso debes contestarlo tú. Si es una relación física (es decir que exista en la definición de las tablas) puedes utilizar el Managment Studio para revisar cada una de las tablas que estás utilizando y las relaciones que tiene definida. Si es una relación lógica (es decir que no tengas definida en la bd la FK) puedes MEDIANTE OBSERVACIÓN DE TUS DATOS determinar qué campo o campos son los que tienen problemas.

Revisa si alguna de las tablas tiene LLAVES COMPUESTAS, si es así debes incluir todos los campos que integran esta llave al hacer la unión. Estás manejando Fechas, pero no veo que relaciones este campo, también este punto.

Finalmente, para este tipo de problemas difícilmente encontrarás una solución en los foros, es decir te podemos dar pistas, algunos trucos etc. pero tú tienes que encontrar la solución.

Saludos y buena suerte
Leo.

Saludos
Leo.

[th3r0rn]

Hola, no tengo llaves primarias, esto podria ser el problema?

[leonardo_josue]

El no tener llaves es un problema desde el punto de vista de diseño de bd, pero no tiene que ver con la duplicidad de los registros. Si observas el ejemplo que puse al crear las tablas en ninguna de ellas estoy definiendo llaves primarias ni foráneas, pero al colocar TODAS LAS RELACIONES LÓGICAS existentes entre los datos entonces no hay problama.

De pura casualidad ya probaste los puntos que te comenté anteriormente??? sobre todo en el punto referente a las fechas???

Saludos
Leo.

[th3r0rn]

Leo, las fechas no puedo empatarlas con ningun otro campo por eso no hago una union, ya revise los campos y no encuentro donde pueda estar el problema. Por favor dame nuevas alternmativas a esto
Porcierto que ves de malo en mi consulta: http://ompldr.org/vN3B5eQ

Código SQL:

USE [wasp]
GO
/****** Object:  StoredProcedure [dbo].[reportes]    Script Date: 03/08/2011 08:48:12 ******/
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
 
 
 
ALTER PROCEDURE [dbo].[reportes]
@finicio datetime,
@ffin datetime,
@linea nvarchar(5),
@tubo nvarchar(5),
@hora nvarchar(15) AS
BEGIN
    SET Nocount ON
SELECT optimas.Linea,optimas.Tubo,htargets.Marca,htargets.Target,pesos.Peso,pesos.Vaso,Optimas.Codigo,pesos.Hora,pesos.Fecha
--  INTO #checho
    FROM pesos INNER JOIN optimas ON optimas.codigo=pesos.codoptima
    INNER JOIN htargets ON pesos.codoptima = htargets.codoptima
    INNER JOIN pesos ON pesos.hora = htargets.hora
        WHERE  optimas.linea = @linea 
    AND optimas.tubo = @tubo AND pesos.fecha  >=@finicio AND pesos.fecha<@ffin
    AND htargets.fecha >=@finicio AND htargets.fecha<@ffin
    AND pesos.hora = @hora AND htargets.codoptima = optimas.codigo AND htargets.hora= @hora
  -- SELECT *,(SELECT count(1)total FROM #checho) as Totality from #checho order by fecha
  
END