Manager - Code Metrics - Inspection of "Alert detail" - Bit-Wasp/bitcoin-p2p-php - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#81)

by thomas

created 2017-06-23 01:39 UTC

Manager A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	142
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Test Coverage

Coverage

88%

Importance

Changes	6
Bugs	1	Features	1

Metric	Value
wmc	9
lcom	1
cbo	7
dl	0
loc	142
ccs	44
cts	50
cp	0.88
rs	10
c	6
b	1
f	1

7 Methods

Rating	Name	Size	Complexity
A	registerListener()	8	1
A	connect()	4	1
A	connectToPeers()	9	2
A	__construct()	4	1
A	registerOutboundPeer()	12	1
A	registerInboundPeer()	9	1
B	connectNextPeer()	31	2

<?php

namespace BitWasp\Bitcoin\Networking\Peer;

use BitWasp\Bitcoin\Networking\Structure\NetworkAddressInterface;
use Evenement\EventEmitter;
use React\Promise\Deferred;

class Manager extends EventEmitter
{
    /**
     * @var Connector
     */
    private $connector;

    /**
     * @var Peer[]
     */
    private $outPeers = [];

    /**
     * @var Peer[]
     */
    private $inPeers = [];

    /**
     * @var int
     */
    private $nOutPeers = 0;

    /**
     * @var int
     */
    private $nInPeers = 0;

    /**
     * Manager constructor.
     * @param Connector $connector
     */
    public function __construct(Connector $connector)
    {
        $this->connector = $connector;
    }

    /**
     * Store the newly connected peer, and trigger a new connection if they go away.
     *
     * @param Peer $peer
     * @return Peer
     */
    public function registerOutboundPeer(Peer $peer)
    {
        $next = $this->nOutPeers++;
        $peer->on('close', function ($peer) use ($next) {
            $this->emit('disconnect', [$peer]);
            unset($this->outPeers[$next]);
        });

        $this->outPeers[$next] = $peer;
        $this->emit('outbound', [$peer]);
        return $peer;
    }

    /**
     * @param Peer $peer
     */
    public function registerInboundPeer(Peer $peer)
    {
        $next = $this->nInPeers++;
        $this->inPeers[$next] = $peer;
        $peer->on('close', function () use ($next) {
            unset($this->inPeers[$next]);
        });
        $this->emit('inbound', [$peer]);
    }

    /**
     * @param Listener $listener
     * @return $this
     */
    public function registerListener(Listener $listener)
    {
        $listener->on('connection', function (Peer $peer) {
            $this->registerInboundPeer($peer);
        });

        return $this;
    }

    /**
     * @param NetworkAddressInterface $address
     * @return \React\Promise\PromiseInterface
     * @throws \Exception
     */
    public function connect(NetworkAddressInterface $address)
    {
        return $this->connector->connect($address);
    }

    /**
     * @param Locator $locator
     * @return \React\Promise\ExtendedPromiseInterface|\React\Promise\Promise
     */
    public function connectNextPeer(Locator $locator)
    {
        $deferred = new Deferred();

        // Otherwise, rely on the Locator.
        try {
            $deferred->resolve($locator->popAddress());
        } catch (\Exception $e) {
            $locator->queryDnsSeeds()->then(
                function () use ($deferred, $locator) {
                    $deferred->resolve($locator->popAddress());
                }
            );
        }

        return $deferred
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
            ->promise()
            ->then(
                function (NetworkAddressInterface $address) {
                    return $this->connect($address)->then(
                        function (Peer $peer) {
                            $this->registerOutboundPeer($peer);
                            return $peer;
                        }
                    );
                }
            )
            ->otherwise(function () use ($locator) {
                return $this->connectNextPeer($locator);
            });
    }

    /**
     * @param Locator $locator
     * @param int $n
     * @return \React\Promise\Promise
     */
    public function connectToPeers(Locator $locator, $n)
    {
        $peers = [];
        for ($i = 0; $i < $n; $i++) {
            $peers[$i] = $this->connectNextPeer($locator);
        }

        return \React\Promise\all($peers);
    }
}


1		<?php
2
3		namespace BitWasp\Bitcoin\Networking\Peer;
4
5		use BitWasp\Bitcoin\Networking\Structure\NetworkAddressInterface;
6		use Evenement\EventEmitter;
7		use React\Promise\Deferred;
8
9		class Manager extends EventEmitter
10		{
11		/**
12		* @var Connector
13		*/
14		private $connector;
15
16		/**
17		* @var Peer[]
18		*/
19		private $outPeers = [];
20
21		/**
22		* @var Peer[]
23		*/
24		private $inPeers = [];
25
26		/**
27		* @var int
28		*/
29		private $nOutPeers = 0;
30
31		/**
32		* @var int
33		*/
34		private $nInPeers = 0;
35
36		/**
37		* Manager constructor.
38		* @param Connector $connector
39		*/
40	6	public function __construct(Connector $connector)
41		{
42	6	$this->connector = $connector;
43	6	}
44
45		/**
46		* Store the newly connected peer, and trigger a new connection if they go away.
47		*
48		* @param Peer $peer
49		* @return Peer
50		*/
51	3	public function registerOutboundPeer(Peer $peer)
52		{
53	3	$next = $this->nOutPeers++;
54		$peer->on('close', function ($peer) use ($next) {
55	3	$this->emit('disconnect', [$peer]);
56	3	unset($this->outPeers[$next]);
57	3	});
58
59	3	$this->outPeers[$next] = $peer;
60	3	$this->emit('outbound', [$peer]);
61	3	return $peer;
62		}
63
64		/**
65		* @param Peer $peer
66		*/
67	3	public function registerInboundPeer(Peer $peer)
68		{
69	3	$next = $this->nInPeers++;
70	3	$this->inPeers[$next] = $peer;
71		$peer->on('close', function () use ($next) {
72		unset($this->inPeers[$next]);
73	3	});
74	3	$this->emit('inbound', [$peer]);
75	3	}
76
77		/**
78		* @param Listener $listener
79		* @return $this
80		*/
81	3	public function registerListener(Listener $listener)
82		{
83		$listener->on('connection', function (Peer $peer) {
84	3	$this->registerInboundPeer($peer);
85	3	});
86
87	3	return $this;
88		}
89
90		/**
91		* @param NetworkAddressInterface $address
92		* @return \React\Promise\PromiseInterface
93		* @throws \Exception
94		*/
95	3	public function connect(NetworkAddressInterface $address)
96		{
97	3	return $this->connector->connect($address);
98		}
99
100		/**
101		* @param Locator $locator
102		* @return \React\Promise\ExtendedPromiseInterface\|\React\Promise\Promise
103		*/
104	3	public function connectNextPeer(Locator $locator)
105		{
106	3	$deferred = new Deferred();
107
108		// Otherwise, rely on the Locator.
109		try {
110	3	$deferred->resolve($locator->popAddress());
111	1	} catch (\Exception $e) {
112		$locator->queryDnsSeeds()->then(
113		function () use ($deferred, $locator) {
114		$deferred->resolve($locator->popAddress());
115		}
116		);
117		}
118
119		return $deferred
		0 ignored issues – show Bug introduced 2015-11-30 23:16 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `React\Promise\PromiseInterface` as the method `otherwise()` does only exist in the following implementations of said interface: `React\Promise\FulfilledPromise`, `React\Promise\LazyPromise`, `React\Promise\Promise`, `React\Promise\RejectedPromise`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
120	3	->promise()
121	3	->then(
122		function (NetworkAddressInterface $address) {
123	3	return $this->connect($address)->then(
124		function (Peer $peer) {
125	3	$this->registerOutboundPeer($peer);
126	3	return $peer;
127	2	}
128	1	);
129	2	}
130	1	)
131	3	->otherwise(function () use ($locator) {
132		return $this->connectNextPeer($locator);
133	3	});
134		}
135
136		/**
137		* @param Locator $locator
138		* @param int $n
139		* @return \React\Promise\Promise
140		*/
141	3	public function connectToPeers(Locator $locator, $n)
142		{
143	3	$peers = [];
144	3	for ($i = 0; $i < $n; $i++) {
145	3	$peers[$i] = $this->connectNextPeer($locator);
146	1	}
147
148	3	return \React\Promise\all($peers);
149		}
150		}
151

Bit-Wasp / bitcoin-p2p-php

Pull Request — master (#81)

Manager A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

7 Methods

Available Fixes

Duplication Side-by-Side

Filter issues like