HansOtt\Lastify\TrackInfo

Complexity

Total Complexity

13

Size/Duplication

Total Lines	65
Duplicated Lines	38.46 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

64.71%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	assertValidName()	9	9	3
B	assertThatTheseAreAllArtists()	16	16	5
A	getName()	0	4	1
A	getArtists()	0	4	1
A	toString()	0	8	1
A	__toString()	0	4	1

<?php

namespace HansOtt\Lastify;

use InvalidArgumentException;
use HansOtt\Lastify\TrackInfo\Artist;

final class TrackInfo
{
    private $name;

    private $artists;

    public function __construct($name, array $artists = [])
    {
        $this->assertValidName($name);
        $this->assertThatTheseAreAllArtists($artists);
        $this->name = $name;
        $this->artists = $artists;
    }

    private function assertValidName($name)

This method seems to be duplicated in your project.

    {
        if (!is_string($name)) {
            throw new InvalidArgumentException('The name should be a string, instead got:' . gettype($name));
        }
        if (empty($name)) {
            throw new InvalidArgumentException('The name cannot be empty');
        }
    }

    private function assertThatTheseAreAllArtists(array $artists)

This method seems to be duplicated in your project.

    {
        if (empty($artists)) {
            throw new InvalidArgumentException('A track should always have an artist');
        }

        foreach ($artists as $artist) {
            if (! ($artist instanceof Artist)) {
                throw new \InvalidArgumentException(sprintf(
                    'Expected "%s", but instead got: "%s"',
                    Artist::class,
                    is_object($artist) ? get_class($artist) : gettype($artist)
                ));
            }
        }
    }

    public function getName()
    {
        return $this->name;
    }

    public function getArtists()
    {
        return $this->artists;
    }

    public function toString()
    {
        $artists = array_map(function (Artist $artist) {
            return $artist->getName();
        }, $this->artists);

        return sprintf('%s - %s', $this->name, implode(', ', $artists));
    }

    public function __toString()
    {
        return $this->toString();
    }
}