VDB\Spider\Discoverer\Discoverer

Complexity

Total Complexity

2

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setDiscovererSet()	0	4	1
A	getName()	0	4	1
	discover()	0	1	?

<?php

namespace VDB\Spider\Discoverer;

use VDB\Spider\Resource;
use VDB\Spider\Uri\DiscoveredUri;

/**
 * @author Matthijs van den Bos
 * @copyright 2013 Matthijs van den Bos
 */
abstract class Discoverer
{
    /** @var DiscovererSet */
    protected $discovererSet;

    /**
     * @param DiscovererSet $discovererSet

Should the type for parameter $discovererSet not be null|DiscovererSet?

     */
    public function setDiscovererSet(DiscovererSet $discovererSet = null)
    {
        $this->discovererSet = $discovererSet;
    }

    public function getName()
    {
        return get_class($this);
    }

    abstract public function discover(Resource $resource);

For interfaces and abstract methods it is generally a good practice to add a @return annotation even if it is just @return void or @return null, so that implementors know what to do in the overridden method.

}