HansOtt\Lastify\TrackCollection

Complexity

Total Complexity

11

Size/Duplication

Total Lines	69
Duplicated Lines	17.39 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

41.38%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	assertThatTheseAreAllTracks()	12	12	4
A	current()	0	4	1
A	next()	0	4	1
A	key()	0	4	1
A	valid()	0	4	1
A	rewind()	0	4	1
A	createSyncProgress()	0	6	1

<?php

namespace HansOtt\Lastify;

use Iterator;

final class TrackCollection implements Iterator
{
    private $tracks;

    private $position = 0;

    /**
     * TrackCollection constructor.
     *
     * @param array $tracks An array of TrackInfo objects.
     */
    public function __construct(array $tracks = [])
    {
        $this->assertThatTheseAreAllTracks($tracks);
        $this->tracks = $tracks;
    }

    private function assertThatTheseAreAllTracks(array $tracks)

This method seems to be duplicated in your project.

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

    public function current()
    {
        return $this->tracks[$this->position];
    }

    public function next()
    {
        $this->position++;
    }

    public function key()
    {
        return $this->position;
    }

    public function valid()
    {
        return isset($this->tracks[$this->position]);
    }

    public function rewind()
    {
        $this->position = 0;
    }

    /**
     * Create a sync progress.
     *
     * @param SyncProgressCallback|null $callback
     *
     * @return SyncProgress
     */
    public function createSyncProgress(SyncProgressCallback $callback = null)
    {
        $total = count($this->tracks);

        return new SyncProgress(0, $total, $callback);
    }
}