Carno\HRPC\Client\Chips\Layers

Complexity

Total Complexity

3

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	exception()	0	3	1
A	outbound()	0	3	1
A	inbound()	0	5	1

<?php
/**
 * Chain layers for RPC client
 * User: moyo
 * Date: 12/12/2017
 * Time: 11:58 AM
 */

namespace Carno\HRPC\Client\Chips;

use function Carno\Coroutine\async;
use Carno\Coroutine\Context;
use Carno\Promise\Promised;
use Carno\RPC\Protocol\Request;
use Carno\RPC\Protocol\Response;
use Throwable;

trait Layers
{
    /**
     * @param Request $request
     * @param Context $ctx
     * @return Promised
     */
    public function inbound($request, Context $ctx) : Promised
    {
        return async(function (Request $request) {
            return $this->call($request);

It seems like call() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }, $ctx, $request);
    }

    /**
     * @param Response $response
     * @param Context $ctx
     * @return Response
     */
    public function outbound($response, Context $ctx)

The parameter $ctx is not used and could be removed.

    {
        return $response;
    }

    /**
     * @param Throwable $e
     * @param Context $ctx
     * @throws Throwable
     */
    public function exception(Throwable $e, Context $ctx) : void

The parameter $ctx is not used and could be removed.

    {
        throw $e;
    }
}