拆分大型 PHP 类

Question

我有一个大类（1500 行，但很快会是数倍），我想拆分它，以便它更适合SRP（并且每个文件更小，更易于管理。）

该类包含 50-100 个属性，并有几种不同类型的对其执行的操作 - 其中之一是 update，它依次执行多个步骤，例如更新数据库和发送电子邮件。

所以我想我想要 4 节课。

我应该如何构建这些类？

---

这是我现在所拥有的简化版本：

class Foo {
    public function __construct ($params) {}

    public function update () {
        $this->updateDatabase();
        $this->sendEmails();
    }

    private function updateDatabase () {}
    private function sendEmails () {}
}

$foo = new Foo($params);
$foo->update();

updateDatabase() 和 sendEmails () 都调用了许多其他方法 - 每个都有数百行代码，并且它们有几个同级方法执行其他任务。

使用静态方法的基本重写

class Foo {
    public function __construct ($params) {}
}

class FooUpdate {
    public static function update ($fooParam) {
        FooUpdateDatabase::main($fooParam);
        FooSendEmails::main($fooParam);
    }
}

class FooUpdateDatabase {
    public static function main ($fooParam) {}
}

class FooSendEmails {
    public static function main ($fooParam) {}
}

$foo = new Foo($params);
FooUpdate::update($foo);

使用实例化对象的基本重写

class Foo {
    public function __construct () {}
}

class FooUpdate {
    private $foo;
    public function __construct ($fooParam) {
        $this->foo = $fooParam;
    }
    public function main () {
        $fooTemp = FooUpdateDatabase($this->fooParam);
        $fooTemp->main();
        $fooTemp = FooSendEmails($this->fooParam);
        $fooTemp->main();
    }
}

class FooUpdateDatabase {
    private $foo;
    public function __construct ($fooParam) {
        $this->foo = $fooParam;
    }
    public function main () {}
}

class FooSendEmails {
    private $foo;
    public function __construct ($fooParam) {
        $this->foo = $fooParam;
    }
    public function main () {}
}

$foo = new Foo($bar, ...);
$fooTemp = new FooUpdate($foo);
$fooTemp->update();

或者我应该以某种方式使用继承或特征？

Answer 1

我想发送电子邮件是一回事，代表您的数据是另一回事，而对数据库的读写操作则是第三个。

所以class Foo、class FooPersistence、class FooMailer。
谁调用FooPersistence::update($foo)也应该调用FooMailer::sendUpdateNotification($foo)。

附注：
如果您设置了事件之类的东西，我会在持久性类中触发“更新事件”并为其添加一个发送电子邮件的侦听器。

Answer 2

正如@SparK 所说，

• 你的对象 (Foo)
• 处理数据库通信的类 (FooRepository)
• 一个发送邮件的类（Mailer）

• 一个包罗万象的类（FooManager）

  $foo = new Foo($params);
  $fooManager = new FooManager(FooRepository, Mailer);
  $fooManager->update($foo);
  $fooManager->notify($foo); //this could be inside the update or an event.
  

这样您还可以分解您的类（即：分离一个处理数据库连接的类并将其注入 FooRepository 等）。但我不认为有代表动作的类是要走的路吗？

类是可以执行操作的对象（除其他外），而不是操作（这只是一个注释，因为您在示例中使用了名称：p）。