Condition::applyRules() - Code Metrics - Inspection of "Added scrutinize & travis config files." - oliverde8/php-etl - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 610136...82575c )

by De Cramer

created 2018-03-23 13:45 UTC

Condition::applyRules() A

↳ Parent: Condition

Complexity

Conditions	1
Paths	1

Size

Total Lines	10
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	1

Importance

Changes

Metric	Value
cc	1
eloc	6
nc	1
nop	4
dl	0
loc	10
ccs	7
cts	7
cp	1
crap	1
rs	9.4285
c	0
b	0
f	0

<?php

namespace Oliverde8\Component\RuleEngine\Rules;

use Oliverde8\Component\RuleEngine\Exceptions\Condition\UnSupportedOperationException;
use Oliverde8\Component\RuleEngine\Exceptions\RuleException;

/**
 * Class Condition
 *
 * @author    de Cramer Oliver<[email protected]>
 * @copyright 2018 Oliverde8
 * @package Oliverde8\Component\RuleEngine\Rules
 */
class Condition extends AbstractRule
{
    /**
     * @inheritdoc
     */
    public function apply($rowData, &$transformedData, $options = [])
    {
        $valueToCmp = $this->applyRules($rowData, $transformedData, $options['if'], $options);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        $value = $this->applyRules($rowData, $transformedData, $options['value'], $options);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();

        $result = $this->compare($valueToCmp, $value, $options['operation'])? $options['then'] : $options['else'];

        return $this->applyRules($rowData, $transformedData, $result, $options);
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
if ($a->getObject()) {
    }

    /**
     * Compare a value.
     *
     * @param mixed $value      The value to compare.
     * @param mixed $with       The value to compare the value with.
     * @param string $operation The operation to use for the comparison.
     *
     * @throws UnSupportedOperationException
     *
     * @return bool
     */
    protected function compare($value, $with, $operation)
    {
        switch ($operation) {
            case 'eq' :
                return $value == $with;
            case 'neq' :
                return $value != $with;
            case 'in' :
                return in_array($value, $with);
            default:
                throw new UnSupportedOperationException("Operation '$operation' is not supported!");
        }
    }

    /**
     * Apply sub rules.
     *
     * @param array $rowData         Row to apply the transformation on
     * @param array $transformedData Data already transformed.
     * @param array $rules           Rule definition to apply.
     * @param array $options         Options for the rules.
     *
     * @return null
     * @throws RuleException
     */
    protected function applyRules($rowData, $transformedData, $rules, $options)
    {
        // Remove fields used by the current rule and not needed by childs.
        unset($options['if']);
        unset($options['value']);
        unset($options['operation']);
        unset($options['then']);
        unset($options['else']);

        return $this->ruleApplier->apply($rowData, $transformedData, $rules, $options);

    }

    /**
     * @inheritdoc
     */
    public function validate($options)
    {
        $this->requireOption('if', $options);
        $this->requireOption('value', $options);
        $this->requireOption('operation', $options);
        $this->requireOption('then', $options);
        $this->requireOption('else', $options);
    }

    /**
     * @inheritdoc
     */
    public function getRuleCode()
    {
        return 'condition';
    }
}

1		<?php
2
3		namespace Oliverde8\Component\RuleEngine\Rules;
4
5		use Oliverde8\Component\RuleEngine\Exceptions\Condition\UnSupportedOperationException;
6		use Oliverde8\Component\RuleEngine\Exceptions\RuleException;
7
8		/**
9		* Class Condition
10		*
11		* @author de Cramer Oliver<[email protected]>
12		* @copyright 2018 Oliverde8
13		* @package Oliverde8\Component\RuleEngine\Rules
14		*/
15		class Condition extends AbstractRule
16		{
17		/**
18		* @inheritdoc
19		*/
20	6	public function apply($rowData, &$transformedData, $options = [])
21		{
22	6	$valueToCmp = $this->applyRules($rowData, $transformedData, $options['if'], $options);
		0 ignored issues – show Bug introduced 2018-03-23 13:51 UTC by Report Bug Copy Issue Report Are you sure the assignment to `$valueToCmp` is correct as `$this->applyRules($rowDa...ptions['if'], $options)` targeting `Oliverde8\Component\Rule...Condition::applyRules()` seems to always return null. This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A { function getObject() { return null; } } $a = new A(); $object = $a->getObject(); The method `getObject()` can return nothing but null, so it makes no sense to assign that value to a variable. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
23	6	$value = $this->applyRules($rowData, $transformedData, $options['value'], $options);
		0 ignored issues – show Bug introduced 2018-03-23 13:51 UTC by Report Bug Copy Issue Report Are you sure the assignment to `$value` is correct as `$this->applyRules($rowDa...ons['value'], $options)` targeting `Oliverde8\Component\Rule...Condition::applyRules()` seems to always return null. This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A { function getObject() { return null; } } $a = new A(); $object = $a->getObject(); The method `getObject()` can return nothing but null, so it makes no sense to assign that value to a variable. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
24
25	6	$result = $this->compare($valueToCmp, $value, $options['operation'])? $options['then'] : $options['else'];
26
27	5	return $this->applyRules($rowData, $transformedData, $result, $options);
		0 ignored issues – show Bug introduced 2018-03-23 13:51 UTC by Report Bug Copy Issue Report Are you sure the usage of `$this->applyRules($rowDa...ata, $result, $options)` targeting `Oliverde8\Component\Rule...Condition::applyRules()` seems to always return null. This check looks for function or method calls that always return null and whose return value is used. class A { function getObject() { return null; } } $a = new A(); if ($a->getObject()) { The method `getObject()` can return nothing but null, so it makes no sense to use the return value. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
28		}
29
30		/**
31		* Compare a value.
32		*
33		* @param mixed $value The value to compare.
34		* @param mixed $with The value to compare the value with.
35		* @param string $operation The operation to use for the comparison.
36		*
37		* @throws UnSupportedOperationException
38		*
39		* @return bool
40		*/
41	6	protected function compare($value, $with, $operation)
42		{
43		switch ($operation) {
44	6	case 'eq' :
45	2	return $value == $with;
46	4	case 'neq' :
47	1	return $value != $with;
48	3	case 'in' :
49	2	return in_array($value, $with);
50		default:
51	1	throw new UnSupportedOperationException("Operation '$operation' is not supported!");
52		}
53		}
54
55		/**
56		* Apply sub rules.
57		*
58		* @param array $rowData Row to apply the transformation on
59		* @param array $transformedData Data already transformed.
60		* @param array $rules Rule definition to apply.
61		* @param array $options Options for the rules.
62		*
63		* @return null
64		* @throws RuleException
65		*/
66	6	protected function applyRules($rowData, $transformedData, $rules, $options)
67		{
68		// Remove fields used by the current rule and not needed by childs.
69	6	unset($options['if']);
70	6	unset($options['value']);
71	6	unset($options['operation']);
72	6	unset($options['then']);
73	6	unset($options['else']);
74
75	6	return $this->ruleApplier->apply($rowData, $transformedData, $rules, $options);
		0 ignored issues – show Bug Best Practice introduced 2018-03-23 13:51 UTC by Report Bug Copy Issue Report The expression `return $this->ruleApplie...Data, $rules, $options)` also could return the type `string` which is incompatible with the documented return type `null`. Loading history...
76		}
77
78		/**
79		* @inheritdoc
80		*/
81	6	public function validate($options)
82		{
83	6	$this->requireOption('if', $options);
84	5	$this->requireOption('value', $options);
85	5	$this->requireOption('operation', $options);
86	5	$this->requireOption('then', $options);
87	5	$this->requireOption('else', $options);
88	5	}
89
90		/**
91		* @inheritdoc
92		*/
93	2	public function getRuleCode()
94		{
95	2	return 'condition';
96		}
97		}

oliverde8 / php-etl

Push — master ( 610136...82575c )

Condition::applyRules() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like