BlueRegister\Log

Complexity

Total Complexity

9

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	__construct()	0	20	5
A	classExists()	0	7	2
A	makeLog()	0	7	2

<?php

namespace BlueRegister;

use \SimpleLog\LogInterface;

class Log
{
    /**
     * @var null|LogInterface
     */
    protected $log;

    /**
     * @param mixed $logObject
     * @throws \LogicException
     * @throws \InvalidArgumentException
     */
    public function __construct($logObject)
    {
        switch (true) {
            case $logObject instanceof LogInterface:
                $this->log = $logObject;
                break;

            case is_string($logObject) && $this->classExists($logObject):
                $this->log = new $logObject();

                if (!$this->log instanceof LogInterface) {
                    $message = 'Log should be instance of SimpleLog\LogInterface: ' . get_class($this->log);
                    throw new \LogicException($message);
                }

                break;

            default:
                throw new \LogicException('Cannot create Log instance: ' . get_class($logObject));

It seems like $logObject can also be of type string; however, parameter $object of get_class() does only seem to accept object, maybe add an additional type check?

                break;
        }
    }

    /**
     * check that class exists and throw exception if not
     *
     * @param string $namespace
     * @return $this
     * @throws \InvalidArgumentException
     */
    protected function classExists($namespace)
    {
        if (!class_exists($namespace)) {
            throw new \InvalidArgumentException('Class don\'t exists: ' . $namespace);
        }

        return $this;
    }

    /**
     * @param string|array $message
     * @return $this
     */
    public function makeLog($message)
    {
        if (!is_null($this->log)) {
            $this->log->makeLog($message);
        }

        return $this;
    }
}