静态功能很糟糕 - 但是替代方案是什么?

For*_*ero 4 php oop static yii2

在我的示例中,我使用的是PHP框架Yii2,但我认为这适用于大多数OO语言.

我有一个ActiveRecord基类,我的大多数业务对象都是从例如Project.

目前,如果我想要一个Project我打电话的实例

Project::findOne(['id' => $id]);
Run Code Online (Sandbox Code Playgroud)

findOne是一个静态方法ActiveRecord(它是Yii2框架的一部分).所以这是不好的形式,因为在编写单元测试时我不能轻易地模拟/存根这个调用的返回.

但是解决这个问题的最佳方法是什么?

我可以创建一个CActiveRecord继承自ActiveRecord静态调用并在非静态调用中包装静态调用的类,并在任何地方使用它 - 但是我必须实例化一个抛弃Project对象才能获得实际的实例.如果Project对象需要实例化一些繁重的配置怎么办- 我会将随机的废话传递给构造函数只是为了得到一个实例.

简介: 简单地将静态更改为非静态似乎是错误的 - 我不应该将函数移动到其他地方吗?如果是的话,在哪里?

dec*_*eze 8

静态调用的问题是与特定其他代码段的硬耦合.只是在"动态"调用中包装它不会使这更好:

$c = new CProject;
$c->findOne(); // Calls Project::findOne()
Run Code Online (Sandbox Code Playgroud)

这非常毫无意义.问题不在于->vs 的语法::,问题是这个特定的代码引用了一个特定的其他类,并且你不能轻易地将这个类替换为其他类.您正在构建类/对象之间的刚性,硬编码依赖关系,这使得它们难以分开,这使得您的代码难以测试,并且这使得代码适应不同情况变得更加困难.

替代方案是依赖注入:

function foo(Project $project) {
    $p = $project->findOne();
}
Run Code Online (Sandbox Code Playgroud)

此函数不会耦合到任何一个特定的 Project类,而是耦合到一个只提供类似接口的类Project.事实上,Project甚至可能只是一个interface.在这里调用哪个特定的类和方法然后在某处完全不同,就像你的依赖注入容器一样; 或者只是这段代码的调用者.

这使得将这些代码分开并以不同方式将其重新组合在一起变得更加容易,这对于手头的情况是必要的.这并不是说它不起作用,你根本不应该使用静态调用,但你真的需要知道你在每个硬编码的类名中建立的交叉依赖关系,以及这是否可能导致一个问题就行了.因为即使中等复杂和/或增长的软件项目,这几乎肯定会导致某种形式的摩擦最终会.

有关更深入的文章,请参阅如何使用Statics杀死可测试性.

  • ActiveRecord本身就是一个问题.如果`Project`既是表示数据实例*的对象,也是数据库连接器本身,实际上没有好办法处理这个问题.至少你应该使用*Factory*.`$ projectFactory-> findOne()`返回一个`Project`实例.`ProjectFactory`本身将被实例化为`new ProjectFactory($ database)`.`$ database`是`PDO`或其他的实例.你只需要实例化一次`ProjectFactory`并将它传递给需要检索`Project`对象的每个函数/类.你看得到差别吗? (2认同)
  • 这样思考:您编写的每个类都完全独立于其他类。您永远不会直接引用任何其他特定的类。最多你在做一些类型提示。最好这些类型提示仅用于* interfaces *,而不是特定的其他类。现在,您有一堆可以像乐高积木一样放在一起的完全独立的类。现在,您需要非常具体和谨慎地选择那些乐高积木在哪些位置组合在一起。DI容器就是这样一个集中配置的位置,可以轻松地对其进行更改或更换。 (2认同)