Service::__construct() - Code Metrics - Inspection of "+ Add Hooks execution, add deciders" - chrl/myhome - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 3a87d5...4e429b )

by Kirill

created 2017-03-07 17:13 UTC

Service::__construct() A

↳ Parent: Service

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1

Importance

Changes

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

<?php
/**
 * Created by PhpStorm.
 * User: chrl
 * Date: 04/03/17
 * Time: 17:49
 */

namespace AppBundle\Action;

use AppBundle\Action\Executor\ExecutorInterface;
use AppBundle\Entity\Action;
use AppBundle\Entity\ActionHistory;
use Doctrine\Bundle\DoctrineBundle\Registry;

class Service
{

    private $doctrine;

    public function __construct(Registry $doctrine)
    {
        $this->doctrine = $doctrine;
    }

    private function getDoctrine()
    {
        return $this->doctrine;
    }


    public function executeVirtual(Action $action, $source, array $changeSet)
    {
        return $this->execute($action, $source, $changeSet, true);
    }

    public function executeReal(Action $action, $source, $arguments)
    {
        return $this->execute($action, $source, $arguments, false);
    }

    public function execute(Action $action, $source = 'internal', array $changeSet = [], $virtual = false)
    {

        //TODO: Find and run executor here

        if (!$virtual) {
            list($executor, $method) = explode(':', $action->getExecutor());

            $executor = 'AppBundle\Action\Executor\\' . ucfirst($executor);

            if (!class_exists($executor)) {
                throw new \Exception('Unknown executor: ' . $executor);
            }

            /** @var ExecutorInterface $executor */
            $executor = new $executor();
            $executor->setDoctrine($this->getDoctrine());

            if (isset($changeSet['container'])) {
                $executor->setContainer($changeSet['container']);
                unset($changeSet['container']);
            }

            if (!method_exists($executor, $method)) {
                throw new \Exception('Unknown executor method: ' . $action->getExecutor().'()');
            }

            $executor->setParameters($changeSet);

            try {
                $result = $executor->{$method}($action);
            } catch (\Exception $exception) {
                $result = $exception->getMessage();
            }
            $changeSet['result']=$result;
        }

        $state = new ActionHistory();
        $state->setAction($action);
        $state->setTime(new \DateTime());
        $state->setSource($source);
        $state->setPerformed(!$virtual);
        $state->setChangeSet(json_encode($changeSet));

        $device = $action->getDevice()->setState($changeSet);

        $this->getDoctrine()->getManagerForClass('AppBundle:ActionHistory')->persist($state);
        $this->getDoctrine()->getManagerForClass('AppBundle:Device')->persist($device);

        $this->getDoctrine()->getManager()->flush();

        return $this;
    }

    public function runAction($actionName, array $parameters)
    {
        $action = $this->doctrine->getRepository('AppBundle:Action')->findOneBy(['name'=>$actionName]);
        if (!$action) {
            throw new \Exception("Action ".$actionName." not found");
        }


        if (isset($parameter['container'])) {

            $args['container'] = $parameter['container'];
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
        }
        $this->executeReal($action, "internalcall", $parameters);
    }
}


1		<?php
2		/**
3		* Created by PhpStorm.
4		* User: chrl
5		* Date: 04/03/17
6		* Time: 17:49
7		*/
8
9		namespace AppBundle\Action;
10
11		use AppBundle\Action\Executor\ExecutorInterface;
12		use AppBundle\Entity\Action;
13		use AppBundle\Entity\ActionHistory;
14		use Doctrine\Bundle\DoctrineBundle\Registry;
15
16		class Service
17		{
18
19		private $doctrine;
20
21	4	public function __construct(Registry $doctrine)
22		{
23	4	$this->doctrine = $doctrine;
24	4	}
25
26		private function getDoctrine()
27		{
28		return $this->doctrine;
29		}
30
31
32		public function executeVirtual(Action $action, $source, array $changeSet)
33		{
34		return $this->execute($action, $source, $changeSet, true);
35		}
36
37		public function executeReal(Action $action, $source, $arguments)
38		{
39		return $this->execute($action, $source, $arguments, false);
40		}
41
42		public function execute(Action $action, $source = 'internal', array $changeSet = [], $virtual = false)
43		{
44
45		//TODO: Find and run executor here
46
47		if (!$virtual) {
48		list($executor, $method) = explode(':', $action->getExecutor());
49
50		$executor = 'AppBundle\Action\Executor\\' . ucfirst($executor);
51
52		if (!class_exists($executor)) {
53		throw new \Exception('Unknown executor: ' . $executor);
54		}
55
56		/** @var ExecutorInterface $executor */
57		$executor = new $executor();
58		$executor->setDoctrine($this->getDoctrine());
59
60		if (isset($changeSet['container'])) {
61		$executor->setContainer($changeSet['container']);
62		unset($changeSet['container']);
63		}
64
65		if (!method_exists($executor, $method)) {
66		throw new \Exception('Unknown executor method: ' . $action->getExecutor().'()');
67		}
68
69		$executor->setParameters($changeSet);
70
71		try {
72		$result = $executor->{$method}($action);
73		} catch (\Exception $exception) {
74		$result = $exception->getMessage();
75		}
76		$changeSet['result']=$result;
77		}
78
79		$state = new ActionHistory();
80		$state->setAction($action);
81		$state->setTime(new \DateTime());
82		$state->setSource($source);
83		$state->setPerformed(!$virtual);
84		$state->setChangeSet(json_encode($changeSet));
85
86		$device = $action->getDevice()->setState($changeSet);
87
88		$this->getDoctrine()->getManagerForClass('AppBundle:ActionHistory')->persist($state);
89		$this->getDoctrine()->getManagerForClass('AppBundle:Device')->persist($device);
90
91		$this->getDoctrine()->getManager()->flush();
92
93		return $this;
94		}
95
96		public function runAction($actionName, array $parameters)
97		{
98		$action = $this->doctrine->getRepository('AppBundle:Action')->findOneBy(['name'=>$actionName]);
99		if (!$action) {
100		throw new \Exception("Action ".$actionName." not found");
101		}
102
103
104		if (isset($parameter['container'])) {
		0 ignored issues – show Bug introduced 2017-03-04 22:25 UTC by Report Bug Copy Issue Report The variable `$parameter` does not exist. Did you mean `$parameters`? This check looks for variables that are accessed but have not been defined. It raises an issue if it finds another variable that has a similar name. The variable may have been renamed without also renaming all references. Loading history...
105		$args['container'] = $parameter['container'];
		0 ignored issues – show Coding Style Comprehensibility introduced 2017-03-04 22:25 UTC by Report Bug Copy Issue Report `$args` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$args = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
106		}
107		$this->executeReal($action, "internalcall", $parameters);
108		}
109		}
110

chrl / myhome

Push — master ( 3a87d5...4e429b )

Service::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like