Psi\Component\ObjectAgent\Query\Composite

Complexity

Total Complexity

4

Size/Duplication

Total Lines	40
Duplicated Lines	40 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	16	16	2
A	getExpressions()	0	4	1
A	getType()	0	4	1

<?php

declare(strict_types=1);

namespace Psi\Component\ObjectAgent\Query;

class Composite implements Expression
{
    const AND = 'and';
    const OR = 'or';

    private static $validTypes = [
        self::AND,
        self::OR,
    ];

    private $expressions;
    private $type;

    public function __construct(string $type, array $expressions)

This method seems to be duplicated in your project.

    {
        // ensure types
        array_map(function (Expression $expr) {

The parameter $expr is not used and could be removed.

        }, $expressions);

        if (!in_array($type, self::$validTypes)) {
            throw new \InvalidArgumentException(sprintf(
                'Invalid composite type "%s", must be one of "%s"',
                $type, implode('", "', self::$validTypes)
            ));
        }

        $this->expressions = $expressions;
        $this->type = $type;
    }

    public function getExpressions()
    {
        return $this->expressions;
    }

    public function getType()
    {
        return $this->type;
    }
}