减少laravel中的数据库查询大小

Question

我期待在laravel中减少我的查询大小。

我的查询看起来是这样的（我缩短了它，它是这一数额线的10倍左右）：

$users = User::where("interface_art", '=', 1)->where('role', '=', 2)->where('commstatus', '=', $unavailableCheck) 
->orWhere("interface_art", '=', 1)->where('role', '=', 2)->where('commstatus', '=', 1) 
->orWhere("web_art", '=', 1)->where('role', '=', 2)->where('commstatus', '=', $unavailableCheck) 
->orWhere("web_art", '=', 1)->where('role', '=', 2)->where('commstatus', '=', 1)      
->orWhere("illustration_art", '=', 1)->where('role', '=', 2)->where('commstatus', '=', $unavailableCheck) 
->orWhere("illustration_art", '=', 1)->where('role', '=', 2)->where('commstatus', '=', 1)     
->orWhere("brush_art", '=', 1)->where('role', '=', 2)->where('commstatus', '=', $unavailableCheck) 
->orWhere("brush_art", '=', 1)->where('role', '=', 2)->where('commstatus', '=', 1)     
->orWhere("typography_art", '=', 1)->where('role', '=', 2)->where('commstatus', '=', $unavailableCheck) 
->orWhere("typography_art", '=', 1)->where('role', '=', 2)->where('commstatus', '=', 1)     
->orWhere("identity_art", '=', 1)->where('role', '=', 2)->where('commstatus', '=', $unavailableCheck) 
->orWhere("identity_art", '=', 1)->where('role', '=', 2)->where('commstatus', '=', 1)     
->orWhere("vector_art", '=', 1)->where('role', '=', 2)->where('commstatus', '=', $unavailableCheck) 
->orWhere("vector_art", '=', 1)->where('role', '=', 2)->where('commstatus', '=', 1)     
->orderBy($orderByString, 'desc') 
->paginate(1); 

正如你可以看到，这是一个有点多余。

对于每种艺术类型，如果它们的commstatus等于“1”或“$ unavailable”，我希望通过“2”角色获取用户。起初，我试图通过在每个“where”子句末尾添加“role”或“commstatus”来缩短它，并在底部编写另一个$ users = $ users :: where（“role” ，“=”，“2”），但我似乎无法找到正确的语法。

有什么办法可以缩短这个查询吗？

Answer 1

正如其他人所指出的，你的模式可能会从实现正常化重构中受益。不过，我相信你仍然可以重构现有的查询，减少冗余并提高可读性。

Laravel有能力处理Advanced Where Clauses其中包括nested parameter groupings。从文档：

Closure将收到一个查询生成器实例，您可以使用它来设置应包含在括号组中的约束。

考虑到这一点，你应该能够重构像这样的查询：

$users = User::whereIn('commstatus', [$unavailableCheck, 1]) 
    ->where('role', 2) 
    ->where(function ($query) { 
     $query->where("interface_art", 1) 
      ->orWhere("web_art", 1) 
      ->orWhere("illustration_art", 1) 
      ->orWhere("brush_art", 1) 
      ->orWhere("typography_art", 1) 
      ->orWhere("identity_art", 1) 
      ->orWhere("vector_art", 1); 
    }) 
    ->orderBy($orderByString, 'desc') 
    ->paginate(1); 

这将创建一个SQL查询的作用：

SELECT * FROM users 
    WHERE commstatus IN ($unavailableCheck, 1) 
    AND role = 2 
    AND (interface_art = 1 OR illustration_art = 1 OR ... etc) 

Answer 2

对于每种艺术类型，您肯定不需要复制where('role', '=', 2)->where('commstatus', '=', $unavailableCheck)，因为它们是有效的与条件;并考虑whereIn('commstatus', [$unavailableCheck, 1])而不是有两个平等检查。

喜欢的东西：

$users = User::where('role', '=', 2) 
    ->whereIn('commstatus', [$unavailableCheck, 1]) 
    ->where("interface_art", '=', 1) 
    ->orWhere("web_art", '=', 1) 
    ->orWhere("illustration_art", '=', 1) 
    ->orWhere("brush_art", '=', 1) 
    ->orWhere("typography_art", '=', 1) 
    ->orWhere("identity_art", '=', 1) 
    ->orWhere("vector_art", '=', 1) 
    ->orderBy($orderByString, 'desc') 
    ->paginate(1);