TimetableStrategy::buildTimetable() - Code Metrics - Inspection of "Add BaseTimetableStrategy" - timegridio/concierge - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 566e3d...cbf465 )

by Ariel

created 2016-02-06 10:46 UTC

TimetableStrategy::buildTimetable() A

↳ Parent: TimetableStrategy

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	4
rs	10
cc	1
eloc	2
nc	1
nop	3

<?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());

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

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


1			<?php
2
3			namespace Timegridio\Concierge\Timetable\Strategies;
4
5			use Timegridio\Concierge\Exceptions\StrategyNotRecognizedException;
6			use Timegridio\Concierge\Timetable\Timetable;
7
8			class TimetableStrategy
9			{
10			/**
11			* Timetable Strategy.
12			*
13			* @var TimetableTimeslotStrategy\|TimetableDateslotStrategy
14			*/
15			protected $strategy = null;
16
17			/**
18			* Construct Timetable Strategy class.
19			*
20			* @param string $strategyId
21			*
22			* @throws Timegridio\Concierge\Exceptions\StrategyNotRecognizedException
23			*/
24			public function __construct($strategyId)
25			{
26			switch ($strategyId) {
27			case 'timeslot':
28			$this->strategy = new TimetableTimeslotStrategy(new Timetable());
29			break;
30			case 'dateslot':
31			$this->strategy = new TimetableDateslotStrategy(new Timetable());
			0 ignored issues – show Documentation Bug introduced 2016-02-06 10:52 UTC by Report Bug Copy Issue Report It seems like `new \Timegridio\Concierg...\Timetable\Timetable())` of type `object<Timegridio\Concie...etableDateslotStrategy>` is incompatible with the declared type `object<Timegridio\Concie...etableTimeslotStrategy>` of property `$strategy`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
32			break;
33			default:
34			throw new StrategyNotRecognizedException($strategyId);
35			}
36			}
37
38			public function buildTimetable($vacancies, $starting = 'today', $days = 1)
39			{
40			return $this->strategy->buildTimetable($vacancies, $starting, $days);
41			}
42			}
43

timegridio / concierge

Push — master ( 566e3d...cbf465 )

TimetableStrategy::buildTimetable() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like