Как я могу стандартизировать некоторые SQL-запросы в Laravel? - PullRequest
0 голосов
/ 17 апреля 2019

Я работал над некоторым SQL-запросом, и он хорошо работает, но какой-то разработчик полного стека сказал мне, что это не лучшая практика, поэтому он не дает мне никакой обратной связи о том, как я могу стандартизировать этот!

Это моя модель функции.

function getExternalSchools( $school_group_id ) {
$schools = DB::connection( 'sqlsrv' )
            ->table( 'T_CLIENTS AS cli' )
            ->join( 'T_SE AS e', 'e.idse', '=', 'cli.idse' )
            ->join( 'T_CITY AS cid', 'cid.cod_city', '=', 'cli.cod_city' )
            ->leftJoin( 'T_CLIENTS_SP AS csp', 'cli.cod_client', '=', 'csp.cod_client', 'AND', 'cli.idse','=','csp.idse' )
            ->leftJoin( 'T_CLIENT_CONTRACT AS cc', 'cc.cod_client', '=', 'cli.cod_client' )
                ->select(                    
                    'cli.cod_client',
                    'cli.Status',
                    'cli.IU'                 
                )
                ->whereRaw(implode(' ', [
                    "cli.category = 'C'",
                    "AND (cli.status = 'A' OR ISNULL(csp.cod_pae,0) <> 0)",
                    "AND cli.idse IN (1, 2, 10)",
                    "AND cli.FisJur = 'J'",
                    "AND cli.ERP_OK <> 0",
                    "AND cli.Typo = ''",
                    "AND cli.Status = 'A'",
                    "AND cli.school_group_id = '{$school_group_id}'",
                ]))
                ->get();

}

Эта функция извлекает некоторые школы из внешней базы данных SQL SERVER. Поэтому я могу только читать данные из этой базы данных.

Где я могу улучшить это?

Спасибо

...