SUM operation doubling query value - Laravel

Question

SUM operation doubling query value - Laravel

Asked 5 years, 9 months ago

Viewed 166 times

0

I am with a query that is with some problem in which I cannot identify.

I need to bring up the additional sum and journey. The problem is that when I add in the bank the value of the journey and no value for the additional it comes right. From the moment I add more than 1 record in the additional it duplicates the value recorded in journey.

Ex:

In the database it’s like this:

$servidores = DB::table('sch_sismapa.tb_servidor')
->join('sch_decom.tb_servidor', 'sch_sismapa.tb_servidor.nr_matricula', '=', 'sch_decom.tb_servidor.nr_matricula')
->leftJoin('sch_sismapa.tb_insalubridade','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_insalubridade.id_servidor')
->leftJoin('sch_sismapa.tb_falta','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_falta.id_servidor')
->leftJoin('sch_sismapa.tb_ferias','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_ferias.id_servidor')
->leftJoin('sch_sismapa.tb_adicional_noturno','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_adicional_noturno.id_servidor')
->leftJoin('sch_sismapa.tb_extensao_ch','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_extensao_ch.id_servidor')
->select('sch_decom.tb_servidor.nm_servidor',DB::raw('COALESCE(SUM(sch_sismapa.tb_adicional_noturno.nr_adicional),0) as adicional'),
    DB::raw('COALESCE(SUM(sch_sismapa.tb_extensao_ch.nr_extensao),0) as jornada'),
    'sch_sismapa.tb_servidor.*',DB::raw('COUNT(sch_sismapa.tb_falta.id_servidor) as faltas'),DB::raw('COUNT(sch_sismapa.tb_ferias.id_servidor) as ferias'))
->where('sch_sismapa.tb_servidor.cd_secretaria',$request->cd_secretaria)
->where('sch_sismapa.tb_servidor.id_referencia',$request->id_referencia)
->groupBy('sch_decom.tb_servidor.nm_servidor','sch_sismapa.tb_servidor.id','sch_sismapa.tb_servidor.id_referencia','sch_sismapa.tb_servidor.nr_matricula',
    'sch_sismapa.tb_servidor.cd_secretaria','sch_sismapa.tb_servidor.cd_departamento','sch_sismapa.tb_servidor.nr_dias',
    'sch_sismapa.tb_servidor.st_lancado','sch_sismapa.tb_servidor.st_aprovado','sch_sismapa.tb_servidor.ds_observacao',
    'sch_sismapa.tb_servidor.st_grat_reuniao')
->orderby('sch_sismapa.tb_servidor.cd_departamento', 'asc')
->orderBy('sch_decom.tb_servidor.nm_servidor', 'asc')
->skip($partes)
->take(300)
->get();

The return that this query generates is this:

It was to bring the value of 10 in journey. But it is bringing 20.

I think what’s wrong with that part:

DB::raw('COALESCE(SUM(sch_sismapa.tb_adicional_noturno.nr_adicional),0) as adicional'),
DB::raw('COALESCE(SUM(sch_sismapa.tb_extensao_ch.nr_extensao),0) as jornada'),

PS: I am using Postgresql.

SQL below:

SELECT 
    sch_decom.tb_servidor.nm_servidor,
    COALESCE(SUM(sch_sismapa.tb_adicional_noturno.nr_adicional),
            0) AS adicional,
    COALESCE(SUM(sch_sismapa.tb_extensao_ch.nr_extensao),
            0) AS jornada,
    sch_sismapa.tb_servidor.*,
    COUNT(sch_sismapa.tb_falta.id_servidor) AS faltas,
    COUNT(sch_sismapa.tb_ferias.id_servidor) AS ferias
FROM
    sch_sismapa.tb_servidor
        INNER JOIN
    sch_decom.tb_servidor ON sch_sismapa.tb_servidor.nr_matricula = sch_decom.tb_servidor.nr_matricula
        LEFT JOIN
    sch_sismapa.tb_insalubridade ON sch_sismapa.tb_servidor.id = sch_sismapa.tb_insalubridade.id_servidor
        LEFT JOIN
    sch_sismapa.tb_falta ON sch_sismapa.tb_servidor.id = sch_sismapa.tb_falta.id_servidor
        LEFT JOIN
    sch_sismapa.tb_ferias ON sch_sismapa.tb_servidor.id = sch_sismapa.tb_ferias.id_servidor
        LEFT JOIN
    sch_sismapa.tb_adicional_noturno ON sch_sismapa.tb_servidor.id = sch_sismapa.tb_adicional_noturno.id_servidor
        LEFT JOIN
    sch_sismapa.tb_extensao_ch ON sch_sismapa.tb_servidor.id = sch_sismapa.tb_extensao_ch.id_servidor
WHERE
    sch_sismapa.tb_servidor.cd_secretaria = '7'
        AND sch_sismapa.tb_servidor.id_referencia = '9'
GROUP BY sch_decom.tb_servidor.nm_servidor , sch_sismapa.tb_servidor.id , sch_sismapa.tb_servidor.id_referencia , sch_sismapa.tb_servidor.nr_matricula , sch_sismapa.tb_servidor.cd_secretaria , sch_sismapa.tb_servidor.cd_departamento , sch_sismapa.tb_servidor.nr_dias , sch_sismapa.tb_servidor.st_lancado , sch_sismapa.tb_servidor.st_aprovado , sch_sismapa.tb_servidor.ds_observacao , sch_sismapa.tb_servidor.st_grat_reuniao
ORDER BY sch_sismapa.tb_servidor.cd_departamento ASC , sch_decom.tb_servidor.nm_servidor ASC
LIMIT 300 OFFSET 0

If you run the query directly in the database correctly returns the value ?

– Bulfaitelo

2019/01/28 at 12:36
Yes. It returns the value 10.

– douglas pjuizfora

2019/01/28 at 12:37
so do this, update the question with an example query, and tampem does it here: var_dump($servidores); but change the ->get()for ->toSql();

– Bulfaitelo

2019/01/28 at 12:38
Dude it returns the value 20 instead of 10. I said wrong to you. I’ll put the SQL in the text for you to see.

– douglas pjuizfora

2019/01/28 at 12:42
So both the eloquent and the direct sql it returns the wrong value?

– Bulfaitelo

2019/01/28 at 12:57
Yes. Both are returning wrong values @Bulfaitelo

– douglas pjuizfora

2019/01/28 at 13:08

Show 1 more comment

1 answer

Browser other questions tagged laravel postgresql

You are not signed in. Login or sign up in order to post.

by douglas pjuizfora • **162** points · Answer 1 · 2019-01-29T13:35:13+00:00

SOLUTION:

I actually had to rethink the strategy. Instead of doing a single query I made several querys because the error was in the leftjoin where I used the

COALESCE(SUM())

So I separated the querys and then added her result to my main object.

$servidores = DB::table('sch_sismapa.tb_servidor')
->join('sch_decom.tb_servidor', 'sch_sismapa.tb_servidor.nr_matricula', '=', 'sch_decom.tb_servidor.nr_matricula')
->leftJoin('sch_sismapa.tb_insalubridade','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_insalubridade.id_servidor')
->leftJoin('sch_sismapa.tb_periculosidade','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_periculosidade.id_servidor')
->leftJoin('sch_sismapa.tb_ferias','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_ferias.id_servidor')
->leftJoin('sch_sismapa.tb_ferias_premio','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_ferias_premio.id_servidor')
->leftJoin('sch_sismapa.tb_falta','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_falta.id_servidor')
->leftJoin('sch_sismapa.tb_vacancia','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_vacancia.id_servidor')
->leftJoin('sch_sismapa.tb_licenca_sem_vencimento','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_licenca_sem_vencimento.id_servidor')
->leftJoin('sch_sismapa.tb_licenca_medica','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_licenca_medica.id_servidor')
->leftJoin('sch_sismapa.tb_acidente_trabalho','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_acidente_trabalho.id_servidor')
->leftJoin('sch_sismapa.tb_penosidade_norm_a','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_penosidade_norm_a.id_servidor')
->leftJoin('sch_sismapa.tb_penosidade_norm_b','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_penosidade_norm_b.id_servidor')
->leftJoin('sch_sismapa.tb_penosidade_norm_c','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_penosidade_norm_c.id_servidor')
->leftJoin('sch_sismapa.tb_penosidade_norm_d','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_penosidade_norm_d.id_servidor')
->leftJoin('sch_sismapa.tb_penosidade_chefia_a','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_penosidade_chefia_a.id_servidor')
->leftJoin('sch_sismapa.tb_penosidade_chefia_b','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_penosidade_chefia_b.id_servidor')
->leftJoin('sch_sismapa.tb_penosidade_chefia_c','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_penosidade_chefia_c.id_servidor')
->leftJoin('sch_sismapa.tb_penosidade_chefia_d','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_penosidade_chefia_d.id_servidor')
->select('sch_decom.tb_servidor.nm_servidor','sch_sismapa.tb_servidor.*','sch_sismapa.tb_insalubridade.ds_insalubridade',
        'sch_sismapa.tb_periculosidade.ds_periculosidade',DB::raw('COUNT(sch_sismapa.tb_ferias.id_servidor) as ferias'),
        DB::raw('COUNT(sch_sismapa.tb_ferias_premio.id_servidor) as fereiasp'),DB::raw('COUNT(sch_sismapa.tb_falta.id_servidor) as faltas'),
        DB::raw('COUNT(sch_sismapa.tb_vacancia.id_servidor) as vacancias'),DB::raw('COUNT(sch_sismapa.tb_licenca_sem_vencimento.id_servidor) as lsv'),
        DB::raw('COUNT(sch_sismapa.tb_licenca_medica.id_servidor) as lm'),DB::raw('COUNT(sch_sismapa.tb_acidente_trabalho.id_servidor) as acitrab'),
        DB::raw('COUNT(sch_sismapa.tb_penosidade_norm_a.id_servidor) as pna'),DB::raw('COUNT(sch_sismapa.tb_penosidade_norm_b.id_servidor) as pnb'),
        DB::raw('COUNT(sch_sismapa.tb_penosidade_norm_c.id_servidor) as pnc'),DB::raw('COUNT(sch_sismapa.tb_penosidade_norm_d.id_servidor) as pnd'),
        DB::raw('COUNT(sch_sismapa.tb_penosidade_chefia_a.id_servidor) as pca'),DB::raw('COUNT(sch_sismapa.tb_penosidade_chefia_b.id_servidor) as pcb'),
        DB::raw('COUNT(sch_sismapa.tb_penosidade_chefia_c.id_servidor) as pcc'),DB::raw('COUNT(sch_sismapa.tb_penosidade_chefia_d.id_servidor) as pcd'))
->where('sch_sismapa.tb_servidor.cd_secretaria',$request->cd_secretaria)
->where('sch_sismapa.tb_servidor.id_referencia',$request->id_referencia)
->groupBy('sch_decom.tb_servidor.nm_servidor','sch_sismapa.tb_servidor.id','sch_sismapa.tb_servidor.id_referencia','sch_sismapa.tb_servidor.nr_matricula',
        'sch_sismapa.tb_servidor.cd_secretaria','sch_sismapa.tb_servidor.cd_departamento','sch_sismapa.tb_servidor.nr_dias',
        'sch_sismapa.tb_servidor.st_lancado','sch_sismapa.tb_servidor.st_aprovado','sch_sismapa.tb_servidor.ds_observacao',
        'sch_sismapa.tb_servidor.st_grat_reuniao','sch_sismapa.tb_insalubridade.ds_insalubridade','sch_sismapa.tb_periculosidade.ds_periculosidade')
->orderby('sch_sismapa.tb_servidor.cd_departamento', 'asc')
->orderBy('sch_decom.tb_servidor.nm_servidor', 'asc')
->skip($partes)
->take(300)
->get();

$jornadas = DB::table('sch_sismapa.tb_servidor')
                    ->join('sch_decom.tb_servidor', 'sch_sismapa.tb_servidor.nr_matricula', '=', 'sch_decom.tb_servidor.nr_matricula')
                    ->leftJoin('sch_sismapa.tb_extensao_ch','sch_sismapa.tb_servidor.id', '=', 'sch_sismapa.tb_extensao_ch.id_servidor')
                    ->select(DB::raw('COALESCE(SUM(sch_sismapa.tb_extensao_ch.nr_extensao),0) as jornada'))
                    ->where('sch_sismapa.tb_servidor.cd_secretaria',$request->cd_secretaria)
                    ->where('sch_sismapa.tb_servidor.id_referencia',$request->id_referencia)
                    ->groupBy('sch_decom.tb_servidor.nm_servidor','sch_sismapa.tb_servidor.id','sch_sismapa.tb_servidor.cd_secretaria','sch_sismapa.tb_servidor.cd_departamento','sch_sismapa.tb_servidor.nr_dias')
                    ->orderby('sch_sismapa.tb_servidor.cd_departamento', 'asc')
                    ->orderBy('sch_decom.tb_servidor.nm_servidor', 'asc')
                    ->skip($partes)
                    ->take(300)
                    ->get();

foreach ($servidores as $value => $servidor) {

$servidor->jornada = $jornadas[$value]->jornada;
}