大家看看这样写查询有没有问题

问答 / 2003 / 2 / 创建于 7年前

$stat = Auth::user()->appsStat();

$allFlow = $stat->select(DB::raw('sum(flow) as result'))->value('result');

$tmpQuery = clone $stat;
$monthFlow = $tmpQuery->where('month', Carbon::now()->format('Y-m'))->select(DB::raw('sum(flow) as result'))->value('result');

$tmpQuery = clone $stat;
$weekFlow = $tmpQuery->where('week', Carbon::now()->format('Y-W'))->select(DB::raw('sum(flow) as result'))->value('result');

$tmpQuery = clone $stat;
$dayFlow = $tmpQuery->where('date', Carbon::now()->toDateString())->select(DB::raw('sum(flow) as result'))->value('result');

这是公司目前项目的后台管理系统，有一个界面，需要显示【总流水】【月流水】【周流水】【今日流水】，数据来源一张表 stat，关联 User Model。

蛋疼的是 where 语句和 groupBy 语句，在链式调用的影响下，会累加，所以我不得不每次查询前 clone 一个原始对象，虽然程序正常，但这样写感觉不优雅，各位有何解？

10 声望

十几年开发经验，已经记不得学过多少种语言，我要说：PHP是最好的开发语....

《L04 微信小程序从零到发布》

从小程序个人账户申请开始，带你一步步进行开发一个微信小程序，直到提交微信控制台上线发布。

《G01 Go 实战入门》

从零开始带你一步步开发一个 Go 博客项目，让你在最短的时间内学会使用 Go 进行编码。项目结构很大程度上参考了 Laravel。

推荐文章：

更多推荐...

失业两个月找存在感系列——几行代码轻松实现laravel链式追踪，一分钟快速排查系统问题 23 / 19 |

微信小程序、抖音小程序、百度小程序、支付宝小程序、快手小程序集合（支付、手机号解密、获取Token、模版消息、支付异步通知、退款、订单查询） 18 / 13 |

Git速查表 29 / 7 |

对线面试官：12个MySQL慢查询的原因分析 55 / 8 |

消息中间件应用的常见问题与方案 27 / 3 |

10个常见的Redis面试"刁难"问题 25 / 2 |

讨论数量: 2

young

课程读者 175 声望

要是我的话，我会把这些方法放到 appsStat model 里面或者 repository 里面，然后在控制器里面直接调用方法获取合计数量。

7年前评论

zhuzhichao

L5.3 译者 504 声望

Stat::my()->thisMonth()->select(Stat::flowDBRaw())->value('result');

Stat::my()->thisWeek()->select(Stat::flowDBRaw())->value('result');

Stat::my()->today()->select(Stat::flowDBRaw())->value('result');

写几个 scope, 以及简单的方法把 DB::raw('sum(flow) as result')给拿出来。这样会简洁一些。

楼下有什么好的方案没？

7年前评论

讨论应以学习和精进为目的。请勿发布不友善或者负能量的内容，与人为善，比聪明更重要！

帮助