Duda para generar reportes

[jandrogdz]

Buen día foro

Tengo una duda al momento de hacer reportes. Espero me puedan aconsejar cual es la mejor opcion:

Tengo dos metodos uno que hace la condicion y otro que hace las consultas:

Código PHP:

public function condicion($k){
        $where = '';
        $inicio = true;
        if(isset($k['desde']) && $k['desde'] != "" && isset($k['hasta']) && $k['hasta'] != ""):
            $where.=($inicio)?" WHERE f.fecha BETWEEN'".$k['desde']."' AND '".$k['hasta']."'" : "AND f.fecha BETWEEN'".$k['desde']."' AND '".$k['hasta']."'"; 
            $inicio = false;   
        endif;
        if(isset($k['cliente']) && $k['cliente'] != ""):
            $where.=($inicio)?" WHERE c.cliente='".$k['cliente']."'":" AND c.cliente='".$k['cliente']."'";
            $inicio = false;
        endif;
        if(isset($k['asociado']) && $k['asociado'] != ""):
            if($k['asociado'] != 4):
                $where.=($inicio)?" WHERE a.asociadoID='".$k['asociado']."'":" AND a.asociadoID='".$k['asociado']."'";
                $inicio = false;
            endif;    
        endif;
        if(isset($k['estado']) && $k['estado'] != ""):
            if($k['estado'] == "Todos"):
                $where.=($inicio)? " WHERE f.status='Pagada' OR f.status='Pendiente' OR f.status='Cancelada' ":" AND f.status='Pagada' OR f.status='Pendiente' OR f.status='Cancelada'";
                $inicio = false;
            else:
                $where.=($inicio)? " WHERE f.status='".$k['estado']."'":" AND f.status='".$k['estado']."'";
                $inicio = false;
            endif;
        endif;
        if(isset($k['agrupar']) && $k['agrupar'] != ""):
            if($k['asociado'] != 4):
                $where.=($k['agrupar'] == 1) ? " GROUP BY c.cliente WITH ROLLUP":"";
            endif;
        endif;
        return $where;
    }
    
    public function datosFiltro($k){
        $condicion = $this->condicion($k);
        $meses = array(1=>"ene",2=>"feb",3=>"mar",4=>"abr",5=>"may",6=>"jun",7=>"jul",8=>"ago",9=>"sep",10=>"oct",11=>"nov",12=>"dic");
        // CONSULTA...
        if($k['asociado'] != 4):
            $query = "SELECT  c.cliente, f.status, ";
            for($j = 1; $j <= 12; $j++):
                $query.= "SUM(CASE WHEN MONTH(f.fecha) = ".$j." THEN f.totalFactura ELSE 0 END) ".$meses[$j].", ";  
            endfor;
            $query.=" SUM(f.totalFactura) totalgral ";
            $query.= $k['asociado'] == 1 ? "FROM facturaX f ": "FROM facturaY f ";
            $query.=" JOIN cliente c ON f.clienteID = c.clienteID JOIN asociado a ON c.asociadoID=a.asociadoID";
            $query.= $condicion;
            return $query;
        else:
            $query = "SELECT  c.cliente, f.status, ";
            for($j = 1; $j <= 12; $j++):
                $query.= "SUM(CASE WHEN MONTH(f.fecha) = ".$j." THEN f.totalFactura ELSE 0 END) ".$meses[$j].", ";  
            endfor;
            $query.=" SUM(f.totalFactura) totalgral ";
            $query.=" FROM facturaX f ";
            $query.=" JOIN cliente c ON f.clienteID = c.clienteID JOIN asociado a ON c.asociadoID=a.asociadoID";
            $query.= $condicion." AND (a.asociadoID='1') GROUP BY c.cliente WITH ROLLUP";
            $query.=" UNION ";
            $query.= "SELECT  c.cliente, f.status, ";
            for($j = 1; $j <= 12; $j++):
                $query.= "SUM(CASE WHEN MONTH(f.fecha) = ".$j." THEN f.totalFactura ELSE 0 END) ".$meses[$j].", ";  
            endfor;
            $query.=" SUM(f.totalFactura) totalgral ";
            $query.=" FROM facturaY f ";
            $query.=" JOIN cliente c ON f.clienteID = c.clienteID JOIN asociado a ON c.asociadoID=a.asociadoID";
            $query.= $condicion." AND (a.asociadoID='2' OR a.asociadoID='3') GROUP BY c.cliente WITH ROLLUP ";
            return $query;
        endif;
    }
 

Mi duda cual es la mejor opcion para manejar las consultas:
Hacer las consultas por separado o generar todo como lo tengo en ese ejemplo?

Espero me puedan apoyar.
De antemano muchas gracias.

[ocp001a]

Pues no veo mal crear la condición por separado de la ejecución de la consulta, lo que creo que trabajas de más es a la hora d e formar la condición, al tratar de concatenarla y al evaluar si existen los parámetros:

Código PHP:

if(isset($k['desde']) && $k['desde'] != "" && isset($k['hasta']) && $k['hasta'] != ""):
            $where.=($inicio)?" WHERE f.fecha BETWEEN'".$k['desde']."' AND '".$k['hasta']."'" : "AND f.fecha BETWEEN'".$k['desde']."' AND '".$k['hasta']."'"; 
            $inicio = false;   
        endif;
 

Código PHP:

if(!empty($k['desde'])  && !empty ($k['hasta'])):
              $where[]="f.fecha BETWEEN'".$k['desde']."' AND '".$k['hasta'];
 

al final, la variable $where la juntas con implode

Código PHP:

$where='WHERE '.implode(' AND ',$where);
 

Y sólo habría que tratar el group by de forma independiente para que no lleve el and.

Las diferencias: empty verifica si la variable existe y está o no vacía, con lo que no necesitas comparar dos veces

Y finalmente, en vez de preguntar si la variable $where ya contiene algo, vas almacenando en un arreglo los segmentos de la consulta para al final unirlo con AND entre cada segmento.