Bpost\BpostApiClient\Common\ValidatedValue

Complexity

Total Complexity

10

Size/Duplication

Total Lines	78
Duplicated Lines	3.85 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getValue()	0	4	1
A	setValue()	0	4	1
A	__toString()	0	4	1
A	validateLength()	3	6	2
A	validateChoice()	0	6	2
A	validatePattern()	0	6	2
	validate()	0	1	?

<?php

namespace Bpost\BpostApiClient\Common;

use Bpost\BpostApiClient\Exception\BpostLogicException;
use Bpost\BpostApiClient\Exception\BpostLogicException\BpostInvalidLengthException;
use Bpost\BpostApiClient\Exception\BpostLogicException\BpostInvalidPatternException;
use Bpost\BpostApiClient\Exception\BpostLogicException\BpostInvalidValueException;

/**
 * Class ValidatedValue
 */
abstract class ValidatedValue
{
    /** @var mixed */
    private $value;

    /**
     * ValidatedValue constructor.
     * @param mixed $value
     */
    public function __construct($value)
    {
        $this->setValue($value);
        $this->validate();
    }

    /**
     * @return mixed
     */
    public function getValue()
    {
        return $this->value;
    }

    /**
     * @param mixed $value
     */
    public function setValue($value)
    {
        $this->value = $value;
    }

    /**
     * @return string
     */
    public function __toString()
    {
        return (string)$this->getValue();
    }

    /**
     * @param int $length
     * @throws BpostInvalidLengthException
     */
    public function validateLength($length)
    {
        if (mb_strlen($this->getValue()) > $length) {

This code seems to be duplicated across your project.

            throw new BpostInvalidLengthException('', mb_strlen($this->getValue()), $length);
        }
    }

    /**
     * @param array $allowedValues
     * @throws BpostInvalidValueException
     */
    public function validateChoice(array $allowedValues)
    {
        if (!in_array($this->getValue(), $allowedValues)) {
            throw new BpostInvalidValueException('', $this->getValue(), $allowedValues);
        }
    }

    /**
     * @param string $regexPattern
     * @throws BpostInvalidPatternException
     */
    public function validatePattern($regexPattern)
    {
        if (!preg_match("/^$regexPattern\$/", $this->getValue())) {
            throw new BpostInvalidPatternException('', $this->getValue(), $regexPattern);
        }
    }

    /**
     * @throws BpostLogicException
     */
    public abstract function validate();

The abstract declaration must precede the visibility declaration

For interfaces and abstract methods it is generally a good practice to add a @return annotation even if it is just @return void or @return null, so that implementors know what to do in the overridden method.

}