think\Pipeline

Complexity

Total Complexity

10

Size/Duplication

Total Lines	87
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	send()	0	4	1
A	then()	0	14	2
A	handleException()	0	6	2
A	whenException()	0	4	1
A	carry()	0	8	2
A	through()	0	4	2

<?php
// +----------------------------------------------------------------------
// | ThinkPHP [ WE CAN DO IT JUST THINK ]
// +----------------------------------------------------------------------
// | Copyright (c) 2006~2019 http://thinkphp.cn All rights reserved.
// +----------------------------------------------------------------------
// | Licensed ( http://www.apache.org/licenses/LICENSE-2.0 )
// +----------------------------------------------------------------------
// | Author: yunwuxin <448901948@qq.com>
// +----------------------------------------------------------------------
namespace think;

use Closure;
use Exception;

This use statement conflicts with another class in this namespace, think\Exception. Consider defining an alias.

use Throwable;

class Pipeline
{
    protected $passable;

    protected $pipes = [];

    protected $exceptionHandler;

    /**

Parameter $passable should have a doc-comment as per coding-style.

     * 初始数据
     * @param $passable

Missing parameter name

     * @return $this
     */
    public function send($passable)
    {
        $this->passable = $passable;
        return $this;
    }

    /**

Parameter $pipes should have a doc-comment as per coding-style.

     * 调用栈
     * @param $pipes

Missing parameter name

     * @return $this
     */
    public function through($pipes)
    {
        $this->pipes = is_array($pipes) ? $pipes : func_get_args();
        return $this;
    }

    /**
     * 执行
     * @param Closure $destination

Missing parameter comment

     * @return mixed
     */
    public function then(Closure $destination)
    {
        $pipeline = array_reduce(
            array_reverse($this->pipes),
            $this->carry(),
            function ($passable) use ($destination) {
                try {
                    return $destination($passable);
                } catch (Throwable|Exception $e) {
                    return $this->handleException($passable, $e);
                }
            });

This line of the multi-line function call does not seem to be indented correctly. Expected 8 spaces, but found 12.

For multi-line function calls, the closing parenthesis should be on a new line.

        return $pipeline($this->passable);
    }

    /**
     * 设置异常处理器
     * @param callable $handler

Missing parameter comment

     * @return $this
     */
    public function whenException($handler)
    {
        $this->exceptionHandler = $handler;
        return $this;
    }

    protected function carry()

Missing doc comment for function carry()

    {
        return function ($stack, $pipe) {
            return function ($passable) use ($stack, $pipe) {
                try {
                    return $pipe($passable, $stack);
                } catch (Throwable|Exception $e) {
                    return $this->handleException($passable, $e);
                }
            };
        };
    }

    /**

Parameter $passable should have a doc-comment as per coding-style.

Parameter $e should have a doc-comment as per coding-style.

     * 异常处理
     * @param $passable

Missing parameter name

     * @param $e

Missing parameter name

     * @return mixed
     */
    protected function handleException($passable, Throwable $e)
    {
        if ($this->exceptionHandler) {
            return call_user_func($this->exceptionHandler, $passable, $e);
        }
        throw $e;
    }

}