Timegridio\Concierge\Timetable\Strategies\TimetableStrategy

Complexity

Total Complexity

4

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	2
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	13	3
A	buildTimetable()	0	4	1

<?php

namespace Timegridio\Concierge\Timetable\Strategies;

use Timegridio\Concierge\Exceptions\StrategyNotRecognizedException;
use Timegridio\Concierge\Timetable\Timetable;

class TimetableStrategy
{
    /**
     * Timetable Strategy.
     *
     * @var TimetableTimeslotStrategy|TimetableDateslotStrategy
     */
    protected $strategy = null;

    /**
     * Construct Timetable Strategy class.
     *
     * @param string $strategyId
     *
     * @throws  Timegridio\Concierge\Exceptions\StrategyNotRecognizedException
     */
    public function __construct($strategyId)
    {
        switch ($strategyId) {
            case 'timeslot':
                $this->strategy = new TimetableTimeslotStrategy(new Timetable());
                break;
            case 'dateslot':
                $this->strategy = new TimetableDateslotStrategy(new Timetable());

It seems like new \Timegridio\Concierge\Timetable\Strategies\TimetableDateslotStrategy(new \Timegridio\Concierge\Timetable\Timetable()) of type object<Timegridio\Concierge\Timetable\Strategies\TimetableDateslotStrategy> is incompatible with the declared type object<Timegridio\Concierge\Timetable\Strategies\TimetableTimeslotStrategy> of property $strategy.

                break;
            default:
                throw new StrategyNotRecognizedException($strategyId);
        }
    }

    public function buildTimetable($vacancies, $starting = 'today', $days = 1)
    {
        return $this->strategy->buildTimetable($vacancies, $starting, $days);
    }
}