Ray\Di\Exception\Unbound

Complexity

Total Complexity

5

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes	5
Bugs	1	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__toString()	0	10	2
A	buildMessage()	0	13	2
A	getMainMessage()	0	8	1

<?php
/**
 * This file is part of the Ray.Di package.
 *
 * @license http://opensource.org/licenses/MIT MIT
 */
namespace Ray\Di\Exception;

use Ray\Di\Exception;

class Unbound extends \LogicException implements ExceptionInterface
{
    public function __toString()
    {
        $messages = [sprintf("- %s\n", $this->getMessage())];
        $e = $this->getPrevious();
        if (! $e instanceof \Exception) {
            return $this->getMainMessage($this);
        }

        return $this->buildMessage($e, $messages) . "\n" . $e->getTraceAsString();

$e of type object<Exception> is not a sub-type of object<Ray\Di\Exception\Unbound>. It seems like you assume a child class of the class Exception to be always present.

    }

    /**
     * @param Unbound  $e
     * @param string[] $msg
     *
     * @return string
     */
    private function buildMessage(Unbound $e, array $msg)
    {
        $lastE = $e;
        while ($e instanceof Unbound) {
            $msg[] = sprintf("- %s\n", $e->getMessage());
            $lastE = $e;
            $e = $e->getPrevious();
        }
        array_pop($msg);
        $msg = array_reverse($msg);

        return $this->getMainMessage($lastE) . implode('', $msg);
    }

    private function getMainMessage(Unbound $e)
    {
        return sprintf(
            "exception '%s' with message '%s'\n",
            get_class($e),
            $e->getMessage()
        );
    }
}