Elastica\Query\SpanNear

Complexity

Total Complexity

8

Size/Duplication

Total Lines	59
Duplicated Lines	28.81 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	9	15	4
A	setSlop()	0	4	1
A	setInOrder()	0	4	1
A	addClause()	8	8	2

<?php
namespace Elastica\Query;

use Elastica\Exception\InvalidException;

/**
 * SpanNear query.
 *
 * @author Marek Hernik <marek.hernik@gmail.com>
 *
 * @link https://www.elastic.co/guide/en/elasticsearch/reference/current/query-dsl-span-near-query.html
 */
class SpanNear extends AbstractSpanQuery
{
    /**
     * Constructs a SpanNear query object.
     *
     * @param AbstractSpanQuery[] $clauses OPTIONAL
     * @param int $slop OPTIONAL maximum proximity
     * @param bool $inOrder OPTIONAL true if order of searched clauses is important
     */
    public function __construct($clauses = [], $slop = 1, $inOrder = false)
    {
        if (!empty($clauses)) {

This code seems to be duplicated across your project.

            foreach ($clauses as $clause) {
                if (!is_subclass_of($clause, AbstractSpanQuery::class)) {

Due to PHP Bug #53727, is_subclass_of might return inconsistent results on some PHP versions if \Elastica\Query\AbstractSpanQuery::class can be an interface. If so, you could instead use ReflectionClass::implementsInterface.

                    throw new InvalidException(
                        'Invalid parameter. Has to be array or instance of Elastica\Query\SpanQuery'
                    );
                }
            }
        }
        $this->setParams(['clauses' => $clauses]);
        $this->setSlop($slop);
        $this->setInOrder($inOrder);
    }

    /**
     * @param int $slop
     */
    public function setSlop($slop)
    {
        $this->setParam('slop', $slop);
    }

    /**
     * @param bool $inOrder
     */
    public function setInOrder($inOrder)
    {
        $this->setParam('in_order', $inOrder);
    }

    /**
     * Add clause part to query.
     *
     * @param AbstractSpanQuery $clause
     *
     * @throws InvalidException If not valid query
     *
     * @return $this
     */
    public function addClause($clause)

This method seems to be duplicated in your project.

    {
        if (!is_subclass_of($clause, AbstractSpanQuery::class)) {

Due to PHP Bug #53727, is_subclass_of might return inconsistent results on some PHP versions if \Elastica\Query\AbstractSpanQuery::class can be an interface. If so, you could instead use ReflectionClass::implementsInterface.

            throw new InvalidException('Invalid parameter. Has to be array or instance of Elastica\Query\SpanQuery');
        }

        return $this->addParam('clauses', $clause);
    }
}