Я работал над некоторым 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. Поэтому я могу только читать данные из этой базы данных.
Где я могу улучшить это?
Спасибо