HttpClientRouter::addClient() - Code Metrics - Inspection of "[WIP] Add http client router feature" - php-http/client-common - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#13)

by Joel

created 2016-01-31 21:35 UTC

HttpClientRouter::addClient() A

↳ Parent: HttpClientRouter

Complexity

Conditions	1
Paths	1

Size

Total Lines	7
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	1

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

<?php

namespace Http\Client\Common;

use Http\Client\Exception\RequestException;
use Http\Client\HttpAsyncClient;
use Http\Client\HttpClient;
use Psr\Http\Message\RequestInterface;

/**
 *
 */
class HttpClientRouter implements HttpClient, HttpAsyncClient
{
    private $clients = [];

    /**
     * {@inheritdoc}
     */
    public function sendAsyncRequest(RequestInterface $request)
    {
        $client = $this->chooseHttpClient($request);

        return $client->sendAsyncRequest($request);
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
    }

    /**
     * {@inheritdoc}
     */
    public function sendRequest(RequestInterface $request)
    {
        $client = $this->chooseHttpClient($request);

        return $client->sendRequest($request);
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
    }

    /**
     * Add a client to the router
     *
     * @param HttpClient|HttpAsyncClient $client
     * @param RequestMatcher             $requestMatcher
     */
    public function addClient($client, RequestMatcher $requestMatcher)
    {
        $this->clients[] = [
            'matcher' => $requestMatcher,
            'client'  => new HttpClientFlexible($client)
        ];
    }

    /**
     * Choose a http client given a specific request
     *
     * @param RequestInterface $request
     *
     * @return HttpClient|HttpAsyncClient
     */
    protected function chooseHttpClient(RequestInterface $request)
    {
        foreach ($this->clients as $client) {
            if ($client['matcher']->matches($request)) {
                return $client['client'];
            }
        }

        throw new RequestException('No client found for the specified request', $request);
    }
}


1		<?php
2
3		namespace Http\Client\Common;
4
5		use Http\Client\Exception\RequestException;
6		use Http\Client\HttpAsyncClient;
7		use Http\Client\HttpClient;
8		use Psr\Http\Message\RequestInterface;
9
10		/**
11		*
12		*/
13		class HttpClientRouter implements HttpClient, HttpAsyncClient
14		{
15		private $clients = [];
16
17		/**
18		* {@inheritdoc}
19		*/
20	2	public function sendAsyncRequest(RequestInterface $request)
21		{
22	2	$client = $this->chooseHttpClient($request);
23
24	1	return $client->sendAsyncRequest($request);
		0 ignored issues – show Bug introduced 2016-01-31 21:37 UTC by Report Bug Copy Issue Report The method `sendAsyncRequest` does only exist in `Http\Client\HttpAsyncClient`, but not in `Http\Client\HttpClient`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
25		}
26
27		/**
28		* {@inheritdoc}
29		*/
30	2	public function sendRequest(RequestInterface $request)
31		{
32	2	$client = $this->chooseHttpClient($request);
33
34	1	return $client->sendRequest($request);
		0 ignored issues – show Bug introduced 2016-01-31 21:37 UTC by Report Bug Copy Issue Report The method `sendRequest` does only exist in `Http\Client\HttpClient`, but not in `Http\Client\HttpAsyncClient`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
35		}
36
37		/**
38		* Add a client to the router
39		*
40		* @param HttpClient\|HttpAsyncClient $client
41		* @param RequestMatcher $requestMatcher
42		*/
43	4	public function addClient($client, RequestMatcher $requestMatcher)
44		{
45	4	$this->clients[] = [
46	4	'matcher' => $requestMatcher,
47	4	'client' => new HttpClientFlexible($client)
48	4	];
49	4	}
50
51		/**
52		* Choose a http client given a specific request
53		*
54		* @param RequestInterface $request
55		*
56		* @return HttpClient\|HttpAsyncClient
57		*/
58	4	protected function chooseHttpClient(RequestInterface $request)
59		{
60	4	foreach ($this->clients as $client) {
61	4	if ($client['matcher']->matches($request)) {
62	2	return $client['client'];
63		}
64	2	}
65
66	2	throw new RequestException('No client found for the specified request', $request);
67		}
68		}
69

php-http / client-common

Pull Request — master (#13)

HttpClientRouter::addClient() A

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like