Speicher210\BusinessHours\DayBuilder

Complexity

Total Complexity

22

Size/Duplication

Total Lines	68
Duplicated Lines	17.65 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
C	fromArray()	0	27	11
C	isIntervalArrayAllDay()	12	25	11

<?php

namespace Speicher210\BusinessHours;

/**
 * Build a DayInterface concrete implementation.
 */
class DayBuilder
{
    /**
     * Create a DayInterface object from an array.
     *
     * @param array $data The day data.
     * @return DayInterface
     */
    public static function fromArray(array $data)
    {
        if (!isset($data['openingIntervals']) || !is_array($data['openingIntervals']) || !isset($data['dayOfWeek'])) {
            throw new \InvalidArgumentException('Array is not valid.');
        }

        if (count($data['openingIntervals']) === 1 && self::isIntervalArrayAllDay(reset($data['openingIntervals']))) {
            return new AllDay($data['dayOfWeek']);

$data['dayOfWeek'] is of type array, but the function expects a integer.

        }

        $openingIntervals = array();
        foreach ($data['openingIntervals'] as $openingInterval) {
            $start = new Time(
                $openingInterval['start']['hours'],
                isset($openingInterval['start']['minutes']) ? $openingInterval['start']['minutes'] : 0,
                isset($openingInterval['start']['seconds']) ? $openingInterval['start']['seconds'] : 0
            );
            $end = new Time(
                $openingInterval['end']['hours'],
                isset($openingInterval['end']['minutes']) ? $openingInterval['end']['minutes'] : 0,
                isset($openingInterval['end']['seconds']) ? $openingInterval['end']['seconds'] : 0
            );
            $openingIntervals[] = array($start, $end);
        }

        return new Day($data['dayOfWeek'], $openingIntervals);

$data['dayOfWeek'] is of type array, but the function expects a integer.

    }

    /**
     * Check if an interval array is all day.
     *
     * @param array $data The interval as an array to check.
     * @return boolean
     */
    private static function isIntervalArrayAllDay(array $data)
    {
        if ($data['start']['hours'] != 0 || $data['end']['hours'] != 24) {
            return false;
        }

        if (isset($data['start']['minutes']) && $data['start']['minutes'] != 0) {

This code seems to be duplicated across your project.

            return false;
        }

        if (isset($data['end']['minutes']) && $data['end']['minutes'] != 0) {

This code seems to be duplicated across your project.

            return false;
        }


        if (isset($data['start']['seconds']) && $data['start']['seconds'] != 0) {

This code seems to be duplicated across your project.

            return false;
        }

        if (isset($data['end']['seconds']) && $data['end']['seconds'] != 0) {

This code seems to be duplicated across your project.

            return false;
        }

        return true;
    }
}