php - 我是否正确使用了 OOP？

Question

我用 PHP 和 MySQL 开发一个 PBBG(游戏)已有 7 个月了。项目快完成了，可能很快就会发布，但我担心一些问题。

这是我第一次编写真正的项目，也是我第一次使用 OOP。我的代码现在有大约 25k 行，这是我的问题:

我是否正确使用了 OOP？我相信我过度调用类构造。我有大约 20 个不同的类(class)，有时他们需要彼此。这是我如何使用 OOP 的示例代码:(只是一个草图，可能不起作用)

class A {
    
    public $b;
    public $c;

    public function __construct(){
        $b = new B();
        $c = new C();
    }

    public function sampleA(){
        //some stuff that depends $c->sampleC
    }

}

class B { 

    public $c;

    public function __construct(){
        $c = new C();
    }

    public function sampleB($id){
        return $this->c->sampleC($id);
    }

}

class C {

    public function sampleC(){

    }

}

然后我会使用:

$a = new A();
$b = new B();
$c = new C();

$whatIWant = $a->sampleA($b->sampleB($c->sampleC()));

虽然我可以

$a = new A();
$whatIWant = $a->sampleA($a->b->sampleB($a->c->sampleC()));

在这个简单的例子中这看起来是不必要的，但我的脚本有超过 200 个不同的构造函数调用，我猜这会减慢我的服务器并使我的服务器过载。

也许我的例子不是很清楚，所以我打印了我的 xdebug 分析器结果:

我需要的是单例吗？我尝试使用像这样的简单示例与单例，但我得到了相同的结果(每个类超过 1 个构造)。
也许我需要扩展这些类并使用父类的构造函数？但是，有 20 个不同的类(class)，有时独立，有时相互依赖，这怎么可能呢？
或者也许我正在以正确的方式使用它(我不这么认为)

Answer 1

我不会那样做的。我会这样做的方法是使用依赖注入(inject)。我已将类中的实例设置为 protected(或将它们设置为 private)。无需将它们设置为公开。如果您认为您需要，那么您几乎肯定会违反某些规则，您需要重新考虑您的设计。

我改变的另一件事是你做了 $b = new B()。这有两个问题:

1. 我想你是想做 $this->b = new B()
2. 您正在将 B 类与另一个类紧密耦合。

这使得无法进行任何单元测试，因为您无法将 B 类替换为模拟类。因此，您不是在测试单元(类)，而是在测试多个类。

我所做的另一件事是对构造函数参数进行类型提示。当你有类的接口(interface)时，这会更有用。这也使得测试类变得更加容易。另见 this related answer .

您可能感兴趣的一些视频:Dependency Injection , Unit testing , Inheritance, Polymorphism, & Testing .您可能还想阅读一些关于 SOLID programming 的内容.您的代码似乎也违反了 LoD这意味着您正在尝试通过其他对象访问内容。

像这样的事情是我会做的:

class A
{
    protected $b;
    protected $c;

    public function __construct(B $b, C $c)
    {
        $this->b = $b;
        $this->c = $c;
    }

    public function sampleA()
    {
        $this->c->sampleC();
    }
}

class B
{     
    protected $c;

    public function __construct(C $c)
    {
        $this->c = $c;
    }

    public function sampleB($id)
    {
        return $this->c->sampleC($id);
    }
}

class C
{    
    public function sampleC() { }
}

$b = new B();
$c = new C($b);
$a = new A($b, $c);

$whatIWant = $a->sampleA();

如果您想编写正确的 OOP 代码，您不应该(恕我直言)使用 signleton 反模式。基本上，它只是作为 global 的另一个名称(具有相同的特征)。

Maybe I need to extend these classes and use parent's constructor? But how will it be possible with 20 different classes, sometimes independent, and sometimes dependent of each other?

只有当子类可以替代父类时，才应该扩展类。这称为 Liskov substitution principle在固体中。换句话说，您必须能够说 class child 是 class parent。