[SOLUCIONADO] Consulta no funciona correctamente

[temerariomalaga]

Tengo una consulta que va filtrando los codigos de otras tablas para devolver los resultados filtados.
Os explico que contiene cada tabla.
-TEMP: Contiene los códigos de la tabla Equipo_arbitral que estan disponibles el dia de un partido.
-TEMP3: Contiene los códigos de los árbitros que cumplen una diferencia entre horas (si por ejemplo tienen un partido ya asignado a las 17:00 no se le podrá asignar un partido en el rango desde las 15:15 hasta las 18:45)
-TEMP4: Contiene los códigos de los árbitros que no cumplen la diferencia de horas (Por algún motivo en el if en el que compruebo el rango horario en esta tabla también me inserta los que si cumplen el rango por lo que tengo que filtrarlo posteriormente, cosa que hago en estas consultas)
-TEMP5: Contiene los árbitros que tienen algún partido el mismo dia del partido que se va a asignar
-TEMP6: Contiene los códigos de los árbitros que tienen un partido el mismo dia del partido a asignar y cumplen los rangos horarios.
-TEMP7: Contiene los códigos de los árbitros que tienen un partido el mismo dia del partido y no cumplen los rangos horarios.

Ahora aquí esta la consulta:

Código MySQL:

SELECT COUNT(*) INTO @regs FROM EQUIPO_ARBITRAL WHERE Codigo_arbitro IN (SELECT * FROM TEMP) AND Codigo_arbitro NOT IN(SELECT codigo FROM TEMP4  WHERE codigo NOT IN(SELECT codigo FROM TEMP3)) AND Codigo_arbitro NOT IN(SELECT codigo FROM TEMP5 WHERE codigo NOT IN(SELECT codigo FROM TEMP7)) AND Cod_localidad=
                (SELECT Codigo_localidad FROM PABELLON WHERE Codigo_pabellon=pabellon) ORDER BY Apellidos;
    IF(SELECT @regs)>=3 THEN
        SELECT Codigo_arbitro AS 'Cod_arbi', Licencia AS 'Licenc', Nombre AS 'Nom', Apellidos AS 'Ape' FROM EQUIPO_ARBITRAL WHERE Codigo_arbitro IN (SELECT * FROM TEMP) AND Codigo_arbitro NOT IN(SELECT codigo FROM TEMP4  WHERE codigo NOT IN(SELECT codigo FROM TEMP3)) AND Codigo_arbitro NOT IN(SELECT codigo FROM TEMP5 WHERE codigo NOT IN(SELECT codigo FROM TEMP7)) AND Cod_localidad=
                    (SELECT Codigo_localidad FROM PABELLON WHERE Codigo_pabellon=pabellon) ORDER BY Apellidos;
    ELSE
        SELECT COUNT(*) INTO @regs2 FROM EQUIPO_ARBITRAL WHERE Codigo_arbitro IN (SELECT * FROM TEMP) AND Codigo_arbitro NOT IN(SELECT codigo FROM TEMP4  WHERE codigo NOT IN(SELECT codigo FROM TEMP3)) AND Codigo_arbitro NOT IN(SELECT codigo FROM TEMP5 WHERE codigo NOT IN(SELECT codigo FROM TEMP7)) AND Cod_localidad IN
            (SELECT Codigo_localidad FROM LOCALIDAD WHERE Codigo_sede=(SELECT SEDE.Codigo_sede FROM PABELLON INNER JOIN LOCALIDAD ON PABELLON.Codigo_localidad=LOCALIDAD.Codigo_localidad
            INNER JOIN SEDE ON LOCALIDAD.Codigo_sede=SEDE.Codigo_sede
            WHERE Codigo_pabellon=pabellon)) ORDER BY Apellidos;
        IF(SELECT @regs2)>=3 THEN
            SELECT Codigo_arbitro AS 'Cod_arbi', Licencia AS 'Licenc', Nombre AS 'Nom', Apellidos AS 'Ape' FROM EQUIPO_ARBITRAL WHERE Codigo_arbitro IN (SELECT * FROM TEMP) AND Codigo_arbitro NOT IN(SELECT codigo FROM TEMP4  WHERE codigo NOT IN(SELECT codigo FROM TEMP3)) AND Codigo_arbitro NOT IN(SELECT codigo FROM TEMP5 WHERE codigo NOT IN(SELECT codigo FROM TEMP7)) AND Cod_localidad IN
                (SELECT Codigo_localidad FROM LOCALIDAD WHERE Codigo_sede=(SELECT SEDE.Codigo_sede FROM PABELLON INNER JOIN LOCALIDAD ON PABELLON.Codigo_localidad=LOCALIDAD.Codigo_localidad
                INNER JOIN SEDE ON LOCALIDAD.Codigo_sede=SEDE.Codigo_sede
                WHERE Codigo_pabellon=pabellon)) ORDER BY Apellidos;
        ELSE
            SELECT Codigo_arbitro AS 'Cod_arbi', Licencia AS 'Licenc', Nombre AS 'Nom', Apellidos AS 'Ape' FROM EQUIPO_ARBITRAL WHERE Codigo_arbitro IN (SELECT * FROM TEMP) AND Codigo_arbitro NOT IN(SELECT codigo FROM TEMP4  WHERE codigo NOT IN(SELECT codigo FROM TEMP3)) AND Codigo_arbitro NOT IN(SELECT codigo FROM TEMP5 WHERE codigo NOT IN(SELECT codigo FROM TEMP7));
        END IF;
    END IF;
 

No se el por qué pero si intento asignar un partido a las 18:00 de un viernes me aparecen los que ya tienen un partido asignado por ejemplo a las 17:00.

[quimfv]

Ufff, eso es un script con varias consultas, divide y vencerás:

1.- Has mirado si cada una por separado da lo que esperas?
2.- Sabes que IN () es una de las clausulas más ineficientes, se puede substitur por LEFT JOIN ... ON y filtrar por NULL o NOT NULL sobre el campo de relación.
3.- Las tablas TEMP realmente contienen lo que dices que contienen. La selección de "un partido a las 18:00" se hace en el momento de crear las tablas TEMP, luego el error podría veir de ahí...

[temerariomalaga]

Iniciado por quimfv

Ufff, eso es un script con varias consultas, divide y vencerás:

1.- Has mirado si cada una por separado da lo que esperas?
2.- Sabes que IN () es una de las clausulas más ineficientes, se puede substitur por LEFT JOIN ... ON y filtrar por NULL o NOT NULL sobre el campo de relación.
3.- Las tablas TEMP realmente contienen lo que dices que contienen. La selección de "un partido a las 18:00" se hace en el momento de crear las tablas TEMP, luego el error podría veir de ahí...

He modificado un poco el código porque encontré un error con la tabla TEMP5. Seguiré modificando y si sigo con el mismo problema o cambia ya ya lo posteo.

[temerariomalaga]

Bueno voy a explicar un poco más. En la tabla TEMP7 se repiten tambien los de la tabla TEMP6. Ahora no aparece ningún árbitro.
Voy a reducir el código para centrarnos solo en la consulta principal ya que solucionando esa el resto se solucionarán de las misma forma:

Código MySQL:

SELECT Codigo_arbitro AS 'Cod_arbi', Licencia AS 'Licenc', Nombre AS 'Nom', Apellidos AS 'Ape' FROM EQUIPO_ARBITRAL WHERE Codigo_arbitro IN (SELECT * FROM TEMP) AND Codigo_arbitro NOT IN
      (SELECT codigo FROM TEMP4  WHERE codigo NOT IN
            (SELECT codigo FROM TEMP3 WHERE codigo NOT IN
                  (SELECT codigo FROM TEMP5 WHERE codigo NOT IN(SELECT codigo FROM TEMP7 WHERE codigo NOT IN(SELECT codigo FROM TEMP6))))) AND Cod_localidad=
                    (SELECT Codigo_localidad FROM PABELLON WHERE Codigo_pabellon=pabellon) ORDER BY Apellidos;
 

Si cambia algo ya edito el post.

[quimfv]

Código MySQL:

SELECT Codigo_arbitro AS 'Cod_arbi', 
       Licencia AS 'Licenc', 
       Nombre AS 'Nom', 
       Apellidos AS 'Ape' 
    FROM EQUIPO_ARBITRAL 
    WHERE Codigo_arbitro IN (SELECT * FROM TEMP) 
      AND Codigo_arbitro 
                  NOT IN (SELECT codigo 
                                    FROM TEMP4  
                                    WHERE codigo 
                   NOT IN (SELECT codigo 
                                 FROM TEMP3 
                                 WHERE codigo 
                   NOT IN (SELECT codigo 
                                 FROM TEMP5 
                                 WHERE codigo 
                   NOT IN (SELECT codigo 
                                 FROM TEMP7 
                                 WHERE codigo 
                   NOT IN(SELECT codigo 
                                 FROM TEMP6))))) 
        AND Cod_localidad=(SELECT Codigo_localidad FROM PABELLON WHERE Codigo_pabellon=pabellon) 
        ORDER BY Apellidos;
 

Esa query parece sintacticamente correcta.... el problema es que cada tabla TEMP es en si misma una subquery y no vemos como esta formulada.

Insisto en el tema de eliminar los IN ()

[quimfv]

Código MySQL:

SELECT Codigo_arbitro AS 'Cod_arbi', 
       Licencia AS 'Licenc', 
       Nombre AS 'Nom', 
       Apellidos AS 'Ape' 
    FROM EQUIPO_ARBITRAL INNER JOIN 
            (SELECT Codigo_localidad FROM PABELLON WHERE Codigo_pabellon=pabellon) Sbc
                  ON EQUIPO_ARBITRAL.Cod_localidad=Sbc.Cod_localidad 
    WHERE Codigo_arbitro IN (SELECT * FROM TEMP) 
      AND Codigo_arbitro 
                  NOT IN (SELECT codigo 
                                    FROM TEMP4  
                                    WHERE codigo 
                   NOT IN (SELECT codigo 
                                 FROM TEMP3 
                                 WHERE codigo 
                   NOT IN (SELECT codigo 
                                 FROM TEMP5 
                                 WHERE codigo 
                   NOT IN (SELECT codigo 
                                 FROM TEMP7 
                                 WHERE codigo 
                   NOT IN(SELECT codigo 
                                 FROM TEMP6))))) 
        ORDER BY Apellidos;
 

Claro que la seleccion de ciudad se podria hacer al obtener TEMP y no seria necesario ponerla aqui....

Código MySQL:

SELECT Codigo_arbitro AS 'Cod_arbi', 
       Licencia AS 'Licenc', 
       Nombre AS 'Nom', 
       Apellidos AS 'Ape' 
    FROM EQUIPO_ARBITRAL 
    WHERE Codigo_arbitro IN (SELECT * FROM TEMP) 
      AND Codigo_arbitro 
                  NOT IN (SELECT codigo 
                                    FROM TEMP4  
                                    WHERE codigo 
                   NOT IN (SELECT codigo 
                                 FROM TEMP3 
                                 WHERE codigo 
                   NOT IN (SELECT codigo 
                                 FROM TEMP5 
                                 WHERE codigo 
                   NOT IN (SELECT codigo 
                                 FROM TEMP7 
                                 WHERE codigo 
                   NOT IN(SELECT codigo 
                                 FROM TEMP6))))) 
        ORDER BY Apellidos;
 

Si TEMP solo continene EQUIPO_ARBITRALes de la ciudad....

Código MySQL:

SELECT Codigo_arbitro AS 'Cod_arbi', 
       Licencia AS 'Licenc', 
       Nombre AS 'Nom', 
       Apellidos AS 'Ape' 
    FROM EQUIPO_ARBITRAL 
          INNER JOIN (SELECT codigo  
                              FROM TEMP
                              WHERE Codigo
                                   NOT IN (SELECT codigo 
                                                FROM TEMP4  
                                                WHERE codigo 
                                                   NOT IN (SELECT codigo 
                                                                FROM TEMP3 
                                                                WHERE codigo 
                                                                  NOT IN (SELECT codigo 
                                                                                FROM TEMP5 
                                                                                WHERE codigo 
                                                                                NOT IN (SELECT codigo 
                                                                                              FROM TEMP7 
                                                                                              WHERE codigo 
                                                                                              NOT IN (SELECT codigo 
                                                                                                            FROM TEMP6)))))) SbcTemp 
                           ON EQUIPO_ARBITRAL.Codigo_arbitro=SbcTemp.codigo
        ORDER BY Apellidos;
 

Ya hemos eliminado un IN ().... substituyendolo por un mas eficiente INNER JOIN .... ON

Ya te lo dije en otro post esa tabla es de ARBITROS NO de equipos arbítrales....

En TEMP ya podiamos haber recuperado Licencia, Nombre, Apellidos, y ciudad y nos ahorrariamos un nivel de subconsultas.....

Código MySQL:

SELECT TEMP.Codigo_arbitro AS 'Cod_arbi', 
             TEMP.Licencia AS 'Licenc', 
             TEMP.Nombre AS 'Nom', 
             TEMP.Apellidos AS 'Ape' 
             FROM TEMP
             WHERE TEMP.Codigo_arbitro
                   NOT IN (SELECT codigo 
                                FROM TEMP4  
                                WHERE codigo 
                                NOT IN (SELECT codigo 
                                             FROM TEMP3 
                                             WHERE codigo 
                                             NOT IN (SELECT codigo 
                                                          FROM TEMP5 
                                                          WHERE codigo 
                                                          NOT IN (SELECT codigo 
                                                                       FROM TEMP7 
                                                                       WHERE codigo 
                                                                        NOT IN (SELECT codigo 
                                                                                      FROM TEMP6)))))
             ORDER BY TEMP.Apellidos;
 

Simplificando....

NOT IN () se substituye por

...
tabla1 t1 LEFT JOIN tabla2 t2 ON t1.campo=t2.campo
WHERE t2.campo IS NULL
...

[temerariomalaga]

He cojido papel y lápiz y he reestructurado todo y lo he simplificado mycho, funcionando ahora todo a la perfección. Ahora tendria TEMP que son todos los arbitros disponibles, TEMP3 que son los que tienen partido un dia distinto, TEMP4 los que tienen el mismo dia y a raiz de TEMP4 saco TEMP5 que son los que cumplen la diferencia horaria y TEMP6 que son los que no. Teniendo esto ahora todo es mucho más sencillo porque solo tengo que obtener los arbitros disponibles (TEMP) y excluir a los que no cumplen la diferencia horaria (TEMP6).
Todo esto dejo aquí el código para que veas que nos hemos comido demasiado la cabeza los dos. De todas formas muchas gracias por tu ayuda @quimfv

Código MySQL:

SELECT COUNT(*) INTO @regs FROM EQUIPO_ARBITRAL WHERE Codigo_arbitro IN (SELECT * FROM TEMP) AND Codigo_arbitro NOT IN(SELECT codigo FROM TEMP6) AND Cod_localidad=
                (SELECT Codigo_localidad FROM PABELLON WHERE Codigo_pabellon=pabellon) ORDER BY Apellidos;
    IF(SELECT @regs)>=3 THEN
        SELECT Codigo_arbitro AS 'Cod_arbi', Licencia AS 'Licenc', Nombre AS 'Nom', Apellidos AS 'Ape' FROM EQUIPO_ARBITRAL WHERE Codigo_arbitro IN 
            (SELECT * FROM TEMP) AND Codigo_arbitro NOT IN(SELECT codigo FROM TEMP6) AND Cod_localidad=
                    (SELECT Codigo_localidad FROM PABELLON WHERE Codigo_pabellon=pabellon) ORDER BY Apellidos;
    ELSE
        SELECT COUNT(*) INTO @regs2 FROM EQUIPO_ARBITRAL WHERE Codigo_arbitro IN (SELECT * FROM TEMP) AND Codigo_arbitro NOT IN(SELECT codigo FROM TEMP6) AND Cod_localidad IN
            (SELECT Codigo_localidad FROM LOCALIDAD WHERE Codigo_sede=(SELECT SEDE.Codigo_sede FROM PABELLON INNER JOIN LOCALIDAD ON PABELLON.Codigo_localidad=LOCALIDAD.Codigo_localidad
                INNER JOIN SEDE ON LOCALIDAD.Codigo_sede=SEDE.Codigo_sede
                    WHERE Codigo_pabellon=pabellon)) ORDER BY Apellidos;
        IF(SELECT @regs2)>=3 THEN
            SELECT Codigo_arbitro AS 'Cod_arbi', Licencia AS 'Licenc', Nombre AS 'Nom', Apellidos AS 'Ape' FROM EQUIPO_ARBITRAL WHERE Codigo_arbitro IN 
                (SELECT * FROM TEMP) AND Codigo_arbitro NOT IN(SELECT codigo FROM TEMP6) AND Cod_localidad IN
                    (SELECT Codigo_localidad FROM LOCALIDAD WHERE Codigo_sede=(SELECT SEDE.Codigo_sede FROM PABELLON INNER JOIN LOCALIDAD ON PABELLON.Codigo_localidad=LOCALIDAD.Codigo_localidad
                        INNER JOIN SEDE ON LOCALIDAD.Codigo_sede=SEDE.Codigo_sede
                            WHERE Codigo_pabellon=pabellon)) ORDER BY Apellidos;
        ELSE
            SELECT Codigo_arbitro AS 'Cod_arbi', Licencia AS 'Licenc', Nombre AS 'Nom', Apellidos AS 'Ape' FROM EQUIPO_ARBITRAL WHERE Codigo_arbitro IN (SELECT * FROM TEMP) AND Codigo_arbitro NOT IN(SELECT codigo FROM TEMP6);
        END IF;
    END IF;