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

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

《L05 电商实战》
从零开发一个电商项目,功能包括电商后台、商品 & SKU 管理、购物车、订单管理、支付宝支付、微信支付、订单退款流程、优惠券等
《L03 构架 API 服务器》
你将学到如 RESTFul 设计风格、PostMan 的使用、OAuth 流程,JWT 概念及使用 和 API 开发相关的进阶知识。
讨论数量: 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年前 评论

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