ReportForm - Code Metrics - Inspection of "updated" - spb-lan/ebs-sdk - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 257c81...98d67f )

by Denis

created 2017-08-25 16:49 UTC

ReportForm A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	73
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

Importance

Changes

Metric	Value
wmc	13
lcom	1
cbo	1
dl	0
loc	73
ccs	0
cts	25
cp	0
rs	10
c	0
b	0
f	0

6 Methods

Rating	Name	Size	Complexity
A	__construct()	8	2
A	getSecretKey()	12	4
B	getUrl()	25	4
A	getBibFond()	4	1
A	getEBooks()	4	1
A	getSpecPo()	4	1

<?php
/**
 * Created by PhpStorm.
 * User: dp
 * Date: 16.08.17
 * Time: 13:09
 */

namespace Lan\Ebs\Sdk;

use Exception;

final class ReportForm implements Common
{
    private $client;

    /**
     * Security constructor.
     *
     * @param  Client $client
     * @throws Exception
     */
    public function __construct(Client $client)
    {
        if (!$client) {
            throw new Exception('Client not defined');
        }

        $this->client = $client;
    }

    public function getSecretKey($date)
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

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

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    {
        if (is_string($date) && strlen($date) >= 8) {
            $date = substr($date, 0, 8);

            if (!empty($_SESSION[$date])) {
                return $_SESSION[$date];
            }
        }

        return $_SESSION[$date] = $this->client->getResponse($this->getUrl(__FUNCTION__), ['date' => $date])['data'];
    }

    public function getUrl($method, array $params = [])
    {
        switch ($method) {
            case 'getBibFond':
                return [
                    'url' => '/1.0/report/form/bibFond',
                    'method' => 'GET',
                    'code' => 200
                ];
            case 'getEBooks':
                return [
                    'url' => '/1.0/report/form/eBooks',
                    'method' => 'GET',
                    'code' => 200
                ];
            case 'getSpecPo':
                return [
                    'url' => '/1.0/report/form/specPo',
                    'method' => 'GET',
                    'code' => 200
                ];
            default:
                throw new Exception('Route for ' . $method . ' not found');
        }
    }

    public function getBibFond()
    {
        return $this->client->getResponse($this->getUrl(__FUNCTION__))['data'];
    }

    public function getEBooks()
    {
        return $this->client->getResponse($this->getUrl(__FUNCTION__))['data'];
    }

    public function getSpecPo()
    {
        return $this->client->getResponse($this->getUrl(__FUNCTION__))['data'];
    }
}

1			<?php
2			/**
3			* Created by PhpStorm.
4			* User: dp
5			* Date: 16.08.17
6			* Time: 13:09
7			*/
8
9			namespace Lan\Ebs\Sdk;
10
11			use Exception;
12
13			final class ReportForm implements Common
14			{
15			private $client;
16
17			/**
18			* Security constructor.
19			*
20			* @param Client $client
21			* @throws Exception
22			*/
23			public function __construct(Client $client)
24			{
25			if (!$client) {
26			throw new Exception('Client not defined');
27			}
28
29			$this->client = $client;
30			}
31
32			public function getSecretKey($date)
			0 ignored issues – show Coding Style introduced 2017-08-18 07:19 UTC by Report Bug Copy Issue Report `getSecretKey` uses the super-global variable `$_SESSION` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
33			{
34			if (is_string($date) && strlen($date) >= 8) {
35			$date = substr($date, 0, 8);
36
37			if (!empty($_SESSION[$date])) {
38			return $_SESSION[$date];
39			}
40			}
41
42			return $_SESSION[$date] = $this->client->getResponse($this->getUrl(__FUNCTION__), ['date' => $date])['data'];
43			}
44
45			public function getUrl($method, array $params = [])
46			{
47			switch ($method) {
48			case 'getBibFond':
49			return [
50			'url' => '/1.0/report/form/bibFond',
51			'method' => 'GET',
52			'code' => 200
53			];
54			case 'getEBooks':
55			return [
56			'url' => '/1.0/report/form/eBooks',
57			'method' => 'GET',
58			'code' => 200
59			];
60			case 'getSpecPo':
61			return [
62			'url' => '/1.0/report/form/specPo',
63			'method' => 'GET',
64			'code' => 200
65			];
66			default:
67			throw new Exception('Route for ' . $method . ' not found');
68			}
69			}
70
71			public function getBibFond()
72			{
73			return $this->client->getResponse($this->getUrl(__FUNCTION__))['data'];
74			}
75
76			public function getEBooks()
77			{
78			return $this->client->getResponse($this->getUrl(__FUNCTION__))['data'];
79			}
80
81			public function getSpecPo()
82			{
83			return $this->client->getResponse($this->getUrl(__FUNCTION__))['data'];
84			}
85			}

spb-lan / ebs-sdk

Push — master ( 257c81...98d67f )

ReportForm A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

6 Methods

Duplication Side-by-Side

Filter issues like