关于以下两种方案那种更好,大家有更好的方案也可以提供更优秀的

假设现在有一个业务场景是我需要下单购买商品,现在有两种处理方式

class Order
{
    public function createOrder(User $user, array $params)
    {
        $this->pay($user, $params);
        $this->insertOrder($user, $params);
        ...
    }

    private function pay(User $user, array $params)
    {        
    }

    private function insertOrder(User $user, array $params)
    {
    }
}

第二种方式就是像下边这样

class Order
{
    public function __construct(
        private User $user,
        private array $params,
    )
    {
    }

    public function createOrder()
    {
        $this->pay();
        $this->insertOrder();
        ...
    }

    private function pay()
    {

    }

    private function insertOrder()
    {
    }
}

上边两种方式,感觉第一个种使用方式对依赖注入更加友好,可以在构造函数中声明依赖关系,通过 Laravel 自动注入相关的对象。同时内部的方法也可以更加简单的让外部来调用。第二种方式使用让代码内部的逻辑更加聚合,但是感觉对依赖注入不是很友好。不知道大家有没有更加优秀的处理方案。

php
aab
《L05 电商实战》
从零开发一个电商项目,功能包括电商后台、商品 & SKU 管理、购物车、订单管理、支付宝支付、微信支付、订单退款流程、优惠券等
《L02 从零构建论坛系统》
以构建论坛项目 LaraBBS 为线索,展开对 Laravel 框架的全面学习。应用程序架构思路贴近 Laravel 框架的设计哲学。
讨论数量: 12
诺墨

建议使用前者,针对需要解耦的业务再拆解。后者维护起来比较费劲。反而没有体现依赖注入的价值

2年前 评论
aab (楼主) 2年前

我喜欢在 UsercreateOrder

class User
{
    public function createOrder(array $params)
    {
        // ...
    }
}
2年前 评论
aab (楼主) 2年前
pan_zoe 2年前
aab (楼主) 2年前

12结合下呢?

public function createOrder(User $user, array $params)
    {
        $this->setParms($user, $params)->pay()->insertOrder()
    }
2年前 评论
aab (楼主) 2年前
boolstone (作者) 2年前
class Order
{
    public function create($params)
    {
        DB::table('pay')->insert($params);
        DB::table('order')->insert($params);
        // .....
    }
}

不要问我为什么,能不能跑? :grin:

2年前 评论
IT学徒 2年前
她来听我的演唱会 2年前

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