[SOLUCIONADO] Generar array con datos de una consulta

[armandoweb]

Amigos:
Genero una consulta a la base de datos y obtengo estos datos en $trabajos_array:

nombre | fecha_ingreso | hora_ingreso | fecha_egreso | hora_egreso

Los datos que extraigo son: nombre: nombre de la persona, fecha y hora ingreso: es el dia y hora que ingresa un trabajo, y fecha y hora egreso: fecha y hora que se termina.
El array que debo crear debería tener este formato:

Código PHP:

Array
(
    [0] => Array
        (
            [tecnico] => Pedro
            [bien] => 15
            [mal] => 2
        )
 
    [1] => Array
        (
            [tecnico] => Juan
            [bien] => 19
            [mal] => 3
        )
 
    [2] => Array
        (
            [tecnico] => Martin
            [bien] => 18
            [mal] => 5
        )
 
)
 

Los datos a mostrar son: tecnico es el nombre de la persona, y bien y mal son los trabajos hechos que sobrepasan la media hora, es decir, si el trabajo hecho por Pedro por ejemplo es entregado antes de la media hora debe sumarse 1 a bien, de lo contrario se sumara 1 a mal.

Este es codigo que tengo hecho hasta el momento:

Código PHP:

$resultado = array();
$nombres = array();
$bien   = 0;
$mal    = 0;
 
for($i = 0; $i < count($trabajos_array); $i++) {
 
        //aqui hago el calculo para saber si el 
        $date1 = new DateTime(date('Y-m-d', strtotime($trabajos_array[$i]['fecha_ingreso'])) . $trabajos_array[$i]['hora_ingreso']);
        $date2 = new DateTime(date('Y-m-d', strtotime($trabajos_array[$i]['fecha_egreso '])) . $trabajos_array[$i]['hora_egreso']);
        $interval = $date1->diff($date2);
        $diferencia = $interval->format('%i');
 
        $nombres[] = $trabajos_array[$i]['nombre'];  
 
        if ($diferencia <= '30') {
                    
            $bien = $bien + 1;
                
    } else {
                    
        $mal = $mal + 1;
                
        }
 
}
 
//aqui armo el array de nombres unicos          
$tecnicos = array();
foreach ($nombres as $key => $val) {
        if (!in_array($val, $tecnicos, true)) {
        $tecnicos[] = $val;
    }
}
 
//aqui genero el array final
for($i = 0; $i < count($tecnicos); $i++) {
    $resultado[$i]['tecnico'] = $tecnicos[$i];
    $resultado[$i]['bien'] = $bien;
        $resultado[$i]['mal'] = $mal;
}
 

Lo que me genera es último array es:

Código PHP:

Array
(
    [0] => Array
        (
            [tecnico] => Juan Uranda
            [bien] => 3
            [mal] => 0
        )
 
    [1] => Array
        (
            [tecnico] => Martin Perez
            [bien] => 3
            [mal] => 0
        )
 
)
 

Mi código falla en mostrar las cantidades de tareas de cada uno ya que el total es 3, Juan tiene hechas 2 tareas y Martin solo 1.

Como puedo modificar mi código para que me muestre correctamente las cantidades de trabajos de cada uno?

Desde ya muchas gracias por su ayuda!

[quimfv]

Puedes modificar la tabla de esta forma

nombre
fecha_ingreso DATETIME YYYY-mm-dd HH:MM:SS
fecha_egreso DATETIME

Si fuera asi seria muy facil usando datetime funtions de mysql

Código MySQL:

SELECT nombre tecnico, 
    SUM(if(TIMEDIFF(fecha_egreso,fecha_ingreso)<'00:30:00',1,0)) bien,
    SUM(if(TIMEDIFF(fecha_egreso,fecha_ingreso)<'00:30:00',0,1)) mal
FROM tu_Tabla
GROUP BY nombre;
 

De ahi al array que pides es muy facil.

Muestra que tipos de campos tienes quizas se podria solucionar combinando datetime functions....

[armandoweb]

Quimfv gracias por tu aporte...debería probarlo.

Los datos los tomo de 2 tablas, una tabla trabajos y una tabla tecnicos. De esta última solo el nombre del mismo y los demas datos de la tabla trabajos.

Generando esa consulta con los datos que necesito se me vería mucho mas simple y limpio el código.

Estoy trabajando con el framework LARAVEL, asi que probaré tu sugerencia y luego te digo como fue.

Gracias!

[armandoweb]

Quimfv no voy a poder modificar la base de datos, lo tendre que hacer desde el código...alguna idea de como modificar lo que ya tengo hecho?
Gracias!

[quimfv]

postea el resultado de

SHOW CREATE TABLE tabla_trabajos

y veremos que se puede hacer.

No tiene ningun sentido que hagas eso por codigo, mysql lo resuelve mucho mejor.

nombre
fecha_ingreso DATE
hora_ingreso TIME
fecha_egreso DATE
hora_egreso TIME

Código MySQL:

SELECT tec.nombre tecnico, 
    SUM(if(TIMEDIFF(STR_TO_DATE(CONCAT(tra.fecha_egreso, ' ', tra.hora_egreso), '%Y-%m-%d %H:%i:%s'),
                            STR_TO_DATE(CONCAT(tra.fecha_ingreso, ' ', tra.hora_ingreso), '%Y-%m-%d %H:%i:%s'))<'00:30:00',1,0)) bien,
    SUM(if(TIMEDIFF(STR_TO_DATE(CONCAT(tra.fecha_egreso, ' ', tra.hora_egreso), '%Y-%m-%d %H:%i:%s'),
                            STR_TO_DATE(CONCAT(tra.fecha_ingreso, ' ', tra.hora_ingreso), '%Y-%m-%d %H:%i:%s'))<'00:30:00',0,1)) mal
FROM trabajos tra INNER JOIN tecnicos tec ON tra.idTecnicos=tec.idTecnicos
GROUP BY tec.nombre;
 

[armandoweb]

Esta es la consulta que hago:

Código PHP:

$trabajos = DB::table('trabajos')
    ->join('tecnicos', 'trabajos.responsable_diagnostico', '=', 'tecnicos.id')
    ->where('trabajos.created_at', '>=', $desde4, 'and', 'trabajos.created_at', '<=', $hasta4)
    ->select('trabajos.fecha_ingreso', 'trabajos.hora_ingreso', 'tecnicos.nombre', 'trabajos.fecha_egreso', 'trabajos.hora_egreso')
    ->get();
 

y este es el objeto que arroja:

Código PHP:

Array
(
    [0] => stdClass Object
        (
            [fecha_ingreso] => 10/10/2014
            [hora_ingreso] => 12:30
            [nombre] => Juan Uranda
            [fecha_egreso] => 10/10/2014
            [hora_egreso] => 12:50
        )
 
    [1] => stdClass Object
        (
            [fecha_ingreso] => 16/10/2014
            [hora_ingreso] => 12:30
            [nombre] => Juan Uranda
            [fecha_egreso] => 16/10/2014
            [hora_egreso] => 13:00
        )
 
    [2] => stdClass Object
        (
            [fecha_ingreso] => 31/10/2014
            [hora_ingreso] => 12:30
            [nombre] => Martin Perez
            [fecha_egreso] => 31/10/2014
            [hora_egreso] => 13:50
        )
 
)
 

Eso es lo que necesitas saber?

[NSD]

Supongo que usas Laravel por la sintaxis que pusiste, la consulta de @quimfv pasada a Laravel seria algo asi:

Código PHP:

DB::table('trabajos')
    ->join('tecnicos', 'trabajos.responsable_diagnostico', '=', 'tecnicos.id')
    ->where('trabajos.created_at', '>=', $desde4, 'and', 'trabajos.created_at', '<=', $hasta4)
    ->select(DB::raw('tecnicos.nombre tecnico, 
                        SUM(if(TIMEDIFF(STR_TO_DATE(CONCAT(trabajos.fecha_egreso, ' ', trabajos.hora_egreso), '%Y-%m-%d %H:%i:%s'),
                            STR_TO_DATE(CONCAT(trabajos.fecha_ingreso, ' ', trabajos.hora_ingreso), '%Y-%m-%d %H:%i:%s'))<'00:30:00',1,0)) bien,
                        SUM(if(TIMEDIFF(STR_TO_DATE(CONCAT(trabajos.fecha_egreso, ' ', trabajos.hora_egreso), '%Y-%m-%d %H:%i:%s'),
                            STR_TO_DATE(CONCAT(trabajos.fecha_ingreso, ' ', trabajos.hora_ingreso), '%Y-%m-%d %H:%i:%s'))<'00:30:00',0,1)) mal'))
    ->groupBy('tecnicos.nombre')
    ->get();
 

No lo he probado, pero esa es la idea.

[quimfv]

[fecha_egreso] => 10/10/2014
[hora_egreso] => 12:50

Ni las fechas ni las horas parecen tener formato DATE y TIME respectivamente

Cambia esto

STR_TO_DATE(CONCAT(trabajos.fecha_egreso, ' ', trabajos.hora_egreso), '%Y-%m-%d %H:%i:%s')

por esto

STR_TO_DATE(CONCAT(trabajos.fecha_egreso, ' ', trabajos.hora_egreso), '%d/%m/%Y %H:%i')

(en todas las apariciones)

a ver si funciona....

[armandoweb]

Hola NSD y Quimfv...
Luego de acomodar algunas comillas y colocar otras me quedó así:

Código PHP:

$trabajos = DB::table('trabajos')
    ->join('tecnicos', 'trabajos.responsable_diagnostico', '=', 'tecnicos.id')
    ->where('trabajos.created_at', '>=', $desde4, 'and', 'trabajos.created_at', '<=', $hasta4)
    ->select(DB::raw('tecnicos.nombre', 
          SUM(if(TIMEDIFF(STR_TO_DATE(CONCAT('trabajos.fecha_egreso', ' ', 'trabajos.hora_egreso'), '%d/%m/%Y %H:%i'),
                    STR_TO_DATE(CONCAT('trabajos.fecha_ingreso', ' ', 'trabajos.hora_ingreso'), '%d/%m/%Y %H:%i'))<'00:30:00',1,0) bien,
              SUM(if(TIMEDIFF(STR_TO_DATE(CONCAT('trabajos.fecha_egreso', ' ', 'trabajos.hora_egreso'), '%d/%m/%Y %H:%i'),
                    STR_TO_DATE(CONCAT('trabajos.fecha_ingreso', ' ', 'trabajos.hora_ingreso'), '%d/%m/%Y %H:%i'))<'00:30:00',0,1) mal))
    ->groupBy('tecnicos.nombre')
    ->get();
 

Pero esto me da un erro en el primer IF:

Código hh:

syntax error, unexpected 'if' (T_IF), expecting ')'
 

Por lo que dice falta un ) pero no puedo encontrar donde... :(

[NSD]

Te faltan las comillas de DB::Raw, ese método recibe un unico string con todo lo demas dentro, quimfv escribio la consulta en sql puro, lo que yo hice fue pasarlo a la sintaxis de laravel, el if que te da error no es un if de php, es un if de mysql por eso debe ir entre comillas. seria algo asi:

Código PHP:

<?php
$trabajos = DB::table('trabajos')
    ->join('tecnicos', 'trabajos.responsable_diagnostico', '=', 'tecnicos.id')
    ->where('trabajos.created_at', '>=', $desde4, 'and', 'trabajos.created_at', '<=', $hasta4)
    ->select(DB::raw("tecnicos.nombre, 
                SUM(if(TIMEDIFF(STR_TO_DATE(CONCAT('trabajos.fecha_egreso', ' ', 'trabajos.hora_egreso'), '%d/%m/%Y %H:%i'),
                    STR_TO_DATE(CONCAT('trabajos.fecha_ingreso', ' ', 'trabajos.hora_ingreso'), '%d/%m/%Y %H:%i'))<'00:30:00',1,0) bien,
                SUM(if(TIMEDIFF(STR_TO_DATE(CONCAT('trabajos.fecha_egreso', ' ', 'trabajos.hora_egreso'), '%d/%m/%Y %H:%i'),
                    STR_TO_DATE(CONCAT('trabajos.fecha_ingreso', ' ', 'trabajos.hora_ingreso'), '%d/%m/%Y %H:%i'))<'00:30:00',0,1) mal)")
            )
    ->groupBy('tecnicos.nombre')
    ->get();
 

[armandoweb]

Si, faltaban esas comillas, me queda así:

Código PHP:

$trabajos = DB::table('trabajos')
    ->join('tecnicos', 'trabajos.responsable_diagnostico', '=', 'tecnicos.id')
    ->where('trabajos.created_at', '>=', $desde4, 'and', 'trabajos.created_at', '<=', $hasta4)
    ->select(DB::raw("tecnicos.nombre, 
                SUM(if(TIMEDIFF(STR_TO_DATE(CONCAT('trabajos.fecha_egreso', ' ', 'trabajos.hora_egreso'), '%d/%m/%Y %H:%i'),
                    STR_TO_DATE(CONCAT('trabajos.fecha_ingreso', ' ', 'trabajos.hora_ingreso'), '%d/%m/%Y %H:%i'))<'00:30:00',1,0) bien,
                SUM(if(TIMEDIFF(STR_TO_DATE(CONCAT('trabajos.fecha_egreso', ' ', 'trabajos.hora_egreso'), '%d/%m/%Y %H:%i'),
                    STR_TO_DATE(CONCAT('trabajos.fecha_ingreso', ' ', 'trabajos.hora_ingreso'), '%d/%m/%Y %H:%i'))<'00:30:00',0,1) mal)"))
    ->groupBy('tecnicos.nombre')
    ->get();
 

Pero ahora me da error en las comillas que encierran el espacio de la concatenación, probé de dejarlas así '" "' pero no es correcto...

[armandoweb]

Así queda el código, ya funciona perfectamente!

Código PHP:

$trabajos = DB::table('trabajos')
    ->join('tecnicos', 'trabajos.responsable_diagnostico', '=', 'tecnicos.id')
    ->where('trabajos.created_at', '>=', $desde4)
        ->where('trabajos.created_at', '<=', $hasta4)
    ->select(DB::raw("tecnicos.nombre,
            SUM(if(TIMEDIFF(STR_TO_DATE(CONCAT('trabajos.fecha_diag', ' ', 'trabajos.hora_diag'), '%d/%m/%Y %H:%i'),
        STR_TO_DATE(CONCAT('trabajos.fecha_ingreso', ' ', 'trabajos.hora_ingreso'), '%d/%m/%Y %H:%i'))<'00:30:00',1,0)) as bien,
        SUM(if(TIMEDIFF(STR_TO_DATE(CONCAT('trabajos.fecha_diag', ' ', 'trabajos.hora_diag'), '%d/%m/%Y %H:%i'),
        STR_TO_DATE(CONCAT('trabajos.fecha_ingreso', ' ', 'trabajos.hora_ingreso'), '%d/%m/%Y %H:%i'))<'00:30:00',0,1)) as mal"))
    ->groupBy('tecnicos.nombre')
    ->get();
 

Muchas gracias NSD y Quimfv por su tiempo, han sido de mucha ayuda!
Saludos!