AsyncClient::send() - Code Metrics - Inspection of "Refactored AsyncClient so that it's more declarati..." - php-api-clients/pusher - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#7)

by David

created 2017-09-06 13:04 UTC

AsyncClient::send() A

↳ Parent: AsyncClient

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
ccs	3
cts	3
cp	1
rs	10
c	0
b	0
f	0
cc	1
eloc	2
nc	1
nop	1
crap	1

<?php declare(strict_types=1);

namespace ApiClients\Client\Pusher;

use React\Dns\Resolver\Resolver;
use React\EventLoop\LoopInterface;
use RuntimeException;
use Rx\Observable;
use Rx\Scheduler;
use Rx\Subject\Subject;
use Rx\Websocket\WebsocketErrorException;
use Throwable;

final class AsyncClient
{
    const DEFAULT_DELAY = 200;
    const NO_ACTIVITY_TIMEOUT = 120;
    const NO_PING_RESPONSE_TIMEOUT = 30;

    /**
     * @var Observable
     */
    private $messages;

    /**
     * @var array
     */
    private $channels = [];

    /**
     * @var int
     */
    private $delay = self::DEFAULT_DELAY;

    /**
     * @var Subject
     */
    private $client;

    /**
     * @var Observable
     */
    private $connected;

    /**
     * @internal
     * @param  Subject                   $client
     * @throws \InvalidArgumentException
     */
    public function __construct(Subject $client)
    {
        $this->client = $client;

        /** @var Observable $events */
        $events = $client
            ->_ApiClients_jsonDecode()
            ->map([Event::class, 'createFromMessage'])
            ->singleInstance();

        $pusherErrors = $events
            ->filter([Event::class, 'isError'])
            ->flatMap(function (Event $event) {
                $throwable = new PusherErrorException($event->getData()['message'], (int)$event->getData()['code']);

                return Observable::error($throwable);
            });

        $this->connected = $events
            ->filter([Event::class, 'connectionEstablished'])
            ->singleInstance();

        $this->messages = $events
            ->merge($this->timeout($events))
            ->merge($pusherErrors)
            ->singleInstance();
    }

    /**
     * @param  LoopInterface             $loop
     * @param  string                    $app      Application ID
     * @param  Resolver                  $resolver Optional DNS resolver
     * @throws \InvalidArgumentException
     * @return AsyncClient
     */
    public static function create(LoopInterface $loop, string $app, Resolver $resolver = null): AsyncClient
    {
        try {
            Scheduler::setDefaultFactory(function () use ($loop) {
                return new Scheduler\EventLoopScheduler($loop);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            });
        } catch (Throwable $t) {

        }

        return new self(
            new WebSocket(ApiSettings::createUrl($app), false, [], $loop, $resolver)
        );
    }

    /**
     * Listen on a channel.
     *
     * @param  string                    $channel Channel to listen on
     * @throws \InvalidArgumentException
     * @return Observable
     */
    public function channel(string $channel): Observable
    {
        // Only join a channel once
        if (isset($this->channels[$channel])) {
            return $this->channels[$channel];
        }

        // Ensure we only get messages for the given channel
        $channelMessages = $this->messages->filter(function (Event $event) use ($channel) {
            return $event->getChannel() !== '' && $event->getChannel() === $channel;
        });

        $subscribe = $this->connected
            ->do(function () use ($channel) {
                // Subscribe to pusher channel after connected
                $this->send(Event::subscribeOn($channel));
            })
            ->flatMapTo(Observable::empty());

        // Observable representing channel events
        $this->channels[$channel] = $channelMessages
            ->merge($subscribe)
            ->filter([Event::class, 'subscriptionSucceeded'])
            ->retryWhen(function (Observable $errors) {
                return $errors->flatMap(function (Throwable $throwable) {
                    return $this->handleLowLevelError($throwable);
                });
            })
            ->finally(function () use ($channel) {
                // Send unsubscribe event
                $this->send(Event::unsubscribeOn($channel));

                // Remove our channel from the channel list so we don't resubscribe in case we reconnect
                unset($this->channels[$channel]);
            })
            ->singleInstance();

        return $this->channels[$channel];
    }

    /**
     * Send a message through the client.
     *
     * @param array $message Message to send, will be json encoded
     *
     */
    public function send(array $message)
    {
        $this->client->onNext(json_encode($message));
    }

    /**
     * Returns an observable of TimeoutException.
     * The timeout observable will get cancelled every time a new event is received.
     *
     * @param  Observable $events
     * @return Observable
     */
    public function timeout(Observable $events): Observable
    {
        $timeoutDuration = $this->connected->map(function (Event $event) {
            return ($event->getData()['activity_timeout'] ?? self::NO_ACTIVITY_TIMEOUT) * 1000;
        });

        return $timeoutDuration
            ->combineLatest([$events])
            ->pluck(0)
            ->concat(Observable::of(-1))
            ->flatMapLatest(function (int $time) {

                // If the events observable ends, return an empty observable so we don't keep the stream alive
                if ($time === -1) {
                    return Observable::empty();
                }

                return Observable::never()
                    ->timeout($time)
                    ->catch(function () use ($time) {
                        // ping (do something that causes incoming stream to get a message)
                        $this->send(Event::ping());
                        // this timeout will actually timeout with a TimeoutException - causing
                        // everything above this to dispose
                        return Observable::never()->timeout($time);
                    });
            });
    }

    /**
     * Handle errors as described at https://pusher.com/docs/pusher_protocol#error-codes.
     * @param  Throwable  $throwable
     * @return Observable
     */
    private function handleLowLevelError(Throwable $throwable): Observable
    {
        // Only allow certain, relevant, exceptions
        if (!($throwable instanceof WebsocketErrorException) &&
            !($throwable instanceof RuntimeException) &&
            !($throwable instanceof PusherErrorException)
        ) {
            return Observable::error($throwable);
        }

        $code = $throwable->getCode();
        $pusherError = ($throwable instanceof WebsocketErrorException || $throwable instanceof PusherErrorException);

        // Errors 4000-4099, don't retry connecting
        if ($pusherError && $code >= 4000 && $code <= 4099) {
            return Observable::error($throwable);
        }

        // Errors 4100-4199 reconnect after 1 or more seconds, we do it after 1.001 second
        if ($pusherError && $code >= 4100 && $code <= 4199) {
            return Observable::timer(1001);
        }

        // Errors 4200-4299 connection closed by Pusher, reconnect immediately, we wait 0.001 second
        if ($pusherError && $code >= 4200 && $code <= 4299) {
            return Observable::timer(1);
        }

        // Double our delay each time we get here
        $this->delay *= 2;

        return Observable::timer($this->delay);
    }
}


1		<?php declare(strict_types=1);
2
3		namespace ApiClients\Client\Pusher;
4
5		use React\Dns\Resolver\Resolver;
6		use React\EventLoop\LoopInterface;
7		use RuntimeException;
8		use Rx\Observable;
9		use Rx\Scheduler;
10		use Rx\Subject\Subject;
11		use Rx\Websocket\WebsocketErrorException;
12		use Throwable;
13
14		final class AsyncClient
15		{
16		const DEFAULT_DELAY = 200;
17		const NO_ACTIVITY_TIMEOUT = 120;
18		const NO_PING_RESPONSE_TIMEOUT = 30;
19
20		/**
21		* @var Observable
22		*/
23		private $messages;
24
25		/**
26		* @var array
27		*/
28		private $channels = [];
29
30		/**
31		* @var int
32		*/
33		private $delay = self::DEFAULT_DELAY;
34
35		/**
36		* @var Subject
37		*/
38		private $client;
39
40		/**
41		* @var Observable
42		*/
43		private $connected;
44
45		/**
46		* @internal
47		* @param Subject $client
48		* @throws \InvalidArgumentException
49		*/
50	18	public function __construct(Subject $client)
51		{
52	18	$this->client = $client;
53
54		/** @var Observable $events */
55		$events = $client
56	18	->_ApiClients_jsonDecode()
57	18	->map([Event::class, 'createFromMessage'])
58	18	->singleInstance();
59
60		$pusherErrors = $events
61	18	->filter([Event::class, 'isError'])
62	18	->flatMap(function (Event $event) {
63		$throwable = new PusherErrorException($event->getData()['message'], (int)$event->getData()['code']);
64
65		return Observable::error($throwable);
66	18	});
67
68	18	$this->connected = $events
69	18	->filter([Event::class, 'connectionEstablished'])
70	18	->singleInstance();
71
72	18	$this->messages = $events
73	18	->merge($this->timeout($events))
74	18	->merge($pusherErrors)
75	18	->singleInstance();
76	18	}
77
78		/**
79		* @param LoopInterface $loop
80		* @param string $app Application ID
81		* @param Resolver $resolver Optional DNS resolver
82		* @throws \InvalidArgumentException
83		* @return AsyncClient
84		*/
85		public static function create(LoopInterface $loop, string $app, Resolver $resolver = null): AsyncClient
86		{
87		try {
88	4	Scheduler::setDefaultFactory(function () use ($loop) {
89	3	return new Scheduler\EventLoopScheduler($loop);
		0 ignored issues – show Documentation introduced 2017-04-05 19:21 UTC by Report Bug Copy Issue Report `$loop` is of type `object<React\EventLoop\LoopInterface>`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
90	4	});
91	2	} catch (Throwable $t) {
		0 ignored issues – show Coding Style Comprehensibility introduced 2017-04-05 19:21 UTC by Report Bug Copy Issue Report Consider adding a comment why this CATCH block is empty. Loading history...
92		}
93
94	4	return new self(
95	4	new WebSocket(ApiSettings::createUrl($app), false, [], $loop, $resolver)
96		);
97		}
98
99		/**
100		* Listen on a channel.
101		*
102		* @param string $channel Channel to listen on
103		* @throws \InvalidArgumentException
104		* @return Observable
105		*/
106	15	public function channel(string $channel): Observable
107		{
108		// Only join a channel once
109	15	if (isset($this->channels[$channel])) {
110	1	return $this->channels[$channel];
111		}
112
113		// Ensure we only get messages for the given channel
114	15	$channelMessages = $this->messages->filter(function (Event $event) use ($channel) {
115	10	return $event->getChannel() !== '' && $event->getChannel() === $channel;
116	15	});
117
118	15	$subscribe = $this->connected
119	15	->do(function () use ($channel) {
120		// Subscribe to pusher channel after connected
121	10	$this->send(Event::subscribeOn($channel));
122	15	})
123	15	->flatMapTo(Observable::empty());
124
125		// Observable representing channel events
126	15	$this->channels[$channel] = $channelMessages
127	15	->merge($subscribe)
128	15	->filter([Event::class, 'subscriptionSucceeded'])
129		->retryWhen(function (Observable $errors) {
130	15	return $errors->flatMap(function (Throwable $throwable) {
131	7	return $this->handleLowLevelError($throwable);
132	15	});
133	15	})
134	15	->finally(function () use ($channel) {
135		// Send unsubscribe event
136	13	$this->send(Event::unsubscribeOn($channel));
137
138		// Remove our channel from the channel list so we don't resubscribe in case we reconnect
139	13	unset($this->channels[$channel]);
140	15	})
141	15	->singleInstance();
142
143	15	return $this->channels[$channel];
144		}
145
146		/**
147		* Send a message through the client.
148		*
149		* @param array $message Message to send, will be json encoded
150		*
151		*/
152	13	public function send(array $message)
153		{
154	13	$this->client->onNext(json_encode($message));
155	13	}
156
157		/**
158		* Returns an observable of TimeoutException.
159		* The timeout observable will get cancelled every time a new event is received.
160		*
161		* @param Observable $events
162		* @return Observable
163		*/
164		public function timeout(Observable $events): Observable
165		{
166	18	$timeoutDuration = $this->connected->map(function (Event $event) {
167	10	return ($event->getData()['activity_timeout'] ?? self::NO_ACTIVITY_TIMEOUT) * 1000;
168	18	});
169
170		return $timeoutDuration
171	18	->combineLatest([$events])
172	18	->pluck(0)
173	18	->concat(Observable::of(-1))
174	18	->flatMapLatest(function (int $time) {
175
176		// If the events observable ends, return an empty observable so we don't keep the stream alive
177	11	if ($time === -1) {
178	5	return Observable::empty();
179		}
180
181	10	return Observable::never()
182	10	->timeout($time)
183	10	->catch(function () use ($time) {
184		// ping (do something that causes incoming stream to get a message)
185	2	$this->send(Event::ping());
186		// this timeout will actually timeout with a TimeoutException - causing
187		// everything above this to dispose
188	2	return Observable::never()->timeout($time);
189	10	});
190	18	});
191		}
192
193		/**
194		* Handle errors as described at https://pusher.com/docs/pusher_protocol#error-codes.
195		* @param Throwable $throwable
196		* @return Observable
197		*/
198	7	private function handleLowLevelError(Throwable $throwable): Observable
199		{
200		// Only allow certain, relevant, exceptions
201	7	if (!($throwable instanceof WebsocketErrorException) &&
202	7	!($throwable instanceof RuntimeException) &&
203	7	!($throwable instanceof PusherErrorException)
204		) {
205	2	return Observable::error($throwable);
206		}
207
208	5	$code = $throwable->getCode();
209	5	$pusherError = ($throwable instanceof WebsocketErrorException \|\| $throwable instanceof PusherErrorException);
210
211		// Errors 4000-4099, don't retry connecting
212	5	if ($pusherError && $code >= 4000 && $code <= 4099) {
213	1	return Observable::error($throwable);
214		}
215
216		// Errors 4100-4199 reconnect after 1 or more seconds, we do it after 1.001 second
217	4	if ($pusherError && $code >= 4100 && $code <= 4199) {
218	1	return Observable::timer(1001);
219		}
220
221		// Errors 4200-4299 connection closed by Pusher, reconnect immediately, we wait 0.001 second
222	3	if ($pusherError && $code >= 4200 && $code <= 4299) {
223	1	return Observable::timer(1);
224		}
225
226		// Double our delay each time we get here
227	2	$this->delay *= 2;
228
229	2	return Observable::timer($this->delay);
230		}
231		}
232

php-api-clients / pusher

GitHub Access Token became invalid

Pull Request — master (#7)

AsyncClient::send() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like