PHP：単一のタスクで関数を関数に分割する？

Question

私はOOPの原則に厳密に従っている約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']); 
     } 
    } 

Answer 1

コードをあまりに細かく修正する必要はありません。私が正しく理解していれば、特定の間隔で通知のステータスを更新しようとしています。

この二つのタスクに侵入することができます

1. は、ここではいくつかの基準

に基づいて、各通知のステータスが私である各そのうちの一つ

Updateを通じてすべての通知およびループの取得

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; 
    } 
}