[SOLUCIONADO] Optimizar consulta

negro1985 · #1 (**permalink**) 13/04/2013, 15:41

Me podrían dar su opinión, critica, recomendación acerca de esta consulta

Código MySQL:

Ver originalSELECT
                        (SELECT
                            IF(COUNT(cc.ID_CONTRATO)>0,COUNT(cc.ID_CONTRATO),'')
                        FROM
                            contrato cc
                        INNER JOIN especialidad ee ON ee.ID_ESPECIALIDAD = cc.ID_ESPECIALIDAD
                        INNER JOIN departamento dd ON dd.ID_DEPARTAMENTO = ee.ID_DEPARTAMENTO
                        WHERE
                            cc.ID_SITUACION = 18
                        AND
                            cc.ID_TIPO_CONTRATO = '".$this->getId_tipo_contrato()."'
                        AND
                            dd.NOMBRE_CORTO = '".$this->getNombre_corto_departamento()."'
                        ) AS 'EJECUCION',
                        (SELECT
                            IF(COUNT(cc.ID_CONTRATO)>0,COUNT(cc.ID_CONTRATO),'')
                        FROM
                            contrato cc
                        INNER JOIN especialidad ee ON ee.ID_ESPECIALIDAD = cc.ID_ESPECIALIDAD
                        INNER JOIN departamento dd ON dd.ID_DEPARTAMENTO = ee.ID_DEPARTAMENTO
                        WHERE
                            cc.ID_SITUACION = 19
                        AND
                            cc.ID_TIPO_CONTRATO = '".$this->getId_tipo_contrato()."'
                        AND
                            dd.NOMBRE_CORTO = '".$this->getNombre_corto_departamento()."'
                        ) AS 'SUSPENDIDO',
                        (SELECT
                            IF(COUNT(cc.ID_CONTRATO)>0,COUNT(cc.ID_CONTRATO),'')
                        FROM
                            contrato cc
                        INNER JOIN especialidad ee ON ee.ID_ESPECIALIDAD = cc.ID_ESPECIALIDAD
                        INNER JOIN departamento dd ON dd.ID_DEPARTAMENTO = ee.ID_DEPARTAMENTO
                        WHERE
                            cc.ID_SITUACION = 20
                        AND
                            cc.ID_TIPO_CONTRATO = '".$this->getId_tipo_contrato()."'
                        AND
                            dd.NOMBRE_CORTO = '".$this->getNombre_corto_departamento()."'
                        ) AS 'MORA',
                        (SELECT
                            IF(COUNT(cc.ID_CONTRATO)>0,COUNT(cc.ID_CONTRATO),'')
                        FROM
                            contrato cc
                        INNER JOIN especialidad ee ON ee.ID_ESPECIALIDAD = cc.ID_ESPECIALIDAD
                        INNER JOIN departamento dd ON dd.ID_DEPARTAMENTO = ee.ID_DEPARTAMENTO
                        WHERE
                            cc.ID_SITUACION = 39
                        AND
                            cc.ID_TIPO_CONTRATO = '".$this->getId_tipo_contrato()."'
                        AND
                            dd.NOMBRE_CORTO = '".$this->getNombre_corto_departamento()."'
                        ) AS 'FINIQUITO',
                        (SELECT
                            IF(COUNT(cc.ID_CONTRATO)>0,COUNT(cc.ID_CONTRATO),'')
                        FROM
                            contrato cc
                        INNER JOIN especialidad ee ON ee.ID_ESPECIALIDAD = cc.ID_ESPECIALIDAD
                        INNER JOIN departamento dd ON dd.ID_DEPARTAMENTO = ee.ID_DEPARTAMENTO
                        WHERE
                            cc.ID_SITUACION = 46
                        AND
                            cc.ID_TIPO_CONTRATO = '".$this->getId_tipo_contrato()."'
                        AND
                            dd.NOMBRE_CORTO = '".$this->getNombre_corto_departamento()."'
                        ) AS 'PROBLEMA'
                        FROM 
                            contrato c
                        INNER JOIN especialidad e ON e.ID_ESPECIALIDAD = c.ID_ESPECIALIDAD
                        INNER JOIN departamento d ON d.ID_DEPARTAMENTO = e.ID_DEPARTAMENTO
                        WHERE
                            d.NOMBRE_CORTO = '".$this->getNombre_corto_departamento()."'

Que devuelte esta consulta, devuelve los tipos de contratos(genéricos y especificos) por departamento, al igual que devuelve la situación de cada contrato (ejecución,mora,problema,suspendido,finiquito).

se aceptan todo tipo de comentarios, yo se que se puede mejorar pero necesito su ayuda para esto, gracias de antemano.

gnzsoloyo · #2 (**permalink**) 13/04/2013, 16:46

Considerando que todas las subconsultas llevan los mismos parámetros, con excepción de uno, y ese sólo es el relevante para la cuenta, simplemente contaría ese. Además, me sorprendería que no estuviese devolviendo errores de sintaxis.
Yo lo podria así:

Código MySQL:

Ver originalSELECT
    IF(E > 0, E, '') EJECUCION,
    IF(S > 0, S, '') SUSPENDIDO,
    IF(M > 0, M, '') MORA,
    IF(P > 0, P, '') PROBLEMA
FROM    
    (SELECT
        COUNT(IF(cc.ID_SITUACION = 18, cc.ID_CONTRATO,0)) E,
        COUNT(IF(cc.ID_SITUACION = 19, cc.ID_CONTRATO,0)) S,
        COUNT(IF(cc.ID_SITUACION = 39, cc.ID_CONTRATO,0)) M,
        COUNT(IF(cc.ID_SITUACION = 46, cc.ID_CONTRATO,0)) P
    FROM 
        contrato c
        INNER JOIN especialidad e ON e.ID_ESPECIALIDAD = c.ID_ESPECIALIDAD
        INNER JOIN departamento d ON d.ID_DEPARTAMENTO = e.ID_DEPARTAMENTO
    WHERE
        d.NOMBRE_CORTO = '".$this->getNombre_corto_departamento()."'
        AND c.ID_TIPO_CONTRATO = '".$this->getId_tipo_contrato()."') TABLA;

Por cierto, un sano consejo es jamás pongas subconsultas en el SELECT si puedes evitarlo. Son desastrosas para la performance, porque se ejecutan una vez por cada registro devuelto en la consulta principal, por lo que el resultado en realidad podría estar siendo calamitoso...

negro1985 · #3 (**permalink**) 15/04/2013, 08:10

Gracias por tu ayuda solo que la consulta me devuelve el mismo resultado en todos if, modifique la consulta y obtuve el resultado que deseaba, quedo asi:

Código:

SELECT
    EJECUCION,
    SUSPENDIDO,
    MORA,
    PROBLEMA
FROM    
    (SELECT
        IF(cc.ID_SITUACION = 18, COUNT(cc.ID_CONTRATO),0) EJECUCION,
        IF(cc.ID_SITUACION = 19, COUNT(cc.ID_CONTRATO),0) SUSPENDIDO,
        IF(cc.ID_SITUACION = 39, COUNT(cc.ID_CONTRATO),0) MORA,
        IF(cc.ID_SITUACION = 46, COUNT(cc.ID_CONTRATO),0) PROBLEMA
    FROM 
        contrato cc
        INNER JOIN especialidad e ON e.ID_ESPECIALIDAD = cc.ID_ESPECIALIDAD
        INNER JOIN departamento d ON d.ID_DEPARTAMENTO = e.ID_DEPARTAMENTO
    WHERE
        d.NOMBRE_CORTO = "MISO"
        AND cc.ID_TIPO_CONTRATO = 1) TABLA;

Gracias por tu ayuda gnzsoloyo.