ClientRegistry::replace() - Code Metrics - Inspection of "[WIP] Release 2.0" - bankiru/doctrine-api-client - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#14)

by Pavel

created 2017-04-28 10:11 UTC

ClientRegistry::replace() A

↳ Parent: ClientRegistry

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	2
crap	1

<?php

namespace Bankiru\Api\Doctrine;

use ScayTrase\Api\Rpc\RpcClientInterface;

final class ClientRegistry implements ClientRegistryInterface
{
    /** @var  RpcClientInterface[] */
    private $clients = [];

    /**
     * @param string $name
     *
     * @return RpcClientInterface
     * @throws \OutOfBoundsException
     */
    public function get($name)
    {
        if (!$this->has($name)) {
            throw new \OutOfBoundsException(sprintf('Client "%s" not registered', $name));
        }

        return $this->clients[$name];
    }

    /**
     * @param string $name
     *
     * @return bool
     */
    public function has($name)
    {
        return array_key_exists($name, $this->clients);
    }

    /**
     * @param string             $name
     * @param RpcClientInterface $client
     *
     * @return void
     * @throws \InvalidArgumentException
     */
    public function add($name, RpcClientInterface $client)
    {
        if ($this->has($name)) {
            throw new \InvalidArgumentException();
        }

        $this->replace($name, $client);
    }

    /**
     * @param string             $name
     * @param RpcClientInterface $client
     *
     * @return void
     */
    public function replace($name, RpcClientInterface $client)
    {
        $this->clients[$name] = $client;
    }

    /**
     * @return RpcClientInterface[]
     */
    public function all()
    {
        return $this->clients;
    }
}


1		<?php
2
3		namespace Bankiru\Api\Doctrine;
4
5		use ScayTrase\Api\Rpc\RpcClientInterface;
6
7		final class ClientRegistry implements ClientRegistryInterface
8		{
9		/** @var RpcClientInterface[] */
10		private $clients = [];
11
12		/**
13		* @param string $name
14		*
15		* @return RpcClientInterface
16		* @throws \OutOfBoundsException
17		*/
18	23	public function get($name)
19		{
20	23	if (!$this->has($name)) {
21		throw new \OutOfBoundsException(sprintf('Client "%s" not registered', $name));
22		}
23
24	23	return $this->clients[$name];
25		}
26
27		/**
28		* @param string $name
29		*
30		* @return bool
31		*/
32	24	public function has($name)
33		{
34	24	return array_key_exists($name, $this->clients);
35		}
36
37		/**
38		* @param string $name
39		* @param RpcClientInterface $client
40		*
41		* @return void
42		* @throws \InvalidArgumentException
43		*/
44	24	public function add($name, RpcClientInterface $client)
45		{
46	24	if ($this->has($name)) {
47		throw new \InvalidArgumentException();
48		}
49
50	24	$this->replace($name, $client);
51	24	}
52
53		/**
54		* @param string $name
55		* @param RpcClientInterface $client
56		*
57		* @return void
58		*/
59	24	public function replace($name, RpcClientInterface $client)
60		{
61	24	$this->clients[$name] = $client;
62	24	}
63
64		/**
65		* @return RpcClientInterface[]
66		*/
67		public function all()
68		{
69		return $this->clients;
70		}
71		}
72

bankiru / doctrine-api-client

Pull Request — master (#14)

ClientRegistry::replace() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like