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

$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 一个原始对象,虽然程序正常,但这样写感觉不优雅,各位有何解?

《L04 微信小程序从零到发布》
从小程序个人账户申请开始,带你一步步进行开发一个微信小程序,直到提交微信控制台上线发布。
《L02 从零构建论坛系统》
以构建论坛项目 LaraBBS 为线索,展开对 Laravel 框架的全面学习。应用程序架构思路贴近 Laravel 框架的设计哲学。
讨论数量: 2

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

8年前 评论
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')给拿出来。这样会简洁一些。

楼下有什么好的方案没?

8年前 评论

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