V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX 提问指南
tlerbao
V2EX  ›  问与答

PHP 这种查询怎么能写的好看点,优雅点,高级点呢

  •  
  •   tlerbao · 100 天前 · 603 次点击
    这是一个创建于 100 天前的主题,其中的信息可能已经有所发展或是发生改变。

    代码如下(这是 thinkphp 写的):

    return $this->alias('cus')
                    ->join('admin a', 'cus.userID = a.id')
                    ->field('cus.userID,a.cjName,count(*) AS count')
                    ->when($month, function ($query) use ($month) {
                        $query->whereMonth('cjTime', $month);
                    },function ($query) {
                        $query->whereYear('cjTime', date('Y'));
                    })
                    ->when($brand_id, function ($query) use ($brand_id) {
                        $query->where('cus.cjbrandID', $brand_id);
                    })
                    ->when($shop_id, function ($query) use ($shop_id) {
                        $query->where('cus.cjMenDian', $shop_id);
                    })
                    ->group('cus.userID')
                    ->order('count desc')
                    ->limit(10)
                    ->select()->each(function ($item) {
                        // ....
                        return $item;
                    })->toArray();
    
    10 条回复    2022-10-27 20:51:18 +08:00
    westoy
        1
    westoy  
       100 天前
    封装进函数

    看不见的就是优雅的
    qinrui
        2
    qinrui  
       100 天前 via iPhone
    我用 medoo 查数据库,配合 smarty 模版引擎

    不会用 thinkphp 等框架
    sarices
        3
    sarices  
       100 天前
    直接在 MySQL 中建视图
    baobao1270
        4
    baobao1270  
       100 天前
    TP 已经比手写 SQL 好很多了
    不爽上 ORM
    Rache1
        5
    Rache1  
       100 天前
    添加 scope ,tp 里面叫搜索器,,把这些 when 操作放到里面去处理,后期改起来也方便一些,代码看上去就会简洁很多了。

    如果使用 PHP 7.4 及以上,你的代码可以更加简单一些。

    ->when($brand_id, fn ($query) => $query->where('cus.cjbrandID', $brand_id))
    ->when($shop_id, fn ($query) => $query->where('cus.cjMenDian', $shop_id))
    Rache1
        6
    Rache1  
       100 天前
    另外,不要把所有东西都揉到一起,应该分开来出来。比如你后面的 each ,如果是我,就会单独弄一个方法去做。
    xiaomada
        7
    xiaomada  
       100 天前
    scope+放弃全链接式调用
    xiaomada
        8
    xiaomada  
       100 天前
    ```
    $query = $this->alias('cus')->join('admin a', 'cus.userID = a.id');

    //条件 A
    $query->when($month, function ($query) use ($month) {
    $query->whereMonth('cjTime', $month);
    },function ($query) {
    $query->whereYear('cjTime', date('Y'));
    });

    //条件 B
    $query->when($brand_id, function ($query) use ($brand_id) {
    $query->where('cus.cjbrandID', $brand_id);
    });

    //条件 C
    $query->when($shop_id, function ($query) use ($shop_id) {
    $query->where('cus.cjMenDian', $shop_id);
    });

    //条件可以封装 scope
    $query->onlyshop($shop_id);

    //执行查询
    $models = $query->group('cus.userID')
    ->field('cus.userID,a.cjName,count(*) AS count')
    ->order('count desc')
    ->limit(10);

    //结果集处理
    $models->each(function ($item) {
    // ....
    return $item;
    });

    return $models;
    ```
    tlerbao
        9
    tlerbao  
    OP
       99 天前
    感谢楼上各位,还有没有大神指教
    tlerbao
        10
    tlerbao  
    OP
       99 天前
    @westoy 其实现在我只要不进模型,不该这部分代码,还是心静自然凉的哈哈。
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   广告投放   ·   实用小工具   ·   1543 人在线   最高记录 5497   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 38ms · UTC 08:31 · PVG 16:31 · LAX 00:31 · JFK 03:31
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.