WebStream\Delegate\ExceptionDelegator

Complexity

Total Complexity

12

Size/Duplication

Total Lines	92
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	setExceptionHandler()	0	3	1
F	raise()	0	45	10

<?php
namespace WebStream\Delegate;

use WebStream\Core\CoreInterface;
use WebStream\Exception\SystemException;
use WebStream\Exception\DelegateException;
use WebStream\DI\Injector;

/**
 * ExceptionDelegator
 * @author Ryuichi TANAKA.
 * @since 2014/05/05
 * @version 0.7
 */
class ExceptionDelegator
{
    use Injector;

    /**
     * @var CoreInterface インスタンス
     */
    private $instance;

    /**
     * @var string メソッド名
     */
    private $method;

    /**
     * @var \Exception 例外オブジェクト
     */
    private $exceptionObject;

    /**
     * @var array<AnnotationContainer> 例外ハンドリングリスト
     */
    private $exceptionHandler;

    /**
     * constructor
     */
    public function __construct(CoreInterface $instance, \Exception $exceptionObject, $method = null)
    {
        $this->instance = $instance;
        $this->method = $method;
        $this->exceptionObject = $exceptionObject;
        $this->exceptionHandler = [];
    }

    /**
     * 例外ハンドリングリストを設定する
     * @param array<AnnotationContainer> 例外ハンドリングリスト
     */

The doc comment 例外ハンドリングリスト at position 0 could not be parsed: Unknown type name '例外ハンドリングリスト' at position 0 in 例外ハンドリングリスト.

    public function setExceptionHandler(array $exceptionHandler)
    {
        $this->exceptionHandler = $exceptionHandler;
    }

    /**
     * 例外を実行する
     */
    public function raise()
    {
        // 捕捉可能な例外の場合のみDelegateExceptionでラップする
        // そうでない場合はそのままスロー
        if ($this->exceptionObject instanceof SystemException) {
            throw $this->exceptionObject;
        } else {
            $originException = $this->exceptionObject;
            $delegateException = null;

            if ($originException instanceof DelegateException) {
                // 複数レイヤ間で例外がやりとりされる場合、すでにDelegateExceptionでラップ済みなので戻す
                $originException = $delegateException->getOriginException();

The method getOriginException() does not exist on null.

            }

            $invokeMethods = [];
            foreach ($this->exceptionHandler as $exceptionHandlerAnnotation) {

The variable name $exceptionHandlerAnnotation exceeds the maximum configured length of 20.

                $exceptions = $exceptionHandlerAnnotation->exceptions;
                $refMethod = $exceptionHandlerAnnotation->method;
                foreach ($exceptions as $exception) {
                    if (is_a($originException, $exception)) {
                        // 一つのメソッドに複数の捕捉例外が指定された場合(派生例外クラス含む)、先勝で1回のみ実行する
                        // そうでなければ複数回メソッドが実行されるため
                        // ただし同一クラス内に限る(親クラスの同一名のメソッドは実行する)
                        // TODO ここはテストを追加する
                        $classpath = $refMethod->class . "#" . $refMethod->name;
                        if (!array_key_exists($classpath, $invokeMethods)) {
                            $invokeMethods[$classpath] = $refMethod;
                        }
                    }
                }
            }

            if (count($invokeMethods) > 0) {
                $delegateException = new DelegateException($this->exceptionObject);
                $delegateException->enableHandled();
            }

            foreach ($invokeMethods as $classpath => $invokeMethod) {
                $params = ["class" => get_class($this->instance), "method" => $this->method, "exception" => $originException];
                $invokeMethod->invokeArgs($this->instance, [$params]);
                $this->logger->debug("Execution of handling is success: " . $classpath);

The property logger does not exist on WebStream\Delegate\ExceptionDelegator. Since you implemented __get, consider adding a @property annotation.

            }

            throw $delegateException ?: $originException;
        }
    }
}