JSKOS\Constructor

Complexity

Total Complexity

5

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	__construct()	0	13	5

<?php declare(strict_types=1);

namespace JSKOS;

use InvalidArgumentException;

/**
 * Initialize field values of a new JSKOS Object.
 */
trait Constructor
{
    /**
     * Create a new JSKOS object with given field values.
     *
     * @param Array|Object fields to copy
     * @param bool strict throw error on unknown fields
     */
    public function __construct($data=null, bool $strict=false)
    {
        if (is_array($data) || is_object($data)) {
            foreach ($data as $key => $value) {
                $this->setField($key, $value, $strict);

It seems like setField() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            }
        } elseif ($data !== null) {
            throw new InvalidArgumentException(
                get_called_class() .
                ' constructor expects array, object, or null'
            );
        }
    }
}