App\Services\AboutMeService

Complexity

Total Complexity

4

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	hasContent()	0	4	1
A	getHtml()	0	4	1
A	getContent()	0	4	1

<?php

namespace App\Services;

use App\Interfaces\AboutMeServiceInterface;

class AboutMeService implements AboutMeServiceInterface
{
    /** @var string */
    protected $content = '';

    public function __construct()
    {
        $this->content = $this->getContent();

It seems like $this->getContent() can also be of type boolean. However, the property $content is declared as type string. Maybe add an additional type check?

    }

    /**
     * Check if there's content to show.
     *
     * @return bool
     */
    public function hasContent()
    {
        return $this->content != '';
    }

    /**
     * Get the content parsed into html.
     *
     * @return string
     */
    public function getHtml()
    {
        return (new \Parsedown())->text($this->content);
    }

    /**
     * Get the text from the specified resource.
     *
     * @return bool|string
     */
    public function getContent()
    {
        return @file_get_contents(resource_path('about_me.md'));
    }
}