Flagbit\Plantuml\TokenReflection\ConstantWriter

Complexity

Total Complexity

3

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	writeElement()	0	4	1
A	writeElements()	0	14	2

<?php

namespace Flagbit\Plantuml\TokenReflection;

use TokenReflection\IReflectionConstant;

class ConstantWriter extends WriterAbstract
{
    /**
     * @param IReflectionConstant $constant
     *
     * @return string
     */
    public function writeElement(IReflectionConstant $constant)
    {
        return $this->formatLine('+{static}' . $constant->getName() . ' = ' . $this->formatValue($constant->getValue()));
    }

    /**
     * @param IReflectionConstant[] $constants
     *
     * @return string
     */
    public function writeElements(array $constants)
    {
        // see https://bugs.php.net/bug.php?id=50688
        @usort($constants, function(IReflectionConstant $a, IReflectionConstant $b) {

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

            return strnatcasecmp($a->getName(), $b->getName());
        });

        $constantsString = '';
        foreach ($constants as $constant) {
            /** @var $property IReflectionConstant */
            $constantsString .= $this->writeElement($constant);
        }
        return $constantsString;
    }
}