GuardInvalidArgumentsTrait - Code Metrics - Inspection of "Merge pull request #4 from Miljar/feature/datamapp..." - PHPExif/php-exif-native - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 80d75d...d22e39 )

by Tom Van

created 2016-11-13 16:36 UTC

GuardInvalidArgumentsTrait A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	25
Duplicated Lines	100 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

Metric	Value
wmc	3
lcom	0
cbo	2
dl	25
loc	25
rs	10
c	0
b	0
f	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	guardInvalidArguments()	10	10	3

How to fix Duplicated Code

<?php
/**
 * Trait with method to validate arguments of ::mapField
 * from Exif\*Mapper classes
 *
 * @link        http://github.com/PHPExif/php-exif-native for the canonical source repository
 * @copyright   Copyright (c) 2016 Tom Van Herreweghe <[email protected]>
 * @license     http://github.com/PHPExif/php-exif-native/blob/master/LICENSE MIT License
 * @category    PHPExif
 * @package     Native
 */

namespace PHPExif\Adapter\Native\Reader\Mapper\Exif;

use PHPExif\Common\Data\ExifInterface;
use PHPExif\Common\Exception\Mapper\UnsupportedFieldException;
use PHPExif\Common\Exception\Mapper\UnsupportedOutputException;

/**
 * Common method to check the arguments of an Exif\* mapper
 *
 * @category    PHPExif
 * @package     Native
 */
trait GuardInvalidArgumentsTrait

{
    /**
     * Guard function for invalid arguments
     *
     * @param string $field
     * @param array $input
     * @param object $output
     *
     * @throws UnsupportedFieldException when asked to map an unsupported field
     * @throws UnsupportedOutputException when asked to map data on an unsupported output object
     *
     * @return void
     */
    private function guardInvalidArguments($field, array $input, $output)

    {
        if (!in_array($field, $this->getSupportedFields())) {
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            throw UnsupportedFieldException::forField($field);
        }

        if (!($output instanceof ExifInterface)) {
            throw UnsupportedOutputException::forOutput($output);
        }
    }
}


1		<?php
2		/**
3		* Trait with method to validate arguments of ::mapField
4		* from Exif\*Mapper classes
5		*
6		* @link http://github.com/PHPExif/php-exif-native for the canonical source repository
7		* @copyright Copyright (c) 2016 Tom Van Herreweghe <[email protected]>
8		* @license http://github.com/PHPExif/php-exif-native/blob/master/LICENSE MIT License
9		* @category PHPExif
10		* @package Native
11		*/
12
13		namespace PHPExif\Adapter\Native\Reader\Mapper\Exif;
14
15		use PHPExif\Common\Data\ExifInterface;
16		use PHPExif\Common\Exception\Mapper\UnsupportedFieldException;
17		use PHPExif\Common\Exception\Mapper\UnsupportedOutputException;
18
19		/**
20		* Common method to check the arguments of an Exif\* mapper
21		*
22		* @category PHPExif
23		* @package Native
24		*/
25	View Code Duplication	trait GuardInvalidArgumentsTrait
		0 ignored issues – show Duplication introduced 2016-11-11 11:17 UTC by Report Bug Copy Issue Report This class seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
26		{
27		/**
28		* Guard function for invalid arguments
29		*
30		* @param string $field
31		* @param array $input
32		* @param object $output
33		*
34		* @throws UnsupportedFieldException when asked to map an unsupported field
35		* @throws UnsupportedOutputException when asked to map data on an unsupported output object
36		*
37		* @return void
38		*/
39		private function guardInvalidArguments($field, array $input, $output)
		0 ignored issues – show Unused Code introduced 2016-11-03 15:13 UTC by Report Bug Copy Issue Report The parameter `$input` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
40		{
41		if (!in_array($field, $this->getSupportedFields())) {
		0 ignored issues – show Bug introduced 2016-11-03 15:13 UTC by Report Bug Copy Issue Report It seems like `getSupportedFields()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
42		throw UnsupportedFieldException::forField($field);
43		}
44
45		if (!($output instanceof ExifInterface)) {
46		throw UnsupportedOutputException::forOutput($output);
47		}
48		}
49		}
50

PHPExif / php-exif-native

GitHub Access Token became invalid

Push — develop ( 80d75d...d22e39 )

GuardInvalidArgumentsTrait A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

1 Method

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like