Thinktomorrow\Chief\HealthMonitor\Monitor

Complexity

Total Complexity

3

Size/Duplication

Total Lines	16
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	check()	0	10	3

<?php

declare(strict_types = 1);

namespace Thinktomorrow\Chief\HealthMonitor;

use Thinktomorrow\Chief\HealthMonitor\Checks\HomepageCheck;
use Thinktomorrow\Chief\HealthMonitor\Notifiers\AlertBarNotifier;

class Monitor
{
    private static $checks = [
        HomepageCheck::class => AlertBarNotifier::class
    ];

    public static function check()
    {
        foreach(static::$checks as $check => $notifier)

Since $checks is declared private, accessing it with static will lead to errors in possible sub-classes; you can either use self, or increase the visibility of $checks to at least protected.

        {
            $checker = (new $check);
            $notifier = (new $notifier);
            
            if(!$checker->check())
            {
                return $notifier->notify($checker->notify());
            }
        } 
    }
}