BestServedCold\PhalueObjects\ValueObject

Complexity

Total Complexity

9

Size/Duplication

Total Lines	91
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	0

Test Coverage

Coverage

90.48%

Importance

Changes	9
Bugs	0	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	getShortName()	0	4	1
A	hash()	0	4	1
A	cloneObject()	0	4	1
A	__construct()	0	5	1
A	__set()	0	6	1
A	fromString()	0	4	1
A	getValue()	0	4	1
A	__toString()	0	4	1
A	equals()	0	4	1

<?php

namespace BestServedCold\PhalueObjects;

/**
 * Class ValueObject
 *
 * @package   BestServedCold\PhalueObjects
 * @author    Adam Lewis <adam.lewis@bestservedcold.com>
 * @copyright Copyright (c) 2015 Best Served Cold Media Limited
 * @license	  http://http://opensource.org/licenses/GPL-3.0 GPL License
 * @link	  http://bestservedcold.com
 * @since	  0.0.1-alpha
 * @version   0.0.2-alpha
 */
class ValueObject implements ValueObjectInterface
{
    /**
     * @var \ReflectionClass
     */
    protected $reflection;

    /**
     * @var mixed
     */
    protected $value;

    /**
     * Class Constructor
     */
    public function __construct($value)
    {
        $this->value = $value;
        $this->reflection = new \ReflectionClass($this);
    }

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

    /**
     * @param $field
     * @param $value
     */
    public function __set($field, $value)
    {
        throw new \RuntimeException(
            "You cannot set a value of a Value Object, that's the whole point!"
        );
    }

    /**
     * @return string
     */
    public function hash()
    {
        return spl_object_hash($this);
    }

    /**
     * @param  $object
     * @return mixed

Consider making the return type a bit more specific; maybe use object.

     */
    public function cloneObject($object)
    {
        return clone($object);
    }

    /**
     * @param $string
     * @return static
     */
    public static function fromString($string)
    {
        return new static((string) $string);
    }

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

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

    /**
     * @param  ValueObject $object
     * @return bool
     */
    public function equals(ValueObject $object)

function equals() does not seem to conform to the naming convention (^(?:is|has|should|may|supports)).

    {
        return $this->value === $object->value;
    }
}