MediaInfoContainer::add() - Code Metrics - Inspection of "Merge pull request #54 from mhor/analysis-Xpe3B6" - mhor/php-mediainfo - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — master (b4f9b9)

by Maxime

created 2016-05-18 20:35 UTC

MediaInfoContainer::add() C

↳ Parent: MediaInfoContainer

Complexity

Conditions	8
Paths	8

Size

Total Lines	28
Code Lines	25

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	27
CRAP Score	8

Importance

Changes	5
Bugs	2	Features	3

Metric	Value
c	5
b	2
f	3
dl	0
loc	28
ccs	27
cts	27
cp	1
rs	5.3846
cc	8
eloc	25
nc	8
nop	1
crap	8

<?php

namespace Mhor\MediaInfo\Container;

use Mhor\MediaInfo\DumpTrait;
use Mhor\MediaInfo\Type\AbstractType;
use Mhor\MediaInfo\Type\Audio;
use Mhor\MediaInfo\Type\General;
use Mhor\MediaInfo\Type\Image;
use Mhor\MediaInfo\Type\Menu;
use Mhor\MediaInfo\Type\Other;
use Mhor\MediaInfo\Type\Subtitle;
use Mhor\MediaInfo\Type\Video;

class MediaInfoContainer implements \JsonSerializable
{
    use DumpTrait;

    const GENERAL_CLASS = 'Mhor\MediaInfo\Type\General';
    const AUDIO_CLASS = 'Mhor\MediaInfo\Type\Audio';
    const IMAGE_CLASS = 'Mhor\MediaInfo\Type\Image';
    const VIDEO_CLASS = 'Mhor\MediaInfo\Type\Video';
    const SUBTITLE_CLASS = 'Mhor\MediaInfo\Type\Subtitle';
    const MENU_CLASS = 'Mhor\MediaInfo\Type\Menu';
    const OTHER_CLASS = 'Mhor\MediaInfo\Type\Other';

    /**
     * @var string
     */
    private $version;

    /**
     * @var General
     */
    private $general;

    /**
     * @var Audio[]
     */
    private $audios = array();

    /**
     * @var Video[]
     */
    private $videos = array();

    /**
     * @var Subtitle[]
     */
    private $subtitles = array();

    /**
     * @var Image[]
     */
    private $images = array();

    /**
     * @var Menu[]
     */
    private $menus = array();

    /**
     * @var Other[]
     */
    private $others = array();

    /**
     * @return General
     */
    public function getGeneral()
    {
        return $this->general;
    }

    /**
     * @return Audio[]
     */
    public function getAudios()
    {
        return $this->audios;
    }

    /**
     * @return Image[]
     */
    public function getImages()
    {
        return $this->images;
    }

    /**
     * @return Menu[]
     */
    public function getMenus()
    {
        return $this->menus;
    }

    /**
     * @return Other[]
     */
    public function getOthers()
    {
        return $this->others;
    }

    /**
     * @param string $version
     */
    public function setVersion($version)
    {
        $this->version = $version;
    }

    /**
     * @return string
     */
    public function getVersion()
    {
        return $this->version;
    }

    /**
     * @return Video[]
     */
    public function getVideos()
    {
        return $this->videos;
    }

    /**
     * @return Subtitle[]
     */
    public function getSubtitles()
    {
        return $this->subtitles;
    }

    /**
     * @param General $general
     */
    public function setGeneral($general)
    {
        $this->general = $general;
    }

    /**
     * @param AbstractType $trackType
     *
     * @throws \Exception
     */
    public function add(AbstractType $trackType)
    {
        switch (get_class($trackType)) {
            case self::AUDIO_CLASS:
                $this->addAudio($trackType);

                break;
            case self::VIDEO_CLASS:
                $this->addVideo($trackType);

                break;
            case self::IMAGE_CLASS:
                $this->addImage($trackType);

                break;
            case self::GENERAL_CLASS:
                $this->setGeneral($trackType);

                break;
            case self::SUBTITLE_CLASS:
                $this->addSubtitle($trackType);

                break;
            case self::MENU_CLASS:
                $this->addMenu($trackType);

                break;
            case self::OTHER_CLASS:
                $this->addOther($trackType);

                break;
            default:
                throw new \Exception('Unknown type');
        }
    }

    /**
     * @param Audio $audio
     */
    private function addAudio(Audio $audio)
    {
        $this->audios[] = $audio;
    }

    /**
     * @param Video $video
     */
    private function addVideo(Video $video)
    {
        $this->videos[] = $video;
    }

    /**
     * @param Image $image
     */
    private function addImage(Image $image)
    {
        $this->images[] = $image;
    }

    /**
     * @param Subtitle $subtitle
     */
    private function addSubtitle(Subtitle $subtitle)
    {
        $this->subtitles[] = $subtitle;
    }

    /**
     * @param Menu $menu
     */
    private function addMenu(Menu $menu)
    {
        $this->menus[] = $menu;
    }

    /**
     * @param Other $other
     */
    private function addOther(Other $other)
    {
        $this->others[] = $other;
    }
}


1		<?php
2
3		namespace Mhor\MediaInfo\Container;
4
5		use Mhor\MediaInfo\DumpTrait;
6		use Mhor\MediaInfo\Type\AbstractType;
7		use Mhor\MediaInfo\Type\Audio;
8		use Mhor\MediaInfo\Type\General;
9		use Mhor\MediaInfo\Type\Image;
10		use Mhor\MediaInfo\Type\Menu;
11		use Mhor\MediaInfo\Type\Other;
12		use Mhor\MediaInfo\Type\Subtitle;
13		use Mhor\MediaInfo\Type\Video;
14
15		class MediaInfoContainer implements \JsonSerializable
16		{
17		use DumpTrait;
18
19		const GENERAL_CLASS = 'Mhor\MediaInfo\Type\General';
20		const AUDIO_CLASS = 'Mhor\MediaInfo\Type\Audio';
21		const IMAGE_CLASS = 'Mhor\MediaInfo\Type\Image';
22		const VIDEO_CLASS = 'Mhor\MediaInfo\Type\Video';
23		const SUBTITLE_CLASS = 'Mhor\MediaInfo\Type\Subtitle';
24		const MENU_CLASS = 'Mhor\MediaInfo\Type\Menu';
25		const OTHER_CLASS = 'Mhor\MediaInfo\Type\Other';
26
27		/**
28		* @var string
29		*/
30		private $version;
31
32		/**
33		* @var General
34		*/
35		private $general;
36
37		/**
38		* @var Audio[]
39		*/
40		private $audios = array();
41
42		/**
43		* @var Video[]
44		*/
45		private $videos = array();
46
47		/**
48		* @var Subtitle[]
49		*/
50		private $subtitles = array();
51
52		/**
53		* @var Image[]
54		*/
55		private $images = array();
56
57		/**
58		* @var Menu[]
59		*/
60		private $menus = array();
61
62		/**
63		* @var Other[]
64		*/
65		private $others = array();
66
67		/**
68		* @return General
69		*/
70	5	public function getGeneral()
71		{
72	5	return $this->general;
73		}
74
75		/**
76		* @return Audio[]
77		*/
78	5	public function getAudios()
79		{
80	5	return $this->audios;
81		}
82
83		/**
84		* @return Image[]
85		*/
86	2	public function getImages()
87		{
88	2	return $this->images;
89		}
90
91		/**
92		* @return Menu[]
93		*/
94	1	public function getMenus()
95		{
96	1	return $this->menus;
97		}
98
99		/**
100		* @return Other[]
101		*/
102	1	public function getOthers()
103		{
104	1	return $this->others;
105		}
106
107		/**
108		* @param string $version
109		*/
110	4	public function setVersion($version)
111		{
112	4	$this->version = $version;
113	4	}
114
115		/**
116		* @return string
117		*/
118	1	public function getVersion()
119		{
120	1	return $this->version;
121		}
122
123		/**
124		* @return Video[]
125		*/
126	2	public function getVideos()
127		{
128	2	return $this->videos;
129		}
130
131		/**
132		* @return Subtitle[]
133		*/
134	2	public function getSubtitles()
135		{
136	2	return $this->subtitles;
137		}
138
139		/**
140		* @param General $general
141		*/
142	10	public function setGeneral($general)
143		{
144	10	$this->general = $general;
145	10	}
146
147		/**
148		* @param AbstractType $trackType
149		*
150		* @throws \Exception
151		*/
152	14	public function add(AbstractType $trackType)
153		{
154	14	switch (get_class($trackType)) {
155	14	case self::AUDIO_CLASS:
156	13	$this->addAudio($trackType);
		0 ignored issues – show Compatibility introduced 2015-12-03 14:36 UTC by Report Bug Copy Issue Report `$trackType` of type `object<Mhor\MediaInfo\Type\AbstractType>` is not a sub-type of `object<Mhor\MediaInfo\Type\Audio>`. It seems like you assume a child class of the class `Mhor\MediaInfo\Type\AbstractType` to be always present. This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass. Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type. Loading history...
157	13	break;
158	11	case self::VIDEO_CLASS:
159	1	$this->addVideo($trackType);
		0 ignored issues – show Compatibility introduced 2015-12-03 14:36 UTC by Report Bug Copy Issue Report `$trackType` of type `object<Mhor\MediaInfo\Type\AbstractType>` is not a sub-type of `object<Mhor\MediaInfo\Type\Video>`. It seems like you assume a child class of the class `Mhor\MediaInfo\Type\AbstractType` to be always present. This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass. Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type. Loading history...
160	1	break;
161	11	case self::IMAGE_CLASS:
162	1	$this->addImage($trackType);
		0 ignored issues – show Compatibility introduced 2015-12-03 14:36 UTC by Report Bug Copy Issue Report `$trackType` of type `object<Mhor\MediaInfo\Type\AbstractType>` is not a sub-type of `object<Mhor\MediaInfo\Type\Image>`. It seems like you assume a child class of the class `Mhor\MediaInfo\Type\AbstractType` to be always present. This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass. Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type. Loading history...
163	1	break;
164	11	case self::GENERAL_CLASS:
165	10	$this->setGeneral($trackType);
		0 ignored issues – show Compatibility introduced 2015-12-03 14:36 UTC by Report Bug Copy Issue Report `$trackType` of type `object<Mhor\MediaInfo\Type\AbstractType>` is not a sub-type of `object<Mhor\MediaInfo\Type\General>`. It seems like you assume a child class of the class `Mhor\MediaInfo\Type\AbstractType` to be always present. This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass. Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type. Loading history...
166	10	break;
167	5	case self::SUBTITLE_CLASS:
168	4	$this->addSubtitle($trackType);
		0 ignored issues – show Compatibility introduced 2015-12-03 14:36 UTC by Report Bug Copy Issue Report `$trackType` of type `object<Mhor\MediaInfo\Type\AbstractType>` is not a sub-type of `object<Mhor\MediaInfo\Type\Subtitle>`. It seems like you assume a child class of the class `Mhor\MediaInfo\Type\AbstractType` to be always present. This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass. Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type. Loading history...
169	4	break;
170	3	case self::MENU_CLASS:
171	1	$this->addMenu($trackType);
		0 ignored issues – show Compatibility introduced 2015-12-03 14:36 UTC by Report Bug Copy Issue Report `$trackType` of type `object<Mhor\MediaInfo\Type\AbstractType>` is not a sub-type of `object<Mhor\MediaInfo\Type\Menu>`. It seems like you assume a child class of the class `Mhor\MediaInfo\Type\AbstractType` to be always present. This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass. Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type. Loading history...
172	1	break;
173	2	case self::OTHER_CLASS:
174	1	$this->addOther($trackType);
		0 ignored issues – show Compatibility introduced 2015-12-03 14:36 UTC by Report Bug Copy Issue Report `$trackType` of type `object<Mhor\MediaInfo\Type\AbstractType>` is not a sub-type of `object<Mhor\MediaInfo\Type\Other>`. It seems like you assume a child class of the class `Mhor\MediaInfo\Type\AbstractType` to be always present. This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass. Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type. Loading history...
175	1	break;
176	1	default:
177	1	throw new \Exception('Unknown type');
178	14	}
179	13	}
180
181		/**
182		* @param Audio $audio
183		*/
184	13	private function addAudio(Audio $audio)
185		{
186	13	$this->audios[] = $audio;
187	13	}
188
189		/**
190		* @param Video $video
191		*/
192	1	private function addVideo(Video $video)
193		{
194	1	$this->videos[] = $video;
195	1	}
196
197		/**
198		* @param Image $image
199		*/
200	1	private function addImage(Image $image)
201		{
202	1	$this->images[] = $image;
203	1	}
204
205		/**
206		* @param Subtitle $subtitle
207		*/
208	4	private function addSubtitle(Subtitle $subtitle)
209		{
210	4	$this->subtitles[] = $subtitle;
211	4	}
212
213		/**
214		* @param Menu $menu
215		*/
216	1	private function addMenu(Menu $menu)
217		{
218	1	$this->menus[] = $menu;
219	1	}
220
221		/**
222		* @param Other $other
223		*/
224	1	private function addOther(Other $other)
225		{
226	1	$this->others[] = $other;
227	1	}
228		}
229

mhor / php-mediainfo

Branch — master (b4f9b9)

MediaInfoContainer::add() C

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like