将PHP函数分解为单一任务的函数？

Question

将PHP函数分解为单一任务的函数？

3

我有一个函数，同时执行大约5个任务，这严重违反了面向对象编程的原则。有人能帮我将其拆分吗？特别是在其他函数中调用函数是一种很棘手的问题。

public function status(){
        $client = new Client();
        $notification=Notification::where('active',1)->get();
        $status = Status::where('name', 'health')->first();
        $default_frequency = 1;

        foreach ($notification as $note) {
            $status_health = $note->status('health');

            $check_frequency = isset($note->check_frequency) ? intval($note->check_frequency) : $default_frequency;

            $date = \Carbon\Carbon::parse($status_health['timestamp']);
            $elapsed_time = $date->diffInMinutes();

            if($elapsed_time < $check_frequency){
                continue;
            }

            $response = $client->get($note->website_url, ['http_errors' => false]);   
            $resCode = $response->getStatusCode();

            $note->statuses()->attach($status,['values'=> $resCode === 200 ? 'up' : 'down']);
        }
    }

- abraham foto

2

你可以将你想要分成独立方法的5个任务列出来，这样有人可能会帮助你。 - RiggsFolly

是的，因为这个类非常重要，我将会在不同的类中反复调用它的函数。这就是为什么我现在想要对它进行拆分。 - abraham foto

是的，我知道，但是请给我们列出您认为需要拆分成5个新方法的代码中的5个任务列表。我没有看到5个单独的任务，所以显然我很愚蠢，需要您的帮助。 - RiggsFolly

这个类负责发送1个HTTP请求，它会检查时间戳与当前时间以便发送HTTP请求。同时，它会附加一些内容（实际上是HTTP请求的一部分）。 - abraham foto

1个回答

网页内容由stack overflow 提供, 点击上面的

可以查看英文原文，
原文链接

- Armin Sam · Answer 1

我认为你不需要过多地优化代码。如果我理解正确，你正在尝试在一些指定的时间间隔内更新通知的状态。

这可以分为两个任务：

获取所有通知并循环遍历每一个
根据某些标准更新每个通知的状态

这是我尝试对你的代码进行简化和优化：

Class YourClass {

    const DEFAULT_FREQUENCY = 1;

    private $client;

    public function __construct(Client $clinet)
    {
        $this->client = $clinet;
    }

    public function status()
    {
        $notifications = Notification::where('active', 1)->get();
        $status = Status::where('name', 'health')->first();

        foreach ($notification as $notification) {
            $this->updateStatus($notification, $status);
        }
    }

    private function updateStatus(Notification $notification, Status $status)
    {
        $status_health = $notification->status('health');

        $frequency = $this->getFrequency($notification);

        $elapsed_time = \Carbon\Carbon::parse($status_health['timestamp'])->diffInMinutes();

        if ($elapsed_time >= $frequency) {
            $response = $this->client->get($notification->website_url, ['http_errors' => false]);   
            $notification->statuses()->attach($status, [
                'values'=> $response->getStatusCode() === 200 ? 'up' : 'down'
            ]);
        }
    }

    private function getFrequency(Notification $notification)
    {
        return isset($notification->check_frequency) 
            ? intval($notification->check_frequency) 
            : self::DEFAULT_FREQUENCY;
    }
}