App\Classes\Breadcrumbs

Complexity

Total Complexity

12

Size/Duplication

Total Lines	121
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	addCrumb()	0	10	1
A	filter()	0	4	1
A	crumbs()	0	4	1
A	contain()	0	7	2
A	hasCount()	0	4	1
A	limit()	0	15	3
A	fromCurrentRoute()	0	22	2

<?php
/**
 * Created by PhpStorm.
 * User: Marky
 * Date: 04/01/2018
 * Time: 01:28
 */

namespace App\Classes;

use Illuminate\Http\Request;
use Illuminate\Support\Collection;

/**
 * Class Breadcrumbs
 *
 * @package App\Classes
 */
class Breadcrumbs
{

    /**
     * A collection of crumbs.
     *
     * @var Collection
     */
    protected $collection;

    /**
     * Breadcrumbs constructor.
     * @param Collection $collection
     */
    public function __construct(Collection $collection)
    {
        $this->collection = $collection;
    }

    /**
     * @param string $title
     * @param string $path
     * @return bool
     */
    public function addCrumb(string $title, string $path)
    {
        $crumb = new \stdClass();
        $crumb->title = $this->filter($title);
        $crumb->path = url($path);

        $this->collection->push($crumb);

        return true;
    }

    /**
     * @param string $title
     * @return string
     */
    public function filter(string $title)
    {
        return ucwords(str_replace(['_', '-'], [' ', ' '], $title));
    }

    /**
     * @return Collection
     */
    public function crumbs()
    {
        return $this->collection;
    }

    /**
     * @param string $name
     * @param int $position
     * @return bool
     */
    public function contain(string $name, int $position)
    {
        if ($this->collection->has($position) == false)

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

            return false;

        return $this->collection->get($position)->title == $this->filter($name);
    }

    /**
     * @param int $integer
     * @return bool
     */
    public function hasCount(int $integer)
    {
        return $this->collection->count() == $integer;
    }

    /**
     * @param int $count
     * @return $this
     */
    public function limit(int $count)
    {
        $collection = new Collection;

        for ($i = 0; $i < $count; $i++)
        {
            if ($this->collection->has($i)) {
                $collection->push($this->collection->get($i));
            }
        }

        $this->collection = $collection;

        return $this;
    }

    /**
     * @return Breadcrumbs
     */
    public static function fromCurrentRoute()
    {
        /** @var Breadcrumbs $instance */
        $instance = app(self::class);

        /** @var array $routes */
        $routes = explode('/', app(Request::class)->getRequestUri());

        $urlPath = url(array_pull($routes, 0));

        // Add the home array.
        $instance->addCrumb('Home', $urlPath);

It seems like $urlPath defined by url(array_pull($routes, 0)) on line 124 can also be of type object<Illuminate\Contracts\Routing\UrlGenerator>; however, App\Classes\Breadcrumbs::addCrumb() does only seem to accept string, maybe add an additional type check?

        foreach ($routes as $route)
        {
            $urlPath = $urlPath . '/' . $route;

            $instance->addCrumb($route, $urlPath);
        }

        return $instance;
    }

}