nebula\component\debug\attach\AttachTrait

Complexity

Total Complexity

11

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	mergeAttribute()	0	3	1
A	interpolate()	0	18	5
A	analyse()	0	11	5

<?php
namespace nebula\component\debug\attach;

/**
 * 多行附加属性
 */
trait AttachTrait
{
    /**
     * 属性数组
     *
     * @var array
     */
    protected $attribute;
    /**
     * 附加数组
     *
     * @var array
     */
    protected $attach;

    
    public function mergeAttribute(array $value)
    {
        $this->attribute = array_merge($this->attribute, $value);
    }

    protected function analyse(array $context)
    {
        $replace = [];
        foreach ($context as $key => $val) {
            if (!is_array($val) && (!is_object($val) || method_exists($val, '__toString'))) {
                $replace['{' . $key . '}'] = $val;
            } else {
                $this->attach[$key] = $val;
            }
        }
        return $replace;
    }

    public function interpolate(string $message, array $context)
    {
        $replace =  $this->analyse($context);
        foreach ($this->attribute as $key => $val) {
            $replace['%' . $key . '%'] = $val;
        }
        $message = strtr($message, $replace);
        foreach ($this->attach as $name => $value) {
            $message.=PHP_EOL.$name.'<' . get_class($val).'>'.PHP_EOL;

The variable $val seems to be defined by a foreach iteration on line 44. Are you sure the iterator is never empty, otherwise this variable is not defined?

            if ($value instanceof AttachValueInterface) {
                $message.= $value->getLogAttach().PHP_EOL;
            } elseif ($value instanceof \Exception) {
                $message.= DumpTrait::dumpException($value).PHP_EOL;
            } else {
                $message.= DumpTrait::parameterToString($value).PHP_EOL;
            }
        }
        return $message;
    }
}