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

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

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
《L01 基础入门》
我们将带你从零开发一个项目并部署到线上,本课程教授 Web 开发中专业、实用的技能,如 Git 工作流、Laravel Mix 前端工作流等。
《G01 Go 实战入门》
从零开始带你一步步开发一个 Go 博客项目,让你在最短的时间内学会使用 Go 进行编码。项目结构很大程度上参考了 Laravel。
讨论数量: 12
class Order
{
    public function create($params)
    {
        DB::table('pay')->insert($params);
        DB::table('order')->insert($params);
        // .....
    }
}

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

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

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

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年前

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