Code Review 收获

今天请Ben对项目的重构Branch进行了Review。收获是很多的。

传入参数和返回值,尽量使用Interface

当使用match分支处理情况的时候,也要和switch类似,加上处理默认default的情况

避免使用动态生成的方法和类

动态的方法和类就是指名称包含变量的。比如

setFirstName(), setLastName()

这样的方法,为了偷懒,我使用set加上一个变量的名字的方法,比如 set$var()

这样的方法或者类不利于静态检查,应该避免使用。

Domain领域应该和外界尽可能的隔绝。

Domain内不该知道什么是REST,DB,gRPC等具体实现的技术概念

应该只使用传入的Interface的公开方法

简化复杂的DI的声明

使用静态分析工具 PhpStan