Xhgui_ServiceContainer::_services() - Code Metrics - Inspection of "WIP: Backend decoupling" - perftools/xhgui - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#244)

by Marcel

created 2018-07-13 21:06 UTC

Xhgui_ServiceContainer::_services() B

↳ Parent: Xhgui_ServiceContainer

Complexity

Conditions	4
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	55
rs	8.9818
c	0
b	0
f	0
cc	4
nc	1
nop	0

How to fix Long Method

<?php
use Slim\Slim;
use Slim\Views\Twig;
use Slim\Middleware\SessionCookie;

class Xhgui_ServiceContainer extends Pimple
namespace YourVendor;

class YourClass { }
{
    protected static $_instance;

    public static function instance()
    {
        if (empty(static::$_instance)) {
            static::$_instance = new self();
        }
        return static::$_instance;
    }

    public function __construct()
    {
        $this->_slimApp();
        $this->_services();
        $this->_controllers();
class User
{
    private $email;

    public function getEmail()
    {
        return $this->email;
    }

    public function setEmail($email)
    {
        $this->email = $email;
    }
}
    }

    // Create the Slim app.
    protected function _slimApp()
    {
        $this['view'] = function ($c) {
            $cacheDir = isset($c['config']['cache']) ? $c['config']['cache'] : XHGUI_ROOT_DIR . '/cache';

            // Configure Twig view for slim
            $view = new Twig();

            $view->parserOptions = array(
                'charset' => 'utf-8',
                'cache' => $cacheDir,
                'auto_reload' => true,
                'strict_variables' => false,
                'autoescape' => true
            );

            return $view;
        };

        $this['app'] = $this->share(function ($c) {
            $app = new Slim($c['config']);

            // Enable cookie based sessions
            $app->add(new SessionCookie(array(
                'httponly' => true,
            )));

            // Add renderer.
            $app->add(new Xhgui_Middleware_Render());

            $view = $c['view'];
            $view->parserExtensions = array(
                new Xhgui_Twig_Extension($app)
            );
            $app->view($view);

            return $app;
        });
    }

    /**
     * Add common service objects to the container.
     */
    protected function _services()
    {
        $this['config'] = Xhgui_Config::all();

        $this['db'] = $this->share(function ($c) {
            $config = $c['config'];
            if (empty($config['db.options'])) {
                $config['db.options'] = array();
            }
            $mongo = new MongoClient($config['db.host'], $config['db.options']);
            $mongo->{$config['db.db']}->results->findOne();

            return $mongo->{$config['db.db']};
        });

        $this['pdo'] = $this->share(function ($c) {
            return new PDO(
                $c['config']['pdo']['dsn'],
                $c['config']['pdo']['pass'],
                $c['config']['pdo']['user']
            );
        });

        $this['searcher.mongo'] = function ($c) {
            return new Xhgui_Searcher_Mongo($c['db']);
        };

        $this['searcher.pdo'] = function ($c) {
            return new Xhgui_Searcher_Pdo($c['pdo'], $c['config']['pdo']['table']);
        };

        $this['searcher'] = function ($c) {
            $config = $c['config'];

            switch ($config['save.handler']) {
                case 'pdo':
                    return $c['searcher.pdo'];

                case 'mongodb':
                default:
                    return $c['searcher.mongo'];
            }
        };

        $this['saver.mongo'] = function($c) {
            $config = $c['config'];
            $config['save.handler'] = 'mongodb';

            return Xhgui_Saver::factory($config);
        };

        $this['saver'] = function($c) {
            return Xhgui_Saver::factory($c['config']);
        };
    }

    /**
     * Add controllers to the DI container.
     */
    protected function _controllers()
    {
        $this['watchController'] = function ($c) {
            return new Xhgui_Controller_Watch($c['app'], $c['searcher']);
        };

        $this['runController'] = function ($c) {
            return new Xhgui_Controller_Run($c['app'], $c['searcher']);
        };

        $this['customController'] = function ($c) {
            return new Xhgui_Controller_Custom($c['app'], $c['searcher']);
        };

        $this['waterfallController'] = function ($c) {
            return new Xhgui_Controller_Waterfall($c['app'], $c['searcher']);
        };

        $this['importController'] = function ($c) {
            return new Xhgui_Controller_Import($c['app'], $c['saver']);
        };
    }

}


1			<?php
2			use Slim\Slim;
3			use Slim\Views\Twig;
4			use Slim\Middleware\SessionCookie;
5
6			class Xhgui_ServiceContainer extends Pimple
			0 ignored issues – show Coding Style Compatibility introduced 2016-02-22 04:48 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
7			{
8			protected static $_instance;
9
10			public static function instance()
11			{
12			if (empty(static::$_instance)) {
13			static::$_instance = new self();
14			}
15			return static::$_instance;
16			}
17
18			public function __construct()
19			{
20			$this->_slimApp();
21			$this->_services();
22			$this->_controllers();
			0 ignored issues – show Unused Code introduced 2016-02-22 04:48 UTC by Report Bug Copy Issue Report The call to the method `Xhgui_ServiceContainer::_controllers()` seems un-needed as the method has no side-effects. PHP Analyzer performs a side-effects analysis of your code. A side-effect is basically anything that might be visible after the scope of the method is left. Let’s take a look at an example: class User { private $email; public function getEmail() { return $this->email; } public function setEmail($email) { $this->email = $email; } } If we look at the `getEmail()` method, we can see that it has no side-effect. Whether you call this method or not, no future calls to other methods are affected by this. As such code as the following is useless: $user = new User(); $user->getEmail(); // This line could safely be removed as it has no effect. On the hand, if we look at the `setEmail()`, this method _has_ side-effects. In the following case, we could not remove the method call: $user = new User(); $user->setEmail('email@domain'); // This line has a side-effect (it changes an // instance variable). Loading history...
23			}
24
25			// Create the Slim app.
26			protected function _slimApp()
27			{
28			$this['view'] = function ($c) {
29			$cacheDir = isset($c['config']['cache']) ? $c['config']['cache'] : XHGUI_ROOT_DIR . '/cache';
30
31			// Configure Twig view for slim
32			$view = new Twig();
33
34			$view->parserOptions = array(
35			'charset' => 'utf-8',
36			'cache' => $cacheDir,
37			'auto_reload' => true,
38			'strict_variables' => false,
39			'autoescape' => true
40			);
41
42			return $view;
43			};
44
45			$this['app'] = $this->share(function ($c) {
46			$app = new Slim($c['config']);
47
48			// Enable cookie based sessions
49			$app->add(new SessionCookie(array(
50			'httponly' => true,
51			)));
52
53			// Add renderer.
54			$app->add(new Xhgui_Middleware_Render());
55
56			$view = $c['view'];
57			$view->parserExtensions = array(
58			new Xhgui_Twig_Extension($app)
59			);
60			$app->view($view);
61
62			return $app;
63			});
64			}
65
66			/**
67			* Add common service objects to the container.
68			*/
69			protected function _services()
70			{
71			$this['config'] = Xhgui_Config::all();
72
73			$this['db'] = $this->share(function ($c) {
74			$config = $c['config'];
75			if (empty($config['db.options'])) {
76			$config['db.options'] = array();
77			}
78			$mongo = new MongoClient($config['db.host'], $config['db.options']);
79			$mongo->{$config['db.db']}->results->findOne();
80
81			return $mongo->{$config['db.db']};
82			});
83
84			$this['pdo'] = $this->share(function ($c) {
85			return new PDO(
86			$c['config']['pdo']['dsn'],
87			$c['config']['pdo']['pass'],
88			$c['config']['pdo']['user']
89			);
90			});
91
92			$this['searcher.mongo'] = function ($c) {
93			return new Xhgui_Searcher_Mongo($c['db']);
94			};
95
96			$this['searcher.pdo'] = function ($c) {
97			return new Xhgui_Searcher_Pdo($c['pdo'], $c['config']['pdo']['table']);
98			};
99
100			$this['searcher'] = function ($c) {
101			$config = $c['config'];
102
103			switch ($config['save.handler']) {
104			case 'pdo':
105			return $c['searcher.pdo'];
106
107			case 'mongodb':
108			default:
109			return $c['searcher.mongo'];
110			}
111			};
112
113			$this['saver.mongo'] = function($c) {
114			$config = $c['config'];
115			$config['save.handler'] = 'mongodb';
116
117			return Xhgui_Saver::factory($config);
118			};
119
120			$this['saver'] = function($c) {
121			return Xhgui_Saver::factory($c['config']);
122			};
123			}
124
125			/**
126			* Add controllers to the DI container.
127			*/
128			protected function _controllers()
129			{
130			$this['watchController'] = function ($c) {
131			return new Xhgui_Controller_Watch($c['app'], $c['searcher']);
132			};
133
134			$this['runController'] = function ($c) {
135			return new Xhgui_Controller_Run($c['app'], $c['searcher']);
136			};
137
138			$this['customController'] = function ($c) {
139			return new Xhgui_Controller_Custom($c['app'], $c['searcher']);
140			};
141
142			$this['waterfallController'] = function ($c) {
143			return new Xhgui_Controller_Waterfall($c['app'], $c['searcher']);
144			};
145
146			$this['importController'] = function ($c) {
147			return new Xhgui_Controller_Import($c['app'], $c['saver']);
148			};
149			}
150
151			}
152

perftools / xhgui

Pull Request — master (#244)

Xhgui_ServiceContainer::_services() B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like